On Fri, Apr 11, 2014 at 10:12:59AM +0200, Antoine Ténart wrote: > Hi Jisheng, > > On Fri, Apr 11, 2014 at 02:33:18PM +0800, Jisheng Zhang wrote: > > On Thu, 10 Apr 2014 06:07:53 -0700 > > Antoine Ténart <antoine.tenart@xxxxxxxxxxxxxxxxxx> wrote: > > > +Required subnode-properties: > > > +- berlin,group: a string describing the group's name. > > > +- berlin,function: the function used to mux the group. > > > > Would it be better to s/function/mode? Because in mrvl internal technology > > discussion and docs, we usually say "set pinmux Gy as mode z". > > The pinctrl framework uses 'function', so I guess it's better not to introduce > an other keyword that may confuse people. Agreed. DT is a standard, Marvell datasheets are just datasheets. > But I agree this is not what's used in the Marvell's docs. What do > you think about we mention it in this documentation ? Or ask for the datahsheet to be changed :-) Andrew -- 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