Re: [PATCH 1/3] iio: adc: add support for mcp3911

2018-07-23 Thread Jonathan Cameron
On Sun, 22 Jul 2018 21:00:51 +0200 Marcus Folkesson wrote: > Hi Jonathan, > > Thanks, all good catches. > > On Sun, Jul 22, 2018 at 09:08:38AM +0100, Jonathan Cameron wrote: > > On Sat, 21 Jul 2018 23:19:48 +0200 (CEST) > > Peter Meerwald-Stadler wrote: > > > > > Hello, > > > > > > >

Re: [PATCH 1/3] iio: adc: add support for mcp3911

2018-07-23 Thread Jonathan Cameron
On Sun, 22 Jul 2018 21:00:51 +0200 Marcus Folkesson wrote: > Hi Jonathan, > > Thanks, all good catches. > > On Sun, Jul 22, 2018 at 09:08:38AM +0100, Jonathan Cameron wrote: > > On Sat, 21 Jul 2018 23:19:48 +0200 (CEST) > > Peter Meerwald-Stadler wrote: > > > > > Hello, > > > > > > >

Re: [PATCH 1/3] iio: adc: add support for mcp3911

2018-07-22 Thread Marcus Folkesson
Hi Jonathan, Thanks, all good catches. On Sun, Jul 22, 2018 at 09:08:38AM +0100, Jonathan Cameron wrote: > On Sat, 21 Jul 2018 23:19:48 +0200 (CEST) > Peter Meerwald-Stadler wrote: > > > Hello, > > > > > MCP3911 is a dual channel Analog Front End (AFE) containing two > > > synchronous

Re: [PATCH 1/3] iio: adc: add support for mcp3911

2018-07-22 Thread Marcus Folkesson
Hi Jonathan, Thanks, all good catches. On Sun, Jul 22, 2018 at 09:08:38AM +0100, Jonathan Cameron wrote: > On Sat, 21 Jul 2018 23:19:48 +0200 (CEST) > Peter Meerwald-Stadler wrote: > > > Hello, > > > > > MCP3911 is a dual channel Analog Front End (AFE) containing two > > > synchronous

Re: [PATCH 1/3] iio: adc: add support for mcp3911

2018-07-22 Thread Marcus Folkesson
Hi Peter, Thanks for the comments! On Sat, Jul 21, 2018 at 11:19:48PM +0200, Peter Meerwald-Stadler wrote: > Hello, > > > MCP3911 is a dual channel Analog Front End (AFE) containing two > > synchronous sampling delta-sigma Analog-to-Digital Converters (ADC). > > some comments below... > > >

Re: [PATCH 1/3] iio: adc: add support for mcp3911

2018-07-22 Thread Marcus Folkesson
Hi Peter, Thanks for the comments! On Sat, Jul 21, 2018 at 11:19:48PM +0200, Peter Meerwald-Stadler wrote: > Hello, > > > MCP3911 is a dual channel Analog Front End (AFE) containing two > > synchronous sampling delta-sigma Analog-to-Digital Converters (ADC). > > some comments below... > > >

Re: [PATCH 1/3] iio: adc: add support for mcp3911

2018-07-22 Thread Jonathan Cameron
On Sat, 21 Jul 2018 23:19:48 +0200 (CEST) Peter Meerwald-Stadler wrote: > Hello, > > > MCP3911 is a dual channel Analog Front End (AFE) containing two > > synchronous sampling delta-sigma Analog-to-Digital Converters (ADC). > > some comments below... +CC Mark for the unusual SPI addressing

Re: [PATCH 1/3] iio: adc: add support for mcp3911

2018-07-22 Thread Jonathan Cameron
On Sat, 21 Jul 2018 23:19:48 +0200 (CEST) Peter Meerwald-Stadler wrote: > Hello, > > > MCP3911 is a dual channel Analog Front End (AFE) containing two > > synchronous sampling delta-sigma Analog-to-Digital Converters (ADC). > > some comments below... +CC Mark for the unusual SPI addressing

Re: [PATCH 1/3] iio: adc: add support for mcp3911

2018-07-21 Thread Peter Meerwald-Stadler
Hello, > MCP3911 is a dual channel Analog Front End (AFE) containing two > synchronous sampling delta-sigma Analog-to-Digital Converters (ADC). some comments below... > Signed-off-by: Marcus Folkesson > Signed-off-by: Kent Gustavsson > --- > drivers/iio/adc/Kconfig | 10 ++ >

Re: [PATCH 1/3] iio: adc: add support for mcp3911

2018-07-21 Thread Peter Meerwald-Stadler
Hello, > MCP3911 is a dual channel Analog Front End (AFE) containing two > synchronous sampling delta-sigma Analog-to-Digital Converters (ADC). some comments below... > Signed-off-by: Marcus Folkesson > Signed-off-by: Kent Gustavsson > --- > drivers/iio/adc/Kconfig | 10 ++ >

[PATCH 1/3] iio: adc: add support for mcp3911

2018-07-21 Thread Marcus Folkesson
MCP3911 is a dual channel Analog Front End (AFE) containing two synchronous sampling delta-sigma Analog-to-Digital Converters (ADC). Signed-off-by: Marcus Folkesson Signed-off-by: Kent Gustavsson --- drivers/iio/adc/Kconfig | 10 ++ drivers/iio/adc/Makefile | 1 +

[PATCH 1/3] iio: adc: add support for mcp3911

2018-07-21 Thread Marcus Folkesson
MCP3911 is a dual channel Analog Front End (AFE) containing two synchronous sampling delta-sigma Analog-to-Digital Converters (ADC). Signed-off-by: Marcus Folkesson Signed-off-by: Kent Gustavsson --- drivers/iio/adc/Kconfig | 10 ++ drivers/iio/adc/Makefile | 1 +