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

2012-09-28 Thread Tom Rini
On Fri, Sep 28, 2012 at 08:12:21PM +0200, Marek Vasut wrote: > 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 t

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

2012-09-28 Thread Marek Vasut
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 controlle

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

2012-09-28 Thread 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