Chris Webb chris@arachsys.com wrote:
Tom Rini trini@konsulko.com wrote:
Adding Marek, as the author of commit 48b3ecbedf82 ("gpio: Get rid of gpio_hog_probe_all()").
Thanks! I don't claim this is the correct way to fix this, just that it works.
Specifically, the two things I found that got gpio-hog working were
(a) adding an explicit probe instead of DM_FLAG_PROBE_AFTER_BIND in gpio_post_bind(), or
(b) adding a .bind function in U_BOOT_DRIVER(mt7981_pinctrl) like
static int mtk_pinctrl_mt7981_bind(struct udevice *dev) { dev_or_flags(dev, DM_FLAG_PROBE_AFTER_BIND); return 0; }However, presumably (b) isn't right as it would (presumably) need repeating in lots of other pinctrl drivers?
Now the release is out, I'd be really keen to pick this one up and get it fixed upstream if possible.
The device I originally discovered this on is now deployed, but I could probably grab it back for a bit and resolder a serial console onto it for further testing if neither of the above is correct and a third alternative I didn't try needs confirming on real hardware.
Best wishes,
Chris.