From mboxrd@z Thu Jan 1 00:00:00 1970 From: Quentin Schulz Subject: Re: [PATCH] net: phy: mscc: fix locking in vsc85xx_default_config Date: Tue, 20 Nov 2018 15:17:05 +0100 Message-ID: <20181120141705.44odry44hhdy3cge@qschulz> References: <20181008100728.24959-1-quentin.schulz@bootlin.com> <20181008100728.24959-2-quentin.schulz@bootlin.com> <20181120135508.paxzs63cqy2llc7d@qschulz> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="n5xcw7r3bck7sftm" Cc: davem@davemloft.net, andrew@lunn.ch, f.fainelli@gmail.com, allan.nielsen@microchip.com, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, thomas.petazzoni@bootlin.com, alexandre.belloni@bootlin.com, linux-riscv@lists.infradead.org To: Andreas Schwab Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org --n5xcw7r3bck7sftm Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi Andreas, On Tue, Nov 20, 2018 at 03:01:25PM +0100, Andreas Schwab wrote: > On Nov 20 2018, Quentin Schulz wrote: >=20 > > You also need to prefix your patch with [PATCH net-next] instead of only > > [PATCH]. This is specific to the net subsystem. >=20 > Why next? This is a bug fix that needs to go in now, not next. >=20 Indeed. Use [PATCH net] then. Please refer to https://www.kernel.org/doc/Documentation/networking/netdev-= FAQ.txt > > Use scripts/get_maintainer.pl on your patch to get the people to put in > > Cc and in To. >=20 > The same people as the original patch the added the bug. Why would it > be different now? >=20 First, people may have changed mail addresses (especially maintainers) and that is picked by get_maintainer. Second, other people may have worked on this driver since the patch was introduced and they might want to know about this change too and have their word on it. This is what happened with this driver. Gustavo and Heiner sent patches since then and they have already been merged. This is also part of the contribution process which is defined here: https://www.kernel.org/doc/html/v4.17/process/submitting-patches.html You have not answered my question yet. Does this fix your issue, yes or no? Quentin --n5xcw7r3bck7sftm Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCgAdFiEEXeEYjDsJh38OoyMzhLiadT7g8aMFAlv0F2EACgkQhLiadT7g 8aOEgg//SEW1tvlbP8HIMI9Azhv0DP9ED+wigV9oT0h0ZxbRMpVqwKOSlT2sJX8m Q1qYCfyePxN5+F7ALFT4iYHmneUqPTkX8fbDf+Ij91O+O77lk+kgf69TdJhIugaM oH68FXs9mdkZZElqxMG7iuavsloLOaz8nPlbPpLujWkD+ggVUlz3m4+d32T5jY8P nTxVF/cij8U2gjVBHyHrTde7OpnD1tQmGoL3C+yLSUpJsuWBfxp8twq3g/YXNHmb 5k8YPl23lNSvscbh4j41Vo/yN326Ooager5AViiknhAYfPLbukotSyzH/hAx9KcK SRnuqMnBNlgKZkaRGuWYGqbIf+Rq14f8DLntOCyBUP/pdEHApHAlPLzwF/qHS6k9 QsW1Rxck6fDmmUGdYxlRsJLpjkQjeZmGgn1Q24ZbhxFC6s8Z7f5cmgeL1WBVdkhH AKhWSwmYB3Bsp8OtFcsKvNC/6jMGBU4BAihVO6DUhD8t/OyozJdaAy/pu614uIYN IcnZ5dBXZ+7Jh2sLC1M/Gn82g0XPr7ugf/OQMgzsh3n22Ybpo9ziguz5TeiYFaXK qaB4BoNo003SUR53w9Iwy5KVGYiCs5W7ZdgcCqrkK4Ht9b7uPzBaqYwG0evHHhgw xL/6FgQYLxCPn9Ccg/P6ytLiC+MRN7FXzuoldiNtlmNQeHvzUmI= =85Mb -----END PGP SIGNATURE----- --n5xcw7r3bck7sftm--