On Tue, Mar 31, 2015 at 06:04:43PM +0300, Sergei Shtylyov wrote:
> Hello.
> 
> On 03/31/2015 03:21 AM, Simon Horman wrote:
> 
> >>Define the generic R8A7790 part of the EtherAVB device node.
> 
> >Please repost once this binding (and associated driver) has been accepted.
> 
>    OK, but will you merge patch #1?

I would prefer to wait until we can exercise it with a driver.

> >>Signed-off-by: Sergei Shtylyov <[email protected]>
> >>
> >>---
> >>  arch/arm/boot/dts/r8a7790.dtsi |   10 ++++++++++
> >>  1 file changed, 10 insertions(+)
> >>
> >>Index: renesas/arch/arm/boot/dts/r8a7790.dtsi
> >>===================================================================
> >>--- renesas.orig/arch/arm/boot/dts/r8a7790.dtsi
> >>+++ renesas/arch/arm/boot/dts/r8a7790.dtsi
> >>@@ -626,6 +626,16 @@
> >>            status = "disabled";
> >>    };
> >>
> >>+   avb: ethernet@e6800000 {
> >>+           compatible = "renesas,ravb-r8a7790";
> 
>    I went the usual way here but I suspect that SoC part of the prop should
> be "-rcar-gen2" as we now have this IP core (as well as most of the others,
> except PFC) described in a common manual for the R-Car generation 2 SoCs.
> What do you think?

I like -r8a7790.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to