diff options
| author | Angel Pons <th3fanbus@gmail.com> | 2021-01-28 14:11:55 +0100 |
|---|---|---|
| committer | Angel Pons <th3fanbus@gmail.com> | 2021-01-30 23:07:55 +0000 |
| commit | ee7fb34dcb495460b4a172ed10b3aebcacdaacad (patch) | |
| tree | 5763d0eeaba8b2a2e8fed59182c5c76337a74eef /src/northbridge/intel/ironlake | |
| parent | 60f5328c7d4991d3fe9e4c0243f89f49e231f856 (diff) | |
nb/intel/ironlake: Use RCBA macros
Use defined RCBAx macros over readX/writeX calls.
Tested with BUILD_TIMELESS=1, Packard Bell MS2290 remains identical.
Change-Id: I87cae75268ef5f329001706e4771e98653d40cd1
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/50037
Reviewed-by: Patrick Rudolph <siro@das-labor.org>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/northbridge/intel/ironlake')
| -rw-r--r-- | src/northbridge/intel/ironlake/raminit.c | 25 |
1 files changed, 12 insertions, 13 deletions
diff --git a/src/northbridge/intel/ironlake/raminit.c b/src/northbridge/intel/ironlake/raminit.c index 1a66e1cf5f..c3d1b0d15f 100644 --- a/src/northbridge/intel/ironlake/raminit.c +++ b/src/northbridge/intel/ironlake/raminit.c @@ -1822,19 +1822,18 @@ static void setup_heci_uma(struct raminfo *info) pci_read_config32(NORTHBRIDGE, DMIBAR); if (info->memory_reserved_for_heci_mb) { DMIBAR32(DMIVC0RCTL) &= ~0x80; - write32(DEFAULT_RCBA + 0x14, read32(DEFAULT_RCBA + 0x14) & ~0x80); + RCBA32(0x14) &= ~0x80; DMIBAR32(DMIVC1RCTL) &= ~0x80; - write32(DEFAULT_RCBA + 0x20, read32(DEFAULT_RCBA + 0x20) & ~0x80); + RCBA32(0x20) &= ~0x80; DMIBAR32(DMIVCPRCTL) &= ~0x80; - write32(DEFAULT_RCBA + 0x30, read32(DEFAULT_RCBA + 0x30) & ~0x80); + RCBA32(0x30) &= ~0x80; DMIBAR32(DMIVCMRCTL) &= ~0x80; - write32(DEFAULT_RCBA + 0x40, read32(DEFAULT_RCBA + 0x40) & ~0x80); + RCBA32(0x40) &= ~0x80; - write32(DEFAULT_RCBA + 0x40, 0x87000080); // OK + RCBA32(0x40) = 0x87000080; // OK DMIBAR32(DMIVCMRCTL) = 0x87000080; // OK - while ((read16(DEFAULT_RCBA + 0x46) & 2) && - DMIBAR16(DMIVCMRSTS) & VCMNP) + while ((RCBA16(0x46) & 2) && DMIBAR16(DMIVCMRSTS) & VCMNP) ; } @@ -3667,17 +3666,17 @@ void chipset_init(const int s3resume) MCHBAR32_AND_OR(0x2c44, 0, 0x1053687); pci_read_config8(GMA, MSAC); // = 0x2 pci_write_config8(GMA, MSAC, 0x2); - read8(DEFAULT_RCBA + 0x2318); - write8(DEFAULT_RCBA + 0x2318, 0x47); - read8(DEFAULT_RCBA + 0x2320); - write8(DEFAULT_RCBA + 0x2320, 0xfc); + RCBA8(0x2318); + RCBA8(0x2318) = 0x47; + RCBA8(0x2320); + RCBA8(0x2320) = 0xfc; } MCHBAR32_AND_OR(0x30, 0, 0x40); pci_write_config16(NORTHBRIDGE, GGC, ggc); - gav(read32(DEFAULT_RCBA + 0x3428)); - write32(DEFAULT_RCBA + 0x3428, 0x1d); + gav(RCBA32(0x3428)); + RCBA32(0x3428) = 0x1d; } void raminit(const int s3resume, const u8 *spd_addrmap) |