linux-kernel - Re: [PATCH] arm64: tegra: Bump #address-cells and #size-cells on Tegra186

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: <vegicz45jspxecpaitgju6ivvrefwoufg5yrzlvxudjatno7cr@rnvnqrmqofsk>
Date: Wed, 7 May 2025 17:37:03 +0200
From: Thierry Reding <thierry.reding@...il.com>
To: webgeek1234@...il.com
Cc: Rob Herring <robh@...nel.org>, 
	Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, 
	Jonathan Hunter <jonathanh@...dia.com>, devicetree@...r.kernel.org, linux-tegra@...r.kernel.org, 
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] arm64: tegra: Bump #address-cells and #size-cells on
 Tegra186

On Sat, Apr 19, 2025 at 10:30:31PM -0500, Aaron Kling via B4 Relay wrote:
> From: Aaron Kling <webgeek1234@...il.com>
> 
> This was done for Tegra194 and Tegra234 in 2838cfd, but Tegra186 was not
> part of that change. The same reasoning for that commit also applies to
> Tegra186, plus keeping the archs as close to each other as possible makes
> it easier to compare between them and support features concurrently.

As explained in the commit that you referenced, the reason for making
these changes for Tegra194 and Tegra234 was so that the PCI and GPU
nodes could move back into the bus@0 node. This doesn't exist on
Tegra186, and the top-level already has #address-cells = <2> and
#size-cells = <2>.

Does this actually fix a bug? Just making this look more similar to
Tegra194/234 doesn't seem like the best of justifications for bloating
the DT.

Thierry

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