shkim has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37911 )
Change subject: mb/google/kohaku: Set reset hold time for I2C digitizer device ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/37911/1/src/mainboard/google/hatch/... File src/mainboard/google/hatch/variants/kohaku/overridetree.cb:
https://review.coreboot.org/c/coreboot/+/37911/1/src/mainboard/google/hatch/... PS1, Line 213: post-power-on-delay-ms
So, is the intent of this change to push the handling of reset delay to kernel driver and not rely o […]
Wacom guys advised to put an 100ms to hold reset_gpio as low for the issue b:129159369, and I expected this change to put the delay before reset_gpio to high in driver. But actually it's doing same thing with "reset_delay_ms" as Tim commented. BTW we saw either of this change and "reset_delay_ms = 100" improves the driver binding issue in b:129159369, so we need further check for this issue.