Ok, will resend this patch. Thanks, Yuantian
> -----Original Message----- > From: s...@google.com [mailto:s...@google.com] On Behalf Of Simon Glass > Sent: Saturday, November 19, 2016 3:34 AM > To: Y.T. Tang <yuantian.t...@nxp.com> > Cc: Bin Meng <bmeng...@gmail.com>; U-Boot Mailing List <u- > b...@lists.denx.de>; S.H. Xie <shaohui....@nxp.com> > 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, <yuantian.t...@nxp.com> wrote: > > From: Tang Yuantian <yuantian.t...@nxp.com> > > > > 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 <yuantian.t...@nxp.com> > > --- > > 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 <s...@chromium.org> > 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 <s...@chromium.org> > > Regards, > Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot