Attention is currently required from: Angel Pons, Jonathon Hall, Felix Held.
Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74369 )
Change subject: mb/purism/librem_cnl: Configure SuperIO for Librem Mini v1/v2 ......................................................................
Patch Set 4:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/74369/comment/1f95c0be_75a432c7 PS4, Line 17: open and BRAM bank 1 is accessible. LPC I/O range happens later with CB:74363.
File src/mainboard/purism/librem_cnl/variants/librem_mini/overridetree.cb:
https://review.coreboot.org/c/coreboot/+/74369/comment/9a8dcbd1_4df41bc5 PS4, Line 166: io 0x62 = 0x360 # BRAM1 I/O base address I'm fine with the address. If you only need 0x360/0x361 pair, there might be some smaller LPC decode enable you can use. Like at 0x200 or 0x208. Then again, I don't remember any typical hardware probing and poking at 0x360/0x361 so that should be a safe choice.