[PATCH 1/2] pinctrl: imx: work around select input quirk

Shawn Guo shawn.guo at linaro.org
Sun Aug 4 23:26:14 EDT 2013


On Mon, Aug 05, 2013 at 09:14:50AM +0800, Peter Chen wrote:
> On Sun, Aug 04, 2013 at 08:54:54PM +0800, Shawn Guo wrote:
> > Yes, we changed the prefix to simplify the DTS files for imx6q and
> > imx6dl.
> > 
> > > 
> > > Besides, please add comments for u16 *input_reg at struct imx_pin_group.
> > 
> > I'm not fond of documenting a workaround for a random quirky select
> > input as a feature all over the files where input_reg is documented.
> > It should be good enough to have it well documented at where the quirk
> > is handled.
> 
> If the user finds "odd value" at xxx-pinfunc.h, how he knows what
> it stands for? At least, It should be documented where the user
> can find its meaning.

We have it well documented in pinctrl-imx.c, function imx_pmx_enable()
where the "odd value" is handled.

But I would try to look at the git log of xxx-pinfunc.h at the first
place to see where and how the "odd value" comes.

Shawn




More information about the linux-arm-kernel mailing list