video: annotate switch/case fall-through
authorAndre Przywara <andre.przywara@arm.com>
Thu, 27 Mar 2025 15:33:05 +0000 (15:33 +0000)
committerTom Rini <trini@konsulko.com>
Tue, 8 Apr 2025 22:23:51 +0000 (16:23 -0600)
The generic DM video code uses an implicit switch/case fallthrough to
provide fallback code paths when certain colour depths are not enabled.

Add our "fallthrough;" statement-like macro to the video_fill() function
to avoid a warning when GCC's -Wimplicit-fallthrough warning option is
enabled.

Signed-off-by: Andre Przywara <andre.przywara@arm.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
drivers/video/video-uclass.c

index ff4f219..c684c99 100644 (file)
@@ -245,6 +245,7 @@ int video_fill(struct udevice *dev, u32 colour)
                                *ppix++ = colour;
                        break;
                }
+               fallthrough;
        case VIDEO_BPP32:
                if (CONFIG_IS_ENABLED(VIDEO_BPP32)) {
                        u32 *ppix = priv->fb;
@@ -254,6 +255,7 @@ int video_fill(struct udevice *dev, u32 colour)
                                *ppix++ = colour;
                        break;
                }
+               fallthrough;
        default:
                memset(priv->fb, colour, priv->fb_size);
                break;