Duncan Laurie has posted comments on this change. ( https://review.coreboot.org/29429 )
Change subject: mb/google/poppy/variant/nocturne: use PLTRST for GPP_C11 ......................................................................
Patch Set 4:
(1 comment)
https://review.coreboot.org/#/c/29429/2/src/mainboard/google/poppy/variants/... File src/mainboard/google/poppy/variants/nocturne/gpio.c:
https://review.coreboot.org/#/c/29429/2/src/mainboard/google/poppy/variants/... PS2, Line 138: PLTRST
I had read in another CL yesterday (that I'm having trouble finding again to include here, sorry) th […]
I was wondering the same thing when I added sarien/arcada. It does make sense that you would want the status to get reset on PLRST assertion for regular interrupt pins.
But probably not if the pin is expected to be able to wake? (like Furquan's comment above) Not sure how to reconcile the behavior in that case to ensure it doesn't have issues.