On Wed, 21 Mar 2018 11:28:58 -0300
Hernán Gonzalez <[email protected]> wrote:

> Signed-off-by: Hernán Gonzalez <[email protected]>
Please look at what these do.  It is not what those abi elements are
documented as doing.

Also if it were you would need to support them via read_raw and write_raw
and put them in the relevant info mask elements.

Jonathan
> ---
>  drivers/staging/iio/cdc/ad7746.c | 30 +++++++++++++++---------------
>  1 file changed, 15 insertions(+), 15 deletions(-)
> 
> diff --git a/drivers/staging/iio/cdc/ad7746.c 
> b/drivers/staging/iio/cdc/ad7746.c
> index b6b99e2..c1f76fc 100644
> --- a/drivers/staging/iio/cdc/ad7746.c
> +++ b/drivers/staging/iio/cdc/ad7746.c
> @@ -336,16 +336,16 @@ static ssize_t ad7746_start_gain_calib(struct device 
> *dev,
>                                 AD7746_CONF_MODE_GAIN_CAL);
>  }
>  
> -static IIO_DEVICE_ATTR(in_capacitance0_calibbias_calibration,
> -                    0200, NULL, ad7746_start_offset_calib, CIN1);
> -static IIO_DEVICE_ATTR(in_capacitance1_calibbias_calibration,
> -                    0200, NULL, ad7746_start_offset_calib, CIN2);
> -static IIO_DEVICE_ATTR(in_capacitance0_calibscale_calibration,
> -                    0200, NULL, ad7746_start_gain_calib, CIN1);
> -static IIO_DEVICE_ATTR(in_capacitance1_calibscale_calibration,
> -                    0200, NULL, ad7746_start_gain_calib, CIN2);
> -static IIO_DEVICE_ATTR(in_voltage0_calibscale_calibration,
> -                    0200, NULL, ad7746_start_gain_calib, VIN);
> +static IIO_DEVICE_ATTR(in_capacitance0_calibbias, 0200, NULL,
> +                    ad7746_start_offset_calib, CIN1);
> +static IIO_DEVICE_ATTR(in_capacitance1_calibbias, 0200, NULL,
> +                    ad7746_start_offset_calib, CIN2);
> +static IIO_DEVICE_ATTR(in_capacitance0_calibscale, 0200, NULL,
> +                    ad7746_start_gain_calib, CIN1);
> +static IIO_DEVICE_ATTR(in_capacitance1_calibscale, 0200, NULL,
> +                    ad7746_start_gain_calib, CIN2);
> +static IIO_DEVICE_ATTR(in_voltage0_calibscale, 0200, NULL,
> +                    ad7746_start_gain_calib, VIN);
>  
>  static int ad7746_store_cap_filter_rate_setup(struct ad7746_chip_info *chip,
>                                             int val)
> @@ -388,11 +388,11 @@ static 
> IIO_CONST_ATTR(in_capacitance_sampling_frequency_available,
>                      "91 84 50 26 16 13 11 9");
>  
>  static struct attribute *ad7746_attributes[] = {
> -     &iio_dev_attr_in_capacitance0_calibbias_calibration.dev_attr.attr,
> -     &iio_dev_attr_in_capacitance0_calibscale_calibration.dev_attr.attr,
> -     &iio_dev_attr_in_capacitance1_calibscale_calibration.dev_attr.attr,
> -     &iio_dev_attr_in_capacitance1_calibbias_calibration.dev_attr.attr,
> -     &iio_dev_attr_in_voltage0_calibscale_calibration.dev_attr.attr,
> +     &iio_dev_attr_in_capacitance0_calibbias.dev_attr.attr,
> +     &iio_dev_attr_in_capacitance0_calibscale.dev_attr.attr,
> +     &iio_dev_attr_in_capacitance1_calibscale.dev_attr.attr,
> +     &iio_dev_attr_in_capacitance1_calibbias.dev_attr.attr,
> +     &iio_dev_attr_in_voltage0_calibscale.dev_attr.attr,
>       &iio_const_attr_in_voltage_sampling_frequency_available.dev_attr.attr,
>       
> &iio_const_attr_in_capacitance_sampling_frequency_available.dev_attr.attr,
>       NULL,

Reply via email to