On Sun, 20 Sep 2015, Jonathan Cameron wrote:
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?
Yes I'm interested in working on this. Please provide me the necessary
guidance so that I can start on those lines.
Thanks,
Shraddha
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
