Re: [PATCH v2] staging: iio: accel: Move header file content to source file

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

 



On 28/02/17 19:17, Jonathan Cameron wrote:
> On 28/02/17 18:51, simran singhal wrote:
>> The contents of the header file are used only by this single
>> source file. Move content into .c and remove .h.
>>
>> Signed-off-by: simran singhal <singhalsimran0@xxxxxxxxx>
> Applied to the togreg branch of iio.git and pushed out as testing
> for the autobuilders to play with it.
> 
> Thanks,
> 
> Jonathan

One quick note:

The title should include the name of the driver being modified
as this is probably the bit interested people are most likely
to pick up on in a long list of patches.

I fixed this up.  Thanks,

Jonathan
>> ---
>>
>>  v2:
>>    -Removing ifndef and define
>>
>>  drivers/staging/iio/accel/adis16201.h      | 144 -----------------------------
>>  drivers/staging/iio/accel/adis16201_core.c | 142 +++++++++++++++++++++++++++-
>>  2 files changed, 141 insertions(+), 145 deletions(-)
>>  delete mode 100644 drivers/staging/iio/accel/adis16201.h
>>
>> diff --git a/drivers/staging/iio/accel/adis16201.h b/drivers/staging/iio/accel/adis16201.h
>> deleted file mode 100644
>> index 64844ad..0000000
>> --- a/drivers/staging/iio/accel/adis16201.h
>> +++ /dev/null
>> @@ -1,144 +0,0 @@
>> -#ifndef SPI_ADIS16201_H_
>> -#define SPI_ADIS16201_H_
>> -
>> -#define ADIS16201_STARTUP_DELAY	220 /* ms */
>> -
>> -/* Flash memory write count */
>> -#define ADIS16201_FLASH_CNT      0x00
>> -
>> -/* Output, power supply */
>> -#define ADIS16201_SUPPLY_OUT     0x02
>> -
>> -/* Output, x-axis accelerometer */
>> -#define ADIS16201_XACCL_OUT      0x04
>> -
>> -/* Output, y-axis accelerometer */
>> -#define ADIS16201_YACCL_OUT      0x06
>> -
>> -/* Output, auxiliary ADC input */
>> -#define ADIS16201_AUX_ADC        0x08
>> -
>> -/* Output, temperature */
>> -#define ADIS16201_TEMP_OUT       0x0A
>> -
>> -/* Output, x-axis inclination */
>> -#define ADIS16201_XINCL_OUT      0x0C
>> -
>> -/* Output, y-axis inclination */
>> -#define ADIS16201_YINCL_OUT      0x0E
>> -
>> -/* Calibration, x-axis acceleration offset */
>> -#define ADIS16201_XACCL_OFFS     0x10
>> -
>> -/* Calibration, y-axis acceleration offset */
>> -#define ADIS16201_YACCL_OFFS     0x12
>> -
>> -/* x-axis acceleration scale factor */
>> -#define ADIS16201_XACCL_SCALE    0x14
>> -
>> -/* y-axis acceleration scale factor */
>> -#define ADIS16201_YACCL_SCALE    0x16
>> -
>> -/* Calibration, x-axis inclination offset */
>> -#define ADIS16201_XINCL_OFFS     0x18
>> -
>> -/* Calibration, y-axis inclination offset */
>> -#define ADIS16201_YINCL_OFFS     0x1A
>> -
>> -/* x-axis inclination scale factor */
>> -#define ADIS16201_XINCL_SCALE    0x1C
>> -
>> -/* y-axis inclination scale factor */
>> -#define ADIS16201_YINCL_SCALE    0x1E
>> -
>> -/* Alarm 1 amplitude threshold */
>> -#define ADIS16201_ALM_MAG1       0x20
>> -
>> -/* Alarm 2 amplitude threshold */
>> -#define ADIS16201_ALM_MAG2       0x22
>> -
>> -/* Alarm 1, sample period */
>> -#define ADIS16201_ALM_SMPL1      0x24
>> -
>> -/* Alarm 2, sample period */
>> -#define ADIS16201_ALM_SMPL2      0x26
>> -
>> -/* Alarm control */
>> -#define ADIS16201_ALM_CTRL       0x28
>> -
>> -/* Auxiliary DAC data */
>> -#define ADIS16201_AUX_DAC        0x30
>> -
>> -/* General-purpose digital input/output control */
>> -#define ADIS16201_GPIO_CTRL      0x32
>> -
>> -/* Miscellaneous control */
>> -#define ADIS16201_MSC_CTRL       0x34
>> -
>> -/* Internal sample period (rate) control */
>> -#define ADIS16201_SMPL_PRD       0x36
>> -
>> -/* Operation, filter configuration */
>> -#define ADIS16201_AVG_CNT        0x38
>> -
>> -/* Operation, sleep mode control */
>> -#define ADIS16201_SLP_CNT        0x3A
>> -
>> -/* Diagnostics, system status register */
>> -#define ADIS16201_DIAG_STAT      0x3C
>> -
>> -/* Operation, system command register */
>> -#define ADIS16201_GLOB_CMD       0x3E
>> -
>> -/* MSC_CTRL */
>> -
>> -/* Self-test enable */
>> -#define ADIS16201_MSC_CTRL_SELF_TEST_EN	        BIT(8)
>> -
>> -/* Data-ready enable: 1 = enabled, 0 = disabled */
>> -#define ADIS16201_MSC_CTRL_DATA_RDY_EN	        BIT(2)
>> -
>> -/* Data-ready polarity: 1 = active high, 0 = active low */
>> -#define ADIS16201_MSC_CTRL_ACTIVE_HIGH	        BIT(1)
>> -
>> -/* Data-ready line selection: 1 = DIO1, 0 = DIO0 */
>> -#define ADIS16201_MSC_CTRL_DATA_RDY_DIO1	BIT(0)
>> -
>> -/* DIAG_STAT */
>> -
>> -/* Alarm 2 status: 1 = alarm active, 0 = alarm inactive */
>> -#define ADIS16201_DIAG_STAT_ALARM2        BIT(9)
>> -
>> -/* Alarm 1 status: 1 = alarm active, 0 = alarm inactive */
>> -#define ADIS16201_DIAG_STAT_ALARM1        BIT(8)
>> -
>> -/* SPI communications failure */
>> -#define ADIS16201_DIAG_STAT_SPI_FAIL_BIT   3
>> -
>> -/* Flash update failure */
>> -#define ADIS16201_DIAG_STAT_FLASH_UPT_BIT  2
>> -
>> -/* Power supply above 3.625 V */
>> -#define ADIS16201_DIAG_STAT_POWER_HIGH_BIT 1
>> -
>> -/* Power supply below 3.15 V */
>> -#define ADIS16201_DIAG_STAT_POWER_LOW_BIT  0
>> -
>> -/* GLOB_CMD */
>> -
>> -#define ADIS16201_GLOB_CMD_SW_RESET	BIT(7)
>> -#define ADIS16201_GLOB_CMD_FACTORY_CAL	BIT(1)
>> -
>> -#define ADIS16201_ERROR_ACTIVE          BIT(14)
>> -
>> -enum adis16201_scan {
>> -	ADIS16201_SCAN_ACC_X,
>> -	ADIS16201_SCAN_ACC_Y,
>> -	ADIS16201_SCAN_INCLI_X,
>> -	ADIS16201_SCAN_INCLI_Y,
>> -	ADIS16201_SCAN_SUPPLY,
>> -	ADIS16201_SCAN_AUX_ADC,
>> -	ADIS16201_SCAN_TEMP,
>> -};
>> -
>> -#endif /* SPI_ADIS16201_H_ */
>> diff --git a/drivers/staging/iio/accel/adis16201_core.c b/drivers/staging/iio/accel/adis16201_core.c
>> index 7963d4a..dcf8463 100644
>> --- a/drivers/staging/iio/accel/adis16201_core.c
>> +++ b/drivers/staging/iio/accel/adis16201_core.c
>> @@ -20,7 +20,147 @@
>>  #include <linux/iio/buffer.h>
>>  #include <linux/iio/imu/adis.h>
>>  
>> -#include "adis16201.h"
>> +#define ADIS16201_STARTUP_DELAY	220 /* ms */
>> +
>> +/* Flash memory write count */
>> +#define ADIS16201_FLASH_CNT      0x00
>> +
>> +/* Output, power supply */
>> +#define ADIS16201_SUPPLY_OUT     0x02
>> +
>> +/* Output, x-axis accelerometer */
>> +#define ADIS16201_XACCL_OUT      0x04
>> +
>> +/* Output, y-axis accelerometer */
>> +#define ADIS16201_YACCL_OUT      0x06
>> +
>> +/* Output, auxiliary ADC input */
>> +#define ADIS16201_AUX_ADC        0x08
>> +
>> +/* Output, temperature */
>> +#define ADIS16201_TEMP_OUT       0x0A
>> +
>> +/* Output, x-axis inclination */
>> +#define ADIS16201_XINCL_OUT      0x0C
>> +
>> +/* Output, y-axis inclination */
>> +#define ADIS16201_YINCL_OUT      0x0E
>> +
>> +/* Calibration, x-axis acceleration offset */
>> +#define ADIS16201_XACCL_OFFS     0x10
>> +
>> +/* Calibration, y-axis acceleration offset */
>> +#define ADIS16201_YACCL_OFFS     0x12
>> +
>> +/* x-axis acceleration scale factor */
>> +#define ADIS16201_XACCL_SCALE    0x14
>> +
>> +/* y-axis acceleration scale factor */
>> +#define ADIS16201_YACCL_SCALE    0x16
>> +
>> +/* Calibration, x-axis inclination offset */
>> +#define ADIS16201_XINCL_OFFS     0x18
>> +
>> +/* Calibration, y-axis inclination offset */
>> +#define ADIS16201_YINCL_OFFS     0x1A
>> +
>> +/* x-axis inclination scale factor */
>> +#define ADIS16201_XINCL_SCALE    0x1C
>> +
>> +/* y-axis inclination scale factor */
>> +#define ADIS16201_YINCL_SCALE    0x1E
>> +
>> +/* Alarm 1 amplitude threshold */
>> +#define ADIS16201_ALM_MAG1       0x20
>> +
>> +/* Alarm 2 amplitude threshold */
>> +#define ADIS16201_ALM_MAG2       0x22
>> +
>> +/* Alarm 1, sample period */
>> +#define ADIS16201_ALM_SMPL1      0x24
>> +
>> +/* Alarm 2, sample period */
>> +#define ADIS16201_ALM_SMPL2      0x26
>> +
>> +/* Alarm control */
>> +#define ADIS16201_ALM_CTRL       0x28
>> +
>> +/* Auxiliary DAC data */
>> +#define ADIS16201_AUX_DAC        0x30
>> +
>> +/* General-purpose digital input/output control */
>> +#define ADIS16201_GPIO_CTRL      0x32
>> +
>> +/* Miscellaneous control */
>> +#define ADIS16201_MSC_CTRL       0x34
>> +
>> +/* Internal sample period (rate) control */
>> +#define ADIS16201_SMPL_PRD       0x36
>> +
>> +/* Operation, filter configuration */
>> +#define ADIS16201_AVG_CNT        0x38
>> +
>> +/* Operation, sleep mode control */
>> +#define ADIS16201_SLP_CNT        0x3A
>> +
>> +/* Diagnostics, system status register */
>> +#define ADIS16201_DIAG_STAT      0x3C
>> +
>> +/* Operation, system command register */
>> +#define ADIS16201_GLOB_CMD       0x3E
>> +
>> +/* MSC_CTRL */
>> +
>> +/* Self-test enable */
>> +#define ADIS16201_MSC_CTRL_SELF_TEST_EN	        BIT(8)
>> +
>> +/* Data-ready enable: 1 = enabled, 0 = disabled */
>> +#define ADIS16201_MSC_CTRL_DATA_RDY_EN	        BIT(2)
>> +
>> +/* Data-ready polarity: 1 = active high, 0 = active low */
>> +#define ADIS16201_MSC_CTRL_ACTIVE_HIGH	        BIT(1)
>> +
>> +/* Data-ready line selection: 1 = DIO1, 0 = DIO0 */
>> +#define ADIS16201_MSC_CTRL_DATA_RDY_DIO1	BIT(0)
>> +
>> +/* DIAG_STAT */
>> +
>> +/* Alarm 2 status: 1 = alarm active, 0 = alarm inactive */
>> +#define ADIS16201_DIAG_STAT_ALARM2        BIT(9)
>> +
>> +/* Alarm 1 status: 1 = alarm active, 0 = alarm inactive */
>> +#define ADIS16201_DIAG_STAT_ALARM1        BIT(8)
>> +
>> +/* SPI communications failure */
>> +#define ADIS16201_DIAG_STAT_SPI_FAIL_BIT   3
>> +
>> +/* Flash update failure */
>> +#define ADIS16201_DIAG_STAT_FLASH_UPT_BIT  2
>> +
>> +/* Power supply above 3.625 V */
>> +#define ADIS16201_DIAG_STAT_POWER_HIGH_BIT 1
>> +
>> +/* Power supply below 3.15 V */
>> +#define ADIS16201_DIAG_STAT_POWER_LOW_BIT  0
>> +
>> +/* GLOB_CMD */
>> +
>> +#define ADIS16201_GLOB_CMD_SW_RESET	BIT(7)
>> +#define ADIS16201_GLOB_CMD_FACTORY_CAL	BIT(1)
>> +
>> +#define ADIS16201_ERROR_ACTIVE          BIT(14)
>> +
>> +enum adis16201_scan {
>> +	ADIS16201_SCAN_ACC_X,
>> +	ADIS16201_SCAN_ACC_Y,
>> +	ADIS16201_SCAN_INCLI_X,
>> +	ADIS16201_SCAN_INCLI_Y,
>> +	ADIS16201_SCAN_SUPPLY,
>> +	ADIS16201_SCAN_AUX_ADC,
>> +	ADIS16201_SCAN_TEMP,
>> +};
>> +
>> +#endif /* SPI_ADIS16201_H_ */
>>  
>>  static const u8 adis16201_addresses[] = {
>>  	[ADIS16201_SCAN_ACC_X] = ADIS16201_XACCL_OFFS,
>>
> 

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