video: stm32: dsi: get first panel device

Yannick Fertre · Patrice Chotard · commit 6d0626c68ab7 · 2025-05-16T09:17:00.000+02:00
Stop parsing the device tree if a panel was found. Change-Id: I0aef1cbf818da70f1dbd3d6e2536c36876fbee8b Signed-off-by: Yannick Fertre <yannick.fertre@foss.st.com> Reviewed-on: https://gerrit.st.com/c/mpu/oe/st/u-boot/+/451727 ACI: CITOOLS <MDG-smet-aci-reviews@list.st.com> Reviewed-by: Patrice CHOTARD <patrice.chotard@foss.st.com>
diff --git a/drivers/video/stm32/stm32_dsi.c b/drivers/video/stm32/stm32_dsi.c
@@ -843,11 +843,13 @@ static int stm32_dsi_get_panel(struct udevice *dev, struct udevice **panel)
 			}
 
 			uclass_get_device_by_ofnode(UCLASS_PANEL, remote, panel);
+			/* Stop parsing the device tree if a panel was found. */
 			if (*panel)
-				break;
+				goto out;
 		}
 	}
 
+out:
 	/* Sanity check, we can get out of the loop without having a clean ofnode */
 	if (!(*panel))
 		ret = -EINVAL;

Original file line number	Diff line number	Diff line change
`@@ -843,11 +843,13 @@ static int stm32_dsi_get_panel(struct udevice dev, struct udevice *panel)`
`843`	`843`	`}`
`844`	`844`
`845`	`845`	`uclass_get_device_by_ofnode(UCLASS_PANEL, remote, panel);`
	`846`	`+ /* Stop parsing the device tree if a panel was found. */`
`846`	`847`	`if (*panel)`
`847`		`- break;`
	`848`	`+ goto out;`
`848`	`849`	`}`
`849`	`850`	`}`
`850`	`851`
	`852`	`+out:`
`851`	`853`	`/* Sanity check, we can get out of the loop without having a clean ofnode */`
`852`	`854`	`if (!(*panel))`
`853`	`855`	`ret = -EINVAL;`