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

From: Sanjay Suthar <sanjaysuthar661996@gmail.com>
To: Jonathan Cameron <jic23@kernel.org>
Cc: linux-kernel@vger.kernel.org, devicetree@vger.kernel.org,
	linux-iio@vger.kernel.org, netdev@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-amlogic@lists.infradead.org, ribalda@kernel.org,
	dlechner@baylibre.com, nuno.sa@analog.com, andy@kernel.org,
	robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org,
	andrew+netdev@lunn.ch, davem@davemloft.net, edumazet@google.com,
	kuba@kernel.org, pabeni@redhat.com, neil.armstrong@linaro.org,
	khilman@baylibre.com, jbrunet@baylibre.com,
	martin.blumenstingl@googlemail.com
Subject: Re: [PATCH v3] dt-bindings: cleanup: fix duplicated 'is is' in YAML docs
Date: Thu, 24 Jul 2025 22:38:14 +0530	[thread overview]
Message-ID: <8f4358e8-ecd6-4d86-8326-25d21c3a4ea2@gmail.com> (raw)
In-Reply-To: <20250724111247.669d6955@jic23-huawei>

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.

Best regards,

Sanjay Suthar

_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic