linux-kernel - Re: [PATCH 0/2] arm: dts: OMAP: Clean up devicetree bindings and compatible strings

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: <20250620-unfitting-shredder-7c4c3c3bc8ec@spud>
Date: Fri, 20 Jun 2025 15:15:45 +0100
From: Conor Dooley <conor@...nel.org>
To: Kory Maincent <kory.maincent@...tlin.com>
Cc: Aaro Koskinen <aaro.koskinen@....fi>,
	Andreas Kemnade <andreas@...nade.info>,
	Kevin Hilman <khilman@...libre.com>,
	Roger Quadros <rogerq@...nel.org>, Tony Lindgren <tony@...mide.com>,
	Rob Herring <robh@...nel.org>,
	Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>,
	Paul Barker <paul.barker@...cloud.com>,
	Marc Murphy <marc.murphy@...cloud.com>,
	Thomas Petazzoni <thomas.petazzoni@...tlin.com>,
	linux-omap@...r.kernel.org, devicetree@...r.kernel.org,
	linux-kernel@...r.kernel.org,
	Romain Gantois <romain.gantois@...tlin.com>,
	Thomas Bonnefille <thomas.bonnefille@...tlin.com>,
	Bajjuri Praneeth <praneeth@...com>, Andrew Davis <afd@...com>
Subject: Re: [PATCH 0/2] arm: dts: OMAP: Clean up devicetree bindings and
 compatible strings

On Fri, Jun 20, 2025 at 10:24:07AM +0200, Kory Maincent wrote:
> This series addresses inconsistencies in OMAP devicetree bindings and
> compatible strings for AM33xx-based boards:
> 
> - Add missing compatible strings to binding documentation
> - Update vendor prefix from "ti" to "beagle" for BeagleBone boards
>   to reflect actual hardware vendors
> - Remove redundant compatible strings from device properties
> 
> Note: Originally attempted to align devicetree model descriptions but
> received pushback due to Debian script dependencies (see thread at
> https://lore.kernel.org/all/7e854232-f02f-4ad6-b65e-22c18d1d9fe5@jm0.eu/).
> The compatible string changes may face similar concerns and would
> benefit from devicetree maintainer review.

I think relying on the "model" field is really silly, but there's also
probably very little harm in keeping the old names. I'd be much more
understanding if the complaint was about the compatible strings
changing! Keeping old ?image packing tools? working with new devicetrees
is a platform maintainer thing really, what we care much more about is
old devicetrees with new software.

> 
> This work was split from a larger series to address feedback and allow
> independent progression of each topic area. Original series:
> https://lore.kernel.org/r/20250617-bbg-v4-0-827cbd606db6@bootlin.com
> 
> Signed-off-by: Kory Maincent <kory.maincent@...tlin.com>
> ---
> Kory Maincent (2):
>       dt-bindings: omap: Add missing AM33xx compatible strings
>       arm: dts: omap: Remove incorrect compatible strings from device trees
> 
>  Documentation/devicetree/bindings/arm/ti/omap.yaml | 22 +++++++++++++++++++++-
>  arch/arm/boot/dts/ti/omap/am335x-base0033.dts      |  2 +-
>  arch/arm/boot/dts/ti/omap/am335x-bone.dts          |  2 +-
>  .../boot/dts/ti/omap/am335x-boneblack-wireless.dts |  2 +-
>  arch/arm/boot/dts/ti/omap/am335x-boneblack.dts     |  2 +-
>  arch/arm/boot/dts/ti/omap/am335x-boneblue.dts      |  2 +-
>  .../boot/dts/ti/omap/am335x-bonegreen-wireless.dts |  2 +-
>  arch/arm/boot/dts/ti/omap/am335x-bonegreen.dts     |  2 +-
>  arch/arm/boot/dts/ti/omap/am335x-chiliboard.dts    |  3 +--
>  arch/arm/boot/dts/ti/omap/am335x-myirtech-myd.dts  |  2 +-
>  .../arm/boot/dts/ti/omap/am335x-osd3358-sm-red.dts |  2 +-
>  arch/arm/boot/dts/ti/omap/am335x-pocketbeagle.dts  |  2 +-
>  .../ti/omap/am335x-sancloud-bbe-extended-wifi.dts  |  5 +----
>  .../boot/dts/ti/omap/am335x-sancloud-bbe-lite.dts  |  5 +----
>  arch/arm/boot/dts/ti/omap/am335x-sancloud-bbe.dts  |  2 +-
>  arch/arm/boot/dts/ti/omap/am335x-shc.dts           |  2 +-
>  16 files changed, 36 insertions(+), 23 deletions(-)
> ---
> base-commit: e22b9ddaf3afd031abc350c303c7c07a51c569d8
> change-id: 20250619-ti_dts_clean-d1d86c9e675f
> 
> Best regards,
> --  
> Köry Maincent, Bootlin
> Embedded Linux and kernel engineering
> https://bootlin.com
> 

Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)