[U-Boot] [PATCH v2] sata: Fixes:d97dc8a0(Separate the non-command code into its own file)

[Simon Glass]

Hi Tang,

On 16 November 2016 at 23:54,  <yuantian.tang at nxp.com> wrote:
> From: Tang Yuantian <Yuantian.Tang at 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.tang at 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 <sjg at 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 <sjg at chromium.org>

Regards,
Simon