Re: [PATCH v2] pinctrl: sh-pfc: r8a7794: add VIN pin groups

[Sergei Shtylyov <sergei.shtylyov@xxxxxxxxxxxxxxxxxx>]

Hello.

On 10/2/2015 10:59 AM, Geert Uytterhoeven wrote:

> > +/* - VIN1 ----------------------------------------------------------------- */
> > +static const unsigned int vin1_data8_pins[] = {
> > +       RCAR_GP_PIN(5, 12), RCAR_GP_PIN(5, 13),
> > +       RCAR_GP_PIN(5, 14), RCAR_GP_PIN(5, 15),
> > +       RCAR_GP_PIN(5, 16), RCAR_GP_PIN(5, 17),
> > +       RCAR_GP_PIN(5, 18), RCAR_GP_PIN(5, 19),
> > +};
> > +static const unsigned int vin1_data8_mux[] = {
> > +       VI1_DATA0_MARK, VI1_DATA1_MARK,
> > +       VI1_DATA2_MARK, VI1_DATA3_MARK,
> > +       VI1_DATA4_MARK, VI1_DATA5_MARK,
> > +       VI1_DATA6_MARK, VI1_DATA7_MARK,
> > +};
> > +static const unsigned int vin1_data12_pins[] = {
> > +       RCAR_GP_PIN(5, 12), RCAR_GP_PIN(5, 13),
> > +       RCAR_GP_PIN(5, 14), RCAR_GP_PIN(5, 15),
> > +       RCAR_GP_PIN(5, 16), RCAR_GP_PIN(5, 17),
> > +       RCAR_GP_PIN(5, 18), RCAR_GP_PIN(5, 19),
> > +       RCAR_GP_PIN(1, 10), RCAR_GP_PIN(1, 11),
> > +       RCAR_GP_PIN(1, 12), RCAR_GP_PIN(1, 13),
> > +};
> > +static const unsigned int vin1_data12_mux[] = {
> > +       VI1_DATA0_MARK, VI1_DATA1_MARK,
> > +       VI1_DATA2_MARK, VI1_DATA3_MARK,
> > +       VI1_DATA4_MARK, VI1_DATA5_MARK,
> > +       VI1_DATA6_MARK, VI1_DATA7_MARK,
> > +       VI1_DATA8_MARK, VI1_DATA9_MARK,
> > +       VI1_DATA10_MARK, VI1_DATA11_MARK,
> > +};
>
> Given Laurent asked to slightly rework this series, perhaps
> vin1_data*[] can use a union to decrease memory usage, too?

   You're right -- using 'union vin_data' would save some memory now that we 
have 3 pairs of arrays.

> Gr{oetje,eeting}s,
>
>                          Geert

MBR, Sergei

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