Re: [PATCH 2/2] sata_rcar: Add R-Car Gen2 SATA PHY support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On Oct 29, 2013, at 3:44 AM, Simon Horman wrote:

> On Tue, Oct 29, 2013 at 03:24:16AM -0500, Kumar Gala wrote:
>> 
>> On Oct 28, 2013, at 11:59 PM, Simon Horman wrote:
>> 
>>> On Wed, Oct 16, 2013 at 04:06:01PM +0400, Valentine Barshak wrote:
>>>> R-Car Gen2 SoCs have a different PHY which is not compatible
>>>> with the older R-Car H1 (R8A7779) version.
>>>> This adds OF/platform device id tables and PHY initialization
>>>> callbacks for the following Gen2 SoCs:
>>>> * R-Car H2: R8A7790;
>>>> * R-Car M2: R8A7791.
>>>> 
>>>> PHY initialization method is chosen based on the device id.
>>>> Default PHY settings are applied for Gen2 SoCs, which should
>>>> suit the Gen2 boards available.
>>>> 
>>>> The R8A7779 platform code is modified to use "sata-r8a7779"
>>>> device name.
>>>> 
>>>> Signed-off-by: Valentine Barshak <valentine.barshak@xxxxxxxxxxxxxxxxxx>
>>>> ---
>>>> .../devicetree/bindings/ata/sata_rcar.txt          |   5 +-
>>>> arch/arm/mach-shmobile/clock-r8a7779.c             |   2 +-
>>>> arch/arm/mach-shmobile/setup-r8a7779.c             |   2 +-
>>>> drivers/ata/sata_rcar.c                            | 118 ++++++++++++++++++---
>>> 
>>> Hi Mark, Hi Device-Tree Folks,
>>> 
>>> I'm wondering if you have had a chance to look over the bindings
>>> aspect of this and the other patch in the series. I believe that
>>> this series addresses all previous review in that regards.
>> 
>> What tree is this binding in?
> 
> sata_car.txt is added by the previous patch in this series
> "[PATCH 1/2] sata_rcar: Adjust and document device tree bindings"

Why not just have all the binding in one patch.  There isn't a reason this should be split.

- k

-- 
Employee of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux