linux-kernel - RE: [PATCH v2 4/4] mtd: spi-nor: fsl-quadspi: extend support for some special requerment.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <AM2PR04MB07229FADBBF527A93CC5410489CA0@AM2PR04MB0722.eurprd04.prod.outlook.com>
Date:	Tue, 12 Jan 2016 04:09:04 +0000
From:	Yao Yuan <yao.yuan@....com>
To:	Han Xu <han.xu@...escale.com>, Yuan Yao <yao.yuan@...escale.com>
CC:	"dwmw2@...radead.org" <dwmw2@...radead.org>,
	"computersforpeace@...il.com" <computersforpeace@...il.com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux-mtd@...ts.infradead.org" <linux-mtd@...ts.infradead.org>
Subject: RE: [PATCH v2 4/4] mtd: spi-nor: fsl-quadspi: extend support for some
 special requerment.

On Tue, Jan 05, 2016 at 05:02:34AM +0800, Han Xu wrote:
> On Thu, Dec 24, 2015 at 07:00:21PM +0800, Yuan Yao wrote:
> > Add extra info in LUT table to support some special requerments.
> > Spansion S25FS-S family flash need some special operations.
> 
> What's the special requirement, detail it.

Ok, we will detail it in next version.

> 
> >
> > Signed-off-by: Yuan Yao <yao.yuan@....com>
> > ---
> > Changed in v2:
> > Update my email to <yao.yuan@....com>
> > ---
> >  drivers/mtd/spi-nor/fsl-quadspi.c | 41
> +++++++++++++++++++++++++++++++++++++--
> >  include/linux/mtd/spi-nor.h       |  4 ++++
> >  2 files changed, 43 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/mtd/spi-nor/fsl-quadspi.c
> > b/drivers/mtd/spi-nor/fsl-quadspi.c
> > index 9ab2b51..081ae85 100644
> > --- a/drivers/mtd/spi-nor/fsl-quadspi.c
> > +++ b/drivers/mtd/spi-nor/fsl-quadspi.c
> > @@ -205,6 +205,8 @@
> > @@ -848,9 +881,13 @@ static int fsl_qspi_write_reg(struct spi_nor
> > *nor, u8 opcode, u8 *buf, int len)  {
> >  	struct fsl_qspi *q = nor->priv;
> >  	int ret;
> > +	u32 to = 0;
> > +
> > +	if (opcode == SPINOR_OP_SPANSION_RDAR)
> > +		memcpy(&to, nor->cmd_buf, 4);
> 
> RADR or WRAR? This is hacking code, why need these two commands?

Sorry, it's should be WRAR.
The command RADR is necessary for Spansion S25FS-S family flash. But the WRAR we didn't use this command now. But it seems we should use it in the future. So we add this command. 

Hi Xu Han, David Woodhouse, Brian Norris

By the way, there are some patches depend on this patch for QSPI Spansion S25FS-S family flash support on ARMv8 platform.
in order to the completeness of the patch sets. I want to just send the patch set for QSPI LS1021A and LS1043A SOC support.
Include: mtd-spi-nor-fsl-quadspi-add-big-endian-support.patch mtd-spi-nor-fsl-quadspi-add-support-for-ls1021a.patch
Those patches add the QSPI support on LS1021A and LS1043A SOC.

And another patch set will send to support Spansion S25FS-S family flash.
Include: mtd-spi-nor-fsl-quadspi-add-support-for-layerscape.patch mtd-spi-nor-fsl-quadspi-extend-support-for-some-spec.patch
And some other patches.
Those patches will add the QSPI Spansion S25FS-S family flash support. This is used for NXP LS2080A SOC.

Is that OK?

Thanks.
Best Regards,
Yuan Yao