Re: [PATCH] mtd: devices: fix mchp23k256 read and write

[Angelo Dureghello <angelo.dureghello@xxxxxxxxxxx>]

Hi all,

many thanks.

Let me know if i need to prepare a v1 with proper tags in case.

Regards,
Angelo

On Wed, Nov 6, 2019 at 12:17 AM Andrew Lunn <andrew@xxxxxxx> wrote:
>
> On Tue, Nov 05, 2019 at 07:41:22PM +0100, Miquel Raynal wrote:
> > Hi Angelo,
> >
> > + Andrew
> >
> > Angelo Dureghello <angelo.dureghello@xxxxxxxxxxx> wrote on Wed, 30 Oct
> > 2019 12:39:57 +0100:
> >
> > > Due to the use of sizeof(), command size set for the spi transfer
> > > was wrong. Driver was sending and receiving always 1 byte less
> > > and especially on write, it was hanging.
> > >
> > > echo -n -e "\\x1\\x2\\x3\\x4" > /dev/mtd1
> > >
> > > And read part too now works as expected.
> > >
> > > hexdump -C -n16 /dev/mtd1
> > > 00000000  01 02 03 04 ab f3 ad c2  ab e3 f4 36 dd 38 04 15
> > > 00000010
> > >
> >
> > Looks fine by me, just added Andrew in case he wants to review the
> > patch as he was the original committer.
> >
> > Anyway you're missing a Cc: stable and Fixes tag on
> > 5dc17fa6fb70 ("mtd: mchp23k256: Add driver for this SPI SRAM device")
>
> Hi Miqučl
>
> This is the wrong fixes tag. The correct one is
>
> Fixes: 4379075a870b ("mtd: mchp23k256: Add support for mchp23lcv1024")
>
> Reviewed-by: Andrew Lunn <andrew@xxxxxxx>
>
>     Andrew



-- 
Angelo Dureghello
Timesys
email: angelo.dureghello@xxxxxxxxxxx
cell.:  +39 388 8550663


-- 
Angelo Dureghello
Timesys
email: angelo.dureghello@xxxxxxxxxxx
cell.:  +39 388 8550663

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/