From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 604B53E121A for ; Mon, 18 May 2026 09:27:19 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779096439; cv=none; b=pQdb8QvsmOzizC1LSeERUx0RJM8qZ5mveTs1n1iFmFHUOkz7uP6UuFBeXzy+eW4exCWBLRwy7zqYFHNm/XdEpaq+n5lm0IYKR1QmHi57OA/PgGLiX8eg+sS0gqsJCG8znErGIm5dT2MIBJLPl9XH9ubDHFyTKfvKBticrKM6CbY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779096439; c=relaxed/simple; bh=gxFBsWO2b3aWVIgNHt0OrFhOfRQ248cgzdeUoDcjVnA=; h=From:Subject:To:Cc:In-Reply-To:References:Content-Type:Date: Message-Id; b=j2x14tkf38nnnxSTT3d4BOowuqHu60C9FK7ztyM/VZ1riMnfhhu79oq2q1Wsvh740YJl3qBd/0K+iweVPQ3AxeYic7A6zcg7pnOErjPY62coSnSadZx4sZrf9wlmBgTWb8PIpNIuamoEH+/T6QZ82X8V5+CEA/pWUEMfN9dZGaE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=SlG+t80n; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="SlG+t80n" Received: by smtp.kernel.org (Postfix) with ESMTPSA id B33B0C2BCB7; Mon, 18 May 2026 09:27:18 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1779096438; bh=gxFBsWO2b3aWVIgNHt0OrFhOfRQ248cgzdeUoDcjVnA=; h=From:Subject:Reply-To:To:Cc:In-Reply-To:References:Date:From; b=SlG+t80nyf9jU/GxDSFCCH4gM7F7yXuUhHP2U5ASjpjWSrTZ0vYRoL4/wxem6RW1T BEdbKKQcmtOawcVNtdB7pypphaX/VPkdfxgz0wIfTfle/xa818PNZhQOBzrUTAd19W 5WVqhfBVfb4U6agbkFa8wGQxNxPHFk0npwXzpGdYrVk+KMrE79I6UKVltK7RgDV2TC 740cICrCOaBnJjy1oEMhVmaYGkDdaU5VuhhH1Ko+8Pj61bBoJWGaXmxEx9Gf46jStS IZA5oY+ysxfW0L6x7bhAxa0IqVGX6yG5NpuG9PXMpm9Hx7XJa6cIQGlRjZLJV4ajnK M0rUhcxsoIfaw== From: sashiko-bot@kernel.org Subject: Re: [PATCH] docs: dt: maintainer: Add Devicetree and OF maintainer profile document Reply-To: sashiko-reviews@lists.linux.dev To: "Krzysztof Kozlowski" Cc: robh@kernel.org, conor+dt@kernel.org, devicetree@vger.kernel.org In-Reply-To: <20260518091942.29822-2-krzysztof.kozlowski@oss.qualcomm.com> References: <20260518091942.29822-2-krzysztof.kozlowski@oss.qualcomm.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Date: Mon, 18 May 2026 09:27:18 +0000 Message-Id: <20260518092718.B33B0C2BCB7@smtp.kernel.org> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Thank you for your contribution! Sashiko AI review found 2 potential issue(= s) to consider: - [Low] Typos in the new documentation file: 'Pachwork' instead of 'Patchwo= rk' in the section header, and 'statutes' instead of 'statuses' in the text. - [Low] Incorrect tag ordering in the MAINTAINERS file. -- commit 51df2774f45c1a37759547f78fcc2881a87cc94a Author: Krzysztof Kozlowski docs: dt: maintainer: Add Devicetree and OF maintainer profile document Document how Devicetree and Open Firmware maintainers handle their subsystem, focusing on how bindings are reviewed, applied, and the heavy reliance on Patchwork. > diff --git a/Documentation/process/maintainer-devicetree.rst b/Documentat= ion/process/maintainer-devicetree.rst > new file mode 100644 > index 0000000000000..b983c5f1d851b > --- /dev/null > +++ b/Documentation/process/maintainer-devicetree.rst [ ... ] > +3. DTS and drivers: > + DT maintainers might provide comments, but review is generally not ex= pected. > + > +Pachwork > +~~~~~~~~ This isn't a bug, but there is a typo in the section header. If Pachwork is changed to Patchwork, should the underline also be extended = to 9 characters to avoid a Sphinx build warning for the title underline being too short? > + > +Devicetree maintainers review patches using Patchwork, so the current st= atus of > +a patch can be checked there. For typical driver submissions, Patchwork > +receives the entire patch set, but only a few patches are usually Device= tree > +bindings that are reviewed by DT maintainers. > + > +Explanation of Patchwork statutes: This isn't a bug, but should this be statuses instead of statutes? > + > + - **New**: Not yet processed by the automation toolset. > + - **Needs ACK**: Waiting for review by DT maintainers. [ ... ] > diff --git a/MAINTAINERS b/MAINTAINERS > index 2fb1c75afd163..c0ef1ee78ffcc 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS [ ... ] > @@ -19962,6 +19963,7 @@ M: Conor Dooley > L: devicetree@vger.kernel.org > S: Maintained > Q: http://patchwork.kernel.org/project/devicetree/list/ > +P: Documentation/process/maintainer-devicetree.rst > C: irc://irc.libera.chat/devicetree This isn't a bug, but is this the correct tag ordering? The preferred order defined at the top of the MAINTAINERS file specifies th= at the chat (C:) tag should precede the subsystem profile (P:) tag. > T: git git://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git > F: Documentation/devicetree/ --=20 Sashiko AI review =C2=B7 https://sashiko.dev/#/patchset/20260518091942.2982= 2-2-krzysztof.kozlowski@oss.qualcomm.com?part=3D1