Re: [U-Boot] [PATCH][v2] mtd: nand: fsl_ifc: Fix nand waitfunc return value

2018-05-01 Thread Jagdish Gediya
Hi York, > -Original Message- > From: York Sun > Sent: Friday, April 27, 2018 8:43 AM > To: Prabhakar Kushwaha > Cc: Jagdish Gediya ; u-boot@lists.denx.de; > o...@buserror.net > Subject: Re: [U-Boot] [PATCH][v2] mtd: nand: fsl_ifc: Fix nand waitfunc return > v

Re: [U-Boot] [PATCH][v2] mtd: nand: fsl_ifc: Fix nand waitfunc return value

2018-04-26 Thread York Sun
> On Apr 26, 2018, at 19:58, Prabhakar Kushwaha > wrote: >> > > Even though there are 2 bytes. > But NAND sub-system only consume 1 byte because all NAND flash has 1 byte > status register. > Then the commit message should describe the position of primary status field, instead of sayi

Re: [U-Boot] [PATCH][v2] mtd: nand: fsl_ifc: Fix nand waitfunc return value

2018-04-26 Thread Prabhakar Kushwaha
Hi York, > -Original Message- > From: U-Boot [mailto:u-boot-boun...@lists.denx.de] On Behalf Of York Sun > Sent: Friday, April 27, 2018 2:31 AM > To: Jagdish Gediya ; u-boot@lists.denx.de > Cc: o...@buserror.net > Subject: Re: [U-Boot] [PATCH][v2] mtd: nand: fsl_ifc:

Re: [U-Boot] [PATCH][v2] mtd: nand: fsl_ifc: Fix nand waitfunc return value

2018-04-26 Thread York Sun
On 03/20/2018 10:24 PM, Jagdish Gediya wrote: > As per the IFC hardware manual, Most significant 2 bytes in > nand_fsr register are the outcome of NAND READ STATUS command. > > So status value need to be shifted and aligned as per the nand > framework requirement. > > Signed-off-by: Jagdish Gediy

[U-Boot] [PATCH][v2] mtd: nand: fsl_ifc: Fix nand waitfunc return value

2018-03-20 Thread Jagdish Gediya
As per the IFC hardware manual, Most significant 2 bytes in nand_fsr register are the outcome of NAND READ STATUS command. So status value need to be shifted and aligned as per the nand framework requirement. Signed-off-by: Jagdish Gediya Reviewed-by: Prabhakar Kushwaha --- Changes for v2: