diff options
author | Simon Glass | 2020-04-08 16:57:28 -0600 |
---|---|---|
committer | Bin Meng | 2020-04-16 14:36:28 +0800 |
commit | 163085308576139ac0b0c65e3b81be78ab890aa9 (patch) | |
tree | d7741b23436203012923a15b565f3b0e4fa4e927 /tools/gpimage-common.c | |
parent | 2831405447caeb55b5f6b625c8be9a4a2f8fcaca (diff) |
x86: apl: Move p2sb ofdata reading to the correct method
With P2SB the initial BAR (base-address register) is set up by TPL and
this is used unchanged right through U-Boot.
At present the reading of this address is split between the ofdata() and
probe() methods. There are a few problems that are unique to the p2sb.
One is that its children need to call pcr_read32(), etc. which needs to
have the p2sb address correct. Also some of its children are pinctrl
devices and pinctrl is used when any device is probed. So p2sb really
needs to get its base address set up in ofdata_to_platdata(), before it is
probed.
Another point is that reading the p2sb BAR will not work if the p2sb is
hidden. The FSP-S seems to hide it, presumably to avoid confusing PCI
enumeration.
Reading ofdata in ofdata_to_platdata() is the correct place anyway, so
this is easy to fix.
Move the code into one place and use the early-regs property in all cases
for simplicity and to avoid needing to probe any PCI devices just to read
the BAR.
Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Wolfgang Wallner <wolfgang.wallner@br-automation.com>
Tested-by: Wolfgang Wallner <wolfgang.wallner@br-automation.com>
Diffstat (limited to 'tools/gpimage-common.c')
0 files changed, 0 insertions, 0 deletions