Re: [PATCH 1/7] dt-bindings: mailbox: Don't require #mbox-cells to be 1
Doug Anderson <[email protected]>
| Newsgroups | org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <CAD=FV=XN0=C2G+6UHpR5_60hLq7xA69w-gffTd2Rx2im8owBWA@mail.gmail.com> |
Hi, On Wed, Jul 22, 2026 at 7:02 AM Rob Herring <[email protected]> wrote: > > On Tue, Jul 14, 2026 at 03:21:40PM -0700, Douglas Anderson wrote: > > Existing mailboxes have #mbox-cells and this makes sense if a mailbox > > only exposes one channel. Update the bindings to match. > > > > Signed-off-by: Douglas Anderson <[email protected]> > > --- > > I assume this is worth doing (?). As noted [1], mailbox bindings are > > already in the core schema, so what's here just provides extra context > > and descriptions. > > We should update mbox-consumer.yaml and remove this file instead. > There's only a couple of references to it. > > We should either just add missing descriptions to mbox-consumer.yaml or > change it to mbox.yaml and add #mbox-cells. The latter would only > provide some completeness as we don't have any constraints > on #mbox-cells (there's a global max of 8 already). FWIW, I've posted up both a pull request in git hub to add missing descriptions into dt-schema [1] and a patch against the kernel repository to delete the old .txt file [2]. I'm still waiting for responses to other patches in this series before sending a v2. When I send v2 I'll drop this patch from the series so we can track it separately. :-) [1] https://github.com/devicetree-org/dt-schema/pull/202 [2] http://lore.kernel.org/r/20260804133801.1.I8b3ff71c528133e7d6f81fe926fa39e6451d63fd@changeid