* [U-Boot] [PATCH] fsl_qspi: fix address mask issue
@ 2016-01-19 13:07 Ye Li
2016-01-20 1:13 ` Peng Fan
0 siblings, 1 reply; 3+ messages in thread
From: Ye Li @ 2016-01-19 13:07 UTC (permalink / raw)
To: u-boot
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
--
1.7.4.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [U-Boot] [PATCH] fsl_qspi: fix address mask issue
2016-01-19 13:07 [U-Boot] [PATCH] fsl_qspi: fix address mask issue Ye Li
@ 2016-01-20 1:13 ` Peng Fan
2016-02-02 16:22 ` Jagan Teki
0 siblings, 1 reply; 3+ messages in thread
From: Peng Fan @ 2016-01-20 1:13 UTC (permalink / raw)
To: u-boot
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 at lists.denx.de
>http://lists.denx.de/mailman/listinfo/u-boot
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-02-02 16:22 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-01-19 13:07 [U-Boot] [PATCH] fsl_qspi: fix address mask issue Ye Li
2016-01-20 1:13 ` Peng Fan
2016-02-02 16:22 ` Jagan Teki
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox