[PATCH v5 1/2] staging: iio: accel: adis16240: enforce SPI mode on probe function

2019-11-23 Thread Rodrigo Carvalho
According to the datasheet, this driver supports only SPI mode 3, so we should enforce it and call spi_setup() on probe function. Signed-off-by: Rodrigo Ribeiro Carvalho --- V5: - Add this patch to the patchset drivers/staging/iio/accel/adis16240.c | 7 +++ 1 file changed, 7 insertions(+)

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

2019-11-23 Thread Rodrigo Carvalho
This patch add device tree binding documentation for ADIS16240. Signed-off-by: Rodrigo Ribeiro Carvalho --- V5: - None .../bindings/iio/accel/adi,adis16240.yaml | 49 +++ 1 file changed, 49 insertions(+) create mode 100644 Documentation/devicetree/bindings/iio/accel/adi

Re: [PATCH 3/3] staging: rtl8188eu: remove return variable from rtw_pwr_unassociated_idle

2019-11-23 Thread Larry Finger
On 11/23/19 9:16 AM, Michael Straube wrote: Function rtw_pwr_unassociated_idle returns boolean values. Remove the return variable ret and the exit label to simplify the function a little bit. Return true or false directly instead. Signed-off-by: Michael Straube --- drivers/staging/rtl8188eu/c

Re: [PATCH 2/3] staging: rtl8188eu: cleanup declarations in rtw_pwrctrl.c

2019-11-23 Thread Larry Finger
On 11/23/19 9:16 AM, Michael Straube wrote: Replace tabs with spaces in declarations to cleanup whitespace. Signed-off-by: Michael Straube --- drivers/staging/rtl8188eu/core/rtw_pwrctrl.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) Acked-by: Larry Finger Thanks

Re: [PATCH 1/3] staging: rtl8188eu: remove unnecessary parentheses in rtw_pwrctrl.c

2019-11-23 Thread Larry Finger
On 11/23/19 9:16 AM, Michael Straube wrote: Remove unnecessary parentheses reported by checkpatch. Signed-off-by: Michael Straube --- drivers/staging/rtl8188eu/core/rtw_pwrctrl.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) Acked-by: Larry Finger Thanks, Larry ___

[PATCH 3/3] staging: rtl8188eu: remove return variable from rtw_pwr_unassociated_idle

2019-11-23 Thread Michael Straube
Function rtw_pwr_unassociated_idle returns boolean values. Remove the return variable ret and the exit label to simplify the function a little bit. Return true or false directly instead. Signed-off-by: Michael Straube --- drivers/staging/rtl8188eu/core/rtw_pwrctrl.c | 10 +++--- 1 file chang

[PATCH 1/3] staging: rtl8188eu: remove unnecessary parentheses in rtw_pwrctrl.c

2019-11-23 Thread Michael Straube
Remove unnecessary parentheses reported by checkpatch. Signed-off-by: Michael Straube --- drivers/staging/rtl8188eu/core/rtw_pwrctrl.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_pwrctrl.c b/drivers/staging/rtl8188eu/core/r

[PATCH 2/3] staging: rtl8188eu: cleanup declarations in rtw_pwrctrl.c

2019-11-23 Thread Michael Straube
Replace tabs with spaces in declarations to cleanup whitespace. Signed-off-by: Michael Straube --- drivers/staging/rtl8188eu/core/rtw_pwrctrl.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_pwrctrl.c b/drivers/staging/rtl

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

2019-11-23 Thread Jonathan Cameron
On Sat, 23 Nov 2019 02:19:27 -0300 Rodrigo Carvalho wrote: > This patch add device tree binding documentation for ADIS16240. > > Signed-off-by: Rodrigo Ribeiro Carvalho No problem with this patch, but I definitely want to see an accompanying one enforcing the SPI mode in the driver. Right now