RE: [RFC PATCH v2 3/4] ARM: imx6q: using pinmux subsystem

From: Dong Aisheng-B29396
Date: Thu Dec 15 2011 - 02:36:25 EST


> -----Original Message-----
> From: Linus Walleij [mailto:linus.walleij@xxxxxxxxxx]
> Sent: Thursday, December 15, 2011 2:03 AM
> To: Dong Aisheng-B29396
> Cc: linux-kernel@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx;
> linus.walleij@xxxxxxxxxxxxxx; s.hauer@xxxxxxxxxxxxxx; Guo Shawn-R65073;
> kernel@xxxxxxxxxxxxxx; grant.likely@xxxxxxxxxxxx; rob.herring@xxxxxxxxxxx
> Subject: Re: [RFC PATCH v2 3/4] ARM: imx6q: using pinmux subsystem
> Importance: High
>
> On Wed, Dec 14, 2011 at 5:03 PM, Dong Aisheng <b29396@xxxxxxxxxxxxx>
> wrote:
>
> > From: Dong Aisheng <dong.aisheng@xxxxxxxxxx>
> >
> > Signed-off-by: Dong Aisheng <dong.aisheng@xxxxxxxxxx>
> > Cc: Linus Walleij <linus.walleij@xxxxxxxxxx>
> > Cc: Sascha Hauer <s.hauer@xxxxxxxxxxxxxx>
> > Cc: Shawn Guo <shanw.guo@xxxxxxxxxxxxx>
>
> Acked-by: Linus Walleij <linus.walleij@xxxxxxxxxx>
>
> > +static struct pinmux_map imx6q_pinmux_map[] = {
> > +       PINMUX_MAP_PRIMARY("usdhc4", "sd4", "219c000.usdhc"), };
> > +
> >  static void __init imx6q_init_machine(void)
> >  {
> >        of_platform_populate(NULL, of_default_bus_match_table, NULL,
> > NULL);
> > -
> > +       pinmux_register_mappings(imx6q_pinmux_map,
> > +                                ARRAY_SIZE(imx6q_pinmux_map));
> >        imx6q_pm_init();
> >  }
>
> So ideally this bit of code should also go into the device tree and we
> should add a DT hook inside drivers/pinctrl/pinmux.c
>
> Please make a patch for this if you have a clear idea on how it should
> work!
>
Yes, I will try to see that and will discuss with you once got an idea.

Regards
Dong Aisheng

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/