Re: [PATCH] iio: adc: ad7292: Modify the bool initialization assignment

2021-03-20 Thread Marcelo Schmitt
Okay, looks good to me. Reviewed-by: Marcelo Schmitt On 03/19, Guoqing chi wrote: > From: Guoqing Chi > > A bool initializer is best assigned to false rather than 0. > > Signed-off-by: Guoqing Chi > --- > drivers/iio/adc/ad7292.c | 2 +- > 1 file changed, 1

Re: [PATCH] iio: adc: ad7292: remove unneeded spi_set_drvdata()

2020-11-20 Thread Marcelo Schmitt
LGTM. Tested on raspberry pi kernel - rpi-5.9.y. ad7292 was heavily based on ad7768-1. Maybe this might apply to ad7768-1 as well. Reviewed-by: Marcelo Schmitt Tested-by: Marcelo Schmitt On 11/19, Alexandru Ardelean wrote: > This seems to have been copied from a driver that ca

[PATCH] kunit: tool: adjust parse regex

2020-07-28 Thread Marcelo Schmitt
kunit config subcommand terminates with error if .config has a configuration assigned with a string, for instance: CONFIG_CC_VERSION_TEXT="gcc (distro package version) ..." This patch adjusts the parse regex to consider such string assignments. Signed-off-by: Marcelo Schmitt --- too

Re: [PATCH v2 2/2] dt-bindings: iio: adc: Add DT docs for AD7292

2019-10-22 Thread Marcelo Schmitt
documentation and/or material for those who want to write dt-bindings that validate with dt-schema? Thanks in advance, Marcelo On 10/22, Rob Herring wrote: > On Tue, Oct 22, 2019 at 9:06 AM Marcelo Schmitt > wrote: > > > > Hi, > > > > I ran the DTC and CHECK for AD7

Re: [PATCH v2 1/2] iio: adc: Add driver support for AD7292

2019-10-22 Thread Marcelo Schmitt
Hi Jonathan, Should I send a V3 for the whole patchset or just for the DT binding patch? Thanks, Marcelo On 10/21, Jonathan Cameron wrote: > On Tue, 15 Oct 2019 23:51:22 -0300 > Marcelo Schmitt wrote: > > > The AD7292 is a 10-bit monitor and control system with ADC, DACs,

Re: [PATCH v2 2/2] dt-bindings: iio: adc: Add DT docs for AD7292

2019-10-22 Thread Marcelo Schmitt
it to the project dependencies at https://github.com/robherring/dt-schema. apt-get install libyaml-dev Thanks, Marcelo On 10/20, Rob Herring wrote: > On Sat, Oct 19, 2019 at 10:07 AM Marcelo Schmitt > wrote: > > > > Hi Rob > > > > Thanks for the review. > > > > I

Re: [PATCH v2 2/2] dt-bindings: iio: adc: Add DT docs for AD7292

2019-10-19 Thread Marcelo Schmitt
10/17, Rob Herring wrote: > On Tue, Oct 15, 2019 at 11:52:23PM -0300, Marcelo Schmitt wrote: > > Add a device tree binding doc for AD7292 monitor and control system. > > > > Signed-off-by: Marcelo Schmitt > > --- > > .../bindings/iio/adc/adi,ad7292.yaml

[PATCH v2 2/2] dt-bindings: iio: adc: Add DT docs for AD7292

2019-10-15 Thread Marcelo Schmitt
Add a device tree binding doc for AD7292 monitor and control system. Signed-off-by: Marcelo Schmitt --- .../bindings/iio/adc/adi,ad7292.yaml | 107 ++ MAINTAINERS | 1 + 2 files changed, 108 insertions(+) create mode 100644

[PATCH v2 1/2] iio: adc: Add driver support for AD7292

2019-10-15 Thread Marcelo Schmitt
-by: Marcelo Schmitt --- MAINTAINERS | 7 + drivers/iio/adc/Kconfig | 10 ++ drivers/iio/adc/Makefile | 1 + drivers/iio/adc/ad7292.c | 350 +++ 4 files changed, 368 insertions(+) create mode 100644 drivers/iio/adc/ad7292.c diff --git a/MAINTAINERS

[PATCH v2 0/2] iio: adc: Add driver support for AD7292

2019-10-15 Thread Marcelo Schmitt
regulator_get_voltage() from read_raw() INFO_SCALE - added a comment to make clear when internal voltage reference will be used - use standard channel definition to specify the ADC channels in dt-binding doc Marcelo Schmitt (2): iio: adc: Add driver support for AD7292 dt-bindings: iio: adc: Add DT docs

Re: [PATCH 1/2] iio: adc: Add driver support for AD7292

2019-10-15 Thread Marcelo Schmitt
Hi Jonathan, Thanks for the review. I'll send a v2 soon. On 10/14, Jonathan Cameron wrote: > On Sun, 13 Oct 2019 11:13:22 -0300 > Marcelo Schmitt wrote: > > > The AD7292 is a 10-bit monitor and control system with ADC, DACs, > > temperature sensor, and GPIOs. > > &

[PATCH 1/2] iio: adc: Add driver support for AD7292

2019-10-13 Thread Marcelo Schmitt
-by: Marcelo Schmitt --- MAINTAINERS | 7 + drivers/iio/adc/Kconfig | 10 ++ drivers/iio/adc/Makefile | 1 + drivers/iio/adc/ad7292.c | 350 +++ 4 files changed, 368 insertions(+) create mode 100644 drivers/iio/adc/ad7292.c diff --git a/MAINTAINERS

[PATCH 2/2] dt-bindings: iio: adc: Add DT docs for AD7292

2019-10-13 Thread Marcelo Schmitt
Add a device tree binding doc for AD7292 monitor and control system. Signed-off-by: Marcelo Schmitt --- .../bindings/iio/adc/adi,ad7292.yaml | 71 +++ MAINTAINERS | 1 + 2 files changed, 72 insertions(+) create mode 100644

[PATCH 0/2] iio: adc: Add driver support for AD7292

2019-10-13 Thread Marcelo Schmitt
This patchset adds a basic driver for the AD7292 ADC/DAC system along with device tree binding documentation. Marcelo Schmitt (2): iio: adc: Add driver support for AD7292 dt-bindings: iio: adc: Add DT docs for AD7292 .../bindings/iio/adc/adi,ad7292.yaml | 71 MAINTAINERS

Re: [PATCH v2] dt-bindings: iio: accel: add binding documentation for ADIS16240

2019-09-22 Thread Marcelo Schmitt
Hi, Could anyone with some experience in devicetree give us a hint on what to do here? I have built a binding doc using the same spi-cpha property. https://github.com/analogdevicesinc/linux/commit/bb2945e489dfdf2faa0255dd2cf09ae4ee77d826 On 09/13, Ardelean, Alexandru wrote: > On Thu, 2019-09-12

Re: [PATCH 1/2] dt-bindings: iio: accel: add binding documentation for ADIS16240

2019-09-02 Thread Marcelo Schmitt
Hi Rodrigo, This dt doc looks overal fine IMHO. I would just add some inline comments about the cpha and cpol properties. On 09/01, Rodrigo Carvalho wrote: > This patch add device tree binding documentation for ADIS16240. > > Signed-off-by: Rodrigo Ribeiro Carvalho > --- > I have doubt about

Re: [PATCH] staging: iio: adis16240: add of_match_table entry

2019-05-24 Thread Marcelo Schmitt
Hi Alexandru, On 05/24, Alexandru Ardelean wrote: > On Fri, May 24, 2019 at 6:30 AM Rodrigo Ribeiro wrote: > > > > This patch adds of_match_table entry in device driver in order to > > enable spi fallback probing. > > > > Signed-off-by: Rodrigo Ribeiro

Re: [PATCH 4/4] staging: iio: ad9832: add devicetree documentation

2019-04-02 Thread Marcelo Schmitt
On 04/02, Alexandru Ardelean wrote: > On Mon, Apr 1, 2019 at 5:38 PM Marcelo Schmitt > wrote: > > > > Add a devicetree documentation for the ad9832 direct digital > > synthesizer, waveform generator. > > > > Signed-off-by: Marcelo Schmitt > > ---

Re: Work on iio: stating: frequency: ad9832

2019-04-02 Thread Marcelo Schmitt
On 04/02, Alexandru Ardelean wrote: > On Mon, Apr 1, 2019 at 7:13 PM Jonathan Cameron > wrote: > > > > On Mon, 1 Apr 2019 11:25:29 -0300 > > Marcelo Schmitt wrote: > > > > > Hello, > > > > > > I was looking for some work on staging: i

Re: [PATCH 2/4] staging: iio: ad9832: add SPDX identifier

2019-04-02 Thread Marcelo Schmitt
On 04/01, Greg KH wrote: > On Mon, Apr 01, 2019 at 08:36:50PM +0530, Mukesh Ojha wrote: > > > > On 4/1/2019 8:07 PM, Marcelo Schmitt wrote: > > > Add SPDX identifier of GPL-2.0 for the ad9832 driver. > > > > > > Signed-off-by: Marcelo Schmitt >

[PATCH 4/4] staging: iio: ad9832: add devicetree documentation

2019-04-01 Thread Marcelo Schmitt
Add a devicetree documentation for the ad9832 direct digital synthesizer, waveform generator. Signed-off-by: Marcelo Schmitt --- .../bindings/iio/frequency/ad9832.txt | 26 +++ 1 file changed, 26 insertions(+) create mode 100644 Documentation/devicetree/bindings/iio

[PATCH 3/4] staging: iio: ad9832: use clock framework for clock reference

2019-04-01 Thread Marcelo Schmitt
Previously external clock were set through platform_data struct. Now device uses clk struct defined in include/linux/clk.h to handle external clock source. It also removes mclk from platform_data struct. Signed-off-by: Marcelo Schmitt --- drivers/staging/iio/frequency/ad9832.c | 37

[PATCH 2/4] staging: iio: ad9832: add SPDX identifier

2019-04-01 Thread Marcelo Schmitt
Add SPDX identifier of GPL-2.0 for the ad9832 driver. Signed-off-by: Marcelo Schmitt --- drivers/staging/iio/frequency/ad9832.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/iio/frequency/ad9832.c b/drivers/staging/iio/frequency/ad9832.c index

[PATCH 1/4] staging: iio: ad9832: organize includes

2019-04-01 Thread Marcelo Schmitt
Organize includes to list them in lexicographic order. Signed-off-by: Marcelo Schmitt --- drivers/staging/iio/frequency/ad9832.c | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/drivers/staging/iio/frequency/ad9832.c b/drivers/staging/iio/frequency/ad9832.c

[PATCH 0/4] staging: iio: ad9832: use clock framework

2019-04-01 Thread Marcelo Schmitt
This series of patches do minor codestyle ajusts, set SPDX licence identifier, set use of linux's clock framework to handle external clock, and add a devicetree documentation. Marcelo Schmitt (4): staging: iio: ad9832: organize includes staging: iio: ad9832: add SPDX identifier staging: iio

Work on iio: stating: frequency: ad9832

2019-04-01 Thread Marcelo Schmitt
Hello, I was looking for some work on staging: iio: ad9832 and made some observations while reading the driver. Apparently it had no devicetree documentation so I tried to elaborate one. It uses a platform_data variable to load external clock frequency (I tried to make it use linux's clock

Help on testing ad5933 driver

2019-03-21 Thread Marcelo Schmitt
Hello, would anyone mind helping me test ad5933 driver on actual hardware? I went through this (https://oslongjourney.github.io/linux-kernel/experiment-one-iio-dummy/) tutorial so I was able to load iio_simple_dummy driver, create and inspect some dummy devices. Now, as Jonathan has asked me, I

[PATCH v5 1/2] staging: iio: ad5933: change attributes to match ABI

2019-03-21 Thread Marcelo Schmitt
Change device attributes' names to match ABI documentation. Names were chosen such that they tend to be similar to existing ABI so it should be easier to standardize them when necessary. Signed-off-by: Marcelo Schmitt --- .../staging/iio/impedance-analyzer/ad5933.c | 24 +-- 1

[PATCH v5 2/2] staging: iio: ad5933: move out of staging

2019-03-21 Thread Marcelo Schmitt
. Signed-off-by: Marcelo Schmitt --- drivers/iio/Kconfig | 1 + drivers/iio/Makefile | 1 + drivers/iio/impedance-analyzer/Kconfig| 18 + drivers/iio/impedance-analyzer/Makefile | 5 + drivers/iio/impedance-analyzer/ad5933.c

[PATCH v5 0/2] staging: iio: ad5933: move out of staging

2019-03-21 Thread Marcelo Schmitt
. Marcelo Schmitt (2): staging: iio: ad5933: change attributes to match ABI staging: iio: ad5933: move out of staging drivers/iio/Kconfig | 1 + drivers/iio/Makefile | 1 + drivers/iio/impedance-analyzer/Kconfig| 18 + drivers/iio

[PATCH v4 6/7] staging: iio: ad5933: change attributes to match ABI

2019-03-16 Thread Marcelo Schmitt
Change device attributes' names to match ABI documentation. Names were chosen such that they tend to be similar to existing ABI so it should be easier to standardize them when necessary. Signed-off-by: Marcelo Schmitt --- .../staging/iio/impedance-analyzer/ad5933.c | 20 +-- 1

[PATCH v4 7/7] staging: iio: ad5933: move out of staging

2019-03-16 Thread Marcelo Schmitt
. Signed-off-by: Marcelo Schmitt --- drivers/iio/impedance-analyzer/Kconfig| 18 + drivers/iio/impedance-analyzer/Makefile | 5 + drivers/iio/impedance-analyzer/ad5933.c | 811 ++ .../staging/iio/impedance-analyzer/Kconfig| 18 - .../staging/iio/impedance

[PATCH v4 5/7] staging: iio: ad5933: add ABI documentation

2019-03-16 Thread Marcelo Schmitt
Add an ABI documentation for the ad5933 driver. Signed-off-by: Marcelo Schmitt --- .../sysfs-bus-iio-impedance-analyzer-ad5933 | 35 +++ .../sysfs-bus-iio-impedance-analyzer-ad5933 | 30 2 files changed, 35 insertions(+), 30 deletions(-) create mode 100644

[PATCH v4 3/7] staging: iio: ad5933: add SPDX identifier

2019-03-16 Thread Marcelo Schmitt
Add SPDX identifier of GPL-2.0 for the ad5933 driver. Signed-off-by: Marcelo Schmitt --- drivers/staging/iio/impedance-analyzer/ad5933.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/iio/impedance-analyzer/ad5933.c b/drivers/staging/iio/impedance

[PATCH v4 4/7] staging: iio: ad5933: change help rule message

2019-03-16 Thread Marcelo Schmitt
Remove the previous comment about direct access via sysfs which would lead one think ad5933 driver has limitations it actually doesn't. Signed-off-by: Marcelo Schmitt --- drivers/staging/iio/impedance-analyzer/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[PATCH v4 1/7] staging: iio: ad5933: change multi-line comment style

2019-03-16 Thread Marcelo Schmitt
Make multi-line comments compliant with the preferred code style. Signed-off-by: Marcelo Schmitt --- drivers/staging/iio/impedance-analyzer/ad5933.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/staging/iio/impedance-analyzer/ad5933.c b/drivers/staging

[PATCH v4 2/7] staging: iio: ad5933: organize includes

2019-03-16 Thread Marcelo Schmitt
Organize includes to list them in lexicographic order. Signed-off-by: Marcelo Schmitt --- .../staging/iio/impedance-analyzer/ad5933.c| 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/iio/impedance-analyzer/ad5933.c b/drivers/staging/iio

[PATCH v4 0/7] staging: iio: ad5933: move out of staging

2019-03-16 Thread Marcelo Schmitt
to existing ABI. Finally, it moves the ad5933 driver from staging directory to iio main drivers directory. Marcelo Schmitt (7): staging: iio: ad5933: change multi-line comment style staging: iio: ad5933: organize includes staging: iio: ad5933: add SPDX identifier staging: iio: ad5933: change

Re: [PATCH v3 5/7] staging: iio: ad5933: add ABI documentation

2019-03-12 Thread Marcelo Schmitt
On 03/11, Alexandru Ardelean wrote: > On Sun, Mar 10, 2019 at 7:47 PM Marcelo Schmitt > wrote: > > > > Add an ABI documentation for the ad5933 driver. > > There's already an ABI documentation for this driver. > See: > https://git.kernel.org/pub/scm/linux/kernel/

[PATCH v3 6/7] staging: iio: ad5933: change attributes' ABI call

2019-03-10 Thread Marcelo Schmitt
Change device attributes to use more suitable ABI functions when setting and displaying AC output excitation voltage. It better complies with device hardware since these attributes are used to manage DDS peak-to-peak output voltage. Signed-off-by: Marcelo Schmitt --- drivers/staging/iio

[PATCH v3 7/7] staging: iio: ad5933: move out of staging

2019-03-10 Thread Marcelo Schmitt
. Signed-off-by: Marcelo Schmitt --- drivers/iio/impedance-analyzer/Kconfig| 18 + drivers/iio/impedance-analyzer/Makefile | 5 + drivers/iio/impedance-analyzer/ad5933.c | 811 ++ .../staging/iio/impedance-analyzer/Kconfig| 18 - .../staging/iio/impedance

[PATCH v3 5/7] staging: iio: ad5933: add ABI documentation

2019-03-10 Thread Marcelo Schmitt
Add an ABI documentation for the ad5933 driver. Signed-off-by: Marcelo Schmitt --- .../ABI/testing/sysfs-bus-iio-ad5933 | 39 +++ 1 file changed, 39 insertions(+) create mode 100644 Documentation/ABI/testing/sysfs-bus-iio-ad5933 diff --git a/Documentation/ABI/testing

[PATCH v3 4/7] staging: iio: ad5933: change help rule message

2019-03-10 Thread Marcelo Schmitt
Remove the previous comment about direct access via sysfs which would lead one think ad5933 driver has limitations it actually doesn't. Signed-off-by: Marcelo Schmitt --- drivers/staging/iio/impedance-analyzer/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[PATCH v3 3/7] staging: iio: ad5933: add SPDX identifier

2019-03-10 Thread Marcelo Schmitt
Add SPDX identifier of GPL-2.0 for the ad5933 driver. Signed-off-by: Marcelo Schmitt --- drivers/staging/iio/impedance-analyzer/ad5933.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/iio/impedance-analyzer/ad5933.c b/drivers/staging/iio/impedance

[PATCH v3 1/7] staging: iio: ad5933: change multi-line comment style

2019-03-10 Thread Marcelo Schmitt
Make multi-line comments compliant with the preferred code style. Signed-off-by: Marcelo Schmitt --- drivers/staging/iio/impedance-analyzer/ad5933.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/staging/iio/impedance-analyzer/ad5933.c b/drivers/staging

[PATCH v3 2/7] staging: iio: ad5933: organize includes

2019-03-10 Thread Marcelo Schmitt
Organize includes to list them in lexicographic order. Signed-off-by: Marcelo Schmitt --- .../staging/iio/impedance-analyzer/ad5933.c| 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/iio/impedance-analyzer/ad5933.c b/drivers/staging/iio

[PATCH v3 0/7] move out of staging

2019-03-10 Thread Marcelo Schmitt
functions. Finally, it moves the ad5933 driver from staging directory to iio main drivers directory. Marcelo Schmitt (7): staging: iio: ad5933: change multi-line comment style staging: iio: ad5933: organize includes staging: iio: ad5933: add SPDX identifier staging: iio: ad5933: change help rule

Re: [PATCH v2 3/4] staging: iio: ad5933: add ABI documentation

2019-03-03 Thread Marcelo Schmitt
On 03/03, Jonathan Cameron wrote: > On Thu, 28 Feb 2019 23:53:14 -0300 > Marcelo Schmitt wrote: > > > Add an ABI documentation for the ad5933 driver. > > > > Signed-off-by: Marcelo Schmitt > Hi Marcelo, > > The ABI that you have defined which is actually ne

[PATCH v2 4/4] staging: iio: ad5933: move out of staging

2019-02-28 Thread Marcelo Schmitt
. Signed-off-by: Marcelo Schmitt --- drivers/iio/impedance-analyzer/Kconfig| 18 + drivers/iio/impedance-analyzer/Makefile | 5 + drivers/iio/impedance-analyzer/ad5933.c | 811 ++ .../staging/iio/impedance-analyzer/Kconfig| 18 - .../staging/iio/impedance

[PATCH v2 2/4] staging: iio: ad5933: change help rule message

2019-02-28 Thread Marcelo Schmitt
Remove the previous comment about direct access via sysfs which would lead one think ad5933 driver has limitations it actually doesn't. Signed-off-by: Marcelo Schmitt --- drivers/staging/iio/impedance-analyzer/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[PATCH v2 3/4] staging: iio: ad5933: add ABI documentation

2019-02-28 Thread Marcelo Schmitt
Add an ABI documentation for the ad5933 driver. Signed-off-by: Marcelo Schmitt --- .../ABI/testing/sysfs-bus-iio-ad5933 | 91 +++ 1 file changed, 91 insertions(+) create mode 100644 Documentation/ABI/testing/sysfs-bus-iio-ad5933 diff --git a/Documentation/ABI/testing

[PATCH v2 1/4] staging: iio: ad5933: add SPDX identifier

2019-02-28 Thread Marcelo Schmitt
Add SPDX identifier of GPL-2.0 for the ad5933 driver. Organize imports. Make multi-line comments compliant with the preferred code style. Signed-off-by: Marcelo Schmitt --- .../staging/iio/impedance-analyzer/ad5933.c | 30 ++- 1 file changed, 16 insertions(+), 14 deletions

[PATCH v2 0/4] staging: iio: ad5933: move out of staging

2019-02-28 Thread Marcelo Schmitt
directory. Marcelo Schmitt (4): staging: iio: ad5933: add SPDX identifier staging: iio: ad5933: change help rule message staging: iio: ad5933: add ABI documentation staging: iio: ad5933: move out of staging .../ABI/testing/sysfs-bus-iio-ad5933 | 91 ++ drivers/iio/impedance

Re: [PATCH] staging: iio: ad5933: move out of staging

2019-02-11 Thread Marcelo Schmitt
On 02/11, Jonathan Cameron wrote: > On Mon, 11 Feb 2019 13:43:11 -0200 > Marcelo Schmitt wrote: > > > Move ad5933 impedance-analyzer driver from staging to mainline. > > > > The ad5933 is a high precision impedance converter system > > solution that combine

[PATCH] staging: iio: ad5933: move out of staging

2019-02-11 Thread Marcelo Schmitt
. Signed-off-by: Marcelo Schmitt --- drivers/iio/impedance-analyzer/Kconfig| 18 + drivers/iio/impedance-analyzer/Makefile | 5 + drivers/iio/impedance-analyzer/ad5933.c | 810 ++ .../staging/iio/impedance-analyzer/Kconfig| 18 - .../staging/iio/impedance

Re: [PATCH 0/3] Add devicetree support for ad5933

2018-12-08 Thread Marcelo Schmitt
the pieces of advice needed for this work. Thanks, Marcelo Em sáb, 8 de dez de 2018 às 16:18, Marcelo Schmitt escreveu: > > This series of patches change voltage regulator error handling for the > ad5933. > It also add an option to specify external clock reference using a clock

Re: [PATCH 0/3] Add devicetree support for ad5933

2018-12-08 Thread Marcelo Schmitt
the pieces of advice needed for this work. Thanks, Marcelo Em sáb, 8 de dez de 2018 às 16:18, Marcelo Schmitt escreveu: > > This series of patches change voltage regulator error handling for the > ad5933. > It also add an option to specify external clock reference using a clock

[PATCH 3/3] staging: iio: ad5933: add binding doc for ad5933

2018-12-08 Thread Marcelo Schmitt
Add a devicetree documentation for the ad5933 and ad5934 impedance converter, network analyzer. Signed-off-by: Marcelo Schmitt Signed-off-by: Gabriel Capella Co-Developed-by: Gabriel Capella --- .../iio/impedance-analyzer/ad5933.txt | 26 +++ 1 file changed, 26

[PATCH 3/3] staging: iio: ad5933: add binding doc for ad5933

2018-12-08 Thread Marcelo Schmitt
Add a devicetree documentation for the ad5933 and ad5934 impedance converter, network analyzer. Signed-off-by: Marcelo Schmitt Signed-off-by: Gabriel Capella Co-Developed-by: Gabriel Capella --- .../iio/impedance-analyzer/ad5933.txt | 26 +++ 1 file changed, 26

[PATCH 2/3] staging: iio: ad5933: use clock framework for clock reference

2018-12-08 Thread Marcelo Schmitt
Add the option to specify the external clock (MCLK) using the clock framework. Also remove the old platform data structure. Signed-off-by: Marcelo Schmitt Signed-off-by: Gabriel Capella Co-Developed-by: Gabriel Capella --- .../staging/iio/impedance-analyzer/ad5933.c | 43

[PATCH 2/3] staging: iio: ad5933: use clock framework for clock reference

2018-12-08 Thread Marcelo Schmitt
Add the option to specify the external clock (MCLK) using the clock framework. Also remove the old platform data structure. Signed-off-by: Marcelo Schmitt Signed-off-by: Gabriel Capella Co-Developed-by: Gabriel Capella --- .../staging/iio/impedance-analyzer/ad5933.c | 43

[PATCH 1/3] staging: iio: ad5933: change regulator binging for vref

2018-12-08 Thread Marcelo Schmitt
Set a single voltage regulator for all voltage references. Remove voltage reference value from default platafrom data struct. Signed-off-by: Marcelo Schmitt Signed-off-by: Gabriel Capella Co-Developed-by: Gabriel Capella --- drivers/staging/iio/impedance-analyzer/ad5933.c | 14

[PATCH 1/3] staging: iio: ad5933: change regulator binging for vref

2018-12-08 Thread Marcelo Schmitt
Set a single voltage regulator for all voltage references. Remove voltage reference value from default platafrom data struct. Signed-off-by: Marcelo Schmitt Signed-off-by: Gabriel Capella Co-Developed-by: Gabriel Capella --- drivers/staging/iio/impedance-analyzer/ad5933.c | 14

[PATCH 0/3] Add devicetree support for ad5933

2018-12-08 Thread Marcelo Schmitt
This series of patches change voltage regulator error handling for the ad5933. It also add an option to specify external clock reference using a clock framework and remove the old platform data structure. Finally it adds binding documentation for devicetree. Marcelo Schmitt (3): staging: iio

[PATCH 0/3] Add devicetree support for ad5933

2018-12-08 Thread Marcelo Schmitt
This series of patches change voltage regulator error handling for the ad5933. It also add an option to specify external clock reference using a clock framework and remove the old platform data structure. Finally it adds binding documentation for devicetree. Marcelo Schmitt (3): staging: iio

Re: [PATCH] staging: iio: ad5933: replaced kfifo by triggered_buffer

2018-12-02 Thread Marcelo Schmitt
On 11/25, Jonathan Cameron wrote: > On Thu, 22 Nov 2018 10:53:47 -0200 > Marcelo Schmitt wrote: > > > Previously, there was an implicit creation of a kfifo which was replaced > > by a call to triggered_buffer_setup, which is already implemented in iio > > infrastr

Re: [PATCH] staging: iio: ad5933: replaced kfifo by triggered_buffer

2018-12-02 Thread Marcelo Schmitt
On 11/25, Jonathan Cameron wrote: > On Thu, 22 Nov 2018 10:53:47 -0200 > Marcelo Schmitt wrote: > > > Previously, there was an implicit creation of a kfifo which was replaced > > by a call to triggered_buffer_setup, which is already implemented in iio > > infrastr

[PATCH] staging: iio: ad5933: add binding doc for ad5933

2018-12-02 Thread Marcelo Schmitt
Add a devicetree documentation for the ad5933 and ad5934 impedance converter, network analyzer. Co-Developed-by: Gabriel Capella Signed-off-by: Marcelo Schmitt Signed-off-by: Gabriel Capella --- .../iio/impedance-analyzer/ad5933.txt | 23 +++ 1 file changed, 23

[PATCH] staging: iio: ad5933: add binding doc for ad5933

2018-12-02 Thread Marcelo Schmitt
Add a devicetree documentation for the ad5933 and ad5934 impedance converter, network analyzer. Co-Developed-by: Gabriel Capella Signed-off-by: Marcelo Schmitt Signed-off-by: Gabriel Capella --- .../iio/impedance-analyzer/ad5933.txt | 23 +++ 1 file changed, 23

[PATCH v2] staging: iio: ad5933: add device tree support

2018-11-24 Thread Marcelo Schmitt
Add a of_device_id struct variable and subsequent call to MODULE_DEVICE_TABLE macro to complete device tree support. Signed-off-by: Marcelo Schmitt --- drivers/staging/iio/impedance-analyzer/ad5933.c | 9 + 1 file changed, 9 insertions(+) diff --git a/drivers/staging/iio/impedance

[PATCH v2] staging: iio: ad5933: add device tree support

2018-11-24 Thread Marcelo Schmitt
Add a of_device_id struct variable and subsequent call to MODULE_DEVICE_TABLE macro to complete device tree support. Signed-off-by: Marcelo Schmitt --- drivers/staging/iio/impedance-analyzer/ad5933.c | 9 + 1 file changed, 9 insertions(+) diff --git a/drivers/staging/iio/impedance

[PATCH] staging: iio: ad5933: finalized device-tree support

2018-11-23 Thread Marcelo Schmitt
Added a of_device_id struct variable and subsequent call to MODULE_DEVICE_TABLE macro to complete device-tree support for this driver. Signed-off-by: Marcelo Schmitt --- drivers/staging/iio/impedance-analyzer/ad5933.c | 9 + 1 file changed, 9 insertions(+) diff --git a/drivers/staging

[PATCH] staging: iio: ad5933: finalized device-tree support

2018-11-23 Thread Marcelo Schmitt
Added a of_device_id struct variable and subsequent call to MODULE_DEVICE_TABLE macro to complete device-tree support for this driver. Signed-off-by: Marcelo Schmitt --- drivers/staging/iio/impedance-analyzer/ad5933.c | 9 + 1 file changed, 9 insertions(+) diff --git a/drivers/staging

[PATCH] staging: iio: ad5933: replaced kfifo by triggered_buffer

2018-11-22 Thread Marcelo Schmitt
Previously, there was an implicit creation of a kfifo which was replaced by a call to triggered_buffer_setup, which is already implemented in iio infrastructure. Signed-off-by: Marcelo Schmitt --- .../staging/iio/impedance-analyzer/Kconfig| 2 +- .../staging/iio/impedance-analyzer/ad5933.c

[PATCH] staging: iio: ad5933: replaced kfifo by triggered_buffer

2018-11-22 Thread Marcelo Schmitt
Previously, there was an implicit creation of a kfifo which was replaced by a call to triggered_buffer_setup, which is already implemented in iio infrastructure. Signed-off-by: Marcelo Schmitt --- .../staging/iio/impedance-analyzer/Kconfig| 2 +- .../staging/iio/impedance-analyzer/ad5933.c

[PATCH v2] Staging iio: adc: Match parenthesis alignment

2018-10-17 Thread Marcelo Schmitt
Change close parenthesis alignment to match respective open parenthesis at iio/drivers/staging/iio/adc/ad7606.c line 379. This makes the file more compliant with the preferred coding style for the linux kernel. Signed-of-by: Marcelo Schmitt --- drivers/staging/iio/adc/ad7606.c | 2 +- 1 file

[PATCH v2] Staging iio: adc: Match parenthesis alignment

2018-10-17 Thread Marcelo Schmitt
Change close parenthesis alignment to match respective open parenthesis at iio/drivers/staging/iio/adc/ad7606.c line 379. This makes the file more compliant with the preferred coding style for the linux kernel. Signed-of-by: Marcelo Schmitt --- drivers/staging/iio/adc/ad7606.c | 2 +- 1 file

[PATCH] Staging iio/adc: fixes parenthesis alignment

2018-10-16 Thread Marcelo Schmitt
Fixes close parenthesis alignment to match open parenthesis at iio/drivers/staging/iio/adc/ad7606.c line 379. Signed-of-by: Marcelo Schmitt --- drivers/staging/iio/adc/ad7606.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/iio/adc/ad7606.c b/drivers/staging

[PATCH] Staging iio/adc: fixes parenthesis alignment

2018-10-16 Thread Marcelo Schmitt
Fixes close parenthesis alignment to match open parenthesis at iio/drivers/staging/iio/adc/ad7606.c line 379. Signed-of-by: Marcelo Schmitt --- drivers/staging/iio/adc/ad7606.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/iio/adc/ad7606.c b/drivers/staging