From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jingoo Han Subject: Re: [PATCH 1/3] ahci: exynos: add ahci sata support on Exynos platform Date: Fri, 04 Oct 2013 09:33:12 +0900 Message-ID: <003f01cec099$4eb706b0$ec251410$%han@samsung.com> References: <1380609183-21430-1-git-send-email-yuvaraj.cd@samsung.com> <1380609183-21430-2-git-send-email-yuvaraj.cd@samsung.com> <1534631.KrL0vInDuc@amdc1032> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: In-reply-to: <1534631.KrL0vInDuc@amdc1032> Content-language: ko Sender: linux-samsung-soc-owner@vger.kernel.org To: 'Bartlomiej Zolnierkiewicz' , 'Yuvaraj Kumar C D' Cc: tj@kernel.org, kgene.kim@samsung.com, grant.likely@linaro.org, rob.herring@calxeda.com, linux-ide@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, devicetree@vger.kernel.org, kishon@ti.com, s.nawrocki@samsung.com, ks.giri@samsung.com, aditya.ps@samsung.com, 'Yuvaraj Kumar C D' , 'Jingoo Han' List-Id: devicetree@vger.kernel.org On Thursday, October 03, 2013 8:32 PM, Bartlomiej Zolnierkiewicz wrote: > On Tuesday, October 01, 2013 12:03:01 PM Yuvaraj Kumar C D wrote: > > Exynos5250 contains one Synopsys AHCI SATA controller.The avalaible > > ahci_platform driver is not sufficient to handle the AHCI PHY and PHY > > clock related initialization. > > > > This patch adds exynos specific ahci sata driver,contained the exynos > > specific initialized codes, re-use the generic ahci_platform driver, and > > keep the generic ahci_platform driver clean as much as possible. > > > > This patch depends on the below patch > > [1].drivers: phy: add generic PHY framework > > by Kishon Vijay Abraham I > > > > Signed-off-by: Yuvaraj Kumar C D > > --- > > drivers/ata/Kconfig | 9 ++ > > drivers/ata/Makefile | 1 + > > drivers/ata/ahci_exynos.c | 226 +++++++++++++++++++++++++++++++++++++++++++++ > > 3 files changed, 236 insertions(+) > > create mode 100644 drivers/ata/ahci_exynos.c > > [.....] > > + priv->phy = devm_phy_get(dev , "sata-phy"); > > + if (IS_ERR(priv->phy)) > > + return PTR_ERR(priv->phy); [.....] > Also please take a look at the following patch: > > https://lkml.org/lkml/2013/9/19/173 > > it adds PHY support to ahci_platform driver, maybe it can be used > for your needs as well. I also agree with Bartlomiej Zolnierkiewicz's opinion. 'ahci_exynos.c' just calls PHY API, without any additional control for Exynos AHCI IP. Best regards, Jingoo Han > > > + ret = phy_init(priv->phy); > > + if (ret < 0) { > > + dev_err(dev, "failed to init SATA PHY\n"); > > + return ret; > > + } > > +