michael/u-boot
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

pinctrl: mediatek: ignored error return from pupd/r1/r0

Browse Source 
Ignore the error return value from mtk_pinconf_bias_set_pupd_r1_r0().
The PUPD/R1/R0 registers only include a small subset of the pins, so
it is normal for this function to return an error for most pins.
Therefore, this error should not be propagated.

This fixes not all pins in a pinmux group being configured in some
cases because the propagated error caused the configuration loop to
exit early.

The rest of the function is refactored to return early on errors to
improve readability.

Signed-off-by: David Lechner <dlechner@baylibre.com>
This commit is contained in:
David Lechner
2026-01-13 19:48:31 -06:00
committed by Tom Rini
parent e63e2e797e
commit 2d665b9cd3
1 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
drivers/pinctrl/mediatek/pinctrl-mtk-common.c
View File

@@ -364,11 +364,13 @@ int mtk_pinconf_bias_set_v1(struct udevice *dev, u32 pin, bool disable,
/* set pupd_r1_r0 if pullen_pullsel succeeded */
err = mtk_pinconf_bias_set_pullen_pullsel(dev, pin, disable, pullup,
val);
if (!err)
return mtk_pinconf_bias_set_pupd_r1_r0(dev, pin, disable,
pullup, val);
if (err)
return err;
return err;
/* Not all pins have PUPD/R1/R0 registers, so ignore any error here. */
mtk_pinconf_bias_set_pupd_r1_r0(dev, pin, disable, pullup, val);
return 0;
}
int mtk_pinconf_bias_set_pu_pd(struct udevice *dev, u32 pin, bool disable,