From: Jonathan Cameron <jic23@kernel.org>
To: Himanshu Jha <himanshujha199640@gmail.com>
Cc: lars@metafoo.de, Michael.Hennerich@analog.com, pmeerw@pmeerw.net,
gregkh@linuxfoundation.org, linux-iio@vger.kernel.org,
devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org,
daniel.baluta@gmail.com, 21cnbao@gmail.com
Subject: Re: [PATCH v2 3/9] Staging: iio: accel: adis16201: Add _REG suffix to reisters
Date: Sun, 18 Mar 2018 08:24:47 +0000 [thread overview]
Message-ID: <20180318082447.5cc8d661@archlinux> (raw)
In-Reply-To: <1521230786-18155-4-git-send-email-himanshujha199640@gmail.com>
On Sat, 17 Mar 2018 01:36:20 +0530
Himanshu Jha <himanshujha199640@gmail.com> wrote:
> Add a _REG suffix to distinguish between registers and the register bit
> fileds.
>
> Signed-off-by: Himanshu Jha <himanshujha199640@gmail.com>
Other than the typo in the patch title, this looks good (I fixed that up when
applying). Applied.
Jonathan
> ---
> v2:
> -reordered patch series.
>
> drivers/staging/iio/accel/adis16201.c | 84 +++++++++++++++++------------------
> 1 file changed, 42 insertions(+), 42 deletions(-)
>
> diff --git a/drivers/staging/iio/accel/adis16201.c b/drivers/staging/iio/accel/adis16201.c
> index 6c06c0d..0c63cd0 100644
> --- a/drivers/staging/iio/accel/adis16201.c
> +++ b/drivers/staging/iio/accel/adis16201.c
> @@ -24,62 +24,62 @@
>
> #define ADIS16201_FLASH_CNT 0x00
>
> -#define ADIS16201_SUPPLY_OUT 0x02
> +#define ADIS16201_SUPPLY_OUT_REG 0x02
>
> -#define ADIS16201_XACCL_OUT 0x04
> +#define ADIS16201_XACCL_OUT_REG 0x04
>
> -#define ADIS16201_YACCL_OUT 0x06
> +#define ADIS16201_YACCL_OUT_REG 0x06
>
> -#define ADIS16201_AUX_ADC 0x08
> +#define ADIS16201_AUX_ADC_REG 0x08
>
> -#define ADIS16201_TEMP_OUT 0x0A
> +#define ADIS16201_TEMP_OUT_REG 0x0A
>
> -#define ADIS16201_XINCL_OUT 0x0C
> +#define ADIS16201_XINCL_OUT_REG 0x0C
>
> -#define ADIS16201_YINCL_OUT 0x0E
> +#define ADIS16201_YINCL_OUT_REG 0x0E
>
> -#define ADIS16201_XACCL_OFFS 0x10
> +#define ADIS16201_XACCL_OFFS_REG 0x10
>
> -#define ADIS16201_YACCL_OFFS 0x12
> +#define ADIS16201_YACCL_OFFS_REG 0x12
>
> -#define ADIS16201_XACCL_SCALE 0x14
> +#define ADIS16201_XACCL_SCALE_REG 0x14
>
> -#define ADIS16201_YACCL_SCALE 0x16
> +#define ADIS16201_YACCL_SCALE_REG 0x16
>
> -#define ADIS16201_XINCL_OFFS 0x18
> +#define ADIS16201_XINCL_OFFS_REG 0x18
>
> -#define ADIS16201_YINCL_OFFS 0x1A
> +#define ADIS16201_YINCL_OFFS_REG 0x1A
>
> -#define ADIS16201_XINCL_SCALE 0x1C
> +#define ADIS16201_XINCL_SCALE_REG 0x1C
>
> -#define ADIS16201_YINCL_SCALE 0x1E
> +#define ADIS16201_YINCL_SCALE_REG 0x1E
>
> -#define ADIS16201_ALM_MAG1 0x20
> +#define ADIS16201_ALM_MAG1_REG 0x20
>
> -#define ADIS16201_ALM_MAG2 0x22
> +#define ADIS16201_ALM_MAG2_REG 0x22
>
> -#define ADIS16201_ALM_SMPL1 0x24
> +#define ADIS16201_ALM_SMPL1_REG 0x24
>
> -#define ADIS16201_ALM_SMPL2 0x26
> +#define ADIS16201_ALM_SMPL2_REG 0x26
>
> -#define ADIS16201_ALM_CTRL 0x28
> +#define ADIS16201_ALM_CTRL_REG 0x28
>
> -#define ADIS16201_AUX_DAC 0x30
> +#define ADIS16201_AUX_DAC_REG 0x30
>
> -#define ADIS16201_GPIO_CTRL 0x32
> +#define ADIS16201_GPIO_CTRL_REG 0x32
>
> -#define ADIS16201_MSC_CTRL 0x34
> +#define ADIS16201_MSC_CTRL_REG 0x34
>
> -#define ADIS16201_SMPL_PRD 0x36
> +#define ADIS16201_SMPL_PRD_REG 0x36
>
> /* Operation, filter configuration */
> -#define ADIS16201_AVG_CNT 0x38
> +#define ADIS16201_AVG_CNT_REG 0x38
>
> -#define ADIS16201_SLP_CNT 0x3A
> +#define ADIS16201_SLP_CNT_REG 0x3A
>
> -#define ADIS16201_DIAG_STAT 0x3C
> +#define ADIS16201_DIAG_STAT_REG 0x3C
>
> -#define ADIS16201_GLOB_CMD 0x3E
> +#define ADIS16201_GLOB_CMD_REG 0x3E
>
>
> #define ADIS16201_MSC_CTRL_SELF_TEST_EN BIT(8)
> @@ -125,10 +125,10 @@ enum adis16201_scan {
> };
>
> static const u8 adis16201_addresses[] = {
> - [ADIS16201_SCAN_ACC_X] = ADIS16201_XACCL_OFFS,
> - [ADIS16201_SCAN_ACC_Y] = ADIS16201_YACCL_OFFS,
> - [ADIS16201_SCAN_INCLI_X] = ADIS16201_XINCL_OFFS,
> - [ADIS16201_SCAN_INCLI_Y] = ADIS16201_YINCL_OFFS,
> + [ADIS16201_SCAN_ACC_X] = ADIS16201_XACCL_OFFS_REG,
> + [ADIS16201_SCAN_ACC_Y] = ADIS16201_YACCL_OFFS_REG,
> + [ADIS16201_SCAN_INCLI_X] = ADIS16201_XINCL_OFFS_REG,
> + [ADIS16201_SCAN_INCLI_Y] = ADIS16201_YINCL_OFFS_REG,
> };
>
> static int adis16201_read_raw(struct iio_dev *indio_dev,
> @@ -232,16 +232,16 @@ static int adis16201_write_raw(struct iio_dev *indio_dev,
> }
>
> static const struct iio_chan_spec adis16201_channels[] = {
> - ADIS_SUPPLY_CHAN(ADIS16201_SUPPLY_OUT, ADIS16201_SCAN_SUPPLY, 0, 12),
> - ADIS_TEMP_CHAN(ADIS16201_TEMP_OUT, ADIS16201_SCAN_TEMP, 0, 12),
> - ADIS_ACCEL_CHAN(X, ADIS16201_XACCL_OUT, ADIS16201_SCAN_ACC_X,
> + ADIS_SUPPLY_CHAN(ADIS16201_SUPPLY_OUT_REG, ADIS16201_SCAN_SUPPLY, 0, 12),
> + ADIS_TEMP_CHAN(ADIS16201_TEMP_OUT_REG, ADIS16201_SCAN_TEMP, 0, 12),
> + ADIS_ACCEL_CHAN(X, ADIS16201_XACCL_OUT_REG, ADIS16201_SCAN_ACC_X,
> BIT(IIO_CHAN_INFO_CALIBBIAS), 0, 14),
> - ADIS_ACCEL_CHAN(Y, ADIS16201_YACCL_OUT, ADIS16201_SCAN_ACC_Y,
> + ADIS_ACCEL_CHAN(Y, ADIS16201_YACCL_OUT_REG, ADIS16201_SCAN_ACC_Y,
> BIT(IIO_CHAN_INFO_CALIBBIAS), 0, 14),
> - ADIS_AUX_ADC_CHAN(ADIS16201_AUX_ADC, ADIS16201_SCAN_AUX_ADC, 0, 12),
> - ADIS_INCLI_CHAN(X, ADIS16201_XINCL_OUT, ADIS16201_SCAN_INCLI_X,
> + ADIS_AUX_ADC_CHAN(ADIS16201_AUX_ADC_REG, ADIS16201_SCAN_AUX_ADC, 0, 12),
> + ADIS_INCLI_CHAN(X, ADIS16201_XINCL_OUT_REG, ADIS16201_SCAN_INCLI_X,
> BIT(IIO_CHAN_INFO_CALIBBIAS), 0, 14),
> - ADIS_INCLI_CHAN(X, ADIS16201_YINCL_OUT, ADIS16201_SCAN_INCLI_Y,
> + ADIS_INCLI_CHAN(X, ADIS16201_YINCL_OUT_REG, ADIS16201_SCAN_INCLI_Y,
> BIT(IIO_CHAN_INFO_CALIBBIAS), 0, 14),
> IIO_CHAN_SOFT_TIMESTAMP(7)
> };
> @@ -261,9 +261,9 @@ static const char * const adis16201_status_error_msgs[] = {
>
> static const struct adis_data adis16201_data = {
> .read_delay = 20,
> - .msc_ctrl_reg = ADIS16201_MSC_CTRL,
> - .glob_cmd_reg = ADIS16201_GLOB_CMD,
> - .diag_stat_reg = ADIS16201_DIAG_STAT,
> + .msc_ctrl_reg = ADIS16201_MSC_CTRL_REG,
> + .glob_cmd_reg = ADIS16201_GLOB_CMD_REG,
> + .diag_stat_reg = ADIS16201_DIAG_STAT_REG,
>
> .self_test_mask = ADIS16201_MSC_CTRL_SELF_TEST_EN,
> .self_test_no_autoclear = true,
WARNING: multiple messages have this Message-ID (diff)
From: Jonathan Cameron <jic23@kernel.org>
To: Himanshu Jha <himanshujha199640@gmail.com>
Cc: devel@driverdev.osuosl.org, daniel.baluta@gmail.com,
lars@metafoo.de, Michael.Hennerich@analog.com,
linux-iio@vger.kernel.org, gregkh@linuxfoundation.org,
21cnbao@gmail.com, linux-kernel@vger.kernel.org,
pmeerw@pmeerw.net
Subject: Re: [PATCH v2 3/9] Staging: iio: accel: adis16201: Add _REG suffix to reisters
Date: Sun, 18 Mar 2018 08:24:47 +0000 [thread overview]
Message-ID: <20180318082447.5cc8d661@archlinux> (raw)
In-Reply-To: <1521230786-18155-4-git-send-email-himanshujha199640@gmail.com>
On Sat, 17 Mar 2018 01:36:20 +0530
Himanshu Jha <himanshujha199640@gmail.com> wrote:
> Add a _REG suffix to distinguish between registers and the register bit
> fileds.
>
> Signed-off-by: Himanshu Jha <himanshujha199640@gmail.com>
Other than the typo in the patch title, this looks good (I fixed that up when
applying). Applied.
Jonathan
> ---
> v2:
> -reordered patch series.
>
> drivers/staging/iio/accel/adis16201.c | 84 +++++++++++++++++------------------
> 1 file changed, 42 insertions(+), 42 deletions(-)
>
> diff --git a/drivers/staging/iio/accel/adis16201.c b/drivers/staging/iio/accel/adis16201.c
> index 6c06c0d..0c63cd0 100644
> --- a/drivers/staging/iio/accel/adis16201.c
> +++ b/drivers/staging/iio/accel/adis16201.c
> @@ -24,62 +24,62 @@
>
> #define ADIS16201_FLASH_CNT 0x00
>
> -#define ADIS16201_SUPPLY_OUT 0x02
> +#define ADIS16201_SUPPLY_OUT_REG 0x02
>
> -#define ADIS16201_XACCL_OUT 0x04
> +#define ADIS16201_XACCL_OUT_REG 0x04
>
> -#define ADIS16201_YACCL_OUT 0x06
> +#define ADIS16201_YACCL_OUT_REG 0x06
>
> -#define ADIS16201_AUX_ADC 0x08
> +#define ADIS16201_AUX_ADC_REG 0x08
>
> -#define ADIS16201_TEMP_OUT 0x0A
> +#define ADIS16201_TEMP_OUT_REG 0x0A
>
> -#define ADIS16201_XINCL_OUT 0x0C
> +#define ADIS16201_XINCL_OUT_REG 0x0C
>
> -#define ADIS16201_YINCL_OUT 0x0E
> +#define ADIS16201_YINCL_OUT_REG 0x0E
>
> -#define ADIS16201_XACCL_OFFS 0x10
> +#define ADIS16201_XACCL_OFFS_REG 0x10
>
> -#define ADIS16201_YACCL_OFFS 0x12
> +#define ADIS16201_YACCL_OFFS_REG 0x12
>
> -#define ADIS16201_XACCL_SCALE 0x14
> +#define ADIS16201_XACCL_SCALE_REG 0x14
>
> -#define ADIS16201_YACCL_SCALE 0x16
> +#define ADIS16201_YACCL_SCALE_REG 0x16
>
> -#define ADIS16201_XINCL_OFFS 0x18
> +#define ADIS16201_XINCL_OFFS_REG 0x18
>
> -#define ADIS16201_YINCL_OFFS 0x1A
> +#define ADIS16201_YINCL_OFFS_REG 0x1A
>
> -#define ADIS16201_XINCL_SCALE 0x1C
> +#define ADIS16201_XINCL_SCALE_REG 0x1C
>
> -#define ADIS16201_YINCL_SCALE 0x1E
> +#define ADIS16201_YINCL_SCALE_REG 0x1E
>
> -#define ADIS16201_ALM_MAG1 0x20
> +#define ADIS16201_ALM_MAG1_REG 0x20
>
> -#define ADIS16201_ALM_MAG2 0x22
> +#define ADIS16201_ALM_MAG2_REG 0x22
>
> -#define ADIS16201_ALM_SMPL1 0x24
> +#define ADIS16201_ALM_SMPL1_REG 0x24
>
> -#define ADIS16201_ALM_SMPL2 0x26
> +#define ADIS16201_ALM_SMPL2_REG 0x26
>
> -#define ADIS16201_ALM_CTRL 0x28
> +#define ADIS16201_ALM_CTRL_REG 0x28
>
> -#define ADIS16201_AUX_DAC 0x30
> +#define ADIS16201_AUX_DAC_REG 0x30
>
> -#define ADIS16201_GPIO_CTRL 0x32
> +#define ADIS16201_GPIO_CTRL_REG 0x32
>
> -#define ADIS16201_MSC_CTRL 0x34
> +#define ADIS16201_MSC_CTRL_REG 0x34
>
> -#define ADIS16201_SMPL_PRD 0x36
> +#define ADIS16201_SMPL_PRD_REG 0x36
>
> /* Operation, filter configuration */
> -#define ADIS16201_AVG_CNT 0x38
> +#define ADIS16201_AVG_CNT_REG 0x38
>
> -#define ADIS16201_SLP_CNT 0x3A
> +#define ADIS16201_SLP_CNT_REG 0x3A
>
> -#define ADIS16201_DIAG_STAT 0x3C
> +#define ADIS16201_DIAG_STAT_REG 0x3C
>
> -#define ADIS16201_GLOB_CMD 0x3E
> +#define ADIS16201_GLOB_CMD_REG 0x3E
>
>
> #define ADIS16201_MSC_CTRL_SELF_TEST_EN BIT(8)
> @@ -125,10 +125,10 @@ enum adis16201_scan {
> };
>
> static const u8 adis16201_addresses[] = {
> - [ADIS16201_SCAN_ACC_X] = ADIS16201_XACCL_OFFS,
> - [ADIS16201_SCAN_ACC_Y] = ADIS16201_YACCL_OFFS,
> - [ADIS16201_SCAN_INCLI_X] = ADIS16201_XINCL_OFFS,
> - [ADIS16201_SCAN_INCLI_Y] = ADIS16201_YINCL_OFFS,
> + [ADIS16201_SCAN_ACC_X] = ADIS16201_XACCL_OFFS_REG,
> + [ADIS16201_SCAN_ACC_Y] = ADIS16201_YACCL_OFFS_REG,
> + [ADIS16201_SCAN_INCLI_X] = ADIS16201_XINCL_OFFS_REG,
> + [ADIS16201_SCAN_INCLI_Y] = ADIS16201_YINCL_OFFS_REG,
> };
>
> static int adis16201_read_raw(struct iio_dev *indio_dev,
> @@ -232,16 +232,16 @@ static int adis16201_write_raw(struct iio_dev *indio_dev,
> }
>
> static const struct iio_chan_spec adis16201_channels[] = {
> - ADIS_SUPPLY_CHAN(ADIS16201_SUPPLY_OUT, ADIS16201_SCAN_SUPPLY, 0, 12),
> - ADIS_TEMP_CHAN(ADIS16201_TEMP_OUT, ADIS16201_SCAN_TEMP, 0, 12),
> - ADIS_ACCEL_CHAN(X, ADIS16201_XACCL_OUT, ADIS16201_SCAN_ACC_X,
> + ADIS_SUPPLY_CHAN(ADIS16201_SUPPLY_OUT_REG, ADIS16201_SCAN_SUPPLY, 0, 12),
> + ADIS_TEMP_CHAN(ADIS16201_TEMP_OUT_REG, ADIS16201_SCAN_TEMP, 0, 12),
> + ADIS_ACCEL_CHAN(X, ADIS16201_XACCL_OUT_REG, ADIS16201_SCAN_ACC_X,
> BIT(IIO_CHAN_INFO_CALIBBIAS), 0, 14),
> - ADIS_ACCEL_CHAN(Y, ADIS16201_YACCL_OUT, ADIS16201_SCAN_ACC_Y,
> + ADIS_ACCEL_CHAN(Y, ADIS16201_YACCL_OUT_REG, ADIS16201_SCAN_ACC_Y,
> BIT(IIO_CHAN_INFO_CALIBBIAS), 0, 14),
> - ADIS_AUX_ADC_CHAN(ADIS16201_AUX_ADC, ADIS16201_SCAN_AUX_ADC, 0, 12),
> - ADIS_INCLI_CHAN(X, ADIS16201_XINCL_OUT, ADIS16201_SCAN_INCLI_X,
> + ADIS_AUX_ADC_CHAN(ADIS16201_AUX_ADC_REG, ADIS16201_SCAN_AUX_ADC, 0, 12),
> + ADIS_INCLI_CHAN(X, ADIS16201_XINCL_OUT_REG, ADIS16201_SCAN_INCLI_X,
> BIT(IIO_CHAN_INFO_CALIBBIAS), 0, 14),
> - ADIS_INCLI_CHAN(X, ADIS16201_YINCL_OUT, ADIS16201_SCAN_INCLI_Y,
> + ADIS_INCLI_CHAN(X, ADIS16201_YINCL_OUT_REG, ADIS16201_SCAN_INCLI_Y,
> BIT(IIO_CHAN_INFO_CALIBBIAS), 0, 14),
> IIO_CHAN_SOFT_TIMESTAMP(7)
> };
> @@ -261,9 +261,9 @@ static const char * const adis16201_status_error_msgs[] = {
>
> static const struct adis_data adis16201_data = {
> .read_delay = 20,
> - .msc_ctrl_reg = ADIS16201_MSC_CTRL,
> - .glob_cmd_reg = ADIS16201_GLOB_CMD,
> - .diag_stat_reg = ADIS16201_DIAG_STAT,
> + .msc_ctrl_reg = ADIS16201_MSC_CTRL_REG,
> + .glob_cmd_reg = ADIS16201_GLOB_CMD_REG,
> + .diag_stat_reg = ADIS16201_DIAG_STAT_REG,
>
> .self_test_mask = ADIS16201_MSC_CTRL_SELF_TEST_EN,
> .self_test_no_autoclear = true,
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
next prev parent reply other threads:[~2018-03-18 8:24 UTC|newest]
Thread overview: 55+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-16 20:06 [PATCH v2 0/9] Staging: iio: accel: adis16201 driver cleanup Himanshu Jha
2018-03-16 20:06 ` Himanshu Jha
2018-03-16 20:06 ` [PATCH v2 1/9] Staging: iio: accel: adis16201: Rename few macro definitions Himanshu Jha
2018-03-16 20:06 ` Himanshu Jha
2018-03-18 8:19 ` Jonathan Cameron
2018-03-16 20:06 ` [PATCH v2 2/9] Staging: iio: accel: adis16201: Remove unnecessary comments Himanshu Jha
2018-03-16 20:06 ` Himanshu Jha
2018-03-18 8:22 ` Jonathan Cameron
2018-03-18 8:22 ` Jonathan Cameron
2018-03-16 20:06 ` [PATCH v2 3/9] Staging: iio: accel: adis16201: Add _REG suffix to reisters Himanshu Jha
2018-03-16 20:06 ` Himanshu Jha
2018-03-18 8:24 ` Jonathan Cameron [this message]
2018-03-18 8:24 ` Jonathan Cameron
2018-03-16 20:06 ` [PATCH v2 4/9] Staging: iio: accel: adis16201: Group register definitions Himanshu Jha
2018-03-16 20:06 ` Himanshu Jha
2018-03-18 8:29 ` Jonathan Cameron
2018-03-18 8:29 ` Jonathan Cameron
2018-03-16 20:06 ` [PATCH v2 5/9] Staging: iio: accel: adis16201: Add comments about units in read_raw() Himanshu Jha
2018-03-16 20:06 ` Himanshu Jha
2018-03-18 8:32 ` Jonathan Cameron
2018-03-18 8:32 ` Jonathan Cameron
2018-03-16 20:06 ` [PATCH v2 6/9] Staging: iio: accel: adis16201: Use sign_extend32 function Himanshu Jha
2018-03-16 20:06 ` Himanshu Jha
2018-03-18 8:33 ` Jonathan Cameron
2018-03-18 8:33 ` Jonathan Cameron
2018-03-16 20:06 ` [PATCH v2 7/9] Staging: iio: accel: adis16201: Prefer reverse christmas tree ordering Himanshu Jha
2018-03-16 20:06 ` Himanshu Jha
2018-03-18 8:34 ` Jonathan Cameron
2018-03-18 8:34 ` Jonathan Cameron
2018-03-16 20:06 ` [PATCH v2 8/9] Staging: iio: accel: adis16201: Adjust argument to match open parentheses Himanshu Jha
2018-03-16 20:06 ` Himanshu Jha
2018-03-18 8:35 ` Jonathan Cameron
2018-03-18 8:35 ` Jonathan Cameron
2018-03-16 20:06 ` [PATCH v2 9/9] Staging: iio: accel: adis16201: Move adis16201 driver out of staging Himanshu Jha
2018-03-16 20:06 ` Himanshu Jha
2018-03-18 9:31 ` Jonathan Cameron
2018-03-18 9:31 ` Jonathan Cameron
2018-03-22 19:12 ` [PATCH 1/4] Staging: iio: accel: adis16201: Remove unused headers Himanshu Jha
2018-03-22 19:12 ` Himanshu Jha
2018-03-22 19:12 ` [PATCH 2/4] Staging: iio: accel: adis16201: Use GENMASK Himanshu Jha
2018-03-22 19:12 ` Himanshu Jha
2018-03-24 14:16 ` Jonathan Cameron
2018-03-24 14:16 ` Jonathan Cameron
2018-03-22 19:12 ` [PATCH 3/4] Staging: iio: accel: adis16201: Fix 80 character line limit Himanshu Jha
2018-03-22 19:12 ` Himanshu Jha
2018-03-24 14:16 ` Jonathan Cameron
2018-03-24 14:16 ` Jonathan Cameron
2018-03-22 19:12 ` [PATCH 4/4] Staging: iio: accel: adis16201: Move adis16201 driver out of staging Himanshu Jha
2018-03-22 19:12 ` Himanshu Jha
2018-03-24 14:26 ` Jonathan Cameron
2018-03-24 14:26 ` Jonathan Cameron
2018-03-23 13:32 ` [PATCH 1/4] Staging: iio: accel: adis16201: Remove unused headers Jonathan Cameron
2018-03-23 13:32 ` Jonathan Cameron
2018-03-24 14:15 ` Jonathan Cameron
2018-03-24 14:15 ` Jonathan Cameron
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180318082447.5cc8d661@archlinux \
--to=jic23@kernel.org \
--cc=21cnbao@gmail.com \
--cc=Michael.Hennerich@analog.com \
--cc=daniel.baluta@gmail.com \
--cc=devel@driverdev.osuosl.org \
--cc=gregkh@linuxfoundation.org \
--cc=himanshujha199640@gmail.com \
--cc=lars@metafoo.de \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=pmeerw@pmeerw.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.