On Oct 29, 2013, at 7:28 PM, Simon Horman wrote: > On Tue, Oct 29, 2013 at 12:19:08PM -0500, Kumar Gala wrote: >> >> 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. > > I think that the motivation was that the first patch documents the > existing implementation while the second documents a change to it. > With that in mind would you still like a single patch for the binding? Yes, I see no reason to split this up as you know both right now. - 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 linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html