Ok, will resend this patch. Thanks, Yuantian
> -----Original Message----- > From: [email protected] [mailto:[email protected]] On Behalf Of Simon Glass > Sent: Saturday, November 19, 2016 3:34 AM > To: Y.T. Tang <[email protected]> > Cc: Bin Meng <[email protected]>; U-Boot Mailing List <u- > [email protected]>; S.H. Xie <[email protected]> > Subject: Re: [PATCH v2] sata: Fixes:d97dc8a0(Separate the non-command > code into its own file) > > 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
