[PATCH v3] dt-bindings: cleanup: fix duplicated 'is is' in YAML docs
Jonathan Cameron
jic23 at kernel.org
Sun Jul 27 06:08:28 PDT 2025
On Thu, 24 Jul 2025 22:38:14 +0530
Sanjay Suthar <sanjaysuthar661996 at gmail.com> wrote:
> On 24/07/25 15:42, Jonathan Cameron wrote:
> > On Tue, 22 Jul 2025 22:35:13 +0530
> > Sanjay Suthar <sanjaysuthar661996 at gmail.com> wrote:
> >
> >> Fix minor grammatical issues by removing duplicated "is" in two devicetree
> >> binding documents:
> >>
> >> - net/amlogic,meson-dwmac.yaml
> >> - iio/dac/ti,dac7612.yaml
> >>
> >> Signed-off-by: Sanjay Suthar <sanjaysuthar661996 at gmail.com>
> > I'd have no problem with the argument that this can go through either
> > tree if there was any interaction between the files or the changes, but
> > here there isn't. This is just causing potential mess if either tree ends
> > up with other changes overlapping this for no benefit. Please split into
> > two patches, one for each subsystem. You should be fine to keep the
> > various tags given here for the new patches.
>
> Thanks for the review. I understand your concern regarding splitting the
> changes. My intention was that, since the fix is a trivial and identical
> grammatical correction across both subsystems, it didn’t seem necessary
> to separate them.
>
> However, I’m open to either approach — keeping it as a single patch or
> splitting it if that's preferred. As I’m still new to this process, I’ll
> defer to @krzysztof and the DT maintainers for the final call on how
> this should be handled.
>
> Please let me know the preferred direction, and I’ll be happy to update
> accordingly.
DT bindings ultimately go through the subsystem trees (in vast majority of
cases) to avoid churn issues and keep them aligned with driver changes.
So split it.
Thanks
Jonathan
>
> Best regards,
>
> Sanjay Suthar
>
More information about the linux-arm-kernel
mailing list