linux-kernel - Re: [PATCH 1/2] iio: ad_sigma_delta: Implement a dedicated reset function

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Sun, 10 Sep 2017 16:22:17 +0100
From:   Jonathan Cameron <jic23@...nel.org>
To:     Lars-Peter Clausen <lars@...afoo.de>
Cc:     Dragos Bogdan <dragos.bogdan@...log.com>,
        Michael Hennerich <Michael.Hennerich@...log.com>,
        Hartmut Knaack <knaack.h@....de>,
        Peter Meerwald-Stadler <pmeerw@...erw.net>,
        linux-iio@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/2] iio: ad_sigma_delta: Implement a dedicated reset
 function

On Tue, 5 Sep 2017 14:16:42 +0200
Lars-Peter Clausen <lars@...afoo.de> wrote:

> On 09/05/2017 02:14 PM, Dragos Bogdan wrote:
> > Since most of the SD ADCs have the option of reseting the serial
> > interface by sending a number of SCLKs with CS = 0 and DIN = 1,
> > a dedicated function that can do this is usefull.
> > 
> > Signed-off-by: Dragos Bogdan <dragos.bogdan@...log.com>  
> 
> Acked-by: Lars-Peter Clausen <lars@...afoo.de>

Applied to the fixes-togreg branch of iio.git and marked
for stable.  I added a note that this was a precursor for
the following patch to justify the stable cc.

Jonathan
> 
> > ---
> >  drivers/iio/adc/ad_sigma_delta.c       | 28 ++++++++++++++++++++++++++++
> >  include/linux/iio/adc/ad_sigma_delta.h |  3 +++
> >  2 files changed, 31 insertions(+)
> > 
> > diff --git a/drivers/iio/adc/ad_sigma_delta.c b/drivers/iio/adc/ad_sigma_delta.c
> > index d10bd0c97233..22c4c17cd996 100644
> > --- a/drivers/iio/adc/ad_sigma_delta.c
> > +++ b/drivers/iio/adc/ad_sigma_delta.c
> > @@ -177,6 +177,34 @@ int ad_sd_read_reg(struct ad_sigma_delta *sigma_delta,
> >  }
> >  EXPORT_SYMBOL_GPL(ad_sd_read_reg);
> >  
> > +/**
> > + * ad_sd_reset() - Reset the serial interface
> > + *
> > + * @sigma_delta: The sigma delta device
> > + * @reset_length: Number of SCLKs with DIN = 1
> > + *
> > + * Returns 0 on success, an error code otherwise.
> > + **/
> > +int ad_sd_reset(struct ad_sigma_delta *sigma_delta,
> > +	unsigned int reset_length)
> > +{
> > +	uint8_t *buf;
> > +	unsigned int size;
> > +	int ret;
> > +
> > +	size = DIV_ROUND_UP(reset_length, 8);
> > +	buf = kcalloc(size, sizeof(*buf), GFP_KERNEL);
> > +	if (!buf)
> > +		return -ENOMEM;
> > +
> > +	memset(buf, 0xff, size);
> > +	ret = spi_write(sigma_delta->spi, buf, size);
> > +	kfree(buf);
> > +
> > +	return ret;
> > +}
> > +EXPORT_SYMBOL_GPL(ad_sd_reset);
> > +
> >  static int ad_sd_calibrate(struct ad_sigma_delta *sigma_delta,
> >  	unsigned int mode, unsigned int channel)
> >  {
> > diff --git a/include/linux/iio/adc/ad_sigma_delta.h b/include/linux/iio/adc/ad_sigma_delta.h
> > index 5ba430cc9a87..1fc7abd28b0b 100644
> > --- a/include/linux/iio/adc/ad_sigma_delta.h
> > +++ b/include/linux/iio/adc/ad_sigma_delta.h
> > @@ -111,6 +111,9 @@ int ad_sd_write_reg(struct ad_sigma_delta *sigma_delta, unsigned int reg,
> >  int ad_sd_read_reg(struct ad_sigma_delta *sigma_delta, unsigned int reg,
> >  	unsigned int size, unsigned int *val);
> >  
> > +int ad_sd_reset(struct ad_sigma_delta *sigma_delta,
> > +	unsigned int reset_length);
> > +
> >  int ad_sigma_delta_single_conversion(struct iio_dev *indio_dev,
> >  	const struct iio_chan_spec *chan, int *val);
> >  int ad_sd_calibrate_all(struct ad_sigma_delta *sigma_delta,
> >   
>