[PATCH] drm/omap: dss: Use ERR_CAST instead of ERR_PTR(PTR_ERR())

Use ERR_CAST inlined function instead of ERR_PTR(PTR_ERR(...)). drivers/gpu/drm/omapdrm/dss/dss.c:959:9-16: WARNING: ERR_CAST can be used with d Generated by: scripts/coccinelle/api/err_cast.cocci Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- drivers/gpu/drm/omapdrm/dss

[PATCH] drm/omap: dss: Use ERR_CAST instead of ERR_PTR(PTR_ERR())

Use ERR_CAST inlined function instead of ERR_PTR(PTR_ERR(...)). drivers/gpu/drm/omapdrm/dss/dss.c:959:9-16: WARNING: ERR_CAST can be used with d Generated by: scripts/coccinelle/api/err_cast.cocci Signed-off-by: Hernán Gonzalez --- drivers/gpu/drm/omapdrm/dss/dss.c | 2 +- 1 file changed, 1

[PATCH] net: ethernet: ti: Use ERR_CAST instead of ERR_PTR(PTR_ERR())

Use ERR_CAST inlined function instead of ERR_PTR(PTR_ERR(...)). drivers/net/ethernet/ti/cpts.c:567:9-16: WARNING: ERR_CAST can be used with cpts->refclk Generated by: scripts/coccinelle/api/err_cast.cocci Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- drivers/net/et

[PATCH] net: ethernet: ti: Use ERR_CAST instead of ERR_PTR(PTR_ERR())

Use ERR_CAST inlined function instead of ERR_PTR(PTR_ERR(...)). drivers/net/ethernet/ti/cpts.c:567:9-16: WARNING: ERR_CAST can be used with cpts->refclk Generated by: scripts/coccinelle/api/err_cast.cocci Signed-off-by: Hernán Gonzalez --- drivers/net/ethernet/ti/cpts.c | 2 +- 1 file chan

[PATCH] ARM: dts: omap3-pandora-common: Use IRQ_TYPE specifier

Replace magic number with the proper IRQ_TYPE specifier to improve DT readability. Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- arch/arm/boot/dts/omap3-pandora-common.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/omap3-p

[PATCH] ARM: dts: omap3-pandora-common: Use IRQ_TYPE specifier

Replace magic number with the proper IRQ_TYPE specifier to improve DT readability. Signed-off-by: Hernán Gonzalez --- arch/arm/boot/dts/omap3-pandora-common.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/omap3-pandora-common.dtsi b/arch/arm/boot/dts

[PATCH] ARM: dts: imx6qdl-phytec-pfla02: Use IRQ_TYPE specifier

Replace magic number with the proper IRQ_TYPE specifier to improve DT readability. Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/imx6qdl-

[PATCH] ARM: dts: imx6qdl-phytec-pfla02: Use IRQ_TYPE specifier

Replace magic number with the proper IRQ_TYPE specifier to improve DT readability. Signed-off-by: Hernán Gonzalez --- arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/imx6qdl-phytec-pfla02.dtsi b/arch/arm/boot

[PATCH] ARM: dts: imx53-voipac-dmm-668: Use IRQ_TYPE specifier

Replace magic number with the proper IRQ_TYPE specifier to improve DT readability. Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- arch/arm/boot/dts/imx53-voipac-dmm-668.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/imx53-voip

[PATCH] ARM: dts: imx53-voipac-dmm-668: Use IRQ_TYPE specifier

Replace magic number with the proper IRQ_TYPE specifier to improve DT readability. Signed-off-by: Hernán Gonzalez --- arch/arm/boot/dts/imx53-voipac-dmm-668.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/imx53-voipac-dmm-668.dtsi b/arch/arm/boot/dts

[PATCH] ARM: dts: imx53-qsb: Use IRQ_TYPE specifier

Replace magic number with the proper IRQ_TYPE specifier to improve DT readability. Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- arch/arm/boot/dts/imx53-qsb.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/imx53-qsb.dts b/arch/ar

[PATCH] ARM: dts: imx53-qsb: Use IRQ_TYPE specifier

Replace magic number with the proper IRQ_TYPE specifier to improve DT readability. Signed-off-by: Hernán Gonzalez --- arch/arm/boot/dts/imx53-qsb.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/imx53-qsb.dts b/arch/arm/boot/dts/imx53-qsb.dts index

[PATCH] ARM: dts: vf-colibri-eval-v3.dtsi: Use IRQ_TYPE specifier

GPIO_ACTIVE_LOW was being used to specify an interrupt, use IRQ_TYPE_EDGE_RISING instead. This improves DT readability. Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- arch/arm/boot/dts/vf-colibri-eval-v3.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH] ARM: dts: vf-colibri-eval-v3.dtsi: Use IRQ_TYPE specifier

GPIO_ACTIVE_LOW was being used to specify an interrupt, use IRQ_TYPE_EDGE_RISING instead. This improves DT readability. Signed-off-by: Hernán Gonzalez --- arch/arm/boot/dts/vf-colibri-eval-v3.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/vf-colibri

[PATCH] ARM: dts: at91-sama5d2_xplained: Use IRQ_TYPE specifier

GPIO_ACTIVE_LOW was being used to specify an interrupt, use IRQ_TYPE_EDGE_RISING instead. This improves DT readability. Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- arch/arm/boot/dts/at91-sama5d2_xplained.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH] ARM: dts: at91-sama5d2_xplained: Use IRQ_TYPE specifier

GPIO_ACTIVE_LOW was being used to specify an interrupt, use IRQ_TYPE_EDGE_RISING instead. This improves DT readability. Signed-off-by: Hernán Gonzalez --- arch/arm/boot/dts/at91-sama5d2_xplained.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/at91

[PATCH] ARM: dts: am335x-boneblue: Use IRQ_TYPE specifier

GPIO_ACTIVE_LOW was being used to specify an interrupt, use IRQ_TYPE_EDGE_RISING instead. This improves DT readability. Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- arch/arm/boot/dts/am335x-boneblue.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] ARM: dts: am335x-boneblue: Use IRQ_TYPE specifier

GPIO_ACTIVE_LOW was being used to specify an interrupt, use IRQ_TYPE_EDGE_RISING instead. This improves DT readability. Signed-off-by: Hernán Gonzalez --- arch/arm/boot/dts/am335x-boneblue.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/am335x

[PATCH] ARM: dts: am335x-baltos.dtsi: Use IRQ_TYPE specifier

GPIO_ACTIVE_LOW was being used to specify an interrupt, use IRQ_TYPE_EDGE_RISING instead. This improves DT readability. Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- arch/arm/boot/dts/am335x-baltos.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] ARM: dts: am335x-baltos.dtsi: Use IRQ_TYPE specifier

GPIO_ACTIVE_LOW was being used to specify an interrupt, use IRQ_TYPE_EDGE_RISING instead. This improves DT readability. Signed-off-by: Hernán Gonzalez --- arch/arm/boot/dts/am335x-baltos.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/am335x

[PATCH] ARM: dts: am335x-baltos-ir5221: Use IRQ_TYPE specifier

GPIO_ACTIVE_LOW was being used to specify an interrupt, use IRQ_TYPE_EDGE_RISING instead. This improves DT readability. Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- arch/arm/boot/dts/am335x-baltos-ir5221.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH] ARM: dts: am335x-baltos-ir5221: Use IRQ_TYPE specifier

GPIO_ACTIVE_LOW was being used to specify an interrupt, use IRQ_TYPE_EDGE_RISING instead. This improves DT readability. Signed-off-by: Hernán Gonzalez --- arch/arm/boot/dts/am335x-baltos-ir5221.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/am335x

Re: [PATCH v2 09/14] staging: iio: ad7746: Add remove()

On Sun, Apr 15, 2018 at 12:31 PM, Jonathan Cameron <ji...@kernel.org> wrote: > On Fri, 13 Apr 2018 13:36:46 -0300 > Hernán Gonzalez <her...@vanguardiasur.com.ar> wrote: > >> This allows the driver to be probed and removed as a module powering it >> down on remo

Re: [PATCH v2 09/14] staging: iio: ad7746: Add remove()

On Sun, Apr 15, 2018 at 12:31 PM, Jonathan Cameron wrote: > On Fri, 13 Apr 2018 13:36:46 -0300 > Hernán Gonzalez wrote: > >> This allows the driver to be probed and removed as a module powering it >> down on remove(). >> >> Signed-off-by: Hernán Gonzalez &

Re: [PATCH v2 11/14] staging: iio: ad7746: Add devicetree bindings documentation

On Sun, Apr 15, 2018 at 12:37 PM, Jonathan Cameron <ji...@kernel.org> wrote: > On Fri, 13 Apr 2018 13:36:48 -0300 > Hernán Gonzalez <her...@vanguardiasur.com.ar> wrote: > >> Cc: Rob Herring <robh...@kernel.org> >> Cc: Mark Rutland <mark.rutl...@arm.com>

Re: [PATCH v2 11/14] staging: iio: ad7746: Add devicetree bindings documentation

On Sun, Apr 15, 2018 at 12:37 PM, Jonathan Cameron wrote: > On Fri, 13 Apr 2018 13:36:48 -0300 > Hernán Gonzalez wrote: > >> Cc: Rob Herring >> Cc: Mark Rutland >> Cc: devicet...@vger.kernel.org >> Signed-off-by: Hernán Gonzalez > A few comme

Re: [PATCH v2 07/14] staging: iio: ad7746: Remove unused defines

You're right, got confused from the macro defined in the .c file. I'll leave this alone on the next series Thanks! On Sun, Apr 15, 2018 at 12:12 PM, Jonathan Cameron <ji...@kernel.org> wrote: > On Fri, 13 Apr 2018 13:36:44 -0300 > Hernán Gonzalez <her...@vanguardiasur.com.ar>

Re: [PATCH v2 07/14] staging: iio: ad7746: Remove unused defines

You're right, got confused from the macro defined in the .c file. I'll leave this alone on the next series Thanks! On Sun, Apr 15, 2018 at 12:12 PM, Jonathan Cameron wrote: > On Fri, 13 Apr 2018 13:36:44 -0300 > Hernán Gonzalez wrote: > >> Signed-off-by: Hernán Gonzalez >

Re: [PATCH v2 03/14] staging: iio: ad7746: Fix bound checkings

On Sun, Apr 15, 2018 at 12:05 PM, Jonathan Cameron <ji...@kernel.org> wrote: > On Fri, 13 Apr 2018 13:36:40 -0300 > Hernán Gonzalez <her...@vanguardiasur.com.ar> wrote: > >> Also remove unnecessary parenthesis > I am probably missing something. I'm not sure what yo

Re: [PATCH v2 03/14] staging: iio: ad7746: Fix bound checkings

On Sun, Apr 15, 2018 at 12:05 PM, Jonathan Cameron wrote: > On Fri, 13 Apr 2018 13:36:40 -0300 > Hernán Gonzalez wrote: > >> Also remove unnecessary parenthesis > I am probably missing something. I'm not sure what you mean > by fix bound checking? There are superfluous br

[PATCH v2 14/14] staging: iio: Remove ad7746 from staging

Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- .../devicetree/bindings/staging/iio/cdc/ad7746.txt | 34 - drivers/staging/iio/cdc/Kconfig| 10 - drivers/staging/iio/cdc/Makefile | 1 - drivers/staging/iio/cdc/ad

[PATCH v2 14/14] staging: iio: Remove ad7746 from staging

Signed-off-by: Hernán Gonzalez --- .../devicetree/bindings/staging/iio/cdc/ad7746.txt | 34 - drivers/staging/iio/cdc/Kconfig| 10 - drivers/staging/iio/cdc/Makefile | 1 - drivers/staging/iio/cdc/ad7746.c | 856

[PATCH v2 13/14] Move ad7746 out of staging

Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- .../devicetree/bindings/iio/cdc/ad7746.txt | 34 + drivers/iio/Kconfig| 1 + drivers/iio/Makefile | 1 + drivers/iio/cdc/K

[PATCH v2 13/14] Move ad7746 out of staging

Signed-off-by: Hernán Gonzalez --- .../devicetree/bindings/iio/cdc/ad7746.txt | 34 + drivers/iio/Kconfig| 1 + drivers/iio/Makefile | 1 + drivers/iio/cdc/Kconfig| 16 + drivers/iio/cdc

[PATCH v2 12/14] staging: iio: ad7746: Add ABI documentation

The use cases for this driver don't comply with the current ABI. The ad7746 and ad7152 need an external capacitance or voltage reference to automatically calibrate themselves which is not the normal use case of the calibscale and calibbias ABIs, a new ABI was needed. Signed-off-by: Hernán

[PATCH v2 12/14] staging: iio: ad7746: Add ABI documentation

The use cases for this driver don't comply with the current ABI. The ad7746 and ad7152 need an external capacitance or voltage reference to automatically calibrate themselves which is not the normal use case of the calibscale and calibbias ABIs, a new ABI was needed. Signed-off-by: Hernán

[PATCH v2 11/14] staging: iio: ad7746: Add devicetree bindings documentation

Cc: Rob Herring <robh...@kernel.org> Cc: Mark Rutland <mark.rutl...@arm.com> Cc: devicet...@vger.kernel.org Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- .../devicetree/bindings/staging/iio/cdc/ad7746.txt | 34 ++ 1 file changed, 34 ins

[PATCH v2 11/14] staging: iio: ad7746: Add devicetree bindings documentation

Cc: Rob Herring Cc: Mark Rutland Cc: devicet...@vger.kernel.org Signed-off-by: Hernán Gonzalez --- .../devicetree/bindings/staging/iio/cdc/ad7746.txt | 34 ++ 1 file changed, 34 insertions(+) create mode 100644 Documentation/devicetree/bindings/staging/iio/cdc/ad7746.txt

[PATCH v2 09/14] staging: iio: ad7746: Add remove()

This allows the driver to be probed and removed as a module powering it down on remove(). Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- drivers/staging/iio/cdc/ad7746.c | 26 ++ 1 file changed, 26 insertions(+) diff --git a/drivers/staging/i

[PATCH v2 06/14] staging: iio: ad7746: Reorder variable declarations

Reorder some variable declarations in an inverse-pyramid scheme. Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- 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/i

[PATCH v2 07/14] staging: iio: ad7746: Remove unused defines

Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- drivers/staging/iio/cdc/ad7746.c | 7 --- drivers/staging/iio/cdc/ad7746.h | 5 - 2 files changed, 12 deletions(-) diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/staging/iio/cdc/ad7746.c index f53612a..d

[PATCH v2 08/14] staging: iio: ad7746: Add dt-bindings

This patch adds dt bindings by populating a pdata struct in order to modify as little as possible the existing code. It supports both platform_data and dt-bindings but uses only one depending on CONFIG_OF's value. Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- drivers/s

[PATCH v2 09/14] staging: iio: ad7746: Add remove()

This allows the driver to be probed and removed as a module powering it down on remove(). Signed-off-by: Hernán Gonzalez --- drivers/staging/iio/cdc/ad7746.c | 26 ++ 1 file changed, 26 insertions(+) diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/staging/iio

[PATCH v2 06/14] staging: iio: ad7746: Reorder variable declarations

Reorder some variable declarations in an inverse-pyramid scheme. Signed-off-by: Hernán Gonzalez --- 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 9ef476a

[PATCH v2 07/14] staging: iio: ad7746: Remove unused defines

Signed-off-by: Hernán Gonzalez --- drivers/staging/iio/cdc/ad7746.c | 7 --- drivers/staging/iio/cdc/ad7746.h | 5 - 2 files changed, 12 deletions(-) diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/staging/iio/cdc/ad7746.c index f53612a..d39ab34 100644 --- a/drivers/staging/iio

[PATCH v2 08/14] staging: iio: ad7746: Add dt-bindings

This patch adds dt bindings by populating a pdata struct in order to modify as little as possible the existing code. It supports both platform_data and dt-bindings but uses only one depending on CONFIG_OF's value. Signed-off-by: Hernán Gonzalez --- drivers/staging/iio/cdc/ad7746.c | 54

[PATCH v2 10/14] staging: iio: ad7746: Add comments

Add comments to clarify some of the calculations made, specially when reading or writing values. Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- drivers/staging/iio/cdc/ad7746.c | 32 +++- 1 file changed, 27 insertions(+), 5 deletions(-) diff

[PATCH v2 10/14] staging: iio: ad7746: Add comments

Add comments to clarify some of the calculations made, specially when reading or writing values. Signed-off-by: Hernán Gonzalez --- drivers/staging/iio/cdc/ad7746.c | 32 +++- 1 file changed, 27 insertions(+), 5 deletions(-) diff --git a/drivers/staging/iio/cdc

[PATCH v2 04/14] staging: iio: ad7746: Fix multiple line dereference

Clear checkpatch.pl WARNING about multiple line derefence but creates a new one of line over 80 characters. In my opinion, it improves readability. Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- drivers/staging/iio/cdc/ad7746.c | 3 +-- 1 file changed, 1 insertion

[PATCH v2 01/14] staging: iio: ad7746: Automatically swap values in readings/writings

Data to read or write was being handled with the swab16() macro instead of using i2c_smbus_{read,write}_swapped. Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- drivers/staging/iio/cdc/ad7746.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff

[PATCH v2 04/14] staging: iio: ad7746: Fix multiple line dereference

Clear checkpatch.pl WARNING about multiple line derefence but creates a new one of line over 80 characters. In my opinion, it improves readability. Signed-off-by: Hernán Gonzalez --- drivers/staging/iio/cdc/ad7746.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers

[PATCH v2 01/14] staging: iio: ad7746: Automatically swap values in readings/writings

Data to read or write was being handled with the swab16() macro instead of using i2c_smbus_{read,write}_swapped. Signed-off-by: Hernán Gonzalez --- drivers/staging/iio/cdc/ad7746.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/iio/cdc

[PATCH v2 02/14] staging: iio: ad7746: Adjust arguments to match open parenthesis

Clear a couple more checkpatch.pl CHECKS. Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- drivers/staging/iio/cdc/ad7746.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/staging/iio/cdc/ad7746.c index 5

[PATCH v2 00/14] Move ad7746 driver out of staging

Version 2 of the series trying to move ad7746 our of staging. Changes in v2: (v1-> https://lkml.org/lkml/2018/3/21/406) * Fix some issues pointed out by Jonathan * Power down device on remove * Add new ABI for the use case Hernán Gonzalez (14): staging: iio: ad7746: Automatically swap val

[PATCH v2 02/14] staging: iio: ad7746: Adjust arguments to match open parenthesis

Clear a couple more checkpatch.pl CHECKS. Signed-off-by: Hernán Gonzalez --- drivers/staging/iio/cdc/ad7746.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/staging/iio/cdc/ad7746.c index 53e28ae..516aa93 100644 --- a/drivers

[PATCH v2 00/14] Move ad7746 driver out of staging

Version 2 of the series trying to move ad7746 our of staging. Changes in v2: (v1-> https://lkml.org/lkml/2018/3/21/406) * Fix some issues pointed out by Jonathan * Power down device on remove * Add new ABI for the use case Hernán Gonzalez (14): staging: iio: ad7746: Automatically swap val

[PATCH v2 05/14] staging: iio: ad7746: Reorder includes alphabetically

Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- drivers/staging/iio/cdc/ad7746.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/staging/iio/cdc/ad7746.c index 82fac76..9ef476a 100644 --- a/drivers/s

[PATCH v2 05/14] staging: iio: ad7746: Reorder includes alphabetically

Signed-off-by: Hernán Gonzalez --- drivers/staging/iio/cdc/ad7746.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/staging/iio/cdc/ad7746.c index 82fac76..9ef476a 100644 --- a/drivers/staging/iio/cdc/ad7746.c +++ b

[PATCH v2 03/14] staging: iio: ad7746: Fix bound checkings

Also remove unnecessary parenthesis Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- drivers/staging/iio/cdc/ad7746.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/staging/iio/cdc/ad7746.c index 516aa93..d

[PATCH v2 03/14] staging: iio: ad7746: Fix bound checkings

Also remove unnecessary parenthesis Signed-off-by: Hernán Gonzalez --- drivers/staging/iio/cdc/ad7746.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/staging/iio/cdc/ad7746.c index 516aa93..d793785 100644 --- a/drivers/staging

[PATCH] um: Add HAVE_DEBUG_BUGVERBOSE.

This option restores the DEBUG_BUGVERBOSE functionality as it was previous to commit 9a93848fe787 ("x86/debug: Implement __WARN() using UD0"). Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- arch/um/Kconfig.common | 1 + 1 file changed, 1 insertion(+) diff

[PATCH] um: Add HAVE_DEBUG_BUGVERBOSE.

This option restores the DEBUG_BUGVERBOSE functionality as it was previous to commit 9a93848fe787 ("x86/debug: Implement __WARN() using UD0"). Signed-off-by: Hernán Gonzalez --- arch/um/Kconfig.common | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/um/Kconfig.common

[PATCH 05/11] staging: iio: ad7746: Remove unused defines

Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- drivers/staging/iio/cdc/ad7746.c | 16 drivers/staging/iio/cdc/ad7746.h | 5 - 2 files changed, 21 deletions(-) diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/staging/iio/cdc/ad7746.c index 5

[PATCH 05/11] staging: iio: ad7746: Remove unused defines

Signed-off-by: Hernán Gonzalez --- drivers/staging/iio/cdc/ad7746.c | 16 drivers/staging/iio/cdc/ad7746.h | 5 - 2 files changed, 21 deletions(-) diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/staging/iio/cdc/ad7746.c index 57623db..cba8cd1 100644 --- a/drivers

[PATCH 04/11] staging: iio: ad7746: Reorder variable declarations

Reorder some variable declarations in an inverse-pyramid scheme. Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- 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/i

[PATCH 04/11] staging: iio: ad7746: Reorder variable declarations

Reorder some variable declarations in an inverse-pyramid scheme. Signed-off-by: Hernán Gonzalez --- 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 86919e8

[PATCH 07/11] staging: iio: ad7746: Add remove()

This allows the driver to be probed and removed as a module. Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- drivers/staging/iio/cdc/ad7746.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/staging/iio/cdc/ad7746.c

[PATCH 07/11] staging: iio: ad7746: Add remove()

This allows the driver to be probed and removed as a module. Signed-off-by: Hernán Gonzalez --- drivers/staging/iio/cdc/ad7746.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/staging/iio/cdc/ad7746.c index 815573c..8abba71 100644

[PATCH 11/11] Move cdc ad7746 driver out of staging to mainline iio

Also modify the proper Kconfigs and move documentation. Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- .../devicetree/bindings/{staging => }/iio/cdc/ad7746.txt | 0 drivers/iio/Kconfig | 1 + drivers/iio/cd

[PATCH 11/11] Move cdc ad7746 driver out of staging to mainline iio

Also modify the proper Kconfigs and move documentation. Signed-off-by: Hernán Gonzalez --- .../devicetree/bindings/{staging => }/iio/cdc/ad7746.txt | 0 drivers/iio/Kconfig | 1 + drivers/iio/cdc/Kconfig |

[PATCH 09/11] staging: iio: ad7746: Add devicetree bindings documentation

Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- .../devicetree/bindings/staging/iio/cdc/ad7746.txt | 32 ++ 1 file changed, 32 insertions(+) create mode 100644 Documentation/devicetree/bindings/staging/iio/cdc/ad7746.txt diff --git a/Documentation/devi

[PATCH 06/11] staging: iio: ad7746: Add dt-bindings

This patch adds dt bindings by populating a pdata struct in order to modify as little as possible the existing code. It supports both platform_data and dt-bindings but uses only one depending on CONFIG_OF's value. Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- drivers/s

[PATCH 10/11] staging: iio: ad7746: Rename sysfs attrs to comply with the ABI

Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- drivers/staging/iio/cdc/ad7746.c | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/staging/iio/cdc/ad7746.c index b6b99e2..c1f76fc

[PATCH 09/11] staging: iio: ad7746: Add devicetree bindings documentation

Signed-off-by: Hernán Gonzalez --- .../devicetree/bindings/staging/iio/cdc/ad7746.txt | 32 ++ 1 file changed, 32 insertions(+) create mode 100644 Documentation/devicetree/bindings/staging/iio/cdc/ad7746.txt diff --git a/Documentation/devicetree/bindings/staging/iio/cdc

[PATCH 06/11] staging: iio: ad7746: Add dt-bindings

This patch adds dt bindings by populating a pdata struct in order to modify as little as possible the existing code. It supports both platform_data and dt-bindings but uses only one depending on CONFIG_OF's value. Signed-off-by: Hernán Gonzalez --- drivers/staging/iio/cdc/ad7746.c | 55

[PATCH 10/11] staging: iio: ad7746: Rename sysfs attrs to comply with the ABI

Signed-off-by: Hernán Gonzalez --- drivers/staging/iio/cdc/ad7746.c | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/staging/iio/cdc/ad7746.c index b6b99e2..c1f76fc 100644 --- a/drivers/staging/iio/cdc

[PATCH 08/11] staging: iio: ad7746: Add comments

Add comments to clarify some of the calculations made, specially when reading or writing values. Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- drivers/staging/iio/cdc/ad7746.c | 34 -- 1 file changed, 28 insertions(+), 6 deletions(-) diff

[PATCH 08/11] staging: iio: ad7746: Add comments

Add comments to clarify some of the calculations made, specially when reading or writing values. Signed-off-by: Hernán Gonzalez --- drivers/staging/iio/cdc/ad7746.c | 34 -- 1 file changed, 28 insertions(+), 6 deletions(-) diff --git a/drivers/staging/iio/cdc

[PATCH 01/11] staging: iio: ad7746: Adjust arguments to match open parenthesis

Clear two more checkpatch.pl CHECKS. Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- drivers/staging/iio/cdc/ad7746.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/staging/iio/cdc/ad7746.c index 4

[PATCH 01/11] staging: iio: ad7746: Adjust arguments to match open parenthesis

Clear two more checkpatch.pl CHECKS. Signed-off-by: Hernán Gonzalez --- drivers/staging/iio/cdc/ad7746.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/staging/iio/cdc/ad7746.c index 4882dbc..02e3164 100644 --- a/drivers

[PATCH 03/11] staging: iio: ad7746: Reorder includes alphabetically

Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- drivers/staging/iio/cdc/ad7746.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/staging/iio/cdc/ad7746.c index 557ed4d..86919e8 100644 --- a/drivers/s

[PATCH 03/11] staging: iio: ad7746: Reorder includes alphabetically

Signed-off-by: Hernán Gonzalez --- drivers/staging/iio/cdc/ad7746.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/staging/iio/cdc/ad7746.c index 557ed4d..86919e8 100644 --- a/drivers/staging/iio/cdc/ad7746.c +++ b

[PATCH 02/11] staging: iio: ad7746: Fix multiple line dereference

Clear checkpatch.pl WARNING about multiple line derefence but creates a new one of line over 80 characters. In my opinion, it improves readability. Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- drivers/staging/iio/cdc/ad7746.c | 3 +-- 1 file changed, 1 insertion

[PATCH 02/11] staging: iio: ad7746: Fix multiple line dereference

Clear checkpatch.pl WARNING about multiple line derefence but creates a new one of line over 80 characters. In my opinion, it improves readability. Signed-off-by: Hernán Gonzalez --- drivers/staging/iio/cdc/ad7746.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers

[PATCH 00/11] Move ad7746 out of staging

edback is welcome. I hope this gets out of staging :) Cheers, Hernán Hernán Gonzalez (11): staging: iio: ad7746: Adjust arguments to match open parenthesis staging: iio: ad7746: Fix multiple line dereference staging: iio: ad7746: Reorder includes alphabetically staging: iio: ad7746: Reorder va

[PATCH 00/11] Move ad7746 out of staging

edback is welcome. I hope this gets out of staging :) Cheers, Hernán Hernán Gonzalez (11): staging: iio: ad7746: Adjust arguments to match open parenthesis staging: iio: ad7746: Fix multiple line dereference staging: iio: ad7746: Reorder includes alphabetically staging: iio: ad7746: Reorder va

Plan to move cdc:ad7746 driver out of staging

Hi, This is my plan for moving out the ad7746 driver out of staging. I have some specific questions that would be really helpful if someone can point me in the right direction to go. 1. Pick up on David's clean-up patch. Finish cleaning CHECKs from checkpatch.pl if possible. 2. Reorder includes

Plan to move cdc:ad7746 driver out of staging

Hi, This is my plan for moving out the ad7746 driver out of staging. I have some specific questions that would be really helpful if someone can point me in the right direction to go. 1. Pick up on David's clean-up patch. Finish cleaning CHECKs from checkpatch.pl if possible. 2. Reorder includes

Re: [PATCH] IB/rxe: Remove unused variable (char *rxe_qp_state_name[])

On Tue, Feb 27, 2018 at 7:17 PM, Bart Van Assche <bart.vanass...@wdc.com> wrote: > On Tue, 2018-02-27 at 19:07 -0300, Hernán Gonzalez wrote: >> Note: This is compile only tested as I have no access to the hw. > > Hello Hernán, > > Are you aware that the only hard

Re: [PATCH] IB/rxe: Remove unused variable (char *rxe_qp_state_name[])

On Tue, Feb 27, 2018 at 7:17 PM, Bart Van Assche wrote: > On Tue, 2018-02-27 at 19:07 -0300, Hernán Gonzalez wrote: >> Note: This is compile only tested as I have no access to the hw. > > Hello Hernán, > > Are you aware that the only hardware that is needed to test this dr

[PATCH] qlogic/qed: Constify *pkt_type_str[]

- -24 Total: Before=3599256, After=3599232, chg -0.00% Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- drivers/net/ethernet/qlogic/qed/qed_iwarp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/qlogic/qed/qed_iwarp.c b/drivers/net/et

[PATCH] qlogic/qed: Constify *pkt_type_str[]

- -24 Total: Before=3599256, After=3599232, chg -0.00% Signed-off-by: Hernán Gonzalez --- drivers/net/ethernet/qlogic/qed/qed_iwarp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/qlogic/qed/qed_iwarp.c b/drivers/net/ethernet/qlogic/qed/qed_iwarp.c index

[PATCH] emulex/benet: Constify *be_misconfig_evt_port_state[]

-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- drivers/net/ethernet/emulex/benet/be_cmds.c | 2 +- drivers/net/ethernet/emulex/benet/be_cmds.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/emulex/benet/be_cmds.c b/drivers/net/ethernet/emulex

[PATCH] emulex/benet: Constify *be_misconfig_evt_port_state[]

-by: Hernán Gonzalez --- drivers/net/ethernet/emulex/benet/be_cmds.c | 2 +- drivers/net/ethernet/emulex/benet/be_cmds.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/emulex/benet/be_cmds.c b/drivers/net/ethernet/emulex/benet/be_cmds.c index 1a49297

[PATCH 1/2] security: evm: Move evm_hmac and evm_hash from evm_main.c to evm_crypto.c

273 271 -2 Total: Before=2112094, After=2112092, chg -0.00% Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- security/integrity/evm/evm.h| 2 -- security/integrity/evm/evm_crypto.c | 3 +++ security/integrity/evm/evm_main.c | 2 -- 3 files changed, 3 inse

[PATCH 2/2] security: evm: Constify *integrity_status_msg[]

Note: This is compile only tested. There is no gain from doing this except for some self-documenting. Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- security/integrity/evm/evm_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/security/integri

[PATCH 1/2] security: evm: Move evm_hmac and evm_hash from evm_main.c to evm_crypto.c

273 271 -2 Total: Before=2112094, After=2112092, chg -0.00% Signed-off-by: Hernán Gonzalez --- security/integrity/evm/evm.h| 2 -- security/integrity/evm/evm_crypto.c | 3 +++ security/integrity/evm/evm_main.c | 2 -- 3 files changed, 3 insertions(+), 4 deletions(-) diff

[PATCH 2/2] security: evm: Constify *integrity_status_msg[]

Note: This is compile only tested. There is no gain from doing this except for some self-documenting. Signed-off-by: Hernán Gonzalez --- security/integrity/evm/evm_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/security/integrity/evm/evm_main.c b/security/integrity

[PATCH] IB/rxe: Remove unused variable (char *rxe_qp_state_name[])

24 - -24 Total: Before=3348732, After=3348708, chg -0.00% Signed-off-by: Hernán Gonzalez <her...@vanguardiasur.com.ar> --- drivers/infiniband/sw/rxe/rxe_qp.c| 9 - drivers/infiniband/sw/rxe/rxe_verbs.h | 2 -- 2 files changed, 11 deletions(-) diff --git a/d

[PATCH] IB/rxe: Remove unused variable (char *rxe_qp_state_name[])

24 - -24 Total: Before=3348732, After=3348708, chg -0.00% Signed-off-by: Hernán Gonzalez --- drivers/infiniband/sw/rxe/rxe_qp.c| 9 - drivers/infiniband/sw/rxe/rxe_verbs.h | 2 -- 2 files changed, 11 deletions(-) diff --git a/drivers/infiniband/sw/rxe/rxe_qp.c b

  1   2   >