Hi,

> -----Original Message-----
> From: U-Boot [mailto:u-boot-boun...@lists.denx.de] On Behalf Of Jagan Teki
> Sent: Wednesday, November 16, 2016 9:33 AM
> To: u-boot@lists.denx.de
> Cc: Jagan Teki <ja...@openedev.com>
> Subject: [U-Boot] [PATCH v6 03/21] sf: sandbox: Use JEDEC_MFR|ID in id
> exctract
> 
> Instead of extracting id's separately better to use JEDEC_MFR|ID for code
> simplicity.
> 
> Cc: Bin Meng <bmeng...@gmail.com>
> Signed-off-by: Jagan Teki <ja...@openedev.com>
> Reviewed-by: Simon Glass <s...@chromium.org>
> Reviewed-by: Jagan Teki <ja...@openedev.com>
> Tested-by: Jagan Teki <ja...@openedev.com>
> ---
>  drivers/mtd/spi/sandbox.c | 5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/mtd/spi/sandbox.c b/drivers/mtd/spi/sandbox.c index
> d68ee4a..09ce783 100644
> --- a/drivers/mtd/spi/sandbox.c
> +++ b/drivers/mtd/spi/sandbox.c
> @@ -359,9 +359,8 @@ static int sandbox_sf_xfer(struct udevice *dev,
> unsigned int bitlen,
>                       debug(" id: off:%u tx:", sbsf->off);
>                       if (sbsf->off < IDCODE_LEN) {
>                               /* Extract correct byte from ID 0x00aabbcc */
> -                             id = ((((sbsf->data)->id[0]) << 16) |
> -                                     (((sbsf->data)->id[1]) << 8 |
> -                                     ((sbsf->data)->id[2]))) >>
> +                             id = ((JEDEC_MFR(sbsf->data) << 16) |
> +                                     JEDEC_ID(sbsf->data)) >>
As mentioned in 01/21, no magic numbers here as well. Otherwise, Reviewed-by

Regards,
Siva
>                                       (8 * (IDCODE_LEN - 1 - sbsf->off));
>                       } else {
>                               id = 0;
> --
> 1.9.1
> 
> _______________________________________________
> U-Boot mailing list
> U-Boot@lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to