Re: [PATCH 00/10] dt-bindings: Various cleanups for Tegra-related bindings

[Thierry Reding <thierry.reding@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 4155 bytes --]

On Mon, Feb 23, 2026 at 11:11:40AM -0600, Rob Herring wrote:
> On Mon, Feb 23, 2026 at 03:32:55PM +0100, Thierry Reding wrote:
> > From: Thierry Reding <treding@nvidia.com>
> > 
> > This patch set contains a couple of cleanups and conversions for Tegra-
> > related bindings. In total, on top of next-20260220, these patches get
> > the number of DT validation issues down from 184 to just 88.

Hi Rob,

sorry for the late reply. I've recently switched to the kernel.org
address and for some reason these messages now don't show up in my
inbox and I only see them on the lore archives.

> Great! Really, you were at only 40 unique warnings (I strip the 
> filenames to avoid multiple boards duplicating warnings). You're in 4th 
> (to last) place:
> 
> arch/arm64/boot/dts/hisilicon:116:74
> arch/arm64/boot/dts/mediatek:197:48
> arch/arm64/boot/dts/qcom:132:45
> arch/arm64/boot/dts/nvidia:184:40
> arch/arm64/boot/dts/rockchip:76:27
> arch/arm64/boot/dts/marvell:182:23
> arch/arm64/boot/dts/renesas:83:13
> arch/arm64/boot/dts/xilinx:16:6
> arch/arm64/boot/dts/microchip:22:6
> arch/arm64/boot/dts/broadcom:32:4
> arch/arm64/boot/dts/nuvoton:3:3
> arch/arm64/boot/dts/sprd:2:2
> arch/arm64/boot/dts/intel:2:2
> arch/arm64/boot/dts/apm:3:2
> arch/arm64/boot/dts/realtek:45:1
> arch/arm64/boot/dts/freescale:2:1
> arch/arm64/boot/dts/arm:1:1

\o/ ... I guess...

I've got a bunch more that I plan to flush out. Hopefully I can make a
bit quicker progress this time around.

> This and logs of all the warnings from next and Linus' trees can be 
> retrieved with scripts here:
> 
> https://gitlab.com/robherring/ci-jobs
> 
> 
> > Note that technically these are at different revisions because they had
> > been sent out separately a while ago, Some of these have already been
> > reviewed, but given that they are fairly old I wanted to send them out
> > in case there are new best practices that these don't include. I've run
> > all of these through dt_binding_check. Also I've verified that these do
> > not produce any new warnings/errors while eliminating old ones.
> > 
> > Krzysztof, Rob, I know that you prefer DT binding changes to go through
> > driver trees, but given that these don't have any driver changes to go
> > with them, should we queue these via the Tegra tree (or devicetree tree)
> > once they've passed review?
> 
> I prefer they go via subsystem trees still, but if you don't get a reply 
> in reasonable time just take them. You can take the interrupt-controller 
> one though as DT only changes don't tend to get picked up.

Is this an implied "Reviewed-by"? Some of these have been reviewed, but
some were either not sent out yet, or didn't get any response at the
time?

> > I plan to pick up the two DTS changes into the Tegra tree since they are
> > fairly trivial and unrelated to the bindings changes. I suppose they
> > could've just been a separate series, but I thought I'd post them along
> > with the other changes since this is all a concerted effort to get the
> > number of issues down.
> > 
> > Thanks,
> > Thierry
> > 
> > Thierry Reding (10):
> >   dt-bindings: phy: tegra-xusb: Document Type C support
> >   dt-bindings: pci: tegra: Convert to json-schema
> >   dt-bindings: clock: tegra124-dfll: Convert to json-schema
> >   dt-bindings: interrupt-controller: tegra: Fix reg entries
> >   dt-bindings: arm: tegra: Add missing compatible strings
> >   dt-bindings: phy: tegra: Document Tegra210 USB PHY
> >   dt-bindings: memory: Add Tegra210 memory controller bindings
> >   dt-bindings: memory: tegra210: Mark EMC as cooling device
> >   arm64: tegra: Fix snps,blen properties
> >   arm64: tegra: Drop redundant clock and reset names for TSEC
> 
> arm64: dts: tegra: ...

How strong is the suggestion? We've used just "arm64: tegra:" since
basically forever. I can obviously switch at some point, but it'll be a
break in consistency. Although, looking at the logs for the last few
years, there have been occasional arm64: dts: tegra prefixes and a
(very) few outliers with totally butchered prefixes.

Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]