[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