Re: [PATCH v3] docs: dt: maintainer: Add Devicetree and OF maintainer profile document

[Rob Herring]

On Mon, Jul 20, 2026 at 1:40 PM Krzysztof Kozlowski
<krzysztof.kozlowski@xxxxxxxxxxxxxxxx> 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 <robh@xxxxxxxxxx>
> Cc: Conor Dooley <conor+dt@xxxxxxxxxx>
> Cc: Saravana Kannan <saravanak@xxxxxxxxxx>
> Cc: devicetree@xxxxxxxxxxxxxxx
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxxxxxxxx>
>
> ---
>
> 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.

> +
> +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