* [PATCH v2] staging: iio: adis16203: Move header file content to source file.
@ 2017-02-28 18:33 Varsha Rao
2017-02-28 18:38 ` [Outreachy kernel] " Julia Lawall
0 siblings, 1 reply; 4+ messages in thread
From: Varsha Rao @ 2017-02-28 18:33 UTC (permalink / raw)
To: Lars-Peter Clausen, Michael Hennerich, Jonathan Cameron,
Hartmut Knaack, Peter Meerwald-Stadler, Greg Kroah-Hartman
Cc: outreachy-kernel
The contents of the header file are used only by this single source
file. Moved content into adis16203_core.c file and removed adis16203.h
file. Arranged #include files in alphabetical order.
Signed-off-by: Varsha Rao <rvarsha016@gmail.com>
---
Changes in v2:
- Modified commit message.
- #include files ordered alphabetically.
drivers/staging/iio/accel/adis16203.h | 125 --------------------------
drivers/staging/iio/accel/adis16203_core.c | 136 +++++++++++++++++++++++++++--
2 files changed, 128 insertions(+), 133 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..68189ad 100644
--- a/drivers/staging/iio/accel/adis16203_core.c
+++ b/drivers/staging/iio/accel/adis16203_core.c
@@ -7,20 +7,140 @@
*/
#include <linux/delay.h>
-#include <linux/mutex.h>
#include <linux/device.h>
+
+#include <linux/iio/buffer.h>
+#include <linux/iio/iio.h>
+#include <linux/iio/imu/adis.h>
+#include <linux/iio/sysfs.h>
+
#include <linux/kernel.h>
-#include <linux/spi/spi.h>
+#include <linux/module.h>
+#include <linux/mutex.h>
#include <linux/slab.h>
+#include <linux/spi/spi.h>
#include <linux/sysfs.h>
-#include <linux/module.h>
-#include <linux/iio/iio.h>
-#include <linux/iio/sysfs.h>
-#include <linux/iio/buffer.h>
-#include <linux/iio/imu/adis.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
-#include "adis16203.h"
+/* 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,
+};
#define DRIVER_NAME "adis16203"
--
2.9.3
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [Outreachy kernel] [PATCH v2] staging: iio: adis16203: Move header file content to source file.
2017-02-28 18:33 [PATCH v2] staging: iio: adis16203: Move header file content to source file Varsha Rao
@ 2017-02-28 18:38 ` Julia Lawall
2017-02-28 19:25 ` Jonathan Cameron
0 siblings, 1 reply; 4+ messages in thread
From: Julia Lawall @ 2017-02-28 18:38 UTC (permalink / raw)
To: Varsha Rao
Cc: Lars-Peter Clausen, Michael Hennerich, Jonathan Cameron,
Hartmut Knaack, Peter Meerwald-Stadler, Greg Kroah-Hartman,
outreachy-kernel
On Wed, 1 Mar 2017, Varsha Rao wrote:
> The contents of the header file are used only by this single source
> file. Moved content into adis16203_core.c file and removed adis16203.h
> file. Arranged #include files in alphabetical order.
I don't remember the previous commit message, but this one seems quite
nice. Concise, to the point, and gives a good overview of what is done.
julia
>
> Signed-off-by: Varsha Rao <rvarsha016@gmail.com>
> ---
> Changes in v2:
> - Modified commit message.
> - #include files ordered alphabetically.
>
> drivers/staging/iio/accel/adis16203.h | 125 --------------------------
> drivers/staging/iio/accel/adis16203_core.c | 136 +++++++++++++++++++++++++++--
> 2 files changed, 128 insertions(+), 133 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..68189ad 100644
> --- a/drivers/staging/iio/accel/adis16203_core.c
> +++ b/drivers/staging/iio/accel/adis16203_core.c
> @@ -7,20 +7,140 @@
> */
>
> #include <linux/delay.h>
> -#include <linux/mutex.h>
> #include <linux/device.h>
> +
> +#include <linux/iio/buffer.h>
> +#include <linux/iio/iio.h>
> +#include <linux/iio/imu/adis.h>
> +#include <linux/iio/sysfs.h>
> +
> #include <linux/kernel.h>
> -#include <linux/spi/spi.h>
> +#include <linux/module.h>
> +#include <linux/mutex.h>
> #include <linux/slab.h>
> +#include <linux/spi/spi.h>
> #include <linux/sysfs.h>
> -#include <linux/module.h>
>
> -#include <linux/iio/iio.h>
> -#include <linux/iio/sysfs.h>
> -#include <linux/iio/buffer.h>
> -#include <linux/iio/imu/adis.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
>
> -#include "adis16203.h"
> +/* 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,
> +};
>
> #define DRIVER_NAME "adis16203"
>
> --
> 2.9.3
>
> --
> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com.
> To post to this group, send email to outreachy-kernel@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/58b5c28f.4a44620a.8f50f.a6b6%40mx.google.com.
> For more options, visit https://groups.google.com/d/optout.
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Outreachy kernel] [PATCH v2] staging: iio: adis16203: Move header file content to source file.
2017-02-28 18:38 ` [Outreachy kernel] " Julia Lawall
@ 2017-02-28 19:25 ` Jonathan Cameron
2017-03-01 20:03 ` Julia Lawall
0 siblings, 1 reply; 4+ messages in thread
From: Jonathan Cameron @ 2017-02-28 19:25 UTC (permalink / raw)
To: Julia Lawall, Varsha Rao
Cc: Lars-Peter Clausen, Michael Hennerich, Hartmut Knaack,
Peter Meerwald-Stadler, Greg Kroah-Hartman, outreachy-kernel,
linux-iio
On 28/02/17 18:38, Julia Lawall wrote:
> On Wed, 1 Mar 2017, Varsha Rao wrote:
>
>> The contents of the header file are used only by this single source
>> file. Moved content into adis16203_core.c file and removed adis16203.h
>> file. Arranged #include files in alphabetical order.
>
> I don't remember the previous commit message, but this one seems quite
> nice. Concise, to the point, and gives a good overview of what is done.
Indeed. Very nice.
Applied to the togreg branch of iio.git and pushed out as testing for
the autobuilders to play with it.
One note though, please cc linux-iio@vger.kernel.org for IIO patches
including those effecting drivers in staging.
Thanks,
Jonathan
>
> julia
>
>>
>> Signed-off-by: Varsha Rao <rvarsha016@gmail.com>
>> ---
>> Changes in v2:
>> - Modified commit message.
>> - #include files ordered alphabetically.
>>
>> drivers/staging/iio/accel/adis16203.h | 125 --------------------------
>> drivers/staging/iio/accel/adis16203_core.c | 136 +++++++++++++++++++++++++++--
>> 2 files changed, 128 insertions(+), 133 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..68189ad 100644
>> --- a/drivers/staging/iio/accel/adis16203_core.c
>> +++ b/drivers/staging/iio/accel/adis16203_core.c
>> @@ -7,20 +7,140 @@
>> */
>>
>> #include <linux/delay.h>
>> -#include <linux/mutex.h>
>> #include <linux/device.h>
>> +
>> +#include <linux/iio/buffer.h>
>> +#include <linux/iio/iio.h>
>> +#include <linux/iio/imu/adis.h>
>> +#include <linux/iio/sysfs.h>
>> +
>> #include <linux/kernel.h>
>> -#include <linux/spi/spi.h>
>> +#include <linux/module.h>
>> +#include <linux/mutex.h>
>> #include <linux/slab.h>
>> +#include <linux/spi/spi.h>
>> #include <linux/sysfs.h>
>> -#include <linux/module.h>
>>
>> -#include <linux/iio/iio.h>
>> -#include <linux/iio/sysfs.h>
>> -#include <linux/iio/buffer.h>
>> -#include <linux/iio/imu/adis.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
>>
>> -#include "adis16203.h"
>> +/* 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,
>> +};
>>
>> #define DRIVER_NAME "adis16203"
>>
>> --
>> 2.9.3
>>
>> --
>> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
>> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com.
>> To post to this group, send email to outreachy-kernel@googlegroups.com.
>> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/58b5c28f.4a44620a.8f50f.a6b6%40mx.google.com.
>> For more options, visit https://groups.google.com/d/optout.
>>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Outreachy kernel] [PATCH v2] staging: iio: adis16203: Move header file content to source file.
2017-02-28 19:25 ` Jonathan Cameron
@ 2017-03-01 20:03 ` Julia Lawall
0 siblings, 0 replies; 4+ messages in thread
From: Julia Lawall @ 2017-03-01 20:03 UTC (permalink / raw)
To: Jonathan Cameron
Cc: Varsha Rao, Lars-Peter Clausen, Michael Hennerich,
Hartmut Knaack, Peter Meerwald-Stadler, Greg Kroah-Hartman,
outreachy-kernel, linux-iio
On Tue, 28 Feb 2017, Jonathan Cameron wrote:
> On 28/02/17 18:38, Julia Lawall wrote:
> > On Wed, 1 Mar 2017, Varsha Rao wrote:
> >
> >> The contents of the header file are used only by this single source
> >> file. Moved content into adis16203_core.c file and removed adis16203.h
> >> file. Arranged #include files in alphabetical order.
> >
> > I don't remember the previous commit message, but this one seems quite
> > nice. Concise, to the point, and gives a good overview of what is done.
> Indeed. Very nice.
>
> Applied to the togreg branch of iio.git and pushed out as testing for
> the autobuilders to play with it.
>
> One note though, please cc linux-iio@vger.kernel.org for IIO patches
> including those effecting drivers in staging.
I've added this information at a few more places in the tutorial...
julia
>
> Thanks,
>
> Jonathan
> >
> > julia
> >
> >>
> >> Signed-off-by: Varsha Rao <rvarsha016@gmail.com>
> >> ---
> >> Changes in v2:
> >> - Modified commit message.
> >> - #include files ordered alphabetically.
> >>
> >> drivers/staging/iio/accel/adis16203.h | 125 --------------------------
> >> drivers/staging/iio/accel/adis16203_core.c | 136 +++++++++++++++++++++++++++--
> >> 2 files changed, 128 insertions(+), 133 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..68189ad 100644
> >> --- a/drivers/staging/iio/accel/adis16203_core.c
> >> +++ b/drivers/staging/iio/accel/adis16203_core.c
> >> @@ -7,20 +7,140 @@
> >> */
> >>
> >> #include <linux/delay.h>
> >> -#include <linux/mutex.h>
> >> #include <linux/device.h>
> >> +
> >> +#include <linux/iio/buffer.h>
> >> +#include <linux/iio/iio.h>
> >> +#include <linux/iio/imu/adis.h>
> >> +#include <linux/iio/sysfs.h>
> >> +
> >> #include <linux/kernel.h>
> >> -#include <linux/spi/spi.h>
> >> +#include <linux/module.h>
> >> +#include <linux/mutex.h>
> >> #include <linux/slab.h>
> >> +#include <linux/spi/spi.h>
> >> #include <linux/sysfs.h>
> >> -#include <linux/module.h>
> >>
> >> -#include <linux/iio/iio.h>
> >> -#include <linux/iio/sysfs.h>
> >> -#include <linux/iio/buffer.h>
> >> -#include <linux/iio/imu/adis.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
> >>
> >> -#include "adis16203.h"
> >> +/* 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,
> >> +};
> >>
> >> #define DRIVER_NAME "adis16203"
> >>
> >> --
> >> 2.9.3
> >>
> >> --
> >> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> >> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com.
> >> To post to this group, send email to outreachy-kernel@googlegroups.com.
> >> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/58b5c28f.4a44620a.8f50f.a6b6%40mx.google.com.
> >> For more options, visit https://groups.google.com/d/optout.
> >>
>
> --
> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com.
> To post to this group, send email to outreachy-kernel@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/94389eae-1bde-de58-0f92-fb004a155fe4%40kernel.org.
> For more options, visit https://groups.google.com/d/optout.
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-03-01 20:03 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-02-28 18:33 [PATCH v2] staging: iio: adis16203: Move header file content to source file Varsha Rao
2017-02-28 18:38 ` [Outreachy kernel] " Julia Lawall
2017-02-28 19:25 ` Jonathan Cameron
2017-03-01 20:03 ` Julia Lawall
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.