Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39121 )
Change subject: mb/google/dedede: configure ESPI IO decode range for chrome EC ......................................................................
Patch Set 2:
(1 comment)
Patch Set 2:
(1 comment)
Patch Set 2:
LGTM, but just a thought, do we want to move the LPC generic IO decode ranges to our EC driver? Then we don't have to keep adding it to every mainboard's devicetree.
This I/O decode range configuration seems to be Intel SoC specific - not the range itself, but the register value. So moving to the EC driver may be tricky and might require SoC knowledge in there.
Actually, these are the I/O windows that Chrome EC uses -- 0x800, 0x900, 0x200. The way they get mapped to LPC I/O regs is specific to Intel, but I think we can do some work to avoid having to duplicate this across mainboards. I believe this was done differently on APL/GLK, but it still required mainboard to provide a function to do the work.
https://review.coreboot.org/c/coreboot/+/39121/2/src/mainboard/google/dedede... File src/mainboard/google/dedede/variants/baseboard/devicetree.cb:
https://review.coreboot.org/c/coreboot/+/39121/2/src/mainboard/google/dedede... PS2, Line 23: f
Discussed offline with Tim and we found that 0xfc translates to 0x3f number of words as accessible i […]
dwords*