On Friday 23 August 2013 02:13 PM, Daniel Mack wrote: > This third memory region just denotes one single register in the CONTROL > module block. The driver uses that in order to set the correct physical > ethernet interface modes. > > Also update the compatible string to make use of the am335x specific > features of the cpsw driver. > > Signed-off-by: Daniel Mack <zonque@xxxxxxxxx> > --- > arch/arm/boot/dts/am33xx.dtsi | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi > index a785b95..1a1e9dd 100644 > --- a/arch/arm/boot/dts/am33xx.dtsi > +++ b/arch/arm/boot/dts/am33xx.dtsi > @@ -446,7 +446,7 @@ > }; > > mac: ethernet@4a100000 { > - compatible = "ti,cpsw"; > + compatible = "ti,am3352-cpsw"; Please keep this as compatible = "ti,am3352-cpsw", "ti,cpsw"; so users can upgrade the DTB or kernel independently. Apart from this the series looks good to me. Thanks, Sekhar -- 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