[PATCH 0/2] drivers/tty: delete break after return or goto

Greg Kroah-Hartman gregkh at linuxfoundation.org
Thu Nov 5 03:20:54 EST 2020


On Thu, Nov 05, 2020 at 03:34:55PM +0800, Bernard wrote:
> From: Greg Kroah-Hartman <gregkh at linuxfoundation.org>
> 
> Date: 2020-11-04 19:59:03
> To:  Bernard <bernard at vivo.com>
> Cc:  Jiri Slaby <jirislaby at kernel.org>,Shawn Guo <shawnguo at kernel.org>,Sascha Hauer <s.hauer at pengutronix.de>,Pengutronix Kernel Team <kernel at pengutronix.de>,Fabio Estevam <festevam at gmail.com>,NXP Linux Team <linux-imx at nxp.com>,linux-kernel at vger.kernel.org,linux-serial at vger.kernel.org,linux-arm-kernel at lists.infradead.org,opensource.kernel at vivo.com
> Subject: Re: Re: [PATCH 0/2] drivers/tty: delete break after return or goto>On Wed, Nov 04, 2020 at 07:17:56PM +0800, Bernard wrote:
> >> 
> >> 
> >> From: Greg Kroah-Hartman <gregkh at linuxfoundation.org>
> >> Date: 2020-11-04 19:02:53
> >> To:  Bernard Zhao <bernard at vivo.com>
> >> Cc:  Jiri Slaby <jirislaby at kernel.org>,Shawn Guo <shawnguo at kernel.org>,Sascha Hauer <s.hauer at pengutronix.de>,Pengutronix Kernel Team <kernel at pengutronix.de>,Fabio Estevam <festevam at gmail.com>,NXP Linux Team <linux-imx at nxp.com>,linux-kernel at vger.kernel.org,linux-serial at vger.kernel.org,linux-arm-kernel at lists.infradead.org,opensource.kernel at vivo.com
> >> Subject: Re: [PATCH 0/2] drivers/tty: delete break after return or goto>On Wed, Nov 04, 2020 at 02:53:29AM -0800, Bernard Zhao wrote:
> >> >> This patch sereies optimise code like:
> >> >> {
> >> >> case XXX:
> >> >> 	return XXX;
> >> >> 	break; //The break is meanless, so just delete it.
> >> >> case YYY:
> >> >> 	goto YYY;
> >> >> 	break; //The break is meanless, so just delete it.
> >> >> ......
> >> >> }
> >> >> 
> >> >> Signed-off-by: Bernard Zhao <bernard at vivo.com>
> >> >> 
> >> >> ---
> >> >> Bernard Zhao (2):
> >> >>   drivers/tty/nozomi.c: delete no use break after goto
> >> >>   drivers/tty/serial/imx.c: delete no use break after return
> >> >
> >> >That is not the subject of the patches you sent out, what broke?
> >> 
> >> Hi:
> >> 
> >> I am sorry that I am a little confused:
> >> The patch series`s subject is "drivers/tty: delete break after return or goto"
> >> and the blurb is:
> >> This patch sereies optimise code like:
> >> {
> >> case XXX:
> >> 	return XXX;
> >> 	break; //The break is meanless, so just delete it.
> >> case YYY:
> >> 	goto YYY;
> >> 	break; //The break is meanless, so just delete it.
> >> ......
> >> }
> >> last, the modified files are:
> >> Bernard Zhao (2):
> >>   drivers/tty/nozomi.c: delete no use break after goto
> >>   drivers/tty/serial/imx.c: delete no use break after return
> >> 
> >> Is there something wrong that I didn`t catch?
> >
> >The above lines do not match up with the subject lines of the patches
> >you sent out, so something went wrong.
> 
> 
> Hi, Greg:
> 
> Sorry to bother you.
> I am a newcomer to the community, and this is my first time submitting a patch series.

You might want to start in the drivers/staging/ part of the kernel to
get your bearings and work out these types of things.  It's "easier"
there as the code there needs lots of work and it's set up to handle new
developers like yourself.

> I am sorry that I still don't understand:"The above lines do not match up with the subject lines of the patches you sent out, so something went wrong."
> I compared my patch series with other people`s patch series, as shown in the picture below, they look the seem.
> The only difference is that I made a signature here, so is this the issue that you mean?

the output of the git command that caused those lines to be written was
taken from the subject lines of the patches in your tree.  Yet the
subject lines of the patches you emailed us did not match that at all,
so what you sent is not what you actually had here when you generated
that cover letter.  So something went wrong with your process.

Try deleting all patch files in the directory and generating them again,
and then emailing the series to yourself to verify that everything
matches up properly.

hope this helps,

greg k-h



More information about the linux-arm-kernel mailing list