Re: [v2 01/11] irqchip: ls-extirq: Add LS1043A, LS1088A external interrupt

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2020-10-27 10:35, Biwen Li (OSS) wrote:

On 2020-10-27 04:46, Biwen Li wrote:
> From: Hou Zhiqiang <Zhiqiang.Hou@xxxxxxx>
>
> Add an new IRQ chip declaration for LS1043A and LS1088A
> - compatible "fsl,ls1043a-extirq" for LS1043A, LS1046A.
> SCFG_INTPCR[31:0]
>   of these SoCs is stored/read as SCFG_INTPCR[0:31] defaultly(bit
>   reverse)
> - compatible "fsl,ls1088a-extirq" for LS1088A, LS208xA, LX216xA
>
> Signed-off-by: Hou Zhiqiang <Zhiqiang.Hou@xxxxxxx>
> Signed-off-by: Biwen Li <biwen.li@xxxxxxx>

You clearly couldn't be bothered to read what I wrote in my earlier replies. I'm
thus ignoring this series...
Okay, got it.

> ---
> Change in v2:
> 	- add despcription of bit reverse
> 	- update copyright
>
>  drivers/irqchip/irq-ls-extirq.c | 10 +++++++++-
>  1 file changed, 9 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/irqchip/irq-ls-extirq.c
> b/drivers/irqchip/irq-ls-extirq.c index 4d1179fed77c..9587bc2607fc
> 100644
> --- a/drivers/irqchip/irq-ls-extirq.c
> +++ b/drivers/irqchip/irq-ls-extirq.c
> @@ -1,5 +1,8 @@
>  // SPDX-License-Identifier: GPL-2.0
> -
> +/*
> + * Author: Rasmus Villemoes <linux@xxxxxxxxxxxxxxxxxx>
> + * Copyright 2020 NXP

... specially when you keep attributing someone else's copyright to NXP.
Then I don't know how to add the copyright, any suggestions?

Simple. You don't add anything. NXP's copyright doesn't apply to this
file before this patch, and your changes are so trivial that they don't
really warrant a mention. Furthermore, the git history already keeps track
of who did what.

        M.
--
Jazz is not dead. It just smells funny...



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux