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