2 comments:
File src/mainboard/google/kahlee/mainboard.c:
<soc/smi.h>
Why is smi. […]
Looks like that's left over from crud I used to have in here, but seems to build OK w/o the addition. I'll remove.
File src/soc/amd/stoneyridge/include/soc/gpio.h:
Could you leave GPIO_BANK0_CONTROL, GPIO_BANK1_CONTROL and GPI2_BANK0_CONTROL in gpio. […]
It won't have a different MMIO offset as the MMIO offsets have remained consistent over the last 10 years. There are two implementations of GPIOs The older one squeezed everything into the block at +100. The newer deprecated the use of +100 and implements 3 banks at +1500/1600/1700. Every device that falls within each of the two categories behaves like all the others in that category. This is also why I chose to name the newer feature "SOC_AMD_COMMON_BLOCK_BANKED_GPIOS". In the event we ever implemented the older one here, it wouldn't be banked.
To view, visit change 32651. To unsubscribe, or for help writing mail filters, visit settings.