From: Venkatesh Yadav Abbarapu Date: Mon, 23 Jun 2025 08:06:07 +0000 (+0530) Subject: i2c: mux: Fix the crash when the i2c-arbitrator node is present X-Git-Tag: v2025.07~9 X-Git-Url: http://git.openpandora.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=903eb123236ccbd8ef05d43507a2a910b785bd56;p=pandora-u-boot.git i2c: mux: Fix the crash when the i2c-arbitrator node is present Observing the crash when we add the i2c-arbitrator node in the device tree as per the DT bindings. The issue is with the child node of i2c-arbitrator@72 i.e., i2c@f1950000->i2c-arbitrator@72->i2c-arb, as the arbitrator uses the uclass of mux(UCLASS_I2C_MUX) and the mux uclass driver checks for the "reg" property using the i2c_mux_child_post_bind() function, if it won't find the "reg" property it will return -EINVAL which is leading to the crash. So, add the logic to check whether the child node has the "reg" property, if the "reg" property exists then read the "reg" and update the channel. https://www.kernel.org/doc/Documentation/devicetree/bindings/i2c/i2c-arb.txt Signed-off-by: Venkatesh Yadav Abbarapu Reviewed-by: Heiko Schocher --- diff --git a/drivers/i2c/muxes/i2c-mux-uclass.c b/drivers/i2c/muxes/i2c-mux-uclass.c index d1999d21feb..012881de05b 100644 --- a/drivers/i2c/muxes/i2c-mux-uclass.c +++ b/drivers/i2c/muxes/i2c-mux-uclass.c @@ -40,6 +40,11 @@ static int i2c_mux_child_post_bind(struct udevice *dev) struct i2c_mux_bus *plat = dev_get_parent_plat(dev); int channel; + ofnode node = dev_ofnode(dev); + + if (!ofnode_has_property(node, "reg")) + return 0; + channel = dev_read_u32_default(dev, "reg", -1); if (channel < 0) return -EINVAL;