Re: [PATCH 0/7] arm/arm64: dts: Remove unused num-viewport from pcie node

[Mark Kettenis]

> From: Jisheng Zhang <jszhang@xxxxxxxxxx>
> Date: Thu, 30 Dec 2021 00:02:38 +0800
> 
> After commit 281f1f99cf3a("PCI: dwc: Detect number of iATU windows"),
> the number of iATU windows is detected at runtime, what's more,
> the 'num-viewport' property parsing has been removed, so remove the
> unused num-viewport from pcie node(s).
> 
> It's too late for linux-5.17-rc1, I will rebase and send out v2 if
> necessary when 5.17-rc1 is released.

Please no.  This only makes the device trees unnecessarily
incompatible with older kernels and other OSes that do rely on the
"num-viewport" property.  It really doesn't hurt to keep this property
even if future Linux kernels no longer look at it.

Thanks,

Mark

> Jisheng Zhang (7):
>   ARM: dts: ls1021a: remove unused num-viewport from pcie nodes
>   arm64: dts: visconti: Remove unused num-viewport from pcie node
>   arm64: dts: uniphier: Remove unused num-viewport from pcie node
>   arm64: tegra: Remove unused num-viewport from pcie node
>   arm64: dts: marvell: Remove unused num-viewport from pcie node
>   arm64: dts: freescale: Remove unused num-viewport from pcie node
>   arm64: dts: exynos: Remove unused num-viewport from pcie node
> 
>  arch/arm/boot/dts/ls1021a.dtsi                          | 2 --
>  arch/arm64/boot/dts/exynos/exynos5433.dtsi              | 1 -
>  arch/arm64/boot/dts/freescale/fsl-ls1012a.dtsi          | 1 -
>  arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi          | 2 --
>  arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi          | 3 ---
>  arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi          | 3 ---
>  arch/arm64/boot/dts/freescale/fsl-ls1088a.dtsi          | 3 ---
>  arch/arm64/boot/dts/freescale/fsl-ls208xa.dtsi          | 4 ----
>  arch/arm64/boot/dts/marvell/armada-8040-mcbin.dtsi      | 1 -
>  arch/arm64/boot/dts/marvell/armada-8040-puzzle-m801.dts | 1 -
>  arch/arm64/boot/dts/marvell/cn9130-crb-A.dts            | 1 -
>  arch/arm64/boot/dts/marvell/cn9130-crb-B.dts            | 1 -
>  arch/arm64/boot/dts/marvell/cn9130-db.dtsi              | 1 -
>  arch/arm64/boot/dts/marvell/cn9131-db.dtsi              | 1 -
>  arch/arm64/boot/dts/marvell/cn9132-db.dtsi              | 2 --
>  arch/arm64/boot/dts/nvidia/tegra194.dtsi                | 6 ------
>  arch/arm64/boot/dts/socionext/uniphier-ld20.dtsi        | 1 -
>  arch/arm64/boot/dts/socionext/uniphier-pxs3.dtsi        | 1 -
>  arch/arm64/boot/dts/toshiba/tmpv7708.dtsi               | 1 -
>  19 files changed, 36 deletions(-)
> 
> -- 
> 2.34.1
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>