[PATCH] Added device tree binding for TDM and TDM phy

Scott Wood scottwood at freescale.com
Fri Jan 11 06:48:38 EST 2013


On 01/10/2013 03:24:21 AM, Singh Sandeep-B37400 wrote:
> > > > +- compatible
> > > > +    Value type: <string>
> > > > +    Definition: Should contain generic compatibility like
> > > "tdm-phy-slic"
> > > > or
> > > > +    "tdm-phy-e1" or "tdm-phy-t1".
> >
> > Does this "generic" string (plus the other properties) tell you all  
> you
> > need to know about the device?  If there are other possible  
> "generic"
> > compatibles, they should be listed or else different people will  
> make up
> > different strings for the same thing.
> 
> This property will describe the type of device, and will help TDM  
> framework
> to know if it is E1/T1/SLIC device. Further details can be extracted  
> from other
> compatible strings.
> There are only three generic compatibles field types, which are  
> already mentioned
> in definition. Do I need to make this thing more clear.

The word "like" suggests that there are other possibilites.  It would  
be clearer as:

Definition: One of "tdm-phy-slic", "tdm-phy-e1", or "tdm-phy-t1".

-Scott


More information about the devicetree-discuss mailing list