[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <2024061745-enzyme-penny-6c12@gregkh>
Date: Mon, 17 Jun 2024 12:41:42 +0200
From: Greg KH <gregkh@...uxfoundation.org>
To: "Kumar, Udit" <u-kumar1@...com>
Cc: vigneshr@...com, nm@...com, tony@...mide.com, jirislaby@...nel.org,
ronald.wahl@...itan.com, thomas.richard@...tlin.com,
tglx@...utronix.de, linux-kernel@...r.kernel.org,
linux-serial@...r.kernel.org, ilpo.jarvinen@...ux.intel.com
Subject: Re: [PATCH v2] serial: 8250_omap: Implementation of Errata i2310
On Mon, Jun 17, 2024 at 03:17:54PM +0530, Kumar, Udit wrote:
> Hi Greg
>
> On 6/17/2024 11:52 AM, Greg KH wrote:
> > On Mon, Jun 17, 2024 at 10:52:53AM +0530, Udit Kumar wrote:
> > > As per Errata i2310[0], Erroneous timeout can be triggered,
> > > if this Erroneous interrupt is not cleared then it may leads
> > > to storm of interrupts, therefore apply Errata i2310 solution.
> > >
> > > [0] https://www.ti.com/lit/pdf/sprz536 page 23
> > >
> > > Fixes: b67e830d38fa ("serial: 8250: 8250_omap: Fix possible interrupt storm on K3 SoCs")
> > > Signed-off-by: Udit Kumar <u-kumar1@...com>
> > > ---
> > > Test logs
> > > https://gist.github.com/uditkumarti/7a7ab6994581bbb09cf1a4271c592d8e
> > >
> > > Change logs
> > > Changes in v2:
> > > - Added Fixes Tag and typo correction in commit message
> > > - Corrected bit position to UART_OMAP_EFR2_TIMEOUT_BEHAVE
> > > Link to v1
> > > https://lore.kernel.org/all/20240614061314.290840-1-u-kumar1@ti.com/
> > >
> > > drivers/tty/serial/8250/8250_omap.c | 25 ++++++++++++++++++++-----
> > > 1 file changed, 20 insertions(+), 5 deletions(-)
> > Hi,
> >
> > This is the friendly patch-bot of Greg Kroah-Hartman. You have sent him
> > a patch that has triggered this response. He used to manually respond
> > to these common problems, but in order to save his sanity (he kept
> > writing the same thing over and over, yet to different people), I was
> > created. Hopefully you will not take offence and will fix the problem
> > in your patch and resubmit it so that it can be accepted into the Linux
> > kernel tree.
> >
> > You are receiving this message because of the following common error(s)
> > as indicated below:
> >
> > - You have marked a patch with a "Fixes:" tag for a commit that is in an
> > older released kernel, yet you do not have a cc: stable line in the
> > signed-off-by area at all, which means that the patch will not be
> > applied to any older kernel releases. To properly fix this, please
> > follow the documented rules in the
> > Documentation/process/stable-kernel-rules.rst file for how to resolve
> > this.
> >
> > If you wish to discuss this problem further, or you have questions about
> > how to resolve this issue, please feel free to respond to this email and
> > Greg will reply once he has dug out from the pending patches received
> > from other developers.
>
>
> I realized the same due to other patches, this not getting applied cleanly
> on stable (using git am).
>
> Let me know if you are ok, I will port this to stable kernel, once this is
> merged into Linux tree.
Ok, but if you properly put a cc: stable line on your commit, the stable
team will notify you when/if something fails to apply, no need for you
to try to remember to do it sometime in the next few months. Please do
that, just adding a "Fixes:" tag never ensures that this will happen at
all.
thanks,
greg k-h
Powered by blists - more mailing lists