Re: [U-Boot] [RESEND PATCH] mtd: nand: zynq: Add support for the NAND lock/unlock operation

2017-11-16 Thread Michal Simek
On 7.11.2017 03:16, Wilson Lee wrote:
> From: Joe Hershberger 
> 
> Zynq NAND driver is not support for NAND lock or unlock operation.
> Hence, accidentally write into the critical NAND region might cause
> data corruption to occur.
> 
> This commit is to add NAND lock/unlock command into NAND SMC register
> set for NAND lock/unlock operaion.
> 
> Signed-off-by: Joe Hershberger 
> Signed-off-by: Keng Soon Cheah 
> Cc: Chen Yee Chew 
> Cc: Siva Durga Prasad Paladugu 
> Cc: Michal Simek 
> Cc: Scott Wood 
> ---
>  drivers/mtd/nand/zynq_nand.c | 12 
>  1 file changed, 12 insertions(+)

Applied.

Thanks,
Michal
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


[U-Boot] [RESEND PATCH] mtd: nand: zynq: Add support for the NAND lock/unlock operation

2017-11-06 Thread Wilson Lee
From: Joe Hershberger 

Zynq NAND driver is not support for NAND lock or unlock operation.
Hence, accidentally write into the critical NAND region might cause
data corruption to occur.

This commit is to add NAND lock/unlock command into NAND SMC register
set for NAND lock/unlock operaion.

Signed-off-by: Joe Hershberger 
Signed-off-by: Keng Soon Cheah 
Cc: Chen Yee Chew 
Cc: Siva Durga Prasad Paladugu 
Cc: Michal Simek 
Cc: Scott Wood 
---
 drivers/mtd/nand/zynq_nand.c | 12 
 1 file changed, 12 insertions(+)

diff --git a/drivers/mtd/nand/zynq_nand.c b/drivers/mtd/nand/zynq_nand.c
index 948f059..484f1bb 100644
--- a/drivers/mtd/nand/zynq_nand.c
+++ b/drivers/mtd/nand/zynq_nand.c
@@ -81,6 +81,13 @@
 #define ZYNQ_NAND_ECC_BUSY (1 << 6)/* ECC block is busy */
 #define ZYNQ_NAND_ECC_MASK 0x00FF  /* ECC value mask */
 
+#ifndef NAND_CMD_LOCK_TIGHT
+#define NAND_CMD_LOCK_TIGHT 0x2c
+#endif
+
+#ifndef NAND_CMD_LOCK_STATUS
+#define NAND_CMD_LOCK_STATUS 0x7a
+#endif
 
 /* SMC register set */
 struct zynq_nand_smc_regs {
@@ -141,6 +148,11 @@ static const struct zynq_nand_command_format 
zynq_nand_commands[] = {
{NAND_CMD_PARAM, NAND_CMD_NONE, 1, 0},
{NAND_CMD_GET_FEATURES, NAND_CMD_NONE, 1, 0},
{NAND_CMD_SET_FEATURES, NAND_CMD_NONE, 1, 0},
+   {NAND_CMD_LOCK, NAND_CMD_NONE, 0, 0},
+   {NAND_CMD_LOCK_TIGHT, NAND_CMD_NONE, 0, 0},
+   {NAND_CMD_UNLOCK1, NAND_CMD_NONE, 3, 0},
+   {NAND_CMD_UNLOCK2, NAND_CMD_NONE, 3, 0},
+   {NAND_CMD_LOCK_STATUS, NAND_CMD_NONE, 3, 0},
{NAND_CMD_NONE, NAND_CMD_NONE, 0, 0},
/* Add all the flash commands supported by the flash device */
 };
-- 
2.7.4

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot