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(-) 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; } -- 2.43.0