LinuxLists.cc - [PATCH] drm: radeon: replace 0 with NULL

2019-11-26 00:37:43

Subject: [PATCH] drm: radeon: replace 0 with NULL

Replace 0 with NULL to fix sparse tool warning
warning: Using plain integer as NULL pointer

Signed-off-by: Jules Irenge <[email protected]>
---
drivers/gpu/drm/radeon/radeon_audio.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/radeon/radeon_audio.c b/drivers/gpu/drm/radeon/radeon_audio.c
index b9aea5776d3d..2269cfced788 100644
--- a/drivers/gpu/drm/radeon/radeon_audio.c
+++ b/drivers/gpu/drm/radeon/radeon_audio.c
@@ -288,7 +288,7 @@ static void radeon_audio_interface_init(struct radeon_device *rdev)
} else {
rdev->audio.funcs = &r600_funcs;
rdev->audio.hdmi_funcs = &r600_hdmi_funcs;
- rdev->audio.dp_funcs = 0;
+ rdev->audio.dp_funcs = NULL;
}
}

--
2.23.0

2019-11-26 10:12:17

by Christian König

[permalink] [raw]

Subject: Re: [PATCH] drm: radeon: replace 0 with NULL

Am 26.11.19 um 01:35 schrieb Jules Irenge:
> Replace 0 with NULL to fix sparse tool warning
> warning: Using plain integer as NULL pointer
>
> Signed-off-by: Jules Irenge <[email protected]>

Acked-by: Christian König <[email protected]>

> ---
> drivers/gpu/drm/radeon/radeon_audio.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon_audio.c b/drivers/gpu/drm/radeon/radeon_audio.c
> index b9aea5776d3d..2269cfced788 100644
> --- a/drivers/gpu/drm/radeon/radeon_audio.c
> +++ b/drivers/gpu/drm/radeon/radeon_audio.c
> @@ -288,7 +288,7 @@ static void radeon_audio_interface_init(struct radeon_device *rdev)
> } else {
> rdev->audio.funcs = &r600_funcs;
> rdev->audio.hdmi_funcs = &r600_hdmi_funcs;
> - rdev->audio.dp_funcs = 0;
> + rdev->audio.dp_funcs = NULL;
> }
> }
>

2019-11-26 15:01:09

by Liu, Zhan

[permalink] [raw]

Subject: RE: [PATCH] drm: radeon: replace 0 with NULL

> -----Original Message-----
> From: amd-gfx <[email protected]> On Behalf Of
> Christian König
> Sent: 2019/November/26, Tuesday 5:10 AM
> To: Jules Irenge <[email protected]>; Deucher, Alexander
> <[email protected]>
> Cc: Zhou, David(ChunMing) <[email protected]>; [email protected];
> [email protected]; [email protected]; dri-
> [email protected]; [email protected]
> Subject: Re: [PATCH] drm: radeon: replace 0 with NULL
>
> Am 26.11.19 um 01:35 schrieb Jules Irenge:
> > Replace 0 with NULL to fix sparse tool warning
> > warning: Using plain integer as NULL pointer
> >
> > Signed-off-by: Jules Irenge <[email protected]>
>
> Acked-by: Christian König <[email protected]>

Reviewed-by: Zhan Liu <[email protected]>

>
> > ---
> > drivers/gpu/drm/radeon/radeon_audio.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/radeon/radeon_audio.c
> b/drivers/gpu/drm/radeon/radeon_audio.c
> > index b9aea5776d3d..2269cfced788 100644
> > --- a/drivers/gpu/drm/radeon/radeon_audio.c
> > +++ b/drivers/gpu/drm/radeon/radeon_audio.c
> > @@ -288,7 +288,7 @@ static void radeon_audio_interface_init(struct
> radeon_device *rdev)
> > } else {
> > rdev->audio.funcs = &r600_funcs;
> > rdev->audio.hdmi_funcs = &r600_hdmi_funcs;
> > - rdev->audio.dp_funcs = 0;
> > + rdev->audio.dp_funcs = NULL;
> > }
> > }
> >
>
> _______________________________________________
> amd-gfx mailing list
> [email protected]

2019-11-26 16:20:05

by Liu, Zhan

[permalink] [raw]

Subject: RE: [PATCH] drm: radeon: replace 0 with NULL

2019-11-26 16:29:34

by Alex Deucher

[permalink] [raw]

Subject: Re: [PATCH] drm: radeon: replace 0 with NULL

Applied. thanks!

Alex

On Tue, Nov 26, 2019 at 3:15 AM Jules Irenge <[email protected]> wrote:
>
> Replace 0 with NULL to fix sparse tool warning
> warning: Using plain integer as NULL pointer
>
> Signed-off-by: Jules Irenge <[email protected]>
> ---
> drivers/gpu/drm/radeon/radeon_audio.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/radeon/radeon_audio.c b/drivers/gpu/drm/radeon/radeon_audio.c
> index b9aea5776d3d..2269cfced788 100644
> --- a/drivers/gpu/drm/radeon/radeon_audio.c
> +++ b/drivers/gpu/drm/radeon/radeon_audio.c
> @@ -288,7 +288,7 @@ static void radeon_audio_interface_init(struct radeon_device *rdev)
> } else {
> rdev->audio.funcs = &r600_funcs;
> rdev->audio.hdmi_funcs = &r600_hdmi_funcs;
> - rdev->audio.dp_funcs = 0;
> + rdev->audio.dp_funcs = NULL;
> }
> }
>
> --
> 2.23.0
>
> _______________________________________________
> dri-devel mailing list
> [email protected]
> https://lists.freedesktop.org/mailman/listinfo/dri-devel