Re: [PATCH] Staging: iio: cdc: Remove unused macro

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 16/09/15 13:44, Shraddha Barke wrote:
> Remove the macro since it's not used anywhere throughout the kernel
> 
> Signed-off-by: Shraddha Barke <shraddha.6596@xxxxxxxxx>
Whilst I have no objection to cleanup patches like this in general,
I would prefer a more thorough look at the driver.  For example, the
macro 2 lines above isn't used either.

So I'm not going to take this patch as it adds unnecessary noise.
If you want to take a good look at the whole driver and see what
else needs cleaning up then that would be great!

Speaking of which.  Lars, do you guys have any plans to get these devices
out of staging anytime soon? (or know of anyone else who does).
If not perhaps some guidance would help Shradda make that move if he is
interested in doing so?

Thanks,

Jonathan
> ---
>  drivers/staging/iio/cdc/ad7746.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/staging/iio/cdc/ad7746.c b/drivers/staging/iio/cdc/ad7746.c
> index 10fa372..f5a807c 100644
> --- a/drivers/staging/iio/cdc/ad7746.c
> +++ b/drivers/staging/iio/cdc/ad7746.c
> @@ -43,7 +43,6 @@
>  #define AD7746_REG_CAP_GAINH		15
>  #define AD7746_REG_CAP_GAINL		16
>  #define AD7746_REG_VOLT_GAINH		17
> -#define AD7746_REG_VOLT_GAINL		18
>  
>  /* Status Register Bit Designations (AD7746_REG_STATUS) */
>  #define AD7746_STATUS_EXCERR		BIT(3)
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux