drivers/video/fbdev:modify 0 to NULL

From: tangchunyou <redacted>

modify 0 to NULL,info is a pointer,it should be

compared with NULL,not 0

Signed-off-by: tangchunyou <redacted>
---
 drivers/video/fbdev/offb.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/video/fbdev/offb.c b/drivers/video/fbdev/offb.c
index 4501e84..cd1042f 100644
--- a/drivers/video/fbdev/offb.c
+++ b/drivers/video/fbdev/offb.c

@@ -412,7 +412,7 @@ static void __init offb_init_fb(const char *name,
 
 	info = framebuffer_alloc(sizeof(u32) * 16, NULL);
 
-	if (info == 0) {
+	if (info == NULL) {
 		release_mem_region(res_start, res_size);
 		return;
 	}

-- 
1.9.1

On 3/17/21 21:33, ChunyouTang wrote:

if (!info) is better.

--
Gustavo

 		release_mem_region(res_start, res_size);
 		return;
 	}

HI,Gustavo

On Wed, 17 Mar 2021 20:41:15 -0500
"Gustavo A. R. Silva" [off-list ref] wrote:

On 3/17/21 21:33, ChunyouTang wrote:

quoted

From: tangchunyou <redacted>

modify 0 to NULL,info is a pointer,it should be

compared with NULL,not 0

Signed-off-by: tangchunyou <redacted>
---
 drivers/video/fbdev/offb.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/video/fbdev/offb.c b/drivers/video/fbdev/offb.c
index 4501e84..cd1042f 100644
--- a/drivers/video/fbdev/offb.c
+++ b/drivers/video/fbdev/offb.c

@@ -412,7 +412,7 @@ static void __init offb_init_fb(const char

*name, 
 	info = framebuffer_alloc(sizeof(u32) * 16, NULL);
 
-	if (info == 0) {
+	if (info == NULL) {  

if (!info) is better.

--
Gustavo

quoted

 		release_mem_region(res_start, res_size);
 		return;
 	}
  

I think "if (info == NULL)" is more intuitive,and there have many
compare likes "if (info == NULL)" in this file.

--
ChunyouTang

Hi,Gustavo

On Wed, 17 Mar 2021 20:54:41 -0500
"Gustavo A. R. Silva" [off-list ref] wrote:

On 3/17/21 21:47, Chunyou Tang wrote:

quoted

I think "if (info == NULL)" is more intuitive,and there have many
compare likes "if (info == NULL)" in this file.  

In that case, all those instances should be changed to if (!foo),
instead.

--
Gustavo

OK,I change it.

--
ChunyouTang

On 3/17/21 21:47, Chunyou Tang wrote:

I think "if (info == NULL)" is more intuitive,and there have many
compare likes "if (info == NULL)" in this file.

In that case, all those instances should be changed to if (!foo), instead.

--
Gustavo