Re: [PATCH 2/2] ata: ahci: add support for DaVinci DM816 SATA controller

From: Neil Armstrong
Date: Tue Mar 14 2017 - 05:00:22 EST


On 03/14/2017 09:57 AM, Sergei Shtylyov wrote:
> Hello!
>
> On 3/13/2017 7:36 PM, Bartosz Golaszewski wrote:
>
>> This SATA controller is quite similar to the one present on the DA850
>> SoC, but the PHY configuration is different and it supports two HBA
>> ports.
>>
>> The IP suffers from the same PMP issue the DA850 does - if we enable
>> PMP but don't use it - softreset fails. Appropriate workaround was
>> implemented in this driver as well.
>>
>> Signed-off-by: Bartosz Golaszewski <bgolaszewski@xxxxxxxxxxxx>
>> ---
>> drivers/ata/Kconfig | 9 +++
>> drivers/ata/Makefile | 1 +
>> drivers/ata/ahci_dm816.c | 200 +++++++++++++++++++++++++++++++++++++++++++++++
>> 3 files changed, 210 insertions(+)
>> create mode 100644 drivers/ata/ahci_dm816.c
>>
>> diff --git a/drivers/ata/Kconfig b/drivers/ata/Kconfig
>> index 70b57d2..7f59a7a 100644
>> --- a/drivers/ata/Kconfig
>> +++ b/drivers/ata/Kconfig
>> @@ -118,6 +118,15 @@ config AHCI_DA850
>>
>> If unsure, say N.
>>
>> +config AHCI_DM816
>> + tristate "DaVinci DM816 AHCI SATA support"
>
> Is it really DaVinci?
>
>> + depends on ARCH_OMAP2PLUS
>
> Especially if it depends on OMAP...

Hi Sergei,

Yes it's a DaVinci family SoC, but shares a lot of IPs with the OMAP family.

Neil

>
> [...]
>
> MBR, Sergei
>