Re: [PATCH v2 7/7] iio:st_pressure: clean useless static channel initializers

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

 



On 27/06/16 11:38, Gregor Boirie wrote:
> Some static channels are explicitly initialized with default values.
> Remove them to enhance readability.
> 
> Signed-off-by: Gregor Boirie <gregor.boirie@xxxxxxxxxx>
Applied to the togreg branch of iio.git - initially pushed out as
testing for the autobuilders to play with it.

I perhaps got a little carried away with Linus' tested by tag and added
it even to those patches with only docs in them. Ah well - I'm sure he
tested them by reading the docs :)

Anyhow, thanks for these and good to get them in (hopefully) before the
end of this cycle.

Thanks,

Jonathan
> ---
>  drivers/iio/pressure/st_pressure_core.c | 10 ----------
>  1 file changed, 10 deletions(-)
> 
> diff --git a/drivers/iio/pressure/st_pressure_core.c b/drivers/iio/pressure/st_pressure_core.c
> index 274cdec..55df9a7 100644
> --- a/drivers/iio/pressure/st_pressure_core.c
> +++ b/drivers/iio/pressure/st_pressure_core.c
> @@ -224,7 +224,6 @@
>  static const struct iio_chan_spec st_press_1_channels[] = {
>  	{
>  		.type = IIO_PRESSURE,
> -		.channel2 = IIO_NO_MOD,
>  		.address = ST_PRESS_1_OUT_XL_ADDR,
>  		.scan_index = 0,
>  		.scan_type = {
> @@ -235,11 +234,9 @@ static const struct iio_chan_spec st_press_1_channels[] = {
>  		},
>  		.info_mask_separate =
>  			BIT(IIO_CHAN_INFO_RAW) | BIT(IIO_CHAN_INFO_SCALE),
> -		.modified = 0,
>  	},
>  	{
>  		.type = IIO_TEMP,
> -		.channel2 = IIO_NO_MOD,
>  		.address = ST_TEMP_1_OUT_L_ADDR,
>  		.scan_index = 1,
>  		.scan_type = {
> @@ -252,7 +249,6 @@ static const struct iio_chan_spec st_press_1_channels[] = {
>  			BIT(IIO_CHAN_INFO_RAW) |
>  			BIT(IIO_CHAN_INFO_SCALE) |
>  			BIT(IIO_CHAN_INFO_OFFSET),
> -		.modified = 0,
>  	},
>  	IIO_CHAN_SOFT_TIMESTAMP(2)
>  };
> @@ -260,7 +256,6 @@ static const struct iio_chan_spec st_press_1_channels[] = {
>  static const struct iio_chan_spec st_press_lps001wp_channels[] = {
>  	{
>  		.type = IIO_PRESSURE,
> -		.channel2 = IIO_NO_MOD,
>  		.address = ST_PRESS_LPS001WP_OUT_L_ADDR,
>  		.scan_index = 0,
>  		.scan_type = {
> @@ -272,11 +267,9 @@ static const struct iio_chan_spec st_press_lps001wp_channels[] = {
>  		.info_mask_separate =
>  			BIT(IIO_CHAN_INFO_RAW) |
>  			BIT(IIO_CHAN_INFO_SCALE),
> -		.modified = 0,
>  	},
>  	{
>  		.type = IIO_TEMP,
> -		.channel2 = IIO_NO_MOD,
>  		.address = ST_TEMP_LPS001WP_OUT_L_ADDR,
>  		.scan_index = 1,
>  		.scan_type = {
> @@ -288,7 +281,6 @@ static const struct iio_chan_spec st_press_lps001wp_channels[] = {
>  		.info_mask_separate =
>  			BIT(IIO_CHAN_INFO_RAW) |
>  			BIT(IIO_CHAN_INFO_SCALE),
> -		.modified = 0,
>  	},
>  	IIO_CHAN_SOFT_TIMESTAMP(2)
>  };
> @@ -296,7 +288,6 @@ static const struct iio_chan_spec st_press_lps001wp_channels[] = {
>  static const struct iio_chan_spec st_press_lps22hb_channels[] = {
>  	{
>  		.type = IIO_PRESSURE,
> -		.channel2 = IIO_NO_MOD,
>  		.address = ST_PRESS_1_OUT_XL_ADDR,
>  		.scan_index = 0,
>  		.scan_type = {
> @@ -309,7 +300,6 @@ static const struct iio_chan_spec st_press_lps22hb_channels[] = {
>  			BIT(IIO_CHAN_INFO_RAW) |
>  			BIT(IIO_CHAN_INFO_SCALE),
>  		.info_mask_shared_by_all = BIT(IIO_CHAN_INFO_SAMP_FREQ),
> -		.modified = 0,
>  	},
>  	{
>  		.type = IIO_TEMP,
> 

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