From 95a1aa4c016829ebd7b641e8871ba123d36d49a4 Mon Sep 17 00:00:00 2001 From: Dean Camera Date: Tue, 15 Mar 2011 06:51:50 +0000 Subject: [PATCH] Add named macros to the board joystick drivers where a shift is needed to maintain unique joystick direction bits in the returned mask value. --- LUFA/Drivers/Board/EVK527/Joystick.h | 8 +++++--- LUFA/Drivers/Board/STK525/Joystick.h | 6 ++++-- LUFA/Drivers/Board/USBKEY/Joystick.h | 8 +++++--- 3 files changed, 14 insertions(+), 8 deletions(-) diff --git a/LUFA/Drivers/Board/EVK527/Joystick.h b/LUFA/Drivers/Board/EVK527/Joystick.h index be8f35aa3b..cab4423ed9 100644 --- a/LUFA/Drivers/Board/EVK527/Joystick.h +++ b/LUFA/Drivers/Board/EVK527/Joystick.h @@ -65,7 +65,9 @@ #if !defined(__DOXYGEN__) /* Macros: */ #define JOY_FMASK ((1 << 4) | (1 << 5) | (1 << 6) | (1 << 7)) - #define JOY_CMASK (1 << 6)) + #define JOY_CMASK (1 << 6) + + #define JOY_PORTC_MASK_SHIFT 3 #endif /* Public Interface - May be used in end-application: */ @@ -80,7 +82,7 @@ #define JOY_UP (1 << 5) /** Mask for the joystick being pushed in the downward direction. */ - #define JOY_DOWN ((1 << 6) >> 3) + #define JOY_DOWN ((1 << 6) >> JOY_PORTC_MASK_SHIFT) /** Mask for the joystick being pushed inward. */ #define JOY_PRESS (1 << 6) @@ -99,7 +101,7 @@ static inline uint8_t Joystick_GetStatus(void) ATTR_WARN_UNUSED_RESULT; static inline uint8_t Joystick_GetStatus(void) { - return (((uint8_t)~PINF & JOY_FMASK) | (((uint8_t)~PINC & JOY_CMASK) >> 3)); + return (((uint8_t)~PINF & JOY_FMASK) | (((uint8_t)~PINC & JOY_CMASK) >> JOY_PORTC_MASK_SHIFT)); } #endif diff --git a/LUFA/Drivers/Board/STK525/Joystick.h b/LUFA/Drivers/Board/STK525/Joystick.h index c1dd3623ea..c90e7a0d2c 100644 --- a/LUFA/Drivers/Board/STK525/Joystick.h +++ b/LUFA/Drivers/Board/STK525/Joystick.h @@ -66,6 +66,8 @@ /* Macros: */ #define JOY_BMASK ((1 << 5) | (1 << 6) | (1 << 7)) #define JOY_EMASK ((1 << 4) | (1 << 5)) + + #define JOY_PORTE_MASK_SHIFT 1 #endif /* Public Interface - May be used in end-application: */ @@ -80,7 +82,7 @@ #define JOY_UP (1 << 7) /** Mask for the joystick being pushed in the downward direction. */ - #define JOY_DOWN ((1 << 5) >> 1) + #define JOY_DOWN ((1 << 5) >> JOY_PORTE_MASK_SHIFT) /** Mask for the joystick being pushed inward. */ #define JOY_PRESS (1 << 5) @@ -99,7 +101,7 @@ static inline uint8_t Joystick_GetStatus(void) ATTR_WARN_UNUSED_RESULT; static inline uint8_t Joystick_GetStatus(void) { - return (((uint8_t)~PINB & JOY_BMASK) | (((uint8_t)~PINE & JOY_EMASK) >> 1)); + return (((uint8_t)~PINB & JOY_BMASK) | (((uint8_t)~PINE & JOY_EMASK) >> JOY_PORTE_MASK_SHIFT)); } #endif diff --git a/LUFA/Drivers/Board/USBKEY/Joystick.h b/LUFA/Drivers/Board/USBKEY/Joystick.h index 8ca5bc45f6..b8ae7bc3e3 100644 --- a/LUFA/Drivers/Board/USBKEY/Joystick.h +++ b/LUFA/Drivers/Board/USBKEY/Joystick.h @@ -66,6 +66,8 @@ /* Macros: */ #define JOY_BMASK ((1 << 5) | (1 << 6) | (1 << 7)) #define JOY_EMASK ((1 << 4) | (1 << 5)) + + #define JOY_PORTE_MASK_SHIFT 1 #endif /* Public Interface - May be used in end-application: */ @@ -74,13 +76,13 @@ #define JOY_LEFT (1 << 6) /** Mask for the joystick being pushed in the right direction. */ - #define JOY_RIGHT ((1 << 4) >> 1) + #define JOY_RIGHT ((1 << 4) >> JOY_PORTE_MASK_SHIFT) /** Mask for the joystick being pushed in the upward direction. */ #define JOY_UP (1 << 7) /** Mask for the joystick being pushed in the downward direction. */ - #define JOY_DOWN ((1 << 5) >> 1) + #define JOY_DOWN ((1 << 5) >> JOY_PORTE_MASK_SHIFT) /** Mask for the joystick being pushed inward. */ #define JOY_PRESS (1 << 5) @@ -99,7 +101,7 @@ static inline uint8_t Joystick_GetStatus(void) ATTR_WARN_UNUSED_RESULT; static inline uint8_t Joystick_GetStatus(void) { - return (((uint8_t)~PINB & JOY_BMASK) | (((uint8_t)~PINE & JOY_EMASK) >> 1)); + return (((uint8_t)~PINB & JOY_BMASK) | (((uint8_t)~PINE & JOY_EMASK) >> JOY_PORTE_MASK_SHIFT)); } #endif