Hi Tang, On 16 November 2016 at 23:54, <[email protected]> wrote: > From: Tang Yuantian <[email protected]> > > Commit d97dc8a0 separated the non-command code into its own file > which caused variable sata_curr_device can not be set to a correct > value. > > Before commit d97dc8a0, variable sata_curr_device can be set > correctly in sata_initialize(). > After commit d97dc8a0, sata_initialize() is moved out to its own file. > Accordingly, sata_curr_device is removed from sata_initialize() too. > This caused sata_curr_device never gets a chance to be set properly > which prevent other command from being executed. > > This patch sets variable sata_curr_device properly. > > Signed-off-by: Tang Yuantian <[email protected]> > --- > v2: > - refined this patch and updated the commit title and message
The 'Fixes' tag should go at the end next to your sign-off. See this commit for example: commit 623c5cc4ad4e992b53a227c2391eface5aaecc92 Author: Simon Glass <[email protected]> Date: Sun Nov 6 16:07:40 2016 -0700 x86: Correct a build warning in x86 tables There is a build warning for three x86 boards since write_smbios_table_wrapper() is not used. Fix it. Fixes: e824cf3f (smbios: Allow compilation on 64bit systems) Signed-off-by: Simon Glass <[email protected]> Regards, Simon _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
