Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1492650 > unrolled thread
| Started by | shyam saini <mayhs11saini@gmail.com> |
|---|---|
| First post | 2016-09-28 17:30 +0200 |
| Last post | 2016-10-02 18:20 +0200 |
| Articles | 6 — 3 participants |
Back to article view | Back to linux.kernel
[PATCH 1/1] Staging: android: ion: Remove explicit NULL comparison using Coccinelle shyam saini <mayhs11saini@gmail.com> - 2016-09-28 17:30 +0200
Re: [PATCH 1/1] Staging: android: ion: Remove explicit NULL comparison using Coccinelle Greg KH <gregkh@linuxfoundation.org> - 2016-10-02 17:00 +0200
Re: [PATCH 1/1] Staging: android: ion: Remove explicit NULL comparison using Coccinelle Shyam Saini <mayhs11saini@gmail.com> - 2016-10-02 17:30 +0200
Re: [PATCH 1/1] Staging: android: ion: Remove explicit NULL comparison using Coccinelle Greg KH <gregkh@linuxfoundation.org> - 2016-10-02 17:40 +0200
Re: [PATCH 1/1] Staging: android: ion: Remove explicit NULL comparison using Coccinelle Shyam Saini <mayhs11saini@gmail.com> - 2016-10-02 18:10 +0200
Re: [PATCH 1/1] Staging: android: ion: Remove explicit NULL comparison using Coccinelle Greg KH <gregkh@linuxfoundation.org> - 2016-10-02 18:20 +0200
| From | shyam saini <mayhs11saini@gmail.com> |
|---|---|
| Date | 2016-09-28 17:30 +0200 |
| Subject | [PATCH 1/1] Staging: android: ion: Remove explicit NULL comparison using Coccinelle |
| Message-ID | <smoY1-8rL-13@gated-at.bofh.it> |
Remove the explicit NULL comparison and rewrite in a compact form. Signed-off-by: shyam saini <mayhs11saini@gmail.com> --- drivers/staging/android/ion/ion_of.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/android/ion/ion_of.c b/drivers/staging/android/ion/ion_of.c index 01f3067..485fb0f 100644 --- a/drivers/staging/android/ion/ion_of.c +++ b/drivers/staging/android/ion/ion_of.c @@ -36,7 +36,7 @@ int ion_parse_dt_heap_common(struct device_node *heap_node, break; } - if (compatible[i].name == NULL) + if (!compatible[i].name) return -ENODEV; heap->id = compatible[i].heap_id; -- 2.7.4
[toc] | [next] | [standalone]
| From | Greg KH <gregkh@linuxfoundation.org> |
|---|---|
| Date | 2016-10-02 17:00 +0200 |
| Subject | Re: [PATCH 1/1] Staging: android: ion: Remove explicit NULL comparison using Coccinelle |
| Message-ID | <snQpb-8c3-9@gated-at.bofh.it> |
| In reply to | #1492650 |
On Wed, Sep 28, 2016 at 08:58:51PM +0530, shyam saini wrote: > Remove the explicit NULL comparison and rewrite in a compact form. > > Signed-off-by: shyam saini <mayhs11saini@gmail.com> > --- > drivers/staging/android/ion/ion_of.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Does not apply to my tree :(
[toc] | [prev] | [next] | [standalone]
| From | Shyam Saini <mayhs11saini@gmail.com> |
|---|---|
| Date | 2016-10-02 17:30 +0200 |
| Subject | Re: [PATCH 1/1] Staging: android: ion: Remove explicit NULL comparison using Coccinelle |
| Message-ID | <snQSe-c2-7@gated-at.bofh.it> |
| In reply to | #1494582 |
On Sun, 2016-10-02 at 16:58 +0200, Greg KH wrote: > On Wed, Sep 28, 2016 at 08:58:51PM +0530, shyam saini wrote: > > > > Remove the explicit NULL comparison and rewrite in a compact form. > > > > Signed-off-by: shyam saini <mayhs11saini@gmail.com> > > --- > > drivers/staging/android/ion/ion_of.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > Does not apply to my tree :( Do I need to rebase it? Because I have already included all the maintainers in the patch which I got from scripts/get_maintainer.pl. Thanks, Shyam
[toc] | [prev] | [next] | [standalone]
| From | Greg KH <gregkh@linuxfoundation.org> |
|---|---|
| Date | 2016-10-02 17:40 +0200 |
| Subject | Re: [PATCH 1/1] Staging: android: ion: Remove explicit NULL comparison using Coccinelle |
| Message-ID | <snR1T-fs-1@gated-at.bofh.it> |
| In reply to | #1494586 |
On Sun, Oct 02, 2016 at 08:51:29PM +0530, Shyam Saini wrote: > On Sun, 2016-10-02 at 16:58 +0200, Greg KH wrote: > > On Wed, Sep 28, 2016 at 08:58:51PM +0530, shyam saini wrote: > > > > > > Remove the explicit NULL comparison and rewrite in a compact form. > > > > > > Signed-off-by: shyam saini <mayhs11saini@gmail.com> > > > --- > > > drivers/staging/android/ion/ion_of.c | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > Does not apply to my tree :( > > > Do I need to rebase it? Because I have already included all the > maintainers in the patch which I got from scripts/get_maintainer.pl. If you want it applied, yes, you need to rebase it. Listing the maintainers doesn't help the patch to somehow magically be able to be applied to the tree :) Please work against my staging-testing branch of the staging.git tree, as it has a lot of patches I just accepted in it that are not yet in linux-next. thanks, greg k-h
[toc] | [prev] | [next] | [standalone]
| From | Shyam Saini <mayhs11saini@gmail.com> |
|---|---|
| Date | 2016-10-02 18:10 +0200 |
| Subject | Re: [PATCH 1/1] Staging: android: ion: Remove explicit NULL comparison using Coccinelle |
| Message-ID | <snRuV-GE-1@gated-at.bofh.it> |
| In reply to | #1494588 |
On Sun, 2016-10-02 at 17:37 +0200, Greg KH wrote: > On Sun, Oct 02, 2016 at 08:51:29PM +0530, Shyam Saini wrote: > > > > On Sun, 2016-10-02 at 16:58 +0200, Greg KH wrote: > > > > > > On Wed, Sep 28, 2016 at 08:58:51PM +0530, shyam saini wrote: > > > > > > > > > > > > Remove the explicit NULL comparison and rewrite in a compact > > > > form. > > > > > > > > Signed-off-by: shyam saini <mayhs11saini@gmail.com> > > > > --- > > > > drivers/staging/android/ion/ion_of.c | 2 +- > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > Does not apply to my tree :( > > > > Do I need to rebase it? Because I have already included all the > > maintainers in the patch which I got from > > scripts/get_maintainer.pl. > If you want it applied, yes, you need to rebase it. Listing the > maintainers doesn't help the patch to somehow magically be able to be > applied to the tree :) > > Please work against my staging-testing branch of the staging.git > tree, > as it has a lot of patches I just accepted in it that are not yet in > linux-next. > > thanks, > > greg k-h From now onward I will be working on your staging-testing tree. But then why some of my patches which were based on linux-next were accepted? for example today You've accepted my patch "Staging: vt6655: Remove explicit NULL comparison using Coccinelle" with commit id 4d932504b534584e116b400dd6e7b8ce7deb455c, which you can check on git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git This patch was based on linux-next tree. Please clear my confusion. Thanks, Shyam
[toc] | [prev] | [next] | [standalone]
| From | Greg KH <gregkh@linuxfoundation.org> |
|---|---|
| Date | 2016-10-02 18:20 +0200 |
| Subject | Re: [PATCH 1/1] Staging: android: ion: Remove explicit NULL comparison using Coccinelle |
| Message-ID | <snREB-Nm-1@gated-at.bofh.it> |
| In reply to | #1494591 |
On Sun, Oct 02, 2016 at 09:37:17PM +0530, Shyam Saini wrote: > On Sun, 2016-10-02 at 17:37 +0200, Greg KH wrote: > > On Sun, Oct 02, 2016 at 08:51:29PM +0530, Shyam Saini wrote: > > > > > > On Sun, 2016-10-02 at 16:58 +0200, Greg KH wrote: > > > > > > > > On Wed, Sep 28, 2016 at 08:58:51PM +0530, shyam saini wrote: > > > > > > > > > > > > > > > Remove the explicit NULL comparison and rewrite in a compact > > > > > form. > > > > > > > > > > Signed-off-by: shyam saini <mayhs11saini@gmail.com> > > > > > --- > > > > > drivers/staging/android/ion/ion_of.c | 2 +- > > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > Does not apply to my tree :( > > > > > > Do I need to rebase it? Because I have already included all the > > > maintainers in the patch which I got from > > > scripts/get_maintainer.pl. > > If you want it applied, yes, you need to rebase it. Listing the > > maintainers doesn't help the patch to somehow magically be able to be > > applied to the tree :) > > > > Please work against my staging-testing branch of the staging.git > > tree, > > as it has a lot of patches I just accepted in it that are not yet in > > linux-next. > > > > thanks, > > > > greg k-h > > > >From now onward I will be working on your staging-testing tree. > > But then why some of my patches which were based on linux-next were > accepted? > > for example today You've accepted > my patch "Staging: vt6655: Remove explicit NULL comparison using > Coccinelle" > > with commit id 4d932504b534584e116b400dd6e7b8ce7deb455c, > > which you can check on > git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git > > This patch was based on linux-next tree. Because maybe others have sent patches in for this specific area of the kernel, or there are changes made by other developer's trees (like the iio maintainer) for this driver. linux-next merges all of our trees together, and usually is the correct thing to use. But sometimes it doesn't work out due to the large number of patches I get for the staging tree (about 20-50 a day, sometimes more.) thanks, greg k-h
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web