On 8/13/20 9:27 AM, Patrice CHOTARD wrote: > Hi Christpohe > > On 7/31/20 9:53 AM, Christophe Kerello wrote: >> The chip select defined in the device tree could only be 0 or 1. >> >> Signed-off-by: Christophe Kerello <christophe.kere...@st.com> >> --- >> >> drivers/mtd/nand/raw/stm32_fmc2_nand.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-)
Applied on u-boot-stm/master Thanks >> >> diff --git a/drivers/mtd/nand/raw/stm32_fmc2_nand.c >> b/drivers/mtd/nand/raw/stm32_fmc2_nand.c >> index 3306bd8..2929acf 100644 >> --- a/drivers/mtd/nand/raw/stm32_fmc2_nand.c >> +++ b/drivers/mtd/nand/raw/stm32_fmc2_nand.c >> @@ -846,7 +846,7 @@ static int stm32_fmc2_parse_child(struct stm32_fmc2_nfc >> *fmc2, >> } >> >> for (i = 0; i < nand->ncs; i++) { >> - if (cs[i] > FMC2_MAX_CE) { >> + if (cs[i] >= FMC2_MAX_CE) { >> pr_err("Invalid reg value: %d\n", >> nand->cs_used[i]); >> return -EINVAL; > Reviewed-by: Patrice Chotard <patrice.chot...@st.com> > > Thanks > > Patrice >