Re: [PATCH v6 1/6] Staging: iio: adis16209: Indent the field definitions

From: Jonathan Cameron
Date: Fri Mar 30 2018 - 06:39:25 EST


On Thu, 29 Mar 2018 14:42:02 +0530
Shreeya Patel <shreeya.patel23498@xxxxxxxxx> wrote:

> Have indentation in field definitions to make them
> clearly different from the register addresses.
>
> Signed-off-by: Shreeya Patel <shreeya.patel23498@xxxxxxxxx>
Applied to the togreg branch of iio.git and pushed out as testing
for the autobuilders to play with it.

Note that, given the timing wrt to the current kernel cycle
(merge window will probably open this weekend), this and
everything for the last 2 weeks is now destined to go upstream
in the next cycle.

Thanks,

Jonathan

> ---
>
> Changes in v5
> -Change some macro names and have indentation in the field
> definitions.
>
> Changes in v6
> -Have indentation in the field definitions and do not
> change the names of the macros as the patch for changing
> the names has already been applied.
>
> drivers/staging/iio/accel/adis16209.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/staging/iio/accel/adis16209.c b/drivers/staging/iio/accel/adis16209.c
> index a8453bf..0e6366a 100644
> --- a/drivers/staging/iio/accel/adis16209.c
> +++ b/drivers/staging/iio/accel/adis16209.c
> @@ -71,13 +71,13 @@
> #define ADIS16209_STAT_REG 0x3C
> #define ADIS16209_STAT_ALARM2 BIT(9)
> #define ADIS16209_STAT_ALARM1 BIT(8)
> -#define ADIS16209_STAT_SELFTEST_FAIL_BIT 5
> -#define ADIS16209_STAT_SPI_FAIL_BIT 3
> -#define ADIS16209_STAT_FLASH_UPT_FAIL_BIT 2
> +#define ADIS16209_STAT_SELFTEST_FAIL_BIT 5
> +#define ADIS16209_STAT_SPI_FAIL_BIT 3
> +#define ADIS16209_STAT_FLASH_UPT_FAIL_BIT 2
> /* Power supply above 3.625 V */
> -#define ADIS16209_STAT_POWER_HIGH_BIT 1
> +#define ADIS16209_STAT_POWER_HIGH_BIT 1
> /* Power supply below 3.15 V */
> -#define ADIS16209_STAT_POWER_LOW_BIT 0
> +#define ADIS16209_STAT_POWER_LOW_BIT 0
>
> #define ADIS16209_CMD_REG 0x3E
> #define ADIS16209_CMD_SW_RESET BIT(7)