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

[Gregory CLEMENT <gregory.clement@bootlin.com>]

Le 1 mars 2020 00:06:56 GMT+01:00, Vadym Kochan <vadym.kochan@plvision.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@plvision.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