Dmitry Eremin-Solenikov <dbaryshkov@xxxxxxxxx> writes: > -#define POODLE_LOCOMO_GPIO_AMP_ON LOCOMO_GPIO(8) > -#define POODLE_LOCOMO_GPIO_MUTE_L LOCOMO_GPIO(10) > -#define POODLE_LOCOMO_GPIO_MUTE_R LOCOMO_GPIO(11) > -#define POODLE_LOCOMO_GPIO_232VCC_ON LOCOMO_GPIO(12) > -#define POODLE_LOCOMO_GPIO_JK_B LOCOMO_GPIO(13) > +#define POODLE_LOCOMO_GPIO_BASE (PXA_NR_BUILTIN_GPIO + 12) I missed that in the v1 serie : why twelve ? > +#define POODLE_GPIO_LCD_VSHA_ON (POODLE_LOCOMO_GPIO_BASE + 4) > +#define POODLE_GPIO_LCD_VSHD_ON (POODLE_LOCOMO_GPIO_BASE + 5) > +#define POODLE_GPIO_LCD_VEE_ON (POODLE_LOCOMO_GPIO_BASE + 6) > +#define POODLE_GPIO_LCD_MOD (POODLE_LOCOMO_GPIO_BASE + 7) > +#define POODLE_GPIO_AMP_ON (POODLE_LOCOMO_GPIO_BASE + 8) > +#define POODLE_GPIO_FL_VR (POODLE_LOCOMO_GPIO_BASE + 9) > +#define POODLE_GPIO_MUTE_L (POODLE_LOCOMO_GPIO_BASE + 10) > +#define POODLE_GPIO_MUTE_R (POODLE_LOCOMO_GPIO_BASE + 11) > +#define POODLE_GPIO_232VCC_ON (POODLE_LOCOMO_GPIO_BASE + 12) > +#define POODLE_GPIO_JK_B (POODLE_LOCOMO_GPIO_BASE + 13) Why not keep the LOCOMO_GPIO() defines, and redevice LOCOMO_GPIO(x) to (POODLE_LOCOMO_GPIO_BASE + (x)) ? No comments on the remaining part, as long as it's tested, you know me by now ... Cheers. -- Robert -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
