tree 68b21c9e61abc4e2e45c849debfa4b906a0e0acb
parent b35adab86280e9328f598e5dbd71e8134ea10813
author Nico Huber <nico.h@gmx.de> 1612618794 +0100
committer Patrick Georgi <pgeorgi@google.com> 1612735159 +0000

nb/intel/pineview: Clean up FIXMEs in raminit

Using MCHBAR32_AND_OR() in these two cases changes the order of
additions slightly. Originally, the MCHBAR offset and the base
register offset (0x5a4/0x5b4) were added first. Due to the added
parentheses in the register macros, now the complete register
offset is calculated first and then added to MCHBAR. Associativity
tells us that this doesn't change the result.

Changes in the resulting binary were verified manually on the
object file.

Change-Id: Id10882225c8e82b02583aa73e73d661c25abdef9
Signed-off-by: Nico Huber <nico.h@gmx.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/50355
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>