Received-SPF: pass (google.com: domain of linux-kernel+bounces-83340-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Date: Tue, 27 Feb 2024 14:45:05 +0100
From: Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= <u.kleine-koenig@pengutronix.de>
To: Mark Brown <broonie@kernel.org>
Cc: Rob Herring <robh@kernel.org>, kernel test robot <lkp@intel.com>, 
	Stephen Warren <swarren@wwwdotorg.org>, linux-kernel@vger.kernel.org, linux-spi@vger.kernel.org, 
	H Hartley Sweeten <hsweeten@visionengravers.com>, kernel@pengutronix.de, 
	Jonathan Cameron <Jonathan.Cameron@huawei.com>
Subject: Re: [PATCH 2/3] spi: ppc4xx: Fix fallout from rename in struct
 spi_bitbang
Message-ID: <6nqpvqodqhk5vwwqgdez2lwsdlh4xrzfstkaf7o2mjz46ub2xf@zjacuzux5jdm>
References: <20240210164006.208149-5-u.kleine-koenig@pengutronix.de>
 <20240210164006.208149-7-u.kleine-koenig@pengutronix.de>
 <y2my7hxrpnwg72ols6a5w7n6zqz2yaxtswq4zlv6xpguiyaunm@tguc7ua3ypa5>
 <76fcc5ec-0180-4f75-aaac-5ae74f2f687a@sirena.org.uk>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
	protocol="application/pgp-signature"; boundary="iqcog6wdvmi5qjne"
Content-Disposition: inline
In-Reply-To: <76fcc5ec-0180-4f75-aaac-5ae74f2f687a@sirena.org.uk>


--iqcog6wdvmi5qjne
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hello,

On Tue, Feb 27, 2024 at 12:50:15PM +0000, Mark Brown wrote:
> On Tue, Feb 27, 2024 at 08:23:06AM +0100, Uwe Kleine-K=F6nig wrote:
>=20
> > Assuming we don't want to have this problem in v6.8, I suggest to revert
> > de4af897ddf2 and reapply it on top of your next branch.
>=20
> BTW the issue here is that you sent this without comment in the middle
> of a series of fixes the other two of which *do* apply to mainline,
> ideally it would have just been sent separately since it needs to go
> separately but if you *are* going to send a single series like this
> things that are -next only should go after any fixes that are for
> mainline.

I expected that adding Fixes lines is enough documentation but I agree
that in retrospect it would have been a good idea to mention the
expected target branch for each patch. I'm willing to take half of the
blame you assigned me as in retrospect double checking the Fixes lines
or doing a compile test of the ppc4xx driver would also have been a good
idea for you as maintainer applying the patches. Sorry for my
contribution to this problem. I only looked at next when I sent the
patches and wasn't aware of the trip wire that git applies patch 2 just
fine to mainline though it's not right to put it there.

I ordered patch 3 at the end because I didn't consider this an urgent
fix as it only addresses a W=3D1 warning that we lived with for over 10
years since v3.11-rc1.

Best regards
Uwe

--=20
Pengutronix e.K.                           | Uwe Kleine-K=F6nig            |
Industrial Linux Solutions                 | https://www.pengutronix.de/ |

--iqcog6wdvmi5qjne
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQEzBAABCgAdFiEEP4GsaTp6HlmJrf7Tj4D7WH0S/k4FAmXd52AACgkQj4D7WH0S
/k451Qf/a7kLmcIlWsKBRYXJmSPkhYsGW8ps8c3C6Lrn7GxLAP6GVOPWYMQYF0Aq
kCZ9MlEV0X1P21+7Jv/mTAOxGF8toa95j9rzaq/iB9aGyriH6CeMwYW9ABbXbRkK
0w8M5s9onPdansOiLZ3kP7H1Y9JshIgXW9i6qrtFJULeDfourl41rJDYsLZdp6KP
lkae1J18STzFvi4mH2lAbMxYtxX38pv+jZBk722zBmN0EsIkZwv4cQxSzrEwmwos
DuAgFcjjw84iLhRBHW3T6n1FEYbisrdVnUdFuXY/0WUIcO8hvtoW+Qaqghzwe4X2
Rol1c3SjKcGcxr82TKUe814ScDZoFA==
=pDHU
-----END PGP SIGNATURE-----

--iqcog6wdvmi5qjne--