* [PATCH] atm: ambassador: use NULL instead of 0 for pointer
@ 2014-02-19 5:11 Daeseok Youn
2014-02-19 14:52 ` chas williams - CONTRACTOR
2014-02-19 21:39 ` David Miller
0 siblings, 2 replies; 3+ messages in thread
From: Daeseok Youn @ 2014-02-19 5:11 UTC (permalink / raw)
To: chas; +Cc: linux-atm-general, netdev, linux-kernel
>From 932e928d53b1e588dc17019e7f9fa7a61b8b7468 Mon Sep 17 00:00:00 2001
From: Daeseok Youn <daeseok.youn@gmail.com>
Date: Wed, 19 Feb 2014 10:35:41 +0900
Subject: [PATCH] atm: ambassador: use NULL instead of 0 for pointer
sparse says:
drivers/atm/ambassador.c:1928:24: warning:
Using plain integer as NULL pointer
Signed-off-by: Daeseok Youn <daeseok.youn@gmail.com>
---
drivers/atm/ambassador.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/atm/ambassador.c b/drivers/atm/ambassador.c
index 62a7607..f1a9198 100644
--- a/drivers/atm/ambassador.c
+++ b/drivers/atm/ambassador.c
@@ -1925,7 +1925,7 @@ static int ucode_init(loader_block *lb, amb_dev *dev)
const struct firmware *fw;
unsigned long start_address;
const struct ihex_binrec *rec;
- const char *errmsg = 0;
+ const char *errmsg = NULL;
int res;
res = request_ihex_firmware(&fw, "atmsar11.fw", &dev->pci_dev->dev);
--
1.7.9.5
---
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] atm: ambassador: use NULL instead of 0 for pointer
2014-02-19 5:11 [PATCH] atm: ambassador: use NULL instead of 0 for pointer Daeseok Youn
@ 2014-02-19 14:52 ` chas williams - CONTRACTOR
2014-02-19 21:39 ` David Miller
1 sibling, 0 replies; 3+ messages in thread
From: chas williams - CONTRACTOR @ 2014-02-19 14:52 UTC (permalink / raw)
To: Daeseok Youn; +Cc: linux-atm-general, netdev, linux-kernel
On Wed, 19 Feb 2014 14:11:15 +0900
Daeseok Youn <daeseok.youn@gmail.com> wrote:
> >From 932e928d53b1e588dc17019e7f9fa7a61b8b7468 Mon Sep 17 00:00:00 2001
> From: Daeseok Youn <daeseok.youn@gmail.com>
> Date: Wed, 19 Feb 2014 10:35:41 +0900
> Subject: [PATCH] atm: ambassador: use NULL instead of 0 for pointer
>
> sparse says:
>
> drivers/atm/ambassador.c:1928:24: warning:
> Using plain integer as NULL pointer
>
> Signed-off-by: Daeseok Youn <daeseok.youn@gmail.com>
> ---
> drivers/atm/ambassador.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/atm/ambassador.c b/drivers/atm/ambassador.c
> index 62a7607..f1a9198 100644
> --- a/drivers/atm/ambassador.c
> +++ b/drivers/atm/ambassador.c
> @@ -1925,7 +1925,7 @@ static int ucode_init(loader_block *lb, amb_dev *dev)
> const struct firmware *fw;
> unsigned long start_address;
> const struct ihex_binrec *rec;
> - const char *errmsg = 0;
> + const char *errmsg = NULL;
> int res;
>
> res = request_ihex_firmware(&fw, "atmsar11.fw", &dev->pci_dev->dev);
Acked-by: Chas Williams <chas@cmf.nrl.navy.mil>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] atm: ambassador: use NULL instead of 0 for pointer
2014-02-19 5:11 [PATCH] atm: ambassador: use NULL instead of 0 for pointer Daeseok Youn
2014-02-19 14:52 ` chas williams - CONTRACTOR
@ 2014-02-19 21:39 ` David Miller
1 sibling, 0 replies; 3+ messages in thread
From: David Miller @ 2014-02-19 21:39 UTC (permalink / raw)
To: daeseok.youn; +Cc: chas, linux-atm-general, netdev, linux-kernel
From: Daeseok Youn <daeseok.youn@gmail.com>
Date: Wed, 19 Feb 2014 14:11:15 +0900
> From 932e928d53b1e588dc17019e7f9fa7a61b8b7468 Mon Sep 17 00:00:00 2001
> From: Daeseok Youn <daeseok.youn@gmail.com>
> Date: Wed, 19 Feb 2014 10:35:41 +0900
> Subject: [PATCH] atm: ambassador: use NULL instead of 0 for pointer
>
> sparse says:
>
> drivers/atm/ambassador.c:1928:24: warning:
> Using plain integer as NULL pointer
>
> Signed-off-by: Daeseok Youn <daeseok.youn@gmail.com>
Applied.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-02-19 21:39 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-02-19 5:11 [PATCH] atm: ambassador: use NULL instead of 0 for pointer Daeseok Youn
2014-02-19 14:52 ` chas williams - CONTRACTOR
2014-02-19 21:39 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).