Re: iio: Is storing output values to non volatile registers something we should do automatically or leave to userspace action. [was Re: [PATCH] iio: potentiometer: max5432: update the non-volatile pos

2019-08-22 Thread Phil Reid
On 19/08/2019 03:32, Jonathan Cameron wrote: On Mon, 12 Aug 2019 19:08:12 +0800 Phil Reid wrote: G'day Martin / Jonathan, On 12/08/2019 18:37, Martin Kaiser wrote: Hi Jonathan, Thus wrote Jonathan Cameron (ji...@kernel.org): The patch is fine, but I'm wondering about whether we need

Re: iio: Is storing output values to non volatile registers something we should do automatically or leave to userspace action. [was Re: [PATCH] iio: potentiometer: max5432: update the non-volatile pos

2019-08-18 Thread Jonathan Cameron
On Mon, 12 Aug 2019 19:08:12 +0800 Phil Reid wrote: > G'day Martin / Jonathan, > > On 12/08/2019 18:37, Martin Kaiser wrote: > > Hi Jonathan, > > > > Thus wrote Jonathan Cameron (ji...@kernel.org): > > > >> The patch is fine, but I'm wondering about whether we need some element > >> of

Re: iio: Is storing output values to non volatile registers something we should do automatically or leave to userspace action. [was Re: [PATCH] iio: potentiometer: max5432: update the non-volatile pos

2019-08-12 Thread Phil Reid
G'day Martin / Jonathan, On 12/08/2019 18:37, Martin Kaiser wrote: Hi Jonathan, Thus wrote Jonathan Cameron (ji...@kernel.org): The patch is fine, but I'm wondering about whether we need some element of policy control on this restore to current value. A few things to take into account.