[U-Boot] [PATCH 1/3] dm: ahci: Add a generic PCI-based AHCI driver

[Simon Glass]

Hi Bin,

On 13 July 2017 at 03:22, Simon Glass <sjg at chromium.org> wrote:
> Hi Bin,
>
> On 6 July 2017 at 23:08, Bin Meng <bmeng.cn at gmail.com> wrote:
>> Hi Simon,
>>
>> On Wed, Jun 21, 2017 at 1:00 PM, Bin Meng <bmeng.cn at gmail.com> wrote:
>>> Hi Simon,
>>>
>>> On Wed, Jun 21, 2017 at 11:29 AM, Simon Glass <sjg at chromium.org> wrote:
>>>> Hi Bin,
>>>>
>>>> On 17 June 2017 at 07:35, Bin Meng <bmeng.cn at gmail.com> wrote:
>>>>> This adds support for PCI-based AHCI controller and enables it on
>>>>> x86 by default.
>>>>>
>>>>> Signed-off-by: Bin Meng <bmeng.cn at gmail.com>
>>>>> ---
>>>>>
>>>>>  drivers/ata/Kconfig    |  7 +++++++
>>>>>  drivers/ata/Makefile   |  1 +
>>>>>  drivers/ata/ahci-pci.c | 42 ++++++++++++++++++++++++++++++++++++++++++
>>>>>  3 files changed, 50 insertions(+)
>>>>>  create mode 100644 drivers/ata/ahci-pci.c
>>>>
>>>> I believe that if the device tree specifies this driver then it will
>>>> override the generic PCI probing done here. Is that right?
>>>>
>>>
>>> That's my understanding as well but I have no target to test. Can you
>>> test that on Chromebook Link?
>>
>> Will this series be applied to u-boot-dm?
>
> Yes, just got to testing this. Had some problems booting the device.
>
> Reviewed-by: Simon Glass <sjg at chromium.org>
> Tested-on: chromebook_link
> Tested-by: Simon Glass <sjg at chromium.org>

Unfortunately this produces a build warning when applied to u-boot-dm/master:

+warning: (AHCI_PCI) selects DM_SCSI which has unmet direct
dependencies (BLK || BLK)

Can you please take a look? I'm sorry I did not get to it earlier.

Regards,
Simon