On 10/30/2013 11:05 AM, Kumar Gala wrote:
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
I'll resend the sata_rcar part as a separate patch in a bit.
I'll keep the old-style compatibility string and device id table entry
for compatibility with existing Marzen (r8a7779) platform code/dts.
Once it is accepted, the bindings documentation will be sent as a separate patch.
Thanks.
--
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