On Sat, May 25, 2024 at 06:08:46PM +0100, Conor Dooley wrote:

On Tue, May 21, 2024 at 11:38:02AM -0700, Elliot Berman wrote:

...

+allOf:

• # either describe soc or soc-version; it's confusing to have both

Why not just use the one that has the most information and discard the others? If your dtb picker for this platform doesn't care about the soc version, then just don't look at that cell?

The dtb picker for the platform doesn't know whether to care about the SoC version/platform version/whatever. That's a property of the DTB itself and I don't think it makes much sense to bake that into the DTB picker which would otherwise be unaware of this.

Likewise for platform and PMIC, why can't you ignore the cells you don't care about, rather than having a new property for each variant? Nothing in this patch explains why multiple variants are required rather than just dealing with the most informational.

Sure, I will explain in future revision.

- Elliot