[U-Boot] [PATCH 1/2] remove unnecessary code in ata_piix

Marek Vasut marex at denx.de
Fri Sep 28 20:12:21 CEST 2012


Dear Pavel Herrmann,

> We set sata_curr_device to 0 right after returning from init_sata(), so
> there's no point in setting it to the last scanned driver at this point.
> Note: there are more duplicities with cmd_sata, but those might be
> required, as the code seems to reset the entire controller on every scan,
> ignoring the requested port number.

I think that code is valid. It configures the sata_curr_device to valid value in 
case this was not called from the context of the command. No?

I think it can be pulled from the loops above the return 0 though.

> Signed-off-by: Pavel Herrmann <morpheus.ibis at gmail.com>
> ---
>  drivers/block/ata_piix.c | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/drivers/block/ata_piix.c b/drivers/block/ata_piix.c
> index c81d11a..1c3ab8a 100644
> --- a/drivers/block/ata_piix.c
> +++ b/drivers/block/ata_piix.c
> @@ -204,9 +204,6 @@ init_sata (int dev)
>  				dev_print (&sata_dev_desc[devno]);
>  				/* initialize partition type */
>  				init_part (&sata_dev_desc[devno]);
> -				if (sata_curr_device < 0)
> -					sata_curr_device =
> -					    i * CONFIG_SYS_SATA_DEVS_PER_BUS + 
j;
>  			}
>  		}
>  	}

Best regards,
Marek Vasut


More information about the U-Boot mailing list