On 06/11/2013 04:23 PM, Samuel Ortiz wrote: > Hi Sebastian, Hi Samuel, > On Tue, Jun 11, 2013 at 01:30:50PM +0200, Sebastian Andrzej Siewior wrote: >> diff --git a/include/linux/mfd/ti_am335x_tscadc.h b/include/linux/mfd/ti_am335x_tscadc.h >> index eeead15..2d78af8 100644 >> --- a/include/linux/mfd/ti_am335x_tscadc.h >> +++ b/include/linux/mfd/ti_am335x_tscadc.h >> @@ -116,6 +113,13 @@ >> #define CNTRLREG_8WIRE CNTRLREG_AFE_CTRL(3) >> #define CNTRLREG_TSCENB BIT(7) >> >> +#define XPP STEPCONFIG_XPP >> +#define XNP STEPCONFIG_XNP >> +#define XNN STEPCONFIG_XNN >> +#define YPP STEPCONFIG_YPP >> +#define YPN STEPCONFIG_YPN >> +#define YNN STEPCONFIG_YNN > What is that for ? Isn't STEPCONFIG_XPP explicit enough ? Yeah :P It was added by the original author of the patch, I have no problem getting rid of it. > > Cheers, > Samuel. Sebastian -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html