Re: Patch "arm64: dts: mediatek: mt7622: drop "reset-names" from thermal block" has been added to the 4.19-stable tree

[Matthias Brugger <matthias.bgg@gmail.com>]

On 4/28/24 13:24, Sasha Levin wrote:
> This is a note to let you know that I've just added the patch titled
> 
>      arm64: dts: mediatek: mt7622: drop "reset-names" from thermal block
> 
> to the 4.19-stable tree which can be found at:
>      http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
> 
> The filename of the patch is:
>       arm64-dts-mediatek-mt7622-drop-reset-names-from-ther.patch
> and it can be found in the queue-4.19 subdirectory.
> 
> If you, or anyone else, feels it should not be added to the stable tree,
> please let <stable@vger.kernel.org> know about it.
> 
> 
> 
> commit 9628765c4a0eefa9474ef4a0698691a10395a469
> Author: Rafał Miłecki <rafal@milecki.pl>
> Date:   Sun Mar 17 23:10:50 2024 +0100
> 
>      arm64: dts: mediatek: mt7622: drop "reset-names" from thermal block
>      
>      [ Upstream commit ecb5b0034f5bcc35003b4b965cf50c6e98316e79 ]
>      
>      Binding doesn't specify "reset-names" property and Linux driver also
>      doesn't use it.
>      

I think that's an open discussion item if fixes to DTS checks are valid 
stable backports. From my point of view there is no bug so it shouldn't 
be in stable.

Regards,
Matthias

>      Fix following validation error:
>      arch/arm64/boot/dts/mediatek/mt7622-rfb1.dtb: thermal@1100b000: Unevaluated properties are not allowed ('reset-names' was unexpected)
>              from schema $id: http://devicetree.org/schemas/thermal/mediatek,thermal.yaml#
>      
>      Fixes: ae457b7679c4 ("arm64: dts: mt7622: add SoC and peripheral related device nodes")
>      Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
>      Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
>      Link: https://lore.kernel.org/r/20240317221050.18595-5-zajec5@gmail.com
>      Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
>      Signed-off-by: Sasha Levin <sashal@kernel.org>
> 
> diff --git a/arch/arm64/boot/dts/mediatek/mt7622.dtsi b/arch/arm64/boot/dts/mediatek/mt7622.dtsi
> index 76297dac2d459..f8df34ac1e64d 100644
> --- a/arch/arm64/boot/dts/mediatek/mt7622.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt7622.dtsi
> @@ -459,7 +459,6 @@ thermal: thermal@1100b000 {
>   			 <&pericfg CLK_PERI_AUXADC_PD>;
>   		clock-names = "therm", "auxadc";
>   		resets = <&pericfg MT7622_PERI_THERM_SW_RST>;
> -		reset-names = "therm";
>   		mediatek,auxadc = <&auxadc>;
>   		mediatek,apmixedsys = <&apmixedsys>;
>   		nvmem-cells = <&thermal_calibration>;