Re: [PATCH] irqchip: irq-imx-gpcv2: Remove unnecessary blank lines

From: Marc Zyngier
Date: Sat Mar 21 2020 - 11:21:10 EST


On Tue, 17 Mar 2020 04:56:41 +0000,
Anson Huang <Anson.Huang@xxxxxxx> wrote:
>
> Remove unnecessary blank lines for cleanup.
>
> Signed-off-by: Anson Huang <Anson.Huang@xxxxxxx>
> ---
> 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.