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: <20251009-lizard-slapstick-4de7dfe2322d@spud>
Date: Thu, 9 Oct 2025 18:13:36 +0100
From: Conor Dooley <conor@...nel.org>
To: Roy Luo <royluo@...gle.com>
Cc: Vinod Koul <vkoul@...nel.org>,
	Kishon Vijay Abraham I <kishon@...nel.org>,
	Rob Herring <robh@...nel.org>,
	Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>,
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	Thinh Nguyen <Thinh.Nguyen@...opsys.com>,
	Philipp Zabel <p.zabel@...gutronix.de>,
	Peter Griffin <peter.griffin@...aro.org>,
	André Draszik <andre.draszik@...aro.org>,
	Tudor Ambarus <tudor.ambarus@...aro.org>,
	Joy Chakraborty <joychakr@...gle.com>,
	Naveen Kumar <mnkumar@...gle.com>,
	Badhri Jagan Sridharan <badhri@...gle.com>,
	linux-phy@...ts.infradead.org, devicetree@...r.kernel.org,
	linux-kernel@...r.kernel.org, linux-usb@...r.kernel.org,
	linux-arm-kernel@...ts.infradead.org,
	linux-samsung-soc@...r.kernel.org
Subject: Re: [PATCH v2 1/4] dt-bindings: usb: dwc3: Add Google Tensor G5 DWC3

On Wed, Oct 08, 2025 at 09:40:57PM -0700, Roy Luo wrote:
> On Wed, Oct 8, 2025 at 1:58 PM Conor Dooley <conor@...nel.org> wrote:
> >
> > On Wed, Oct 08, 2025 at 05:59:57AM +0000, Roy Luo wrote:

> > > +allOf:
> > > +  - $ref: snps,dwc3-common.yaml#
> > > +
> > > +unevaluatedProperties: false
> >
> > So every property from snps,dwc3-common.yaml is valid here, with any of
> > the permitted values?
> 
> Conor,
> 
> Appreciate the review.
> Ack to all the comments, will fix them in the next patch.
> And yes, every property from snps,dwc3-common.yaml is valid here.
> You can find more context here [1], essentially the dwc3 glue would be
> operating on the same platform device as the dwc3 core, hence all
> properties are allowed.
> 
> [1] https://lore.kernel.org/all/20250414-dwc3-refactor-v7-0-f015b358722d@oss.qualcomm.com/

I find it exceedingly hard to believe that every property from that
file, with every permitted value, is possible. AFAIU, the tensor g5 is a
phone chip that's only used in pixel devices, not something that people
can just buy and integrate into whatever device they feel like. There
should be a vanishingly small number of possible configurations,
possibly exactly one configuration. There are dozens of properties in
the dwc3 common binding, of which at least 10 are for "quirks" or other
sorts of hardware errata that are not going to be variable from one
phone to another.

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

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ