[PATCH] mtd: cfi_cmdset_0002: Use status register where possible

Joakim Tjernlund Joakim.Tjernlund at infinera.com
Sat Oct 31 07:26:18 EDT 2020


On Fri, 2020-10-30 at 19:52 +0100, Miquel Raynal wrote:
> CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.
> 
> 
> Hi Joakim,
> 
> Joakim Tjernlund <Joakim.Tjernlund at infinera.com> wrote on Fri, 30 Oct
> 2020 18:39:35 +0000:
> 
> > On Fri, 2020-10-30 at 18:47 +0100, Miquel Raynal wrote:
> > > 
> > > Hi Joakim,
> > 
> > Hi Miquel
> > 
> > > 
> > > Please Cc the MTD maintainers, not only the list (get_maintainers.pl).
> > 
> > I figure all maintainers are on the list ?
> 
> I personally don't look at the list very often. I expect patches to be
> directed to me (in the current case, Vignesh) when I am concerned.

Added Vignesh

> 
> > 
> > > 
> > > Joakim Tjernlund <joakim.tjernlund at infinera.com> wrote on Thu, 22 Oct
> > > 2020 17:45:06 +0200:
> > > 
> > > > Commit "mtd: cfi_cmdset_0002: Add support for polling status register"
> > > > added support for polling the status rather than using DQ polling.
> > > > However, status register is used only when DQ polling is missing.
> > > > Lets use status register when available as it is superior to DQ polling.
> > > > 
> > > 
> > > I will let vignesh comment about the content (looks fine by me) but you will
> > > need a Fixes tag here.
> > 
> > This is not a Fixes situation, no bug just a hw enabling thing.
> > Also, I would like to see the Status patches be backported to 4.19 as well.
> 
> Backporting features is IMHO not relevant. I guess stable kernel only
> take fixes...

This is not a feature really and the 5.4 stable did get them, I ask 4.19 get them too.


More information about the linux-mtd mailing list