Received-SPF: pass (google.com: domain of linux-kernel+bounces-100542-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Precedence: bulk
MIME-Version: 1.0
References: <20240311140435.34329-1-animeshagarwal28@gmail.com> <045ae9cf-104d-472e-94f0-ea7cd5298214@linaro.org>
In-Reply-To: <045ae9cf-104d-472e-94f0-ea7cd5298214@linaro.org>
From: Animesh Agarwal <animeshagarwal28@gmail.com>
Date: Tue, 12 Mar 2024 22:19:42 +0530
Message-ID: <CAE3Oz83x54L9j0TOJmM1C8QGO8Ed=ZY107a0eSL2dO7dXOKfRg@mail.gmail.com>
Subject: Re: [PATCH v7] dt-bindings: imx-pata: Convert to dtschema
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc: Damien Le Moal <dlemoal@kernel.org>, Rob Herring <robh+dt@kernel.org>, 
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Conor Dooley <conor+dt@kernel.org>, 
	Shawn Guo <shawnguo@kernel.org>, Sascha Hauer <s.hauer@pengutronix.de>, 
	Pengutronix Kernel Team <kernel@pengutronix.de>, Fabio Estevam <festevam@gmail.com>, 
	NXP Linux Team <linux-imx@nxp.com>, linux-ide@vger.kernel.org, devicetree@vger.kernel.org, 
	linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Tue, Mar 12, 2024 at 9:55=E2=80=AFPM Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:
> Thank you for improving the commit msg. It looks a bit oddly wrapped. In
> the future, please wrap commit message according to Linux coding style /
> submission process (neither too early nor over the limit):
> https://elixir.bootlin.com/linux/v6.4-rc1/source/Documentation/process/su=
bmitting-patches.rst#L597

I was just trying to get the lines wrapped within 75 characters
without making it harder to read. I'll follow getting line wrapped at
75 columns more strictly now. Thanks for pointing it out.

> Oh my... missing space after coma, but more important: what is happening
> here? The space was here before in previous versions, so you removed it!
> At least till v4 it was correct and bam - now it is not. Why? It's the
> same confusing as new line appearing suddenly in v6.
>
> It looks like between each revisions of patchset some random changes
> appear in your code. How is it possible? Don't you work with Git? You
> change file, git add -p and git commit --amend. Why unrelated changes
> keep happening?
>
> While the actual space is a nitpick, it points to the fact that your
> patch experiences some changes in an unexpected way and we cannot trust
> that you did only what you claim in changelog.

As you pointed out I indeed made a mistake at v5 by removing the space
and adding a line at the end. I removed the line in v7 and didn't add
the space back. I acknowledge my mistake and will be more careful.

Warm regards,
Animesh