| Message ID | 20240827165337.1075904-1-sherry.yang@oracle.com |
|---|---|
| State | Accepted |
| Commit | 25b85075150fe8adddb096db8a4b950353045ee1 |
| Headers | show |
| Series | [1/1] drm/msm: fix %s null argument error | expand |
> On Aug 27, 2024, at 10:50 AM, Abhinav Kumar <quic_abhinavk@quicinc.com> wrote: > > > > On 8/27/2024 9:53 AM, Sherry Yang wrote: >> The following build error was triggered because of NULL string argument: >> BUILDSTDERR: drivers/gpu/drm/msm/disp/mdp5/mdp5_smp.c: In function 'mdp5_smp_dump': >> BUILDSTDERR: drivers/gpu/drm/msm/disp/mdp5/mdp5_smp.c:352:51: error: '%s' directive argument is null [-Werror=format-overflow=] >> BUILDSTDERR: 352 | drm_printf(p, "%s:%d\t%d\t%s\n", >> BUILDSTDERR: | ^~ >> BUILDSTDERR: drivers/gpu/drm/msm/disp/mdp5/mdp5_smp.c:352:51: error: '%s' directive argument is null [-Werror=format-overflow=] >> This happens from the commit a61ddb4393ad ("drm: enable (most) W=1 >> warnings by default across the subsystem"). Using "(null)" instead >> to fix it. >> Fixes: bc5289eed481 ("drm/msm/mdp5: add debugfs to show smp block status") >> Signed-off-by: Sherry Yang <sherry.yang@oracle.com> >> --- >> drivers/gpu/drm/msm/disp/mdp5/mdp5_smp.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) > > I am not sure how the patch got titled 1/1 and not just "PATCH". It should be just "PATCH" My bad. I generated the patch with parameter “--numbered” in “git format-patch”, it will output in [PATCH n/m] format, even with a single patch. Will drop it for single patch next time. Thanks, Sherry > >> diff --git a/drivers/gpu/drm/msm/disp/mdp5/mdp5_smp.c b/drivers/gpu/drm/msm/disp/mdp5/mdp5_smp.c >> index 3a7f7edda96b..500b7dc895d0 100644 >> --- a/drivers/gpu/drm/msm/disp/mdp5/mdp5_smp.c >> +++ b/drivers/gpu/drm/msm/disp/mdp5/mdp5_smp.c >> @@ -351,7 +351,7 @@ void mdp5_smp_dump(struct mdp5_smp *smp, struct drm_printer *p, >> drm_printf(p, "%s:%d\t%d\t%s\n", >> pipe2name(pipe), j, inuse, >> - plane ? plane->name : NULL); >> + plane ? plane->name : "(null)"); >> total += inuse; >> } > > Change itself looks fine to me, > > > Reviewed-by: Abhinav Kumar <quic_abhinavk@quicinc.com>
diff --git a/drivers/gpu/drm/msm/disp/mdp5/mdp5_smp.c b/drivers/gpu/drm/msm/disp/mdp5/mdp5_smp.c index 3a7f7edda96b..500b7dc895d0 100644 --- a/drivers/gpu/drm/msm/disp/mdp5/mdp5_smp.c +++ b/drivers/gpu/drm/msm/disp/mdp5/mdp5_smp.c @@ -351,7 +351,7 @@ void mdp5_smp_dump(struct mdp5_smp *smp, struct drm_printer *p, drm_printf(p, "%s:%d\t%d\t%s\n", pipe2name(pipe), j, inuse, - plane ? plane->name : NULL); + plane ? plane->name : "(null)"); total += inuse; }
The following build error was triggered because of NULL string argument: BUILDSTDERR: drivers/gpu/drm/msm/disp/mdp5/mdp5_smp.c: In function 'mdp5_smp_dump': BUILDSTDERR: drivers/gpu/drm/msm/disp/mdp5/mdp5_smp.c:352:51: error: '%s' directive argument is null [-Werror=format-overflow=] BUILDSTDERR: 352 | drm_printf(p, "%s:%d\t%d\t%s\n", BUILDSTDERR: | ^~ BUILDSTDERR: drivers/gpu/drm/msm/disp/mdp5/mdp5_smp.c:352:51: error: '%s' directive argument is null [-Werror=format-overflow=] This happens from the commit a61ddb4393ad ("drm: enable (most) W=1 warnings by default across the subsystem"). Using "(null)" instead to fix it. Fixes: bc5289eed481 ("drm/msm/mdp5: add debugfs to show smp block status") Signed-off-by: Sherry Yang <sherry.yang@oracle.com> --- drivers/gpu/drm/msm/disp/mdp5/mdp5_smp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)