On Tue, Jan 4, 2011 at 6:10 AM, Kevin Hilman <[email protected]>wrote:

> Subhasish Ghosh <[email protected]> writes:
>
> [...]
>
> > @@ -542,7 +549,12 @@ static const struct mux_config da850_pins[] = {
> >       MUX_CFG(DA850, EMA_CLK,         6,      0,      15,     1,
>  false)
> >       MUX_CFG(DA850, EMA_WAIT_1,      6,      24,     15,     1,
>  false)
> >       MUX_CFG(DA850, NEMA_CS_2,       7,      0,      15,     1,
>  false)
> > +     /* PRU functions for soft CAN */
> > +     MUX_CFG(DA850, PRU0_R31_0,      7,      28,     15, 0,  false)
> > +     MUX_CFG(DA850, PRU1_R30_15,     12,     0,      15, 4,  false)
> > +     MUX_CFG(DA850, PRU1_R31_18,     11,     20,     15, 0,  false)
>
> Please ensure these are aligned with existing entries.  You have a
> single space instead of a TAB after the '15,' in your entries, so they
> do not align with the others.
>

    [SG] -- Will correct these.


>
> Kevin
>
>
> >       /* GPIO function */
> > +     MUX_CFG(DA850, GPIO2_0,         6,      28,     15,     8,
>  false)
> >       MUX_CFG(DA850, GPIO2_6,         6,      4,      15,     8,
>  false)
> >       MUX_CFG(DA850, GPIO2_8,         5,      28,     15,     8,
>  false)
> >       MUX_CFG(DA850, GPIO2_15,        5,      0,      15,     8,
>  false)
>
_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to