Re: [PATCH v3] fbdev: platinumfb: add error checking for ioremap calls

[Helge Deller]

On 8/20/26 08:00, Yue Bing Kun wrote:
> From: BingKun Yue <yuebingkun@xxxxxxxxxx>

You are using various combinations of your name:
- BingKun Yue (as signed-off address)
- Yue Bing Kun (in plain email)
Maybe you should decide on one?

> The ioremap() and ioremap_wt() calls in platinumfb_probe() were not
> checked for failure. If any of these mappings fail, the driver would
> dereference NULL pointers, leading to a kernel panic.
>
> Add proper error checking and use goto-based cleanup to avoid code
> duplication across the error paths.

This commit message has nothing to do with this patch.
 
> Reported-by: Nathan Chancellor <nathan@xxxxxxxxxx>
> Link: https://lore.kernel.org/linux-fbdev/20260819235918.GA2021182@ax162/
> Signed-off-by: BingKun Yue <yuebingkun@xxxxxxxxxx>
> ---
>   drivers/video/fbdev/platinumfb.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/video/fbdev/platinumfb.c b/drivers/video/fbdev/platinumfb.c
> index fdffadcc3e73..f74c937c839f 100644
> --- a/drivers/video/fbdev/platinumfb.c
> +++ b/drivers/video/fbdev/platinumfb.c
> @@ -320,7 +320,7 @@ static void platinum_init_info(struct fb_info *info,
>   	fb_alloc_cmap(&info->cmap, 256, 0);
>   
>   	/* Fill fix common fields */
> -	strscpy(info->fix.id, "platinum", sizeof(info->fix.id));
> +	strcpy(info->fix.id, "platinum");

And this one is a) reversed and b) useless & contraproductive (I will not take it).

Helge