Messages in this thread | | | Date | Fri, 11 Apr 2014 16:18:43 +0800 | From | Jisheng Zhang <> | Subject | Re: [PATCH RESEND 4/5] Documentation: add the Marvell Berlin pinctrl documentation |
| |
Hi Antoine,
On Fri, 11 Apr 2014 01:12:59 -0700 Antoine Ténart <antoine.tenart@free-electrons.com> 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@free-electrons.com> 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. 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 ?
That's a good idea.
Thanks, Jisheng -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |