linux-kernel - Re: [PATCH] arm64: boot: dts: ti: k3-am62a-main: Fix GPIO pin count in DT nodes.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Thu, 26 Oct 2023 12:17:17 +0530
From:   Nitin Yadav <n-yadav@...com>
To:     Nishanth Menon <nm@...com>
CC:     <vigneshr@...com>, <kristo@...nel.org>, <robh+dt@...nel.org>,
        <krzysztof.kozlowski+dt@...aro.org>, <conor+dt@...nel.org>,
        <linux-arm-kernel@...ts.infradead.org>,
        <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] arm64: boot: dts: ti: k3-am62a-main: Fix GPIO pin count
 in DT nodes.



On 25/10/23 21:27, Nishanth Menon wrote:
> On 16:32-20231025, Nitin Yadav wrote:
>> Fix number of gpio pins in main_gpio0 & main_gpio1 DT nodes according
>> to AM62a SK datasheet. The Link of datasheet is in the following line:
> 
> SK? line?
> 
> Please rephrase above and just mention the section of the data sheet to
> refer to.
> 
>>
>> https://www.ti.com/lit/gpn/am62a3
>>
>> Section: 6.3.10 GPIO (Page No. 52-55)
> 
>>
>> Fixes: '5fc6b1b62639c ("arm64: dts: ti: Introduce AM62A7 family of SoCs")'
> 
> What is the single quote for?
> Also note the additional comment in the list to drop the extra EoL.
> 
> 
> Did you check the MCU and WKUP GPIO count as well? if there are bugs
> around it, fix it in a single commit please.
yes, checked they are good.
> 
>>
>> Signed-off-by: Nitin Yadav <n-yadav@...com>
>> ---
>>  arch/arm64/boot/dts/ti/k3-am62a-main.dtsi | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/arm64/boot/dts/ti/k3-am62a-main.dtsi b/arch/arm64/boot/dts/ti/k3-am62a-main.dtsi
>> index 3198af08fb9f..de36abb243f1 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am62a-main.dtsi
>> +++ b/arch/arm64/boot/dts/ti/k3-am62a-main.dtsi
>> @@ -462,7 +462,7 @@ main_gpio0: gpio@...000 {
>>  			     <193>, <194>, <195>;
>>  		interrupt-controller;
>>  		#interrupt-cells = <2>;
>> -		ti,ngpio = <87>;
>> +		ti,ngpio = <92>;
>>  		ti,davinci-gpio-unbanked = <0>;
>>  		power-domains = <&k3_pds 77 TI_SCI_PD_EXCLUSIVE>;
>>  		clocks = <&k3_clks 77 0>;
>> @@ -480,7 +480,7 @@ main_gpio1: gpio@...000 {
>>  			     <183>, <184>, <185>;
>>  		interrupt-controller;
>>  		#interrupt-cells = <2>;
>> -		ti,ngpio = <88>;
>> +		ti,ngpio = <52>;
>>  		ti,davinci-gpio-unbanked = <0>;
>>  		power-domains = <&k3_pds 78 TI_SCI_PD_EXCLUSIVE>;
>>  		clocks = <&k3_clks 78 0>;
>> -- 
>> 2.25.1
>>
> 

-- 
Regards,
Nitin