[PATCH v2] staging: iio: accel: adis16240: Improve readability on write_raw function

2019-08-13 Thread Rodrigo Ribeiro
Replace shift and minus operation by GENMASK macro and remove the local variables used to store intermediate data. Signed-off-by: Rodrigo Ribeiro Carvalho --- v2: - Leave switch statement instead of replace by if statement drivers/staging/iio/accel/adis16240.c | 5 + 1 file changed, 1

Re: [PATCH] staging: iio: accel: adis16240: Improve readability on write_raw function

2019-08-11 Thread Rodrigo Ribeiro
itch statement alone, > but put in your GENMASK change as that one is good and gets > rid of the odd local variable 'bits' as well :) > > Thanks, > > Jonathan > > > > > > Signed-off-by: Rodrigo Ribeiro Carvalho > > --- > > drivers/staging/iio/accel/adi

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

2019-05-23 Thread Rodrigo Ribeiro
This patch adds of_match_table entry in device driver in order to enable spi fallback probing. Signed-off-by: Rodrigo Ribeiro Reviewed-by: Marcelo Schmitt --- drivers/staging/iio/accel/adis16240.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/iio/accel/adis16240.c b

Re: [PATCH] staging:iio:ad7152: Rename misspelled RESEVERD -> RESERVED

2019-02-18 Thread Rodrigo Ribeiro
Em seg, 18 de fev de 2019 às 11:46, Jonathan Cameron escreveu: > > On Thu, 14 Feb 2019 10:51:49 + > "Popa, Stefan Serban" wrote: > > > On Mi, 2019-02-13 at 22:25 -0200, Rodrigo Ribeiro wrote: > > > [External] > > > > > > > > > E

Re: [PATCH] staging:iio:ad7152: Rename misspelled RESEVERD -> RESERVED

2019-01-25 Thread Rodrigo Ribeiro
Em sex, 25 de jan de 2019 às 21:46, Rodrigo Ribeiro escreveu: > > Em sex, 25 de jan de 2019 às 06:20, Alexandru Ardelean > escreveu: > > > > On Thu, Jan 24, 2019 at 9:35 PM Rodrigo Ribeiro > > wrote: > > > > > > Remove the checkpatch.pl check: &g

Re: [PATCH] staging:iio:ad7152: Rename misspelled RESEVERD -> RESERVED

2019-01-25 Thread Rodrigo Ribeiro
Em sex, 25 de jan de 2019 às 06:20, Alexandru Ardelean escreveu: > > On Thu, Jan 24, 2019 at 9:35 PM Rodrigo Ribeiro wrote: > > > > Remove the checkpatch.pl check: > > > > CHECK: 'RESEVERD' may be misspelled - perhaps 'RESERVED'? > > Hey, Hi, Thanks

[PATCH] staging:iio:ad7152: Rename misspelled RESEVERD -> RESERVED

2019-01-24 Thread Rodrigo Ribeiro
Remove the checkpatch.pl check: CHECK: 'RESEVERD' may be misspelled - perhaps 'RESERVED'? Signed-off-by: Rodrigo Ribeiro Signed-off-by: Rafael Tsuha --- This macro is not used anywhere. Should we just correct the spelling or remove the macro? drivers/staging/iio/cdc/ad7152.c | 2 +- 1 file