[coreboot-gerrit] Change in coreboot[master]: qualcomm/sc7280: gpio: Support eGPIO scheme

[Taniya Das (Code Review)]

Attention is currently required from: Shelley Chen, Ravi kumar, Paul Menzel, Julius Werner, mturney mturney.
Taniya Das has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55078 )

Change subject: qualcomm/sc7280: gpio: Support eGPIO scheme
......................................................................

Patch Set 12:

(1 comment)

File src/soc/qualcomm/sc7280/sc7280_egpio.c:

https://review.coreboot.org/c/coreboot/+/55078/comment/f118cbe6_7726f3ae 
PS3, Line 56: 		setbits32(&regs->cfg, BIT(EGPIO_CFG_EN));
> Hi Taniya,  was wondering if Qualcomm can respond to Julius' comment above?
> > >

The kernel should always do its own GPIO setup and not rely on firmware to prepare things correctly.
> > >

Yes you are correct Julius. But here these GPIOs cannot be accessed by CPU/APSS subsystem i.e kernel driver unless we move these GPIOs which are dedicated to the LPASS subsystem to APSS/CPU. This was a request which was asked as a feature and thus we are supporting them in CB.