Commit 4b733b2c authored by Tom Rix's avatar Tom Rix Committed by Neil Armstrong

drm/meson: set variables meson_hdmi_* storage-class-specifier to static

smatch has several similar warnings to
drivers/gpu/drm/meson/meson_venc.c:189:28: warning: symbol
  'meson_hdmi_enci_mode_480i' was not declared. Should it be static?

These variables are only used in their defining file so should be static
Signed-off-by: default avatarTom Rix <trix@redhat.com>
Acked-by: default avatarMartin Blumenstingl <martin.blumenstingl@googlemail.com>
Signed-off-by: default avatarNeil Armstrong <neil.armstrong@linaro.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20230423145300.3937831-1-trix@redhat.com
parent 0ed9208c
...@@ -186,7 +186,7 @@ union meson_hdmi_venc_mode { ...@@ -186,7 +186,7 @@ union meson_hdmi_venc_mode {
} encp; } encp;
}; };
union meson_hdmi_venc_mode meson_hdmi_enci_mode_480i = { static union meson_hdmi_venc_mode meson_hdmi_enci_mode_480i = {
.enci = { .enci = {
.hso_begin = 5, .hso_begin = 5,
.hso_end = 129, .hso_end = 129,
...@@ -206,7 +206,7 @@ union meson_hdmi_venc_mode meson_hdmi_enci_mode_480i = { ...@@ -206,7 +206,7 @@ union meson_hdmi_venc_mode meson_hdmi_enci_mode_480i = {
}, },
}; };
union meson_hdmi_venc_mode meson_hdmi_enci_mode_576i = { static union meson_hdmi_venc_mode meson_hdmi_enci_mode_576i = {
.enci = { .enci = {
.hso_begin = 3, .hso_begin = 3,
.hso_end = 129, .hso_end = 129,
...@@ -226,7 +226,7 @@ union meson_hdmi_venc_mode meson_hdmi_enci_mode_576i = { ...@@ -226,7 +226,7 @@ union meson_hdmi_venc_mode meson_hdmi_enci_mode_576i = {
}, },
}; };
union meson_hdmi_venc_mode meson_hdmi_encp_mode_480p = { static union meson_hdmi_venc_mode meson_hdmi_encp_mode_480p = {
.encp = { .encp = {
.dvi_settings = 0x21, .dvi_settings = 0x21,
.video_mode = 0x4000, .video_mode = 0x4000,
...@@ -272,7 +272,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_480p = { ...@@ -272,7 +272,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_480p = {
}, },
}; };
union meson_hdmi_venc_mode meson_hdmi_encp_mode_576p = { static union meson_hdmi_venc_mode meson_hdmi_encp_mode_576p = {
.encp = { .encp = {
.dvi_settings = 0x21, .dvi_settings = 0x21,
.video_mode = 0x4000, .video_mode = 0x4000,
...@@ -318,7 +318,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_576p = { ...@@ -318,7 +318,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_576p = {
}, },
}; };
union meson_hdmi_venc_mode meson_hdmi_encp_mode_720p60 = { static union meson_hdmi_venc_mode meson_hdmi_encp_mode_720p60 = {
.encp = { .encp = {
.dvi_settings = 0x2029, .dvi_settings = 0x2029,
.video_mode = 0x4040, .video_mode = 0x4040,
...@@ -360,7 +360,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_720p60 = { ...@@ -360,7 +360,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_720p60 = {
}, },
}; };
union meson_hdmi_venc_mode meson_hdmi_encp_mode_720p50 = { static union meson_hdmi_venc_mode meson_hdmi_encp_mode_720p50 = {
.encp = { .encp = {
.dvi_settings = 0x202d, .dvi_settings = 0x202d,
.video_mode = 0x4040, .video_mode = 0x4040,
...@@ -405,7 +405,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_720p50 = { ...@@ -405,7 +405,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_720p50 = {
}, },
}; };
union meson_hdmi_venc_mode meson_hdmi_encp_mode_1080i60 = { static union meson_hdmi_venc_mode meson_hdmi_encp_mode_1080i60 = {
.encp = { .encp = {
.dvi_settings = 0x2029, .dvi_settings = 0x2029,
.video_mode = 0x5ffc, .video_mode = 0x5ffc,
...@@ -454,7 +454,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_1080i60 = { ...@@ -454,7 +454,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_1080i60 = {
}, },
}; };
union meson_hdmi_venc_mode meson_hdmi_encp_mode_1080i50 = { static union meson_hdmi_venc_mode meson_hdmi_encp_mode_1080i50 = {
.encp = { .encp = {
.dvi_settings = 0x202d, .dvi_settings = 0x202d,
.video_mode = 0x5ffc, .video_mode = 0x5ffc,
...@@ -503,7 +503,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_1080i50 = { ...@@ -503,7 +503,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_1080i50 = {
}, },
}; };
union meson_hdmi_venc_mode meson_hdmi_encp_mode_1080p24 = { static union meson_hdmi_venc_mode meson_hdmi_encp_mode_1080p24 = {
.encp = { .encp = {
.dvi_settings = 0xd, .dvi_settings = 0xd,
.video_mode = 0x4040, .video_mode = 0x4040,
...@@ -552,7 +552,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_1080p24 = { ...@@ -552,7 +552,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_1080p24 = {
}, },
}; };
union meson_hdmi_venc_mode meson_hdmi_encp_mode_1080p30 = { static union meson_hdmi_venc_mode meson_hdmi_encp_mode_1080p30 = {
.encp = { .encp = {
.dvi_settings = 0x1, .dvi_settings = 0x1,
.video_mode = 0x4040, .video_mode = 0x4040,
...@@ -596,7 +596,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_1080p30 = { ...@@ -596,7 +596,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_1080p30 = {
}, },
}; };
union meson_hdmi_venc_mode meson_hdmi_encp_mode_1080p50 = { static union meson_hdmi_venc_mode meson_hdmi_encp_mode_1080p50 = {
.encp = { .encp = {
.dvi_settings = 0xd, .dvi_settings = 0xd,
.video_mode = 0x4040, .video_mode = 0x4040,
...@@ -644,7 +644,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_1080p50 = { ...@@ -644,7 +644,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_1080p50 = {
}, },
}; };
union meson_hdmi_venc_mode meson_hdmi_encp_mode_1080p60 = { static union meson_hdmi_venc_mode meson_hdmi_encp_mode_1080p60 = {
.encp = { .encp = {
.dvi_settings = 0x1, .dvi_settings = 0x1,
.video_mode = 0x4040, .video_mode = 0x4040,
...@@ -688,7 +688,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_1080p60 = { ...@@ -688,7 +688,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_1080p60 = {
}, },
}; };
union meson_hdmi_venc_mode meson_hdmi_encp_mode_2160p24 = { static union meson_hdmi_venc_mode meson_hdmi_encp_mode_2160p24 = {
.encp = { .encp = {
.dvi_settings = 0x1, .dvi_settings = 0x1,
.video_mode = 0x4040, .video_mode = 0x4040,
...@@ -730,7 +730,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_2160p24 = { ...@@ -730,7 +730,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_2160p24 = {
}, },
}; };
union meson_hdmi_venc_mode meson_hdmi_encp_mode_2160p25 = { static union meson_hdmi_venc_mode meson_hdmi_encp_mode_2160p25 = {
.encp = { .encp = {
.dvi_settings = 0x1, .dvi_settings = 0x1,
.video_mode = 0x4040, .video_mode = 0x4040,
...@@ -772,7 +772,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_2160p25 = { ...@@ -772,7 +772,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_2160p25 = {
}, },
}; };
union meson_hdmi_venc_mode meson_hdmi_encp_mode_2160p30 = { static union meson_hdmi_venc_mode meson_hdmi_encp_mode_2160p30 = {
.encp = { .encp = {
.dvi_settings = 0x1, .dvi_settings = 0x1,
.video_mode = 0x4040, .video_mode = 0x4040,
...@@ -814,7 +814,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_2160p30 = { ...@@ -814,7 +814,7 @@ union meson_hdmi_venc_mode meson_hdmi_encp_mode_2160p30 = {
}, },
}; };
struct meson_hdmi_venc_vic_mode { static struct meson_hdmi_venc_vic_mode {
unsigned int vic; unsigned int vic;
union meson_hdmi_venc_mode *mode; union meson_hdmi_venc_mode *mode;
} meson_hdmi_venc_vic_modes[] = { } meson_hdmi_venc_vic_modes[] = {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment