[coreboot-gerrit] Change in coreboot[master]: mb/google/octopus/variants/fleex: Update GPIOs to fix EMR

[EricR Lai (Code Review)]

EricR Lai has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35881 )

Change subject: mb/google/octopus/variants/fleex: Update GPIOs to fix EMR
......................................................................

Patch Set 2:
> Patch Set 2:
> > Patch Set 2:
> > > Patch Set 2:
> > >
> > > (1 comment)
> >
> > It add delay based on enable_delay_ms between enable and disable reset, I think we miss this in device tree. ELAN0001 and WDHT0002 have this.
>
> Nice catch since EN_PP3300_TOUCHSCREEN control the power of stylus and if you add it as enable pin and add delay as well then it will be good to me. But curious about why without it the power of stylus is still enabled currently.

That's true and HW measure power sequence is Pass. Kind of weird.