Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH v4 6/7] dt-bindings: interrupt-controller: Add
 Loongson-2K1000 LIOINTC
To:     Jiaxun Yang <jiaxun.yang@flygoat.com>,
        Huacai Chen <chenhuacai@kernel.org>,
        Thomas Bogendoerfer <tsbogend@alpha.franken.de>,
        Thomas Gleixner <tglx@linutronix.de>,
        Marc Zyngier <maz@kernel.org>
References: <20210310075639.20372-1-zhangqing@loongson.cn>
 <20210310075639.20372-7-zhangqing@loongson.cn>
 <da399bdd-9454-4d63-a549-546c049c1a29@www.fastmail.com>
Cc:     "linux-mips@vger.kernel.org" <linux-mips@vger.kernel.org>,
        linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
        Ming Wang <wangming01@loongson.cn>
From:   zhangqing <zhangqing@loongson.cn>
Message-ID: <8bd4f6c3-f2cf-4f36-879b-9755117cdac7@loongson.cn>
Date:   Thu, 11 Mar 2021 09:43:14 +0800
User-Agent: Mozilla/5.0 (X11; Linux mips64; rv:45.0) Gecko/20100101
 Thunderbird/45.4.0
MIME-Version: 1.0
In-Reply-To: <da399bdd-9454-4d63-a549-546c049c1a29@www.fastmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 03/10/2021 08:04 PM, Jiaxun Yang wrote:
>
> On Wed, Mar 10, 2021, at 3:56 PM, Qing Zhang wrote:
>> Add liointc-2.0 properties support, so update the maxItems and description.
>>
>> Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com>
> ^ I have nothing todo with this patch so please drop me for this one :-)
>
>> Signed-off-by: Qing Zhang <zhangqing@loongson.cn>
>> Tested-by: Ming Wang <wangming01@loongson.cn>
>> ---
>>
>> v3-v4: Standard submission of information
> ^ It's called commit message.
>
>>   .../bindings/interrupt-controller/loongson,liointc.yaml    | 7 ++++---
>>   1 file changed, 4 insertions(+), 3 deletions(-)
>>
>> diff --git
>> a/Documentation/devicetree/bindings/interrupt-controller/loongson,liointc.yaml b/Documentation/devicetree/bindings/interrupt-controller/loongson,liointc.yaml
>> index f38e0113f360..5280cf60a9a7 100644
>> ---
>> a/Documentation/devicetree/bindings/interrupt-controller/loongson,liointc.yaml
>> +++
>> b/Documentation/devicetree/bindings/interrupt-controller/loongson,liointc.yaml
>> @@ -10,9 +10,9 @@ maintainers:
>>     - Jiaxun Yang <jiaxun.yang@flygoat.com>
>>   
>>   description: |
>> -  This interrupt controller is found in the Loongson-3 family of chips
>> as the primary
>> -  package interrupt controller which can route local I/O interrupt to
>> interrupt lines
>> -  of cores.
>> +  This interrupt controller is found in the Loongson-3 family of chips
>> and
>> +  Loongson-2K1000 chip, as the primary package interrupt controller
>> which
>> +  can route local I/O interrupt to interrupt lines of cores.
>>   
>>   allOf:
>>     - $ref: /schemas/interrupt-controller.yaml#
>> @@ -22,6 +22,7 @@ properties:
>>       oneOf:
>>         - const: loongson,liointc-1.0
>>         - const: loongson,liointc-1.0a
>> +      - const: loongson,liointc-2.0
>>   
>>     reg:
>>       maxItems: 1
> ^ Please document multiple reg prop change as well.
   Hi, Jiaxun

   Thank you for your reply,

   I will do it and send v5.

   Thanks,
   -Qing
>
> Thanks.
>
> - Jiaxun
>
>> -- 
>> 2.20.1
>>
>>