[PATCH 1/3] video: sunxi: Change sunxi_get_mon_desc() to not return NULL for the default case
Tom Rini
trini at konsulko.com
Mon Apr 6 14:41:55 CEST 2020
On Mon, Apr 06, 2020 at 05:36:03AM -0700, Bin Meng wrote:
> When building with gcc 9.2.0, the following build warning was seen:
>
> drivers/video/sunxi/sunxi_display.c: In function 'video_hw_init':
> drivers/video/sunxi/sunxi_display.c:1217:2:
> error: '%s' directive argument is null [-Werror=format-overflow=]
>
> Change sunxi_get_mon_desc() to not return NULL for the default case,
> to fix the compiler warning.
>
> Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
> ---
>
> drivers/video/sunxi/sunxi_display.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/video/sunxi/sunxi_display.c b/drivers/video/sunxi/sunxi_display.c
> index 31f0aa7..5679d83 100644
> --- a/drivers/video/sunxi/sunxi_display.c
> +++ b/drivers/video/sunxi/sunxi_display.c
> @@ -1024,7 +1024,7 @@ static const char *sunxi_get_mon_desc(enum sunxi_monitor monitor)
> case sunxi_monitor_composite_pal_m: return "composite-pal-m";
> case sunxi_monitor_composite_pal_nc: return "composite-pal-nc";
> }
> - return NULL; /* never reached */
> + return "none"; /* never reached */
> }
>
> ulong board_get_usable_ram_top(ulong total_size)
That's fine. But do we want to re-work the function to catch
sunxi_monitor_none in the default case and use "none" there as well
(like it does today) to silence the warning? Thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200406/80bc0938/attachment.sig>
More information about the U-Boot
mailing list