Re: [PATCH 0/4] iio:magnetometer:ak8975 cleanup and move out of staging.

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

 



Laxman, any thoughts on these?

I'm not in an immediate rush with this series, but have never been in
favour of keeping things queued up for too long!

Jonathan

On 03/24/2013 04:58 PM, Jonathan Cameron wrote:
> When I proposed moving this out of staging a while back, Lars-Peter
> pointed out a few 'unusual' elements in the driver.  This series
> addresses those and the interesting question of what the NOSTART flag
> was doing there in the read code (came up in the dicussion of one of
> Lars-Peter's points).
> 
> The caching in the write is a little unnecessary, but isn't technically
> incorrect so I haven't touched that.
> 
> If anyone has one of these to test and make sure the changes haven't
> broken anything that would be great!
> 
> Thanks,
> 
> Jonathan
> 
> Jonathan Cameron (4):
>   staging:iio:magnetometer:ak8975 drop unused eoc_irq
>   staging:iio:magnetometer:ak8975 drop I2C_M_NOSTART flag in read_data
>   staging:iio:magnetometer:ak8975 use standard i2c_smbus read functions.
>   iio:magnetometer:ak8975 move driver out of staging
> 
>  drivers/iio/magnetometer/Kconfig          |  11 +
>  drivers/iio/magnetometer/Makefile         |   1 +
>  drivers/iio/magnetometer/ak8975.c         | 485 ++++++++++++++++++++++++++++
>  drivers/staging/iio/magnetometer/Kconfig  |  11 -
>  drivers/staging/iio/magnetometer/Makefile |   1 -
>  drivers/staging/iio/magnetometer/ak8975.c | 520 ------------------------------
>  6 files changed, 497 insertions(+), 532 deletions(-)
>  create mode 100644 drivers/iio/magnetometer/ak8975.c
>  delete mode 100644 drivers/staging/iio/magnetometer/ak8975.c
> 
--
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