Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1215766 > unrolled thread
| Started by | Anders Fridlund <anders.fridlund@gmail.com> |
|---|---|
| First post | 2015-08-29 14:00 +0200 |
| Last post | 2015-09-05 22:10 +0200 |
| Articles | 4 — 2 participants |
Back to article view | Back to linux.kernel
[PATCH] staging: most: Use NULL instead of 0 in assignment of pointer Anders Fridlund <anders.fridlund@gmail.com> - 2015-08-29 14:00 +0200
Re: [PATCH] staging: most: Use NULL instead of 0 in assignment of pointer Greg KH <gregkh@linuxfoundation.org> - 2015-09-03 18:50 +0200
Re: [PATCH] staging: most: Use NULL instead of 0 in assignment of pointer Anders Fridlund <anders.fridlund@gmail.com> - 2015-09-05 21:30 +0200
Re: [PATCH] staging: most: Use NULL instead of 0 in assignment of pointer Greg KH <gregkh@linuxfoundation.org> - 2015-09-05 22:10 +0200
| From | Anders Fridlund <anders.fridlund@gmail.com> |
|---|---|
| Date | 2015-08-29 14:00 +0200 |
| Subject | [PATCH] staging: most: Use NULL instead of 0 in assignment of pointer |
| Message-ID | <q2MXE-Td-1@gated-at.bofh.it> |
Fix sparse warning 'Using plain integer as NULL pointer' by replacing 0 with NULL in the assignment. Signed-off-by: Anders Fridlund <anders.fridlund@gmail.com> --- drivers/staging/most/aim-network/networking.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/most/aim-network/networking.c b/drivers/staging/most/aim-network/networking.c index c8ab239..a1e1bce 100644 --- a/drivers/staging/most/aim-network/networking.c +++ b/drivers/staging/most/aim-network/networking.c @@ -295,7 +295,7 @@ static void most_net_rm_netdev_safe(struct net_dev_context *nd) unregister_netdev(nd->dev); free_netdev(nd->dev); - nd->dev = 0; + nd->dev = NULL; } static struct net_dev_context *get_net_dev_context( -- 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [next] | [standalone]
| From | Greg KH <gregkh@linuxfoundation.org> |
|---|---|
| Date | 2015-09-03 18:50 +0200 |
| Subject | Re: [PATCH] staging: most: Use NULL instead of 0 in assignment of pointer |
| Message-ID | <q4FS4-M8-35@gated-at.bofh.it> |
| In reply to | #1215766 |
On Sat, Aug 29, 2015 at 01:53:49PM +0200, Anders Fridlund wrote: > Fix sparse warning 'Using plain integer as NULL pointer' by replacing 0 > with NULL in the assignment. > > Signed-off-by: Anders Fridlund <anders.fridlund@gmail.com> > --- > drivers/staging/most/aim-network/networking.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/staging/most/aim-network/networking.c b/drivers/staging/most/aim-network/networking.c > index c8ab239..a1e1bce 100644 > --- a/drivers/staging/most/aim-network/networking.c > +++ b/drivers/staging/most/aim-network/networking.c > @@ -295,7 +295,7 @@ static void most_net_rm_netdev_safe(struct net_dev_context *nd) > > unregister_netdev(nd->dev); > free_netdev(nd->dev); > - nd->dev = 0; > + nd->dev = NULL; > } > > static struct net_dev_context *get_net_dev_context( > -- > 1.9.1 Why did you resend this patch again? -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Anders Fridlund <anders.fridlund@gmail.com> |
|---|---|
| Date | 2015-09-05 21:30 +0200 |
| Message-ID | <q5rjY-1gU-17@gated-at.bofh.it> |
| In reply to | #1218442 |
I resent it since no one responded to it for 10 days. From what I came to understand that is the common practice. If that is not the case I'm sorry (and please let me know so I don't do it again). Cheers, Anders Fridlund On Thu, Sep 3, 2015 at 6:41 PM, Greg KH <gregkh@linuxfoundation.org> wrote: > On Sat, Aug 29, 2015 at 01:53:49PM +0200, Anders Fridlund wrote: >> Fix sparse warning 'Using plain integer as NULL pointer' by replacing 0 >> with NULL in the assignment. >> >> Signed-off-by: Anders Fridlund <anders.fridlund@gmail.com> >> --- >> drivers/staging/most/aim-network/networking.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/staging/most/aim-network/networking.c b/drivers/staging/most/aim-network/networking.c >> index c8ab239..a1e1bce 100644 >> --- a/drivers/staging/most/aim-network/networking.c >> +++ b/drivers/staging/most/aim-network/networking.c >> @@ -295,7 +295,7 @@ static void most_net_rm_netdev_safe(struct net_dev_context *nd) >> >> unregister_netdev(nd->dev); >> free_netdev(nd->dev); >> - nd->dev = 0; >> + nd->dev = NULL; >> } >> >> static struct net_dev_context *get_net_dev_context( >> -- >> 1.9.1 > > Why did you resend this patch again? -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Greg KH <gregkh@linuxfoundation.org> |
|---|---|
| Date | 2015-09-05 22:10 +0200 |
| Subject | Re: [PATCH] staging: most: Use NULL instead of 0 in assignment of pointer |
| Message-ID | <q5rWF-2ff-3@gated-at.bofh.it> |
| In reply to | #1219613 |
On Sat, Sep 05, 2015 at 09:21:58PM +0200, Anders Fridlund wrote: > I resent it since no one responded to it for 10 days. From what I came > to understand that is the common practice. If that is not the case I'm > sorry (and please let me know so I don't do it again). That's fine to do, but at least say why you are resending it in the area under the --- line, otherwise we have no idea what is going on. thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web