[PATCH v2] dt-bindings: iio: accel: adxl372: switch to YAML bindings

Convert the old device tree documentation to yaml format. Signed-off-by: Lucas Oshiro Signed-off-by: Rodrigo Ribeiro Co-developed-by: Rodrigo Ribeiro Reviewed-by: Matheus Tavares Reviewed-by: Marcelo Schmitt --- V2: - Update MAINTAINERS file - Remove description field for reg and

Re: [PATCH] dt-bindings: iio: accel: adxl372: switch to YAML bindings

Thanks, Alexandru. We're going to send a new version for this patch. On 20/05/2019 07:46, Alexandru Ardelean wrote: On Sun, May 19, 2019 at 8:27 PM Jonathan Cameron wrote: On Sat, 18 May 2019 18:55:42 -0300 Lucas Oshiro wrote: Convert the old device tree documentation to yaml f

[PATCH] dt-bindings: iio: accel: adxl372: switch to YAML bindings

Convert the old device tree documentation to yaml format. Signed-off-by: Lucas Oshiro Signed-off-by: Rodrigo Ribeiro Co-developed-by: Rodrigo Ribeiro --- Hello, We've added Stefan Popa as maintainer of the yaml documentation of this driver because we found through git that he was the a

[PATCH v2] staging: iio: adis16240: add device to module device table

Add a of_device_id struct and MODULE_DEVICE_TABLE call, in order to add device-tree support for this driver. Signed-off-by: Lucas Oshiro Signed-off-by: Rodrigo Ribeiro Co-developed-by: Rodrigo Ribeiro --- drivers/staging/iio/accel/adis16240.c | 6 ++ 1 file changed, 6 insertions(+) diff

[PATCH] staging: iio: adis16240: add device to module device table

Add a of_device_id struct and MODULE_DEVICE_TABLE call. Signed-off-by: Lucas Oshiro Signed-off-by: Rodrigo Ribeiro Co-developed-by: Rodrigo Ribeiro --- drivers/staging/iio/accel/adis16240.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/staging/iio/accel/adis16240.c b

[PATCH] staging: iio: cdc: ad7746: Replace bitshift by BIT

Replace bitshifts on lines 54, 56 and 78 of ad7746.c. Signed-off-by: Lucas Oshiro --- drivers/staging/iio/cdc/ad7746.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/staging/iio/cdc/ad7746.c index 0eb28fea876e..ea48b14cee72

Re: [PATCH 5/5] iio:potentiostat:lmp91000: add '\n' on dev_err

Thanks for the review! On 20/02/2019 19:22, Joe Perches wrote: On Wed, 2019-02-20 at 09:49 +, Jonathan Cameron wrote: On Mon, 18 Feb 2019 13:01:23 -0800 Joe Perches wrote: On Mon, 2019-02-18 at 14:22 -0300, Lucas Oshiro wrote: Add missing '\n' at the end of dev_err message o

[PATCH 5/5] iio:potentiostat:lmp91000: add '\n' on dev_err

Add missing '\n' at the end of dev_err message on line 215. Signed-off-by: Lucas Oshiro Signed-off-by: Anderson Reis --- drivers/iio/potentiostat/lmp91000.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/iio/potentiostat/lmp91000.c b/drivers/iio/po

[PATCH 4/5] iio:potentiostat:lmp91000: invert if statement

Invert if statement arms in line 214, in order to make the code cleaner Signed-off-by: Lucas Oshiro Signed-off-by: Anderson Reis --- drivers/iio/potentiostat/lmp91000.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/iio/potentiostat/lmp91000.c b/drivers/iio

[PATCH 3/5] iio:potentiostat:lmp91000: reduce line width and remove blank line

lines Signed-off-by: Lucas Oshiro Signed-off-by: Anderson Reis --- drivers/iio/potentiostat/lmp91000.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/iio/potentiostat/lmp91000.c b/drivers/iio/potentiostat/lmp91000.c index ca31be510940..6dba26121a62 100644 --- a/driver

[PATCH 2/5] iio:potentiostat:lmp91000: insert braces around if arms

Insert braces around all arms of if statement starting on line 214, and solve these checkpath.el CHECKs: - lmp91000.c:214: CHECK: braces {} should be used on all arms of this statement - lmp91000.c:216: CHECK: Unbalanced braces around else statement Signed-off-by: Lucas Oshiro Signed-off-by

[PATCH 1/5] iio:potentiostat:lmp91000: remove unnecessary parentheses

Remove unnecessary parentheses on line 116, and solve these checkpatch.pl CHECKs: - lmp91000.c:116: CHECK: Unnecessary parentheses around 'state != channel' - lmp91000.c:116: CHECK: Unnecessary parentheses around 'channel == LMP91000_REG_MODECN_TEMP' Signed-off-by: Lucas O

[PATCH 0/5] iio:potentiostat:lmp91000: Adjust codestyle, and minor cleanup changes

Hi, We solved some checkpath.el CHECKs and WARNINGs. We also inverted the arms of an if statement, in order to make the code smaller as the else statement was supressed. We added a missing '\n' on a dev_err message. Lucas Oshiro (5): iio:potentiostat:lmp91000: remove unnecessary p

Re: [PATCH] iio:potentiostat:lmp91000: solve codestyle WARNINGs and CHECKs

Thanks! I'll send those changes in my next patchset. On 02/02/2019 08:00, Jonathan Cameron wrote: On Fri, 1 Feb 2019 12:29:11 -0200 LSO wrote: Thanks for the review! On 29/01/2019 20:48, Joe Perches wrote: On Tue, 2019-01-29 at 16:36 -0200, Lucas Oshiro wrote: Solve most o

[PATCH] iio:potentiostat:lmp91000: solve codestyle WARNINGs and CHECKs

CK: braces {} should be used on all arms of this statement lmp91000.c:216: CHECK: Unbalanced braces around else statement lmp91000.c:258: WARNING: line over 80 characters lmp91000.c:279: CHECK: Please don't use multiple blank lines Signed-off-by: Lucas Oshiro Signed-off-by: Anderson Reis ---