Re: [PATCH v4 09/26] staging: iio: tsl2583: fix alignment of #define values

From: Jonathan Cameron
Date: Sun Nov 13 2016 - 07:13:45 EST


On 12/11/16 18:19, Brian Masney wrote:
> Most of the values in the #defines have their values aligned on a single
> column, but some do not. This changes the remaining defines to use
> consistent alignment with the majority to improve code readability.
>
> Signed-off-by: Brian Masney <masneyb@xxxxxxxxxxxxx>
Applied.
> ---
> drivers/staging/iio/light/tsl2583.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/staging/iio/light/tsl2583.c b/drivers/staging/iio/light/tsl2583.c
> index 5a82a26..d482a84 100644
> --- a/drivers/staging/iio/light/tsl2583.c
> +++ b/drivers/staging/iio/light/tsl2583.c
> @@ -44,19 +44,19 @@
> /* tsl2583 cmd reg masks */
> #define TSL2583_CMD_REG 0x80
> #define TSL2583_CMD_SPL_FN 0x60
> -#define TSL2583_CMD_ALS_INT_CLR 0X01
> +#define TSL2583_CMD_ALS_INT_CLR 0x01
>
> /* tsl2583 cntrl reg masks */
> -#define TSL2583_CNTL_ADC_ENBL 0x02
> +#define TSL2583_CNTL_ADC_ENBL 0x02
> #define TSL2583_CNTL_PWR_OFF 0x00
> #define TSL2583_CNTL_PWR_ON 0x01
>
> /* tsl2583 status reg masks */
> -#define TSL2583_STA_ADC_VALID 0x01
> -#define TSL2583_STA_ADC_INTR 0x10
> +#define TSL2583_STA_ADC_VALID 0x01
> +#define TSL2583_STA_ADC_INTR 0x10
>
> /* Lux calculation constants */
> -#define TSL2583_LUX_CALC_OVER_FLOW 65535
> +#define TSL2583_LUX_CALC_OVER_FLOW 65535
>
> #define TSL2583_INTERRUPT_DISABLED 0x00
>
>