From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergei Shtylyov Subject: Re: [PATCH 2/2] ata: ahci: add support for DaVinci DM816 SATA controller Date: Tue, 14 Mar 2017 11:57:08 +0300 Message-ID: References: <1489422982-3461-1-git-send-email-bgolaszewski@baylibre.com> <1489422982-3461-3-git-send-email-bgolaszewski@baylibre.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1489422982-3461-3-git-send-email-bgolaszewski@baylibre.com> Sender: linux-kernel-owner@vger.kernel.org To: Bartosz Golaszewski , Tejun Heo , Rob Herring , Mark Rutland , Neil Armstrong , Michael Turquette , Kevin Hilman , Patrick Titiano Cc: linux-ide@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org List-Id: devicetree@vger.kernel.org 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 > --- > 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... [...] MBR, Sergei