Hi Jonathan, after a long stuggle, I got a working kernel version for my board running as I need it. I tried both the staging/adis16255 and the staging/iio/gyro driver. The latter doesn't work. "adis16260 spi1.1: problem when reading 16 bit register 0x34 iio device0: disable irq failed" A quick look in the code of staging/adis16255 and the data sheet tells me, that you have to read from the higher register address but we read from the lower one. So I changed the value to 0x35, but it doesn't work either. Well in the end I copied "my" read version from staging/adis16255 and put a read call in the probe function (because it is the easiest way to get spi_device structure). Then it seems to work, with higher and lower byte. So my conclusion is, that something went wrong when you casacade and discascade (sorry for the poor english), from spi_device through adis16260_state to device. Sounds stange to me, as it works with the adis16260 chip, right? So maybe it's because I use avr32 architecture? So I don't know if we should dig deeper. The problem is, that I don't have too much time to do it... What do you think? Regards, Matthias 2010/9/27 Jonathan Cameron <jic23@xxxxxxxxx>: > On 09/18/10 16:48, matthias wrote: >> Hi Jonathan, >> >> sorry for not answering yet. I was on vacation and next week I won't >> be able to test the driver. Will try to do it asap.... >> >> Matthias > > Hi Matthias, > > I'm afraid quite a bit has changed over the last few weeks. Feel free to test > this patch set. ÂThe changes since then are merely renames of a couple of > attributes and a lot of stuff for event handling that doesn't effect this > driver. > > If not, I'm hosting a *temporary* git tree with all my various queued up changes > at: > > http://git.kernel.org/?p=linux/kernel/git/jic23/iio_temp.git > > Seems excessive to post this set again until I hear back from you! > > Thanks, > > Jonathan > > p.s. Switched to drivers@xxxxxxxxxx address as that now seems to work. > >> >> 2010/9/18 Jonathan Cameron <jic23@xxxxxxxxx>: >>> On 09/06/10 16:16, Jonathan Cameron wrote: >>>> Mainly a rebase, but a couple of attribute naming fixes as well. >>>> >>>> Note I don't have one of these so if anyone could test that would >>>> be great! >>>> >>>> Signed-off-by: Jonathan Cameron <jic23@xxxxxxxxx> >>>> >>>> Jonathan Cameron (2): >>>>  staging:iio:adis16260 add id table support >>>>  staging:iio:adis16260 add suppport for adis16255 and adis16250. >>>> >>>> Âdrivers/staging/iio/gyro/Kconfig          |  Â8 +- >>>> Âdrivers/staging/iio/gyro/adis16260.h        |  Â3 + >>>> Âdrivers/staging/iio/gyro/adis16260_core.c     Â| Â139 ++++++++++++++------ >>>> Âdrivers/staging/iio/gyro/adis16260_platform_data.h |  19 +++ >>>> Âdrivers/staging/iio/gyro/gyro.h          Â|  Â9 ++ >>>> Â5 files changed, 136 insertions(+), 42 deletions(-) >>>> Âcreate mode 100644 drivers/staging/iio/gyro/adis16260_platform_data.h >>>> >>>> >>> Whilst I haven't tested these (don't have the hardware) I don't think there >>> is anything controversial, so my intent is to push these to Greg before the >>> next merge window. ÂThis is primarily to remove the duplication we currently >>> have with two drivers that effectively cover the same parts. >>> >>> Thanks, >>> >>> Jonathan >>> >> >> >> > > -- motzblog.wordpress.com -- 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
