Message ID | 1453208838-10306-1-git-send-email-ye.li@nxp.com |
---|---|
State | Rejected |
Delegated to: | Jagannadha Sutradharudu Teki |
Headers | show |
On Tue, Jan 19, 2016 at 09:07:18PM +0800, Ye Li wrote: >The OFFSET_BITS_MASK should mask bit from 0-23. >By using GENMASK(24, 0), when using the fast read common (0xb), a >invalid sf_addr 0x1000000 is produced by swab32(txbuf) & OFFSET_BITS_MASK. > >Signed-off-by: Ye Li <ye.li@nxp.com> >--- > drivers/spi/fsl_qspi.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > >diff --git a/drivers/spi/fsl_qspi.c b/drivers/spi/fsl_qspi.c >index feec3e8..542b6cf 100644 >--- a/drivers/spi/fsl_qspi.c >+++ b/drivers/spi/fsl_qspi.c >@@ -25,7 +25,7 @@ DECLARE_GLOBAL_DATA_PTR; > #define TX_BUFFER_SIZE 0x40 > #endif > >-#define OFFSET_BITS_MASK GENMASK(24, 0) >+#define OFFSET_BITS_MASK GENMASK(23, 0) > > #define FLASH_STATUS_WEL 0x02 Reviewed-by: Peng Fan <peng.fan@nxp.com> > >-- >1.7.4.1 > >_______________________________________________ >U-Boot mailing list >U-Boot@lists.denx.de >http://lists.denx.de/mailman/listinfo/u-boot
On 20 January 2016 at 06:43, Peng Fan <van.freenix@gmail.com> wrote: > On Tue, Jan 19, 2016 at 09:07:18PM +0800, Ye Li wrote: >>The OFFSET_BITS_MASK should mask bit from 0-23. >>By using GENMASK(24, 0), when using the fast read common (0xb), a >>invalid sf_addr 0x1000000 is produced by swab32(txbuf) & OFFSET_BITS_MASK. >> >>Signed-off-by: Ye Li <ye.li@nxp.com> It's already at ML [1] [1] http://git.denx.de/?p=u-boot.git;a=commitdiff;h=8770413f380d07439bc9ad60c6b4d108d5a9adf0 -- Jagan.
diff --git a/drivers/spi/fsl_qspi.c b/drivers/spi/fsl_qspi.c index feec3e8..542b6cf 100644 --- a/drivers/spi/fsl_qspi.c +++ b/drivers/spi/fsl_qspi.c @@ -25,7 +25,7 @@ DECLARE_GLOBAL_DATA_PTR; #define TX_BUFFER_SIZE 0x40 #endif -#define OFFSET_BITS_MASK GENMASK(24, 0) +#define OFFSET_BITS_MASK GENMASK(23, 0) #define FLASH_STATUS_WEL 0x02
The OFFSET_BITS_MASK should mask bit from 0-23. By using GENMASK(24, 0), when using the fast read common (0xb), a invalid sf_addr 0x1000000 is produced by swab32(txbuf) & OFFSET_BITS_MASK. Signed-off-by: Ye Li <ye.li@nxp.com> --- drivers/spi/fsl_qspi.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)