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

[Bin Meng]

Hi Simon,

On Sat, Jul 29, 2017 at 2:44 AM, Simon Glass <sjg at chromium.org> wrote:
> 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.

Sure I will take a look.

Regards,
Bin