On Mon, Jul 20, 2026 at 03:12:17PM -0500, Rob Herring wrote:
> On Mon, Jul 20, 2026 at 1:40 PM Krzysztof Kozlowski
> <[email protected]> wrote:
> >
> > Document how Devicetree and Open Firmware maintainers handle their
> > subsystem, especially focusing on two caveats:
> >
> > Devicetree subsystem handles patches with a minor difference comparing
> > to other subsystems: while DT maintainers pick up OF code, they only
> > provide review of DT bindings without applying these.
> >
> > All three DT bindings maintainers rely currently on Patchwork and due to
> > enormous amount of emails per day, regardless how much DT maintainers
> > try, they cannot read all the emails.
> >
> > Cc: Rob Herring <[email protected]>
> > Cc: Conor Dooley <[email protected]>
> > Cc: Saravana Kannan <[email protected]>
> > Cc: [email protected]
> > Signed-off-by: Krzysztof Kozlowski <[email protected]>
> >
> > ---
> >
> > I expect patch to be picked up by Rob, after review.
> >
> > Changes in v3:
> > 1. Add also F: entry
> >
> > Changes in v2:
> > 1. Correct typos and trailing white spaces.
> > 2. Fix order of P: after C: in maintainers.
> > ---
> >  .../process/maintainer-devicetree.rst         | 70 +++++++++++++++++++
> >  MAINTAINERS                                   |  3 +
> >  2 files changed, 73 insertions(+)
> >  create mode 100644 Documentation/process/maintainer-devicetree.rst
> >
> > diff --git a/Documentation/process/maintainer-devicetree.rst 
> > b/Documentation/process/maintainer-devicetree.rst
> > new file mode 100644
> > index 000000000000..d8ffe752bf5d
> > --- /dev/null
> > +++ b/Documentation/process/maintainer-devicetree.rst
> > @@ -0,0 +1,70 @@
> > +.. SPDX-License-Identifier: GPL-2.0
> > +
> > +======================================
> > +Devicetree and Open Firmware Subsystem
> > +======================================
> > +
> > +Other Process Documents
> > +-----------------------
> > +
> > +Please see the documents in Documentation/devicetree/bindings/ for 
> > information
> > +on how to write proper Devicetree bindings and how to submit patches.
> > +
> > +Patch Review and Handling
> > +-------------------------
> > +
> > +Patches handled by Devicetree maintainers are processed differently 
> > depending
> > +on the patch type:
> > +
> > +1. Core OF driver code, e.g. drivers/of/:
> > +   patches are reviewed and applied by DT maintainers.
> > +
> > +2. Devicetree bindings:
> > +   patches are reviewed by DT maintainers but, except in certain cases, 
> > should
> > +   be applied by subsystem maintainers.  See also *For kernel maintainers* 
> > in
> > +   Documentation/devicetree/bindings/submitting-patches.rst.
>
> I would reword:
>
> patches are reviewed by DT maintainers, but should be applied by
> subsystem maintainers except in certain cases.

It would be wonderful if provide a few sample *certain* cases.

Anyways, nice document.

Reviewed-by: Frank Li <[email protected]>
>
> > +
> > +3. DTS and drivers:
> > +   DT maintainers might provide comments, but review is generally not 
> > expected.
>
> For DTS, we expect to pass schema checks or at least not add new warnings.
>
> I can address these 2 when applying.
>
> Rob

Reply via email to