Re: [PATCH 2/4] staging: iio: adis16203: Move contents of the header to the source file

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 04/03/17 13:46, Narcisa Ana Maria Vasile wrote:
> The contents of the header file are used only by this single
> source file. Move content into .c file and remove header.
> 
> Signed-off-by: Narcisa Ana Maria Vasile <narcisaanamaria12@xxxxxxxxx>
Varsha Rao posted a patch for this one the other day and I've already
merged that.

Thanks

Jonathan
> ---
>  drivers/staging/iio/accel/adis16203.h      | 125 -----------------------------
>  drivers/staging/iio/accel/adis16203_core.c | 123 +++++++++++++++++++++++++++-
>  2 files changed, 121 insertions(+), 127 deletions(-)
>  delete mode 100644 drivers/staging/iio/accel/adis16203.h
> 
> diff --git a/drivers/staging/iio/accel/adis16203.h b/drivers/staging/iio/accel/adis16203.h
> deleted file mode 100644
> index b483e4e..0000000
> --- a/drivers/staging/iio/accel/adis16203.h
> +++ /dev/null
> @@ -1,125 +0,0 @@
> -#ifndef SPI_ADIS16203_H_
> -#define SPI_ADIS16203_H_
> -
> -#define ADIS16203_STARTUP_DELAY	220 /* ms */
> -
> -/* Flash memory write count */
> -#define ADIS16203_FLASH_CNT      0x00
> -
> -/* Output, power supply */
> -#define ADIS16203_SUPPLY_OUT     0x02
> -
> -/* Output, auxiliary ADC input */
> -#define ADIS16203_AUX_ADC        0x08
> -
> -/* Output, temperature */
> -#define ADIS16203_TEMP_OUT       0x0A
> -
> -/* Output, x-axis inclination */
> -#define ADIS16203_XINCL_OUT      0x0C
> -
> -/* Output, y-axis inclination */
> -#define ADIS16203_YINCL_OUT      0x0E
> -
> -/* Incline null calibration */
> -#define ADIS16203_INCL_NULL      0x18
> -
> -/* Alarm 1 amplitude threshold */
> -#define ADIS16203_ALM_MAG1       0x20
> -
> -/* Alarm 2 amplitude threshold */
> -#define ADIS16203_ALM_MAG2       0x22
> -
> -/* Alarm 1, sample period */
> -#define ADIS16203_ALM_SMPL1      0x24
> -
> -/* Alarm 2, sample period */
> -#define ADIS16203_ALM_SMPL2      0x26
> -
> -/* Alarm control */
> -#define ADIS16203_ALM_CTRL       0x28
> -
> -/* Auxiliary DAC data */
> -#define ADIS16203_AUX_DAC        0x30
> -
> -/* General-purpose digital input/output control */
> -#define ADIS16203_GPIO_CTRL      0x32
> -
> -/* Miscellaneous control */
> -#define ADIS16203_MSC_CTRL       0x34
> -
> -/* Internal sample period (rate) control */
> -#define ADIS16203_SMPL_PRD       0x36
> -
> -/* Operation, filter configuration */
> -#define ADIS16203_AVG_CNT        0x38
> -
> -/* Operation, sleep mode control */
> -#define ADIS16203_SLP_CNT        0x3A
> -
> -/* Diagnostics, system status register */
> -#define ADIS16203_DIAG_STAT      0x3C
> -
> -/* Operation, system command register */
> -#define ADIS16203_GLOB_CMD       0x3E
> -
> -/* MSC_CTRL */
> -
> -/* Self-test at power-on: 1 = disabled, 0 = enabled */
> -#define ADIS16203_MSC_CTRL_PWRUP_SELF_TEST	BIT(10)
> -
> -/* Reverses rotation of both inclination outputs */
> -#define ADIS16203_MSC_CTRL_REVERSE_ROT_EN	BIT(9)
> -
> -/* Self-test enable */
> -#define ADIS16203_MSC_CTRL_SELF_TEST_EN	        BIT(8)
> -
> -/* Data-ready enable: 1 = enabled, 0 = disabled */
> -#define ADIS16203_MSC_CTRL_DATA_RDY_EN	        BIT(2)
> -
> -/* Data-ready polarity: 1 = active high, 0 = active low */
> -#define ADIS16203_MSC_CTRL_ACTIVE_HIGH	        BIT(1)
> -
> -/* Data-ready line selection: 1 = DIO1, 0 = DIO0 */
> -#define ADIS16203_MSC_CTRL_DATA_RDY_DIO1	BIT(0)
> -
> -/* DIAG_STAT */
> -
> -/* Alarm 2 status: 1 = alarm active, 0 = alarm inactive */
> -#define ADIS16203_DIAG_STAT_ALARM2        BIT(9)
> -
> -/* Alarm 1 status: 1 = alarm active, 0 = alarm inactive */
> -#define ADIS16203_DIAG_STAT_ALARM1        BIT(8)
> -
> -/* Self-test diagnostic error flag */
> -#define ADIS16203_DIAG_STAT_SELFTEST_FAIL_BIT 5
> -
> -/* SPI communications failure */
> -#define ADIS16203_DIAG_STAT_SPI_FAIL_BIT      3
> -
> -/* Flash update failure */
> -#define ADIS16203_DIAG_STAT_FLASH_UPT_BIT     2
> -
> -/* Power supply above 3.625 V */
> -#define ADIS16203_DIAG_STAT_POWER_HIGH_BIT    1
> -
> -/* Power supply below 3.15 V */
> -#define ADIS16203_DIAG_STAT_POWER_LOW_BIT     0
> -
> -/* GLOB_CMD */
> -
> -#define ADIS16203_GLOB_CMD_SW_RESET	BIT(7)
> -#define ADIS16203_GLOB_CMD_CLEAR_STAT	BIT(4)
> -#define ADIS16203_GLOB_CMD_FACTORY_CAL	BIT(1)
> -
> -#define ADIS16203_ERROR_ACTIVE          BIT(14)
> -
> -enum adis16203_scan {
> -	ADIS16203_SCAN_INCLI_X,
> -	ADIS16203_SCAN_INCLI_Y,
> -	ADIS16203_SCAN_SUPPLY,
> -	ADIS16203_SCAN_AUX_ADC,
> -	ADIS16203_SCAN_TEMP,
> -};
> -
> -#endif /* SPI_ADIS16203_H_ */
> diff --git a/drivers/staging/iio/accel/adis16203_core.c b/drivers/staging/iio/accel/adis16203_core.c
> index bd8119a..7018605 100644
> --- a/drivers/staging/iio/accel/adis16203_core.c
> +++ b/drivers/staging/iio/accel/adis16203_core.c
> @@ -20,10 +20,129 @@
>  #include <linux/iio/buffer.h>
>  #include <linux/iio/imu/adis.h>
>  
> -#include "adis16203.h"
> -
>  #define DRIVER_NAME		"adis16203"
>  
> +#define ADIS16203_STARTUP_DELAY	220 /* ms */
> +
> +/* Flash memory write count */
> +#define ADIS16203_FLASH_CNT      0x00
> +
> +/* Output, power supply */
> +#define ADIS16203_SUPPLY_OUT     0x02
> +
> +/* Output, auxiliary ADC input */
> +#define ADIS16203_AUX_ADC        0x08
> +
> +/* Output, temperature */
> +#define ADIS16203_TEMP_OUT       0x0A
> +
> +/* Output, x-axis inclination */
> +#define ADIS16203_XINCL_OUT      0x0C
> +
> +/* Output, y-axis inclination */
> +#define ADIS16203_YINCL_OUT      0x0E
> +
> +/* Incline null calibration */
> +#define ADIS16203_INCL_NULL      0x18
> +
> +/* Alarm 1 amplitude threshold */
> +#define ADIS16203_ALM_MAG1       0x20
> +
> +/* Alarm 2 amplitude threshold */
> +#define ADIS16203_ALM_MAG2       0x22
> +
> +/* Alarm 1, sample period */
> +#define ADIS16203_ALM_SMPL1      0x24
> +
> +/* Alarm 2, sample period */
> +#define ADIS16203_ALM_SMPL2      0x26
> +
> +/* Alarm control */
> +#define ADIS16203_ALM_CTRL       0x28
> +
> +/* Auxiliary DAC data */
> +#define ADIS16203_AUX_DAC        0x30
> +
> +/* General-purpose digital input/output control */
> +#define ADIS16203_GPIO_CTRL      0x32
> +
> +/* Miscellaneous control */
> +#define ADIS16203_MSC_CTRL       0x34
> +
> +/* Internal sample period (rate) control */
> +#define ADIS16203_SMPL_PRD       0x36
> +
> +/* Operation, filter configuration */
> +#define ADIS16203_AVG_CNT        0x38
> +
> +/* Operation, sleep mode control */
> +#define ADIS16203_SLP_CNT        0x3A
> +
> +/* Diagnostics, system status register */
> +#define ADIS16203_DIAG_STAT      0x3C
> +
> +/* Operation, system command register */
> +#define ADIS16203_GLOB_CMD       0x3E
> +
> +/* MSC_CTRL */
> +
> +/* Self-test at power-on: 1 = disabled, 0 = enabled */
> +#define ADIS16203_MSC_CTRL_PWRUP_SELF_TEST	BIT(10)
> +
> +/* Reverses rotation of both inclination outputs */
> +#define ADIS16203_MSC_CTRL_REVERSE_ROT_EN	BIT(9)
> +
> +/* Self-test enable */
> +#define ADIS16203_MSC_CTRL_SELF_TEST_EN	        BIT(8)
> +
> +/* Data-ready enable: 1 = enabled, 0 = disabled */
> +#define ADIS16203_MSC_CTRL_DATA_RDY_EN	        BIT(2)
> +
> +/* Data-ready polarity: 1 = active high, 0 = active low */
> +#define ADIS16203_MSC_CTRL_ACTIVE_HIGH	        BIT(1)
> +
> +/* Data-ready line selection: 1 = DIO1, 0 = DIO0 */
> +#define ADIS16203_MSC_CTRL_DATA_RDY_DIO1	BIT(0)
> +
> +/* DIAG_STAT */
> +
> +/* Alarm 2 status: 1 = alarm active, 0 = alarm inactive */
> +#define ADIS16203_DIAG_STAT_ALARM2        BIT(9)
> +
> +/* Alarm 1 status: 1 = alarm active, 0 = alarm inactive */
> +#define ADIS16203_DIAG_STAT_ALARM1        BIT(8)
> +
> +/* Self-test diagnostic error flag */
> +#define ADIS16203_DIAG_STAT_SELFTEST_FAIL_BIT 5
> +
> +/* SPI communications failure */
> +#define ADIS16203_DIAG_STAT_SPI_FAIL_BIT      3
> +
> +/* Flash update failure */
> +#define ADIS16203_DIAG_STAT_FLASH_UPT_BIT     2
> +
> +/* Power supply above 3.625 V */
> +#define ADIS16203_DIAG_STAT_POWER_HIGH_BIT    1
> +
> +/* Power supply below 3.15 V */
> +#define ADIS16203_DIAG_STAT_POWER_LOW_BIT     0
> +
> +/* GLOB_CMD */
> +
> +#define ADIS16203_GLOB_CMD_SW_RESET	BIT(7)
> +#define ADIS16203_GLOB_CMD_CLEAR_STAT	BIT(4)
> +#define ADIS16203_GLOB_CMD_FACTORY_CAL	BIT(1)
> +
> +#define ADIS16203_ERROR_ACTIVE          BIT(14)
> +
> +enum adis16203_scan {
> +	ADIS16203_SCAN_INCLI_X,
> +	ADIS16203_SCAN_INCLI_Y,
> +	ADIS16203_SCAN_SUPPLY,
> +	ADIS16203_SCAN_AUX_ADC,
> +	ADIS16203_SCAN_TEMP,
> +};
> +
>  static const u8 adis16203_addresses[] = {
>  	[ADIS16203_SCAN_INCLI_X] = ADIS16203_INCL_NULL,
>  };
> 

--
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



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux