[PATCH v2] dt-bindings: iommu: riscv: Add bindings for Tenstorrent RISC-V IOMMU
Conor Dooley
conor at kernel.org
Wed May 20 11:55:29 PDT 2026
On Wed, May 20, 2026 at 11:14:32AM -0700, Drew Fustini wrote:
> On Wed, May 20, 2026 at 05:17:41PM +0100, Conor Dooley wrote:
> > On Tue, May 19, 2026 at 11:16:28PM -0700, Drew Fustini wrote:
> > > +allOf:
> > > + - if:
> > > + properties:
> > > + compatible:
> > > + contains:
> > > + enum:
> > > + - tenstorrent,riscv-iommu
> > > + then:
> > > + properties:
> > > + reg:
> >
> > > + items:
> > > + - description: IOMMU base registers
> > > + - description: Tenstorrent IOMMU machine mode registers.
> >
> > Should this also have minItems: 2?
>
> I think items: with 2 items implies minItems: 2 but I see your later
> point about the description and names don't belong in the allOf: block.
It would usually, but this is a conditional portion of a wider binding
that you've modified to look like
| reg:
| minItems: 1
| maxItems: 2
outside the conditional section, and since you don't further constrain
reg in the conditional section, this minItems: 1 applies.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-riscv/attachments/20260520/5fc44c8f/attachment.sig>
More information about the linux-riscv
mailing list