public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] staging: fsl-mc: fix coding style warning
@ 2017-02-16 18:42 Lucian Zala
  2017-02-24 16:33 ` Greg KH
  0 siblings, 1 reply; 4+ messages in thread
From: Lucian Zala @ 2017-02-16 18:42 UTC (permalink / raw)
  To: stuyoder, gregkh, laurentiu.tudor
  Cc: me, nipun.gupta, bharat.bhushan, bhaktipriya96, bogdan.purcareata,
	linux-kernel, devel, Lucian Zala

Fix for "WARNING: Avoid multiple line dereference - prefer
'mc_msi_domain->host_data'" found by checkpatch.pl in
bus/irq-gic-v3-its-fsl-mc-msi.c.

Signed-off-by: Lucian Zala <zala.lucian@gmail.com>
---
 drivers/staging/fsl-mc/bus/irq-gic-v3-its-fsl-mc-msi.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/fsl-mc/bus/irq-gic-v3-its-fsl-mc-msi.c b/drivers/staging/fsl-mc/bus/irq-gic-v3-its-fsl-mc-msi.c
index 87e4471..49127ac 100644
--- a/drivers/staging/fsl-mc/bus/irq-gic-v3-its-fsl-mc-msi.c
+++ b/drivers/staging/fsl-mc/bus/irq-gic-v3-its-fsl-mc-msi.c
@@ -95,8 +95,8 @@ int __init its_fsl_mc_msi_init(void)
 			continue;
 		}
 
-		WARN_ON(mc_msi_domain->
-				host_data != &its_fsl_mc_msi_domain_info);
+		WARN_ON(mc_msi_domain->host_data !=
+			&its_fsl_mc_msi_domain_info);
 
 		pr_info("fsl-mc MSI: %s domain created\n", np->full_name);
 	}
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] staging: fsl-mc: fix coding style warning
  2017-02-16 18:42 [PATCH] staging: fsl-mc: fix coding style warning Lucian Zala
@ 2017-02-24 16:33 ` Greg KH
  2017-02-24 18:24   ` Lucian Zala
  0 siblings, 1 reply; 4+ messages in thread
From: Greg KH @ 2017-02-24 16:33 UTC (permalink / raw)
  To: Lucian Zala
  Cc: stuyoder, laurentiu.tudor, devel, me, bhaktipriya96,
	bogdan.purcareata, linux-kernel, nipun.gupta, bharat.bhushan

On Thu, Feb 16, 2017 at 08:42:47PM +0200, Lucian Zala wrote:
> Fix for "WARNING: Avoid multiple line dereference - prefer
> 'mc_msi_domain->host_data'" found by checkpatch.pl in
> bus/irq-gic-v3-its-fsl-mc-msi.c.
> 
> Signed-off-by: Lucian Zala <zala.lucian@gmail.com>
> ---
>  drivers/staging/fsl-mc/bus/irq-gic-v3-its-fsl-mc-msi.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

This patch does not apply to my tree at all, sorry :(

greg k-h

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] staging: fsl-mc: fix coding style warning
  2017-02-24 16:33 ` Greg KH
@ 2017-02-24 18:24   ` Lucian Zala
  2017-02-25  7:53     ` Greg KH
  0 siblings, 1 reply; 4+ messages in thread
From: Lucian Zala @ 2017-02-24 18:24 UTC (permalink / raw)
  To: Greg KH
  Cc: stuyoder, laurentiu.tudor, devel, me, bhaktipriya96,
	bogdan.purcareata, linux-kernel, nipun.gupta, bharat.bhushan

Hi Greg,

I am a little bit confused regarding your reply, it seems
that the patch applied afterall, or?

http://git.kernel.org/cgit/linux/kernel/git/gregkh/staging.git/commit/drivers/staging/fsl-mc?h=staging-testing&id=3bee20fffff39e4df8d65e8d34ab9b3ab92e5acc

Have I messed something in it?

BR,
Lucian
On Fri, Feb 24, 2017 at 05:33:55PM +0100, Greg KH wrote:
> On Thu, Feb 16, 2017 at 08:42:47PM +0200, Lucian Zala wrote:
> > Fix for "WARNING: Avoid multiple line dereference - prefer
> > 'mc_msi_domain->host_data'" found by checkpatch.pl in
> > bus/irq-gic-v3-its-fsl-mc-msi.c.
> > 
> > Signed-off-by: Lucian Zala <zala.lucian@gmail.com>
> > ---
> >  drivers/staging/fsl-mc/bus/irq-gic-v3-its-fsl-mc-msi.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> This patch does not apply to my tree at all, sorry :(
> 
> greg k-h

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] staging: fsl-mc: fix coding style warning
  2017-02-24 18:24   ` Lucian Zala
@ 2017-02-25  7:53     ` Greg KH
  0 siblings, 0 replies; 4+ messages in thread
From: Greg KH @ 2017-02-25  7:53 UTC (permalink / raw)
  To: Lucian Zala
  Cc: stuyoder, laurentiu.tudor, devel, me, bhaktipriya96,
	bogdan.purcareata, linux-kernel, nipun.gupta, bharat.bhushan

On Fri, Feb 24, 2017 at 08:24:42PM +0200, Lucian Zala wrote:
> Hi Greg,
> 
> I am a little bit confused regarding your reply, it seems
> that the patch applied afterall, or?
> 
> http://git.kernel.org/cgit/linux/kernel/git/gregkh/staging.git/commit/drivers/staging/fsl-mc?h=staging-testing&id=3bee20fffff39e4df8d65e8d34ab9b3ab92e5acc
> 
> Have I messed something in it?

No, my mistake, I must have already applied it, sorry, dealing with
hundreds of patches causes confusion at times :)

sorry for the noise,

greg k-h

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2017-02-25  7:55 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-02-16 18:42 [PATCH] staging: fsl-mc: fix coding style warning Lucian Zala
2017-02-24 16:33 ` Greg KH
2017-02-24 18:24   ` Lucian Zala
2017-02-25  7:53     ` Greg KH

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox