[PATCH v5 00/14] ARM: da850-lcdk: add SATA support
From: Sekhar Nori <hidden>
Date: 2017-01-20 14:25:51
Also in:
linux-devicetree, linux-ide, lkml
On Friday 20 January 2017 07:32 PM, Bartosz Golaszewski wrote:
2017-01-20 14:56 GMT+01:00 Sekhar Nori [off-list ref]:quoted
Hi Tejun, On Friday 20 January 2017 06:58 PM, Tejun Heo wrote:quoted
On Fri, Jan 20, 2017 at 12:21:51PM +0100, Bartosz Golaszewski wrote:quoted
This series contains all the changes necessary to make SATA work on the da850-lcdk board. The first patch adds DT bindings for the ahci-da850 driver. The second enables relevant modules in davinci_all_defconfig. Patches 03/14-06/14 modify the way the clocks are handled regarding SATA on the da850 platform. We modify the ahci driver to retrieve the clock via con_id and model the external SATA oscillator as a real clock. Patches 07/14-11/14 extend the ahci-da850 driver. Add DT support, implement workarounds necessary to make SATA work on the da850-lcdk board and un-hardcode the external clock multiplier.Please feel free to add Acked-by: Tejun Heo [off-list ref] to the all libata patches. Please let me know how the patches should be routed once other parts are settled.I believe you can queue the libata patches independently (patches 1, 4, 7, 8, 9, 10, 11). It looks like they have been written such that driver continues to work with existing platform code (Bartosz, please disagree if I am wrong). 1/14 still needs the ack from DT maintainers.Patch 11/14 depends on 06/14. Other than that I think it should work independently.
Ah, right. You recently changed 11/14 to error out in case refclkpn is not found instead of using a default value (do agree that current version is better). If there are no build time dependencies still, it might be okay to queue these patches through their respective maintainer trees. Tejun, I am open to queuing the driver changes through ARM-SoC. I guess with that there is little chance that SATA will be broken on linux-next for a longish period of time. I have Cced the ARM-SoC maintainers too. Thanks, Sekhar