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

Jonathan Cameron jic23 at kernel.org
Sun Nov 13 12:13:40 UTC 2016


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 at onstation.org>
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
>  
> 



More information about the devel mailing list