* [PATCH] ARM: zynq: Fix invalid check on NAND_CMD_NONE.
@ 2020-04-24 10:25 Michal Simek
2020-04-30 7:37 ` Michal Simek
0 siblings, 1 reply; 2+ messages in thread
From: Michal Simek @ 2020-04-24 10:25 UTC (permalink / raw)
To: u-boot
From: Patrick van Gelder <patrick.vangelder@nl.bosch.com>
The end_cmd field in the variables cmd_phase_addr and data_phase_addr
contains the value 0xFF when the end_cmd equals NAND_CMD_NONE. This
should be 0x00.
This is caused by comparing NAND_CMD_NONE (int) with end_cmd (u8).
end_cmd will be promoted by the int value -1 and therefore is not equal
to 0xFF. Solved by casting NAND_CMD_NONE to u8 which will avoid int
promotion.
Signed-off-by: Patrick van Gelder <patrick.vangelder@nl.bosch.com>
Reviewed-by: T Karthik Reddy <t.karthik.reddy@xilinx.com>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---
drivers/mtd/nand/raw/zynq_nand.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mtd/nand/raw/zynq_nand.c b/drivers/mtd/nand/raw/zynq_nand.c
index 0aea83dac0e6..394129741831 100644
--- a/drivers/mtd/nand/raw/zynq_nand.c
+++ b/drivers/mtd/nand/raw/zynq_nand.c
@@ -845,7 +845,7 @@ static void zynq_nand_cmd_function(struct mtd_info *mtd, unsigned int command,
if (curr_cmd->end_cmd_valid == ZYNQ_NAND_CMD_PHASE)
end_cmd_valid = 1;
- if (curr_cmd->end_cmd == NAND_CMD_NONE)
+ if (curr_cmd->end_cmd == (u8)NAND_CMD_NONE)
end_cmd = 0x0;
else
end_cmd = curr_cmd->end_cmd;
--
2.26.2
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [PATCH] ARM: zynq: Fix invalid check on NAND_CMD_NONE.
2020-04-24 10:25 [PATCH] ARM: zynq: Fix invalid check on NAND_CMD_NONE Michal Simek
@ 2020-04-30 7:37 ` Michal Simek
0 siblings, 0 replies; 2+ messages in thread
From: Michal Simek @ 2020-04-30 7:37 UTC (permalink / raw)
To: u-boot
p? 24. 4. 2020 v 12:25 odes?latel Michal Simek <michal.simek@xilinx.com> napsal:
>
> From: Patrick van Gelder <patrick.vangelder@nl.bosch.com>
>
> The end_cmd field in the variables cmd_phase_addr and data_phase_addr
> contains the value 0xFF when the end_cmd equals NAND_CMD_NONE. This
> should be 0x00.
>
> This is caused by comparing NAND_CMD_NONE (int) with end_cmd (u8).
> end_cmd will be promoted by the int value -1 and therefore is not equal
> to 0xFF. Solved by casting NAND_CMD_NONE to u8 which will avoid int
> promotion.
>
> Signed-off-by: Patrick van Gelder <patrick.vangelder@nl.bosch.com>
> Reviewed-by: T Karthik Reddy <t.karthik.reddy@xilinx.com>
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
> ---
>
> drivers/mtd/nand/raw/zynq_nand.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/mtd/nand/raw/zynq_nand.c b/drivers/mtd/nand/raw/zynq_nand.c
> index 0aea83dac0e6..394129741831 100644
> --- a/drivers/mtd/nand/raw/zynq_nand.c
> +++ b/drivers/mtd/nand/raw/zynq_nand.c
> @@ -845,7 +845,7 @@ static void zynq_nand_cmd_function(struct mtd_info *mtd, unsigned int command,
> if (curr_cmd->end_cmd_valid == ZYNQ_NAND_CMD_PHASE)
> end_cmd_valid = 1;
>
> - if (curr_cmd->end_cmd == NAND_CMD_NONE)
> + if (curr_cmd->end_cmd == (u8)NAND_CMD_NONE)
> end_cmd = 0x0;
> else
> end_cmd = curr_cmd->end_cmd;
> --
> 2.26.2
>
Applied.
M
--
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP/Versal SoCs
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-04-30 7:37 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-24 10:25 [PATCH] ARM: zynq: Fix invalid check on NAND_CMD_NONE Michal Simek
2020-04-30 7:37 ` Michal Simek
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.