- 23 Aug, 2011 40 commits
-
-
Jonathan Cameron authored
I still have numerous issues with this driver and it needs a substantial rewrite. Whilst doing this patch at least one bug has been fixed (temp reading didn't have an address) and a number of missing channel info elements added. Untested! Signed-off-by:
Jonathan Cameron <jic23@cam.ac.uk> Acked-by:
Michael Hennerich <michael.hennerich@analog.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@suse.de>
-
Jonathan Cameron authored
Currently this is the only driver using some of the defines in accel.h. If these become common we'll just add them to the channel_spec options, for now push down into this driver. Signed-off-by:
-
Jonathan Cameron authored
Still a very odd driver. This patch has the side effect of changing it's name slightly. Signed-off-by:
-
Jonathan Cameron authored
Simple conversion now we've dropped the type attributes. Signed-off-by:
-
Jonathan Cameron authored
Signed-off-by:
-
Jonathan Cameron authored
This is a rare driver that is so simple it actually gets longer as a result of this conversion. Oh well, swings and roundabouts. Signed-off-by:
-
Jonathan Cameron authored
Signed-off-by:
-
Jonathan Cameron authored
This needs confirmation that the devices is happy if another part is talked to in between the request for a register and the read back. Signed-off-by:
-
Jonathan Cameron authored
Signed-off-by:
-
Jonathan Cameron authored
Signed-off-by:
-
Jonathan Cameron authored
Lots of minor bits and pieces. Signed-off-by:
-
Jonathan Cameron authored
The equivalent should always be done using iio_alllocate_device with to create a private area for the driver and then iio_priv to access it. There may be other uses for a private data pointer but right now it just leads driver writers astray. Signed-off-by:
-
Jonathan Cameron authored
There are some unusual corners in the probe function of this driver, so may need another look. V2: Now with the check for allocation success not inverted. V3: Now with the i2c devdata calls actually being correctly cast. Signed-off-by:
Jon Brenner <jbrenner@TAOSinc.com> Signed-off-by:
-
Jonathan Cameron authored
Also push one down into the only driver that actually uses it. Signed-off-by:
-
Jonathan Cameron authored
Reorder to remove need for definitions currently in header. Remove ida related utility function defs from header. Signed-off-by:
-
Jonathan Cameron authored
Fix trivial wrong index (no effect on any current drivers). Signed-off-by:
-
Jonathan Cameron authored
Signed-off-by:
-
Jonathan Cameron authored
Signed-off-by:
-
Jonathan Cameron authored
Clear out all the excessive includes in the *_ring.c files. Mixture of cut and paste bloat and things needed for code that has gone away. Signed-off-by:
-
Jonathan Cameron authored
I will assume these were cut and pastes messup. Signed-off-by:
-
Jonathan Cameron authored
Signed-off-by:
-
Jonathan Cameron authored
More headers now not needed after iio_chan_spec conversions. A couple of drivers were using helpers from adc.h. Given these were trivial, they are pushed down into the drivers. Signed-off-by:
-
Jonathan Cameron authored
Missing from the iio_chan_spec conversion patches. Signed-off-by:
-
Jonathan Cameron authored
There is no clear use case for this functionality in a driver that only supports slow reading via sysfs. The interface use was non compliant with the abi motiving it being dropped. It can go back in if anyone ever implements buffered reading support for this device. Then it will be controlled as part of the buffering abi where this should indeed be correctly supported. Signed-off-by:
-
Jonathan Cameron authored
These should have gone with the iio_chan_spec conversion. Signed-off-by:
-
Jonathan Cameron authored
These should have gone with the iio_chan_spec conversion patch. Signed-off-by:
-
Jonathan Cameron authored
Right now this results in increased code, but I still think it is worth doing to avoid replication across instances of drivers etc and move as much stuff as possible to constant. Ops structure is optional for the occasional driver that uses none of it (currently only the ad7793). Signed-off-by:
-
Jonathan Cameron authored
These should really have gone away with the code that needed them. Signed-off-by:
-
Jonathan Cameron authored
Signed-off-by:
-
Jonathan Cameron authored
Getting rid of messages that make it harder to spot important issues. Some code removed that will be useful one day. Can put it back then. Signed-off-by:
-
Jonathan Cameron authored
Signed-off-by: -
Jonathan Cameron authored
No idea how this particularly 'quirk' got in there in the first place. Signed-off-by:
-
Jonathan Cameron authored
Signed-off-by:
-
Jonathan Cameron authored
Needed to ensure the reference counting is correct. Temporary fix whilst discussions are ongoing. Signed-off-by:
-
Jonathan Cameron authored
Smatch caught bug. Signed-off-by:
-
Jonathan Cameron authored
Introduced when I did the iio_chan_spec conversion. Detected by smatch. Signed-off-by:
-
Jonathan Cameron authored
Coccinelle found this one. I'm not certain what the intent is, but this fix makes no functional changes. Signed-off-by:
-
Jonathan Cameron authored
Vivien pointed out that 99e5dc45 removed the IIO_DEVICE_ATTR_2 macro but didn't clean up the other macros that used it. As it turns out, no one is using val2 in tree, so lets scrap that until it is needed. Signed-off-by:
Vivien Didelot <vivien.didelot@savoirfairelinux.com> Signed-off-by:
-
Jonathan Cameron authored
The unwinding index was j, not i and it is much simpler to do these as a single loop unwinding those elements necessary in before jumping to the error handler. Signed-off-by:
-
Jonathan Cameron authored
indio_dev->channels was accidentally removed thus preventing any sysfs registrations occuring for direct reading. That hid the other issue. Unfortunately the incorrect handling leads to misspresentation of data in sysfs reads. Signed-off-by:
-
