linux-kernel - Re: [PATCH 4/7] ARM: dts: amlogic: meson6: delete undocumented nodes

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: <CAFBinCD+txhz0x20LXhydZpLk=u4d9FT4=1sOX1bKuDeQqsMjQ@mail.gmail.com>
Date: Sun, 15 Sep 2024 21:12:10 +0200
From: Martin Blumenstingl <martin.blumenstingl@...glemail.com>
To: Neil Armstrong <neil.armstrong@...aro.org>
Cc: Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, 
	Conor Dooley <conor+dt@...nel.org>, Kevin Hilman <khilman@...libre.com>, 
	Jerome Brunet <jbrunet@...libre.com>, devicetree@...r.kernel.org, 
	linux-arm-kernel@...ts.infradead.org, linux-amlogic@...ts.infradead.org, 
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 4/7] ARM: dts: amlogic: meson6: delete undocumented nodes

Hi Neil,

On Wed, Sep 11, 2024 at 12:19 PM Neil Armstrong
<neil.armstrong@...aro.org> wrote:
>
> Delete nodes in meson6.dtsi that are clearly undocumented & unused for meson6,
> fixing the following check errors:
> audio-controller@...0: compatible:0: 'amlogic,aiu' is not one of ['amlogic,aiu-gxbb', 'amlogic,aiu-gxl', 'amlogic,aiu-meson8', 'amlogic,aiu-meson8b']
>         from schema $id: http://devicetree.org/schemas/sound/amlogic,aiu.yaml#
> audio-controller@...0: compatible: ['amlogic,aiu'] is too short
>         from schema $id: http://devicetree.org/schemas/sound/amlogic,aiu.yaml#
> /soc/bus@...00000/pwm@...0: failed to match any schema with compatible: ['amlogic,meson-pwm']
> /soc/bus@...00000/pwm@...0: failed to match any schema with compatible: ['amlogic,meson-pwm']
> phy@...0: compatible: 'oneOf' conditional failed, one must be fixed:
>         ['amlogic,meson-mx-usb2-phy'] is too short
>         'amlogic,meson-mx-usb2-phy' is not one of ['amlogic,meson8-usb2-phy', 'amlogic,meson8b-usb2-phy', 'amlogic,meson8m2-usb2-phy']
>         'amlogic,meson-gxbb-usb2-phy' was expected
>         from schema $id: http://devicetree.org/schemas/phy/amlogic,meson8b-usb2-phy.yaml#
> phy@...0: compatible: 'oneOf' conditional failed, one must be fixed:
>         ['amlogic,meson-mx-usb2-phy'] is too short
>         'amlogic,meson-mx-usb2-phy' is not one of ['amlogic,meson8-usb2-phy', 'amlogic,meson8b-usb2-phy', 'amlogic,meson8m2-usb2-phy']
>         'amlogic,meson-gxbb-usb2-phy' was expected
>         from schema $id: http://devicetree.org/schemas/phy/amlogic,meson8b-usb2-phy.yaml#
> /soc/bus@...00000/mmc@...0: failed to match any schema with compatible: ['amlogic,meson-mx-sdio']
> mmc@...0: compatible:0: 'amlogic,meson-mx-sdhc' is not one of ['amlogic,meson8-sdhc', 'amlogic,meson8b-sdhc', 'amlogic,meson8m2-sdhc']
>         from schema $id: http://devicetree.org/schemas/mmc/amlogic,meson-mx-sdhc.yaml#
> mmc@...0: compatible: ['amlogic,meson-mx-sdhc'] is too short
>         from schema $id: http://devicetree.org/schemas/mmc/amlogic,meson-mx-sdhc.yaml#
> remoteproc@1c: compatible:0: 'amlogic,meson-mx-ao-arc' is not one of ['amlogic,meson8-ao-arc', 'amlogic,meson8b-ao-arc']
>         from schema $id: http://devicetree.org/schemas/remoteproc/amlogic,meson-mx-ao-arc.yaml#
> remoteproc@1c: compatible: ['amlogic,meson-mx-ao-arc'] is too short
>         from schema $id: http://devicetree.org/schemas/remoteproc/amlogic,meson-mx-ao-arc.yaml#
>
> Signed-off-by: Neil Armstrong <neil.armstrong@...aro.org>
Generally I'm fine with this. I'm wondering if we should go one step
further by decommissioning Meson6 support?
To be specific here, I'm thinking of removing MACH_MESON6 from
arch/arm/mach-meson/Kconfig and deleting meson6{.dtsi,-atv1200.dts}

I don't have any boards with Meson6 SoC, there's no upstream GPIO,
pinctrl, clock, MMC, <you name it> driver for that SoC either.
So all we have upstream is the ability to boot to initramfs and use
serial console(s).
If someone is interested in Meson6 support they should bring it back
along with a few more patches for upstream support of further
peripherals.


Best regards,
Martin