linux-kernel - Re: [PATCH] arm64: dts: marvell: fix non-existed cpu referrence in armada-ap806-dual.dtsi

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]

Message-ID: <647B367B-4278-4D00-98D3-1FCA76D87488@bootlin.com>
Date:   Sun, 01 Mar 2020 08:44:19 +0100
From:   Gregory CLEMENT <gregory.clement@...tlin.com>
To:     Vadym Kochan <vadym.kochan@...ision.eu>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        Mark Rutland <mark.rutland@....com>,
        Rob Herring <robh+dt@...nel.org>,
        Sebastian Hesselbarth <sebastian.hesselbarth@...il.com>,
        Andrew Lunn <andrew@...n.ch>,
        Jason Cooper <jason@...edaemon.net>
CC:     "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] arm64: dts: marvell: fix non-existed cpu referrence in armada-ap806-dual.dtsi



Le 1 mars 2020 00:06:56 GMT+01:00, Vadym Kochan <vadym.kochan@...ision.eu> a écrit :
>Hi,
>

Hello
>Just softly ping if I sent it to the right direction.
It looks ok, and was on the list of patches to apply 

Sorry for the delay
>
>On Sun, Feb 09, 2020 at 11:20:30PM +0200, Vadym Kochan wrote:
>> armada-ap806-dual.dtsi includes armada-ap806.dtsi which describes
>> thermal zones for 4 cpus but only cpu0 and cpu1 only exists for dual
>> configuration, this makes dtb compilation fail. Fix it by removing
>> thermal zone nodes for non-existed cpus for dual configuration.
>> 
>> Signed-off-by: Vadym Kochan <vadym.kochan@...ision.eu>
>> ---
>>  arch/arm64/boot/dts/marvell/armada-ap806-dual.dtsi | 5 +++++
>>  1 file changed, 5 insertions(+)
>> 
>> diff --git a/arch/arm64/boot/dts/marvell/armada-ap806-dual.dtsi
>b/arch/arm64/boot/dts/marvell/armada-ap806-dual.dtsi
>> index 09849558a776..fcab5173fe67 100644
>> --- a/arch/arm64/boot/dts/marvell/armada-ap806-dual.dtsi
>> +++ b/arch/arm64/boot/dts/marvell/armada-ap806-dual.dtsi
>> @@ -53,4 +53,9 @@
>>  			cache-sets = <512>;
>>  		};
>>  	};
>> +
>> +	thermal-zones {
>> +		/delete-node/ ap-thermal-cpu2;
>> +		/delete-node/ ap-thermal-cpu3;
>> +	};
>>  };
>> -- 
>> 2.17.1
>> 
>
>Regards,
>Vadym Kochan

-- 
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com