On Thu, 20 May 2021, Ian Abbott wrote: > On 20/05/2021 13:25, Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > > > drivers/staging/comedi/drivers/jr3_pci.c: In function ‘jr3_pci_poll_subdevice’: > > drivers/staging/comedi/drivers/jr3_pci.c:507:22: warning: variable ‘min_full_scale’ set but not used [-Wunused-but-set-variable] > > > > Cc: Ian Abbott <abbotti@xxxxxxxxx> > > Cc: H Hartley Sweeten <hsweeten@xxxxxxxxxxxxxxxxxxx> > > Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > > Cc: "Alexander A. Klimov" <grandmaster@xxxxxxxxxxxx> > > Cc: Anders Blomdell <anders.blomdell@xxxxxxxxxxxxxx> > > Cc: linux-staging@xxxxxxxxxxxxxxx > > Signed-off-by: Lee Jones <lee.jones@xxxxxxxxxx> > > --- > > drivers/comedi/drivers/jr3_pci.c | 3 +-- > > 1 file changed, 1 insertion(+), 2 deletions(-) > > Actually, that one is stale now (or will be when Greg's staging-next branch > gets merged into linux-next soon): > > https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git/commit/?h=staging-next&id=b9f38e913a926b455e5048a95f53a993b515509f > > | staging: comedi: Remove unused variable ‘min_full_scale’ and function > 'get_min_full_scales' > | > | The variable ‘min_full_scale’ and function 'get_min_full_scales' are > | not used, So delete them. > No worries, if it's fixed, it's fixed. Could you please take the patches that you are happy with. I'll rework or drop the remainder, thanks. -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog