[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