lkml.org 
[lkml]   [2012]   [Apr]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/3] pinctrl: pinctrl-imx: add imx pinctrl core driver
On Mon, Apr 16, 2012 at 11:47:14AM +0800, Dong Aisheng wrote:
...
> Good suggestion.
> I can do it and it's easy to change.
> But as i said above, before the dtc macro support is available, the pinctrl
> setting in dts file will be hard to read since it's all raw data.
> It becomes
> fsl,pins = <1 2 3 4 5 6 ....>;
> People have to search the binding doc to see what the data means.
>
It does not make the thing any worse. You were asking people to search
pinctrl-imx6q.c for the meaning of the following.

fsl,mux = <0 0 1 1 1 1 1 1 1 1>;

IMO, it makes a bit more sense to look at binding doc than code to
understand the dts file.

> The question is how long the dtc macro support will be available?
> I'm very willing to do like as you said if it's just a short time.
> But if it's long time, do we still want to lose the readability right now?
>
The point is with my suggestion taken, we do not need to change code
when the dtc macro support is available.

--
Regards,
Shawn


\
 
 \ /
  Last update: 2012-04-16 05:49    [W:0.565 / U:0.064 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site