Re: [PATCH v3] dt-bindings: cleanup: fix duplicated 'is is' in YAML docs

[Jonathan Cameron <jic23@kernel.org>]

On Thu, 24 Jul 2025 22:38:14 +0530
Sanjay Suthar <sanjaysuthar661996@gmail.com> wrote:

> On 24/07/25 15:42, Jonathan Cameron wrote:
> > On Tue, 22 Jul 2025 22:35:13 +0530
> > Sanjay Suthar <sanjaysuthar661996@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@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
>