| Message ID | 20231209124401.93814-1-krzysztof.kozlowski@linaro.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | arm64: dts: amlogic: drop redundant status=okay | expand |
On 09/12/2023 13:44, Krzysztof Kozlowski wrote: > New device nodes are enabled by default, so no need for status=okay. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > --- > > Tested by comparing decompiled DTBs. > --- > arch/arm64/boot/dts/amlogic/meson-axg-s400.dts | 5 ----- > arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts | 1 - > arch/arm64/boot/dts/amlogic/meson-g12a-x96-max.dts | 1 - > arch/arm64/boot/dts/amlogic/meson-g12b-gtking.dts | 1 - > arch/arm64/boot/dts/amlogic/meson-g12b-ugoos-am6.dts | 1 - > arch/arm64/boot/dts/amlogic/meson-gx-p23x-q20x.dtsi | 1 - > arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dts | 1 - > arch/arm64/boot/dts/amlogic/meson-gxbb-p200.dts | 1 - > arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95.dtsi | 1 - > arch/arm64/boot/dts/amlogic/meson-gxbb-wetek-play2.dts | 1 - > arch/arm64/boot/dts/amlogic/meson-gxm-nexbox-a1.dts | 1 - > arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts | 1 - > arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dts | 1 - > 13 files changed, 17 deletions(-) > > diff --git a/arch/arm64/boot/dts/amlogic/meson-axg-s400.dts b/arch/arm64/boot/dts/amlogic/meson-axg-s400.dts > index df16eead2c80..c8905663bc75 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-axg-s400.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-axg-s400.dts > @@ -66,7 +66,6 @@ linein: audio-codec-0 { > VDDA-supply = <&vcc_3v3>; > VDDP-supply = <&vcc_3v3>; > VDDD-supply = <&vcc_3v3>; > - status = "okay"; > sound-name-prefix = "Linein"; > }; > > @@ -75,14 +74,12 @@ lineout: audio-codec-1 { > compatible = "everest,es7154"; > VDD-supply = <&vcc_3v3>; > PVDD-supply = <&vcc_5v>; > - status = "okay"; > sound-name-prefix = "Lineout"; > }; > > spdif_dit: audio-codec-2 { > #sound-dai-cells = <0>; > compatible = "linux,spdif-dit"; > - status = "okay"; > sound-name-prefix = "DIT"; > }; > > @@ -91,14 +88,12 @@ dmics: audio-codec-3 { > compatible = "dmic-codec"; > num-channels = <7>; > wakeup-delay-ms = <50>; > - status = "okay"; > sound-name-prefix = "MIC"; > }; > > spdif_dir: audio-codec-4 { > #sound-dai-cells = <0>; > compatible = "linux,spdif-dir"; > - status = "okay"; > sound-name-prefix = "DIR"; > }; > > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts b/arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts > index 0ad0c2b7dfef..4c4550dd4711 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts > @@ -45,7 +45,6 @@ dmics: audio-codec-1 { > compatible = "dmic-codec"; > num-channels = <2>; > wakeup-delay-ms = <50>; > - status = "okay"; > sound-name-prefix = "MIC"; > }; > > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12a-x96-max.dts b/arch/arm64/boot/dts/amlogic/meson-g12a-x96-max.dts > index 4969a76460fa..9b55982b6a6b 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-g12a-x96-max.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-g12a-x96-max.dts > @@ -22,7 +22,6 @@ aliases { > spdif_dit: audio-codec-1 { > #sound-dai-cells = <0>; > compatible = "linux,spdif-dit"; > - status = "okay"; > sound-name-prefix = "DIT"; > }; > > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12b-gtking.dts b/arch/arm64/boot/dts/amlogic/meson-g12b-gtking.dts > index 8fc2e143cb54..0da386cabe1a 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-g12b-gtking.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-g12b-gtking.dts > @@ -22,7 +22,6 @@ aliases { > spdif_dit: audio-codec-1 { > #sound-dai-cells = <0>; > compatible = "linux,spdif-dit"; > - status = "okay"; > sound-name-prefix = "DIT"; > }; > > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12b-ugoos-am6.dts b/arch/arm64/boot/dts/amlogic/meson-g12b-ugoos-am6.dts > index ce548b373296..6396f190d703 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-g12b-ugoos-am6.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-g12b-ugoos-am6.dts > @@ -17,7 +17,6 @@ / { > spdif_dit: audio-codec-1 { > #sound-dai-cells = <0>; > compatible = "linux,spdif-dit"; > - status = "okay"; > sound-name-prefix = "DIT"; > }; > > diff --git a/arch/arm64/boot/dts/amlogic/meson-gx-p23x-q20x.dtsi b/arch/arm64/boot/dts/amlogic/meson-gx-p23x-q20x.dtsi > index 18f7b730289e..e59c3c92b1e7 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-gx-p23x-q20x.dtsi > +++ b/arch/arm64/boot/dts/amlogic/meson-gx-p23x-q20x.dtsi > @@ -26,7 +26,6 @@ dio2133: analog-amplifier { > spdif_dit: audio-codec-0 { > #sound-dai-cells = <0>; > compatible = "linux,spdif-dit"; > - status = "okay"; > sound-name-prefix = "DIT"; > }; > > diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dts b/arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dts > index e238f1f10124..f28452b9f00f 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dts > @@ -18,7 +18,6 @@ / { > spdif_dit: audio-codec-0 { > #sound-dai-cells = <0>; > compatible = "linux,spdif-dit"; > - status = "okay"; > sound-name-prefix = "DIT"; > }; > > diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb-p200.dts b/arch/arm64/boot/dts/amlogic/meson-gxbb-p200.dts > index 292c718ee19c..591455c50e88 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-gxbb-p200.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb-p200.dts > @@ -18,7 +18,6 @@ / { > spdif_dit: audio-codec-0 { > #sound-dai-cells = <0>; > compatible = "linux,spdif-dit"; > - status = "okay"; > sound-name-prefix = "DIT"; > }; > > diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95.dtsi b/arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95.dtsi > index e8303089bff6..74df32534231 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95.dtsi > +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95.dtsi > @@ -21,7 +21,6 @@ chosen { > spdif_dit: audio-codec-0 { > #sound-dai-cells = <0>; > compatible = "linux,spdif-dit"; > - status = "okay"; > sound-name-prefix = "DIT"; > }; > > diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb-wetek-play2.dts b/arch/arm64/boot/dts/amlogic/meson-gxbb-wetek-play2.dts > index e1b74b174915..376760d86766 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-gxbb-wetek-play2.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb-wetek-play2.dts > @@ -17,7 +17,6 @@ / { > spdif_dit: audio-codec-0 { > #sound-dai-cells = <0>; > compatible = "linux,spdif-dit"; > - status = "okay"; > sound-name-prefix = "DIT"; > }; > > diff --git a/arch/arm64/boot/dts/amlogic/meson-gxm-nexbox-a1.dts b/arch/arm64/boot/dts/amlogic/meson-gxm-nexbox-a1.dts > index 7d525bdb0e06..ad2dd4ad0a31 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-gxm-nexbox-a1.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-gxm-nexbox-a1.dts > @@ -28,7 +28,6 @@ chosen { > spdif_dit: audio-codec-0 { > #sound-dai-cells = <0>; > compatible = "linux,spdif-dit"; > - status = "okay"; > sound-name-prefix = "DIT"; > }; > > diff --git a/arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts b/arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts > index 50d49aec41bd..d05dde8da5c5 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts > @@ -37,7 +37,6 @@ memory@0 { > spdif_dit: audio-codec-0 { > #sound-dai-cells = <0>; > compatible = "linux,spdif-dit"; > - status = "okay"; > sound-name-prefix = "DIT"; > }; > > diff --git a/arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dts b/arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dts > index 095579c55f18..109932068dbe 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dts > @@ -32,7 +32,6 @@ dmics: audio-codec-1 { > compatible = "dmic-codec"; > num-channels = <2>; > wakeup-delay-ms = <50>; > - status = "okay"; > sound-name-prefix = "MIC"; > }; > Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
Hi, On Sat, 09 Dec 2023 13:44:01 +0100, Krzysztof Kozlowski wrote: > New device nodes are enabled by default, so no need for status=okay. > > Thanks, Applied to https://git.kernel.org/pub/scm/linux/kernel/git/amlogic/linux.git (v6.8/arm64-dt) [1/1] arm64: dts: amlogic: drop redundant status=okay https://git.kernel.org/amlogic/c/e3c58a44ea7daef26c8c53c93489888ac983f327 These changes has been applied on the intermediate git tree [1]. The v6.8/arm64-dt branch will then be sent via a formal Pull Request to the Linux SoC maintainers for inclusion in their intermediate git branches in order to be sent to Linus during the next merge window, or sooner if it's a set of fixes. In the cases of fixes, those will be merged in the current release candidate kernel and as soon they appear on the Linux master branch they will be backported to the previous Stable and Long-Stable kernels [2]. The intermediate git branches are merged daily in the linux-next tree [3], people are encouraged testing these pre-release kernels and report issues on the relevant mailing-lists. If problems are discovered on those changes, please submit a signed-off-by revert patch followed by a corrective changeset. [1] https://git.kernel.org/pub/scm/linux/kernel/git/amlogic/linux.git [2] https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git [3] https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
diff --git a/arch/arm64/boot/dts/amlogic/meson-axg-s400.dts b/arch/arm64/boot/dts/amlogic/meson-axg-s400.dts index df16eead2c80..c8905663bc75 100644 --- a/arch/arm64/boot/dts/amlogic/meson-axg-s400.dts +++ b/arch/arm64/boot/dts/amlogic/meson-axg-s400.dts @@ -66,7 +66,6 @@ linein: audio-codec-0 { VDDA-supply = <&vcc_3v3>; VDDP-supply = <&vcc_3v3>; VDDD-supply = <&vcc_3v3>; - status = "okay"; sound-name-prefix = "Linein"; }; @@ -75,14 +74,12 @@ lineout: audio-codec-1 { compatible = "everest,es7154"; VDD-supply = <&vcc_3v3>; PVDD-supply = <&vcc_5v>; - status = "okay"; sound-name-prefix = "Lineout"; }; spdif_dit: audio-codec-2 { #sound-dai-cells = <0>; compatible = "linux,spdif-dit"; - status = "okay"; sound-name-prefix = "DIT"; }; @@ -91,14 +88,12 @@ dmics: audio-codec-3 { compatible = "dmic-codec"; num-channels = <7>; wakeup-delay-ms = <50>; - status = "okay"; sound-name-prefix = "MIC"; }; spdif_dir: audio-codec-4 { #sound-dai-cells = <0>; compatible = "linux,spdif-dir"; - status = "okay"; sound-name-prefix = "DIR"; }; diff --git a/arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts b/arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts index 0ad0c2b7dfef..4c4550dd4711 100644 --- a/arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts +++ b/arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts @@ -45,7 +45,6 @@ dmics: audio-codec-1 { compatible = "dmic-codec"; num-channels = <2>; wakeup-delay-ms = <50>; - status = "okay"; sound-name-prefix = "MIC"; }; diff --git a/arch/arm64/boot/dts/amlogic/meson-g12a-x96-max.dts b/arch/arm64/boot/dts/amlogic/meson-g12a-x96-max.dts index 4969a76460fa..9b55982b6a6b 100644 --- a/arch/arm64/boot/dts/amlogic/meson-g12a-x96-max.dts +++ b/arch/arm64/boot/dts/amlogic/meson-g12a-x96-max.dts @@ -22,7 +22,6 @@ aliases { spdif_dit: audio-codec-1 { #sound-dai-cells = <0>; compatible = "linux,spdif-dit"; - status = "okay"; sound-name-prefix = "DIT"; }; diff --git a/arch/arm64/boot/dts/amlogic/meson-g12b-gtking.dts b/arch/arm64/boot/dts/amlogic/meson-g12b-gtking.dts index 8fc2e143cb54..0da386cabe1a 100644 --- a/arch/arm64/boot/dts/amlogic/meson-g12b-gtking.dts +++ b/arch/arm64/boot/dts/amlogic/meson-g12b-gtking.dts @@ -22,7 +22,6 @@ aliases { spdif_dit: audio-codec-1 { #sound-dai-cells = <0>; compatible = "linux,spdif-dit"; - status = "okay"; sound-name-prefix = "DIT"; }; diff --git a/arch/arm64/boot/dts/amlogic/meson-g12b-ugoos-am6.dts b/arch/arm64/boot/dts/amlogic/meson-g12b-ugoos-am6.dts index ce548b373296..6396f190d703 100644 --- a/arch/arm64/boot/dts/amlogic/meson-g12b-ugoos-am6.dts +++ b/arch/arm64/boot/dts/amlogic/meson-g12b-ugoos-am6.dts @@ -17,7 +17,6 @@ / { spdif_dit: audio-codec-1 { #sound-dai-cells = <0>; compatible = "linux,spdif-dit"; - status = "okay"; sound-name-prefix = "DIT"; }; diff --git a/arch/arm64/boot/dts/amlogic/meson-gx-p23x-q20x.dtsi b/arch/arm64/boot/dts/amlogic/meson-gx-p23x-q20x.dtsi index 18f7b730289e..e59c3c92b1e7 100644 --- a/arch/arm64/boot/dts/amlogic/meson-gx-p23x-q20x.dtsi +++ b/arch/arm64/boot/dts/amlogic/meson-gx-p23x-q20x.dtsi @@ -26,7 +26,6 @@ dio2133: analog-amplifier { spdif_dit: audio-codec-0 { #sound-dai-cells = <0>; compatible = "linux,spdif-dit"; - status = "okay"; sound-name-prefix = "DIT"; }; diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dts b/arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dts index e238f1f10124..f28452b9f00f 100644 --- a/arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dts +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dts @@ -18,7 +18,6 @@ / { spdif_dit: audio-codec-0 { #sound-dai-cells = <0>; compatible = "linux,spdif-dit"; - status = "okay"; sound-name-prefix = "DIT"; }; diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb-p200.dts b/arch/arm64/boot/dts/amlogic/meson-gxbb-p200.dts index 292c718ee19c..591455c50e88 100644 --- a/arch/arm64/boot/dts/amlogic/meson-gxbb-p200.dts +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb-p200.dts @@ -18,7 +18,6 @@ / { spdif_dit: audio-codec-0 { #sound-dai-cells = <0>; compatible = "linux,spdif-dit"; - status = "okay"; sound-name-prefix = "DIT"; }; diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95.dtsi b/arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95.dtsi index e8303089bff6..74df32534231 100644 --- a/arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95.dtsi +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95.dtsi @@ -21,7 +21,6 @@ chosen { spdif_dit: audio-codec-0 { #sound-dai-cells = <0>; compatible = "linux,spdif-dit"; - status = "okay"; sound-name-prefix = "DIT"; }; diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb-wetek-play2.dts b/arch/arm64/boot/dts/amlogic/meson-gxbb-wetek-play2.dts index e1b74b174915..376760d86766 100644 --- a/arch/arm64/boot/dts/amlogic/meson-gxbb-wetek-play2.dts +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb-wetek-play2.dts @@ -17,7 +17,6 @@ / { spdif_dit: audio-codec-0 { #sound-dai-cells = <0>; compatible = "linux,spdif-dit"; - status = "okay"; sound-name-prefix = "DIT"; }; diff --git a/arch/arm64/boot/dts/amlogic/meson-gxm-nexbox-a1.dts b/arch/arm64/boot/dts/amlogic/meson-gxm-nexbox-a1.dts index 7d525bdb0e06..ad2dd4ad0a31 100644 --- a/arch/arm64/boot/dts/amlogic/meson-gxm-nexbox-a1.dts +++ b/arch/arm64/boot/dts/amlogic/meson-gxm-nexbox-a1.dts @@ -28,7 +28,6 @@ chosen { spdif_dit: audio-codec-0 { #sound-dai-cells = <0>; compatible = "linux,spdif-dit"; - status = "okay"; sound-name-prefix = "DIT"; }; diff --git a/arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts b/arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts index 50d49aec41bd..d05dde8da5c5 100644 --- a/arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts +++ b/arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts @@ -37,7 +37,6 @@ memory@0 { spdif_dit: audio-codec-0 { #sound-dai-cells = <0>; compatible = "linux,spdif-dit"; - status = "okay"; sound-name-prefix = "DIT"; }; diff --git a/arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dts b/arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dts index 095579c55f18..109932068dbe 100644 --- a/arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dts +++ b/arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dts @@ -32,7 +32,6 @@ dmics: audio-codec-1 { compatible = "dmic-codec"; num-channels = <2>; wakeup-delay-ms = <50>; - status = "okay"; sound-name-prefix = "MIC"; };
New device nodes are enabled by default, so no need for status=okay. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- Tested by comparing decompiled DTBs. --- arch/arm64/boot/dts/amlogic/meson-axg-s400.dts | 5 ----- arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts | 1 - arch/arm64/boot/dts/amlogic/meson-g12a-x96-max.dts | 1 - arch/arm64/boot/dts/amlogic/meson-g12b-gtking.dts | 1 - arch/arm64/boot/dts/amlogic/meson-g12b-ugoos-am6.dts | 1 - arch/arm64/boot/dts/amlogic/meson-gx-p23x-q20x.dtsi | 1 - arch/arm64/boot/dts/amlogic/meson-gxbb-kii-pro.dts | 1 - arch/arm64/boot/dts/amlogic/meson-gxbb-p200.dts | 1 - arch/arm64/boot/dts/amlogic/meson-gxbb-vega-s95.dtsi | 1 - arch/arm64/boot/dts/amlogic/meson-gxbb-wetek-play2.dts | 1 - arch/arm64/boot/dts/amlogic/meson-gxm-nexbox-a1.dts | 1 - arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts | 1 - arch/arm64/boot/dts/amlogic/meson-sm1-sei610.dts | 1 - 13 files changed, 17 deletions(-)