* [PATCH] irqchip: irq-imx-gpcv2: Remove unnecessary blank lines @ 2020-03-17 4:56 Anson Huang 2020-03-21 15:21 ` Marc Zyngier 0 siblings, 1 reply; 4+ messages in thread From: Anson Huang @ 2020-03-17 4:56 UTC (permalink / raw) To: tglx, jason, maz, shawnguo, s.hauer, kernel, festevam, linux-kernel, linux-arm-kernel Cc: Linux-imx Remove unnecessary blank lines for cleanup. Signed-off-by: Anson Huang <Anson.Huang@nxp.com> --- drivers/irqchip/irq-imx-gpcv2.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/irqchip/irq-imx-gpcv2.c b/drivers/irqchip/irq-imx-gpcv2.c index 4f74c15..4f11b9b 100644 --- a/drivers/irqchip/irq-imx-gpcv2.c +++ b/drivers/irqchip/irq-imx-gpcv2.c @@ -17,7 +17,6 @@ #define GPC_IMR1_CORE2 0x1c0 #define GPC_IMR1_CORE3 0x1d0 - struct gpcv2_irqchip_data { struct raw_spinlock rlock; void __iomem *gpc_base; @@ -287,6 +286,5 @@ static int __init imx_gpcv2_irqchip_init(struct device_node *node, of_node_clear_flag(node, OF_POPULATED); return 0; } - IRQCHIP_DECLARE(imx_gpcv2_imx7d, "fsl,imx7d-gpc", imx_gpcv2_irqchip_init); IRQCHIP_DECLARE(imx_gpcv2_imx8mq, "fsl,imx8mq-gpc", imx_gpcv2_irqchip_init); -- 2.7.4 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] irqchip: irq-imx-gpcv2: Remove unnecessary blank lines 2020-03-17 4:56 [PATCH] irqchip: irq-imx-gpcv2: Remove unnecessary blank lines Anson Huang @ 2020-03-21 15:21 ` Marc Zyngier 2020-03-22 19:08 ` Daniel Baluta 0 siblings, 1 reply; 4+ messages in thread From: Marc Zyngier @ 2020-03-21 15:21 UTC (permalink / raw) To: Anson Huang Cc: jason, shawnguo, s.hauer, linux-kernel, Linux-imx, kernel, tglx, festevam, linux-arm-kernel On Tue, 17 Mar 2020 04:56:41 +0000, Anson Huang <Anson.Huang@nxp.com> wrote: > > Remove unnecessary blank lines for cleanup. > > Signed-off-by: Anson Huang <Anson.Huang@nxp.com> > --- > drivers/irqchip/irq-imx-gpcv2.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/irqchip/irq-imx-gpcv2.c b/drivers/irqchip/irq-imx-gpcv2.c > index 4f74c15..4f11b9b 100644 > --- a/drivers/irqchip/irq-imx-gpcv2.c > +++ b/drivers/irqchip/irq-imx-gpcv2.c > @@ -17,7 +17,6 @@ > #define GPC_IMR1_CORE2 0x1c0 > #define GPC_IMR1_CORE3 0x1d0 > > - > struct gpcv2_irqchip_data { > struct raw_spinlock rlock; > void __iomem *gpc_base; > @@ -287,6 +286,5 @@ static int __init imx_gpcv2_irqchip_init(struct device_node *node, > of_node_clear_flag(node, OF_POPULATED); > return 0; > } > - > IRQCHIP_DECLARE(imx_gpcv2_imx7d, "fsl,imx7d-gpc", imx_gpcv2_irqchip_init); > IRQCHIP_DECLARE(imx_gpcv2_imx8mq, "fsl,imx8mq-gpc", imx_gpcv2_irqchip_init); I honestly don't think this deserves a patch. Next time you work on this driver, add the cleanups to it. But on its own, it's only churn. Thanks, M. -- Jazz is not dead, it just smells funny. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] irqchip: irq-imx-gpcv2: Remove unnecessary blank lines 2020-03-21 15:21 ` Marc Zyngier @ 2020-03-22 19:08 ` Daniel Baluta 2020-03-22 19:35 ` Marc Zyngier 0 siblings, 1 reply; 4+ messages in thread From: Daniel Baluta @ 2020-03-22 19:08 UTC (permalink / raw) To: Marc Zyngier Cc: jason, Anson Huang, Fabio Estevam, Sascha Hauer, Linux Kernel Mailing List, dl-linux-imx, Pengutronix Kernel Team, tglx, Shawn Guo, linux-arm-kernel On Sat, Mar 21, 2020 at 5:22 PM Marc Zyngier <maz@kernel.org> wrote: > > On Tue, 17 Mar 2020 04:56:41 +0000, > Anson Huang <Anson.Huang@nxp.com> wrote: > > > > Remove unnecessary blank lines for cleanup. > > > > Signed-off-by: Anson Huang <Anson.Huang@nxp.com> > > --- > > drivers/irqchip/irq-imx-gpcv2.c | 2 -- > > 1 file changed, 2 deletions(-) > > > > diff --git a/drivers/irqchip/irq-imx-gpcv2.c b/drivers/irqchip/irq-imx-gpcv2.c > > index 4f74c15..4f11b9b 100644 > > --- a/drivers/irqchip/irq-imx-gpcv2.c > > +++ b/drivers/irqchip/irq-imx-gpcv2.c > > @@ -17,7 +17,6 @@ > > #define GPC_IMR1_CORE2 0x1c0 > > #define GPC_IMR1_CORE3 0x1d0 > > > > - > > struct gpcv2_irqchip_data { > > struct raw_spinlock rlock; > > void __iomem *gpc_base; > > @@ -287,6 +286,5 @@ static int __init imx_gpcv2_irqchip_init(struct device_node *node, > > of_node_clear_flag(node, OF_POPULATED); > > return 0; > > } > > - > > IRQCHIP_DECLARE(imx_gpcv2_imx7d, "fsl,imx7d-gpc", imx_gpcv2_irqchip_init); > > IRQCHIP_DECLARE(imx_gpcv2_imx8mq, "fsl,imx8mq-gpc", imx_gpcv2_irqchip_init); > > I honestly don't think this deserves a patch. Next time you work on > this driver, add the cleanups to it. But on its own, it's only churn. While you are technically right, it's only churn I think we need this for code consistency and cleanup. Even if we fix this when we work on the driver we still need to make the cleanup in a separate patch. My 2 cents, Daniel. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] irqchip: irq-imx-gpcv2: Remove unnecessary blank lines 2020-03-22 19:08 ` Daniel Baluta @ 2020-03-22 19:35 ` Marc Zyngier 0 siblings, 0 replies; 4+ messages in thread From: Marc Zyngier @ 2020-03-22 19:35 UTC (permalink / raw) To: Daniel Baluta Cc: jason, Anson Huang, Fabio Estevam, Sascha Hauer, Linux Kernel Mailing List, dl-linux-imx, Pengutronix Kernel Team, tglx, Shawn Guo, linux-arm-kernel Daniel, On 2020-03-22 19:08, Daniel Baluta wrote: > On Sat, Mar 21, 2020 at 5:22 PM Marc Zyngier <maz@kernel.org> wrote: >> >> On Tue, 17 Mar 2020 04:56:41 +0000, >> Anson Huang <Anson.Huang@nxp.com> wrote: >> > >> > Remove unnecessary blank lines for cleanup. >> > >> > Signed-off-by: Anson Huang <Anson.Huang@nxp.com> >> > --- >> > drivers/irqchip/irq-imx-gpcv2.c | 2 -- >> > 1 file changed, 2 deletions(-) >> > >> > diff --git a/drivers/irqchip/irq-imx-gpcv2.c b/drivers/irqchip/irq-imx-gpcv2.c >> > index 4f74c15..4f11b9b 100644 >> > --- a/drivers/irqchip/irq-imx-gpcv2.c >> > +++ b/drivers/irqchip/irq-imx-gpcv2.c >> > @@ -17,7 +17,6 @@ >> > #define GPC_IMR1_CORE2 0x1c0 >> > #define GPC_IMR1_CORE3 0x1d0 >> > >> > - >> > struct gpcv2_irqchip_data { >> > struct raw_spinlock rlock; >> > void __iomem *gpc_base; >> > @@ -287,6 +286,5 @@ static int __init imx_gpcv2_irqchip_init(struct device_node *node, >> > of_node_clear_flag(node, OF_POPULATED); >> > return 0; >> > } >> > - >> > IRQCHIP_DECLARE(imx_gpcv2_imx7d, "fsl,imx7d-gpc", imx_gpcv2_irqchip_init); >> > IRQCHIP_DECLARE(imx_gpcv2_imx8mq, "fsl,imx8mq-gpc", imx_gpcv2_irqchip_init); >> >> I honestly don't think this deserves a patch. Next time you work on >> this driver, add the cleanups to it. But on its own, it's only churn. > > While you are technically right, it's only churn I think we need this > for code consistency and cleanup. Get real. We really don't. Two blank lines do not lead to a misinterpretation of the code, do not get in the way of normal maintenance, do not lead to *any* practical issue. What's next? Cc stable? > Even if we fix this when we work on the driver we still need > to make the cleanup in a separate patch. Neither. As well as removing blank lines, you could also remove the dead code in this driver. That would be a good cleanup. You could also have a look at what feels like a potential deadlock in the mask/unmask callbacks. That'd be a good thing to do too. Certainly more useful than just dropping two blank lines. M. -- Jazz is not dead. It just smells funny... _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-03-22 19:36 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2020-03-17 4:56 [PATCH] irqchip: irq-imx-gpcv2: Remove unnecessary blank lines Anson Huang 2020-03-21 15:21 ` Marc Zyngier 2020-03-22 19:08 ` Daniel Baluta 2020-03-22 19:35 ` Marc Zyngier
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).