On 02/10/2012 09:47 PM, Dan Carpenter wrote: > On Fri, Feb 10, 2012 at 11:45:31PM +0300, Dan Carpenter wrote: >> On Fri, Feb 10, 2012 at 07:15:00PM +0100, Lars-Peter Clausen wrote: >>> Add macros for extracting whether the event is for a differential channel and >>> the second channel number from the event code. These were the only two fields >>> which did not have such an macro yet. >>> >>> Signed-off-by: Lars-Peter Clausen <lars@xxxxxxxxxx> >>> Acked-by: Jonathan Cameron <jic23@xxxxxxxxxx> >>> --- >>> drivers/staging/iio/events.h | 2 ++ >>> 1 files changed, 2 insertions(+), 0 deletions(-) >>> >>> diff --git a/drivers/staging/iio/events.h b/drivers/staging/iio/events.h >>> index 416800e..c25f0e3 100644 >>> --- a/drivers/staging/iio/events.h >>> +++ b/drivers/staging/iio/events.h >>> @@ -97,7 +97,9 @@ enum iio_event_direction { >>> /* Event code number extraction depends on which type of event we have. >>> * Perhaps review this function in the future*/ >>> #define IIO_EVENT_CODE_EXTRACT_CHAN(mask) ((__s16)(mask & 0xFFFF)) >>> +#define IIO_EVENT_CODE_EXTRACT_CHAN2(mask) ((__s16)(((mask) >> 16) & 0xFFFF)) >>> >>> #define IIO_EVENT_CODE_EXTRACT_MODIFIER(mask) ((mask >> 40) & 0xFF) >>> +#define IIO_EVENT_CODE_EXTRACT_DIFF(mask) (((mask) >> 55) & 0x1) >>> >> >> Breaks the build. >> > > Oops. This isn't a rename. It doesn't break the build. Why are > we adding macros which nobody uses? Patch 4 in this series adds a user. -- 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