Re: [PATCH 2/5] ARM: dts: AM4372: Add USB nodes

From: Tony Lindgren
Date: Thu Mar 06 2014 - 13:37:49 EST


* Roger Quadros <rogerq@xxxxxx> [140306 00:52]:
> On 03/03/2014 03:53 PM, George Cherian wrote:
> > --- a/arch/arm/boot/dts/am4372.dtsi
> > +++ b/arch/arm/boot/dts/am4372.dtsi
> > @@ -698,6 +698,105 @@
> > <&edma 11>;
> > dma-names = "tx", "rx";
> > };
> > +
> > + am43xx_control_usb2phy1: control-phy@44e10620 {
> > + compatible = "ti,control-phy-am437usb2";
>
> Since this is the first usage of this compatible ID, we still have room to
> change it to be better. Tony's suggestion was to use "ti,control-phy-am437-usb2"

Or from grepping point of view ti,control-phy-usb2-am437 might be better?

Regards,

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/