On 07/13/11 16:49, Randy Dunlap wrote:
>
> static ssize_t ade7753_read_frequency(struct device *dev,
> struct device_attribute *attr,
> char *buf)
> {
> int ret, len = 0;
> u8 t;
> int sps;
> ret = ade7753_spi_read_reg_8(dev, ADE7753_MODE, &t);
> if (ret)
> return ret;
>
> t = (t >> 11) & 0x3; /// HUH?
>
Excellent question. Honestly I've passed that line by several times but always whilst
working on something else and had no real idea of what it is actually meant to be
doing. Clearly it's garbage.
So time for some datasheet trawling. The mode register is 16 bit, so I'm guessing that
should have been u16 t; and ade7753_spi_read_reg_16. I think this is a cut and paste
bug form the ade7754 driver where the equivalent is an 8 bit register. Patch below.
Michael, do you have access to test hardware for this one? I might do a more general
tidy up of that driver whilst I'm here. Something fun to do before coffee ;)
Thanks,
Jonathan
Subject: [PATCH] staging:iio:meter:ade7753 should be 16 bit read not 8 bit for mode register.
Build tested only.
Signed-off-by: Jonathan Cameron <jic23@xxxxxxxxx>
Reported-by: Randy Dunlap <rdunlap@xxxxxxxxxxxx>
---
drivers/staging/iio/meter/ade7753.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/iio/meter/ade7753.c b/drivers/staging/iio/meter/ade7753.c
index 59f47fd..482e16a 100644
--- a/drivers/staging/iio/meter/ade7753.c
+++ b/drivers/staging/iio/meter/ade7753.c
@@ -398,9 +398,9 @@ static ssize_t ade7753_read_frequency(struct device *dev,
char *buf)
{
int ret, len = 0;
- u8 t;
+ u16 t;
int sps;
- ret = ade7753_spi_read_reg_8(dev, ADE7753_MODE, &t);
+ ret = ade7753_spi_read_reg_16(dev, ADE7753_MODE, &t);
if (ret)
return ret;
--
1.7.3.4
--
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
