[Concept,16/42] video: Convert to use uclass_id_foreach_dev()

Message ID 20250919201507.4024144-17-sjg@u-boot.org
State New
Headers
Series video: Support a cursor more generally |

Commit Message

Simon Glass Sept. 19, 2025, 8:14 p.m. UTC
  From: Simon Glass <sjg@chromium.org>

Convert a few functions in the video uclass to use the new
uclass_id_foreach_dev() macro.

Signed-off-by: Simon Glass <sjg@chromium.org>
---

 drivers/video/video-uclass.c | 15 ++++++---------
 1 file changed, 6 insertions(+), 9 deletions(-)
  

Patch

diff --git a/drivers/video/video-uclass.c b/drivers/video/video-uclass.c
index 324817aa5ce..cd5c8dd8fad 100644
--- a/drivers/video/video-uclass.c
+++ b/drivers/video/video-uclass.c
@@ -127,15 +127,14 @@  static ulong alloc_fb(struct udevice *dev, ulong *addrp)
 int video_reserve(ulong *addrp)
 {
 	struct udevice *dev;
+	struct uclass *uc;
 	ulong size;
 
 	if (IS_ENABLED(CONFIG_SPL_VIDEO_HANDOFF) && xpl_phase() == PHASE_BOARD_F)
 		return 0;
 
 	gd->video_top = *addrp;
-	for (uclass_find_first_device(UCLASS_VIDEO, &dev);
-	     dev;
-	     uclass_find_next_device(&dev)) {
+	uclass_id_foreach_dev(UCLASS_VIDEO, dev, uc) {
 		size = alloc_fb(dev, addrp);
 		debug("%s: Reserving %lx bytes at %lx for video device '%s'\n",
 		      __func__, size, *addrp, dev->name);
@@ -539,11 +538,10 @@  int video_sync(struct udevice *vid, bool force)
 void video_sync_all(void)
 {
 	struct udevice *dev;
+	struct uclass *uc;
 	int ret;
 
-	for (uclass_find_first_device(UCLASS_VIDEO, &dev);
-	     dev;
-	     uclass_find_next_device(&dev)) {
+	uclass_id_foreach_dev(UCLASS_VIDEO, dev, uc) {
 		if (device_active(dev)) {
 			ret = video_sync(dev, true);
 			if (ret)
@@ -555,14 +553,13 @@  void video_sync_all(void)
 bool video_is_active(void)
 {
 	struct udevice *dev;
+	struct uclass *uc;
 
 	/* Assume video to be active if SPL passed video hand-off to U-boot */
 	if (IS_ENABLED(CONFIG_SPL_VIDEO_HANDOFF) && xpl_phase() > PHASE_SPL)
 		return true;
 
-	for (uclass_find_first_device(UCLASS_VIDEO, &dev);
-	     dev;
-	     uclass_find_next_device(&dev)) {
+	uclass_id_foreach_dev(UCLASS_VIDEO, dev, uc) {
 		if (device_active(dev))
 			return true;
 	}