RE: [PATCH v5 01/19] dt-bindings: crypto: add Rambus CryptoManager Hub

From: Ousherovitch, Alex

Date: Fri Oct 02 2026 - 20:01:59 EST


On Sun, Sep 28, 2026 at 06:18:48PM -0600, Rob Herring wrote:
> > + clock-names:
> > + oneOf:
> [...]
> Can be simplified to:
>
> items:
> - const: core
> - enum: [ core-div2, rt ]
> - const: rt

Thanks, I'll use that, with minItems: 1 so the lone "core" case still
validates:

clock-names:
minItems: 1
items:
- const: core
- enum: [ core-div2, rt ]
- const: rt

> > + reg:
> > + maxItems: 1
> [...]
> Is there a maximum number of instances? If so, 'maximum: N'.

Yes -- the hardware supports up to 64 mailbox instances, so the index is
0..63. I'll add 'maximum: 63' to the queue reg.

Both will be in v6.

Thanks,
Alex