Re: [PATCH] iio: dac: remove redundant variable 'shift'

2018-07-07 Thread Jonathan Cameron
On Mon, 2 Jul 2018 12:43:20 +0200 Sean Nyekjær wrote: > On 2018-07-02 12:24, Colin King wrote: > > From: Colin Ian King > > > > Variable shift is being assigned but is never used hence it is > > redundant and can be removed. > > > > Cleans up two clang warnings: > > warning: variable ‘shift’

Re: [PATCH] iio: dac: remove redundant variable 'shift'

2018-07-07 Thread Jonathan Cameron
On Mon, 2 Jul 2018 12:43:20 +0200 Sean Nyekjær wrote: > On 2018-07-02 12:24, Colin King wrote: > > From: Colin Ian King > > > > Variable shift is being assigned but is never used hence it is > > redundant and can be removed. > > > > Cleans up two clang warnings: > > warning: variable ‘shift’

Re: [PATCH] iio: dac: remove redundant variable 'shift'

2018-07-02 Thread Sean Nyekjær
On 2018-07-02 12:24, Colin King wrote: From: Colin Ian King Variable shift is being assigned but is never used hence it is redundant and can be removed. Cleans up two clang warnings: warning: variable ‘shift’ set but not used [-Wunused-but-set-variable] Signed-off-by: Colin Ian King

Re: [PATCH] iio: dac: remove redundant variable 'shift'

2018-07-02 Thread Sean Nyekjær
On 2018-07-02 12:24, Colin King wrote: From: Colin Ian King Variable shift is being assigned but is never used hence it is redundant and can be removed. Cleans up two clang warnings: warning: variable ‘shift’ set but not used [-Wunused-but-set-variable] Signed-off-by: Colin Ian King

[PATCH] iio: dac: remove redundant variable 'shift'

2018-07-02 Thread Colin King
From: Colin Ian King Variable shift is being assigned but is never used hence it is redundant and can be removed. Cleans up two clang warnings: warning: variable ‘shift’ set but not used [-Wunused-but-set-variable] Signed-off-by: Colin Ian King --- drivers/iio/dac/ti-dac5571.c | 6 -- 1

[PATCH] iio: dac: remove redundant variable 'shift'

2018-07-02 Thread Colin King
From: Colin Ian King Variable shift is being assigned but is never used hence it is redundant and can be removed. Cleans up two clang warnings: warning: variable ‘shift’ set but not used [-Wunused-but-set-variable] Signed-off-by: Colin Ian King --- drivers/iio/dac/ti-dac5571.c | 6 -- 1