Re: [PATCH v4 0/5] staging:iio:hmc5843: Few adjustments and support for hmc5983

From: Jonathan Cameron
Date: Tue Jul 15 2014 - 13:57:23 EST

On 15/07/14 13:24, Josef Gajdusek wrote:
This patch series modifies the hmc5843 driver to support the hmc5983 i2c and
spi interfaces.

* Reverted the changed order of iio unregister and setting hmc mode to sleep

* Fixed bug introduced in the first patch
* Readded few comment lines which have gone missing in the hmc5843.c ->
hmc5843_core.c move

* Changes suggested by Lars in
* The i2c and spi drivers are now user-selectable with the core driver
being selected automatically
* The regmap_config structs in both _spi and _i2c files are now static
* The iio_dev allocation is now done in the common function
* pm ops are now defined in the header file and the same definition is
shared between both interface-specific drivers
* regmap tables in hmc5843.h are not longer static (this fixed bunch of
variable not used warnings)

Jonathan, the patch series applies cleanly against mainline (for me at least),
maybe you already have some hmc5843 related patches in your tree which are not
yet in upstream? (probably
You are correct (although curiously the error from patch this time is a much cleaner
fail than before where it was detecting a repeated apply - ah well).
Could you rebase this series against staging/staging-next
from please. I'm far more likely to get something wrong doing
this than you are. Also you have the ability to test the result!

Sorry for the extra work but it's good to see lots of simultaneous interest
in this driver!



Josef Gajdusek (5):
staging:iio:hmc5843: Added regmap support
staging:iio:hmc5843: Split hmc5843.c to multiple files
staging:iio:hmc5843: register <-> value arrays now can have different
staging:iio:hmc5843: Add support for i2c hmc5983
staging:iio:hmc5843: Add support for spi hmc5983

drivers/staging/iio/magnetometer/Kconfig | 30 +-
drivers/staging/iio/magnetometer/Makefile | 4 +-
drivers/staging/iio/magnetometer/hmc5843.c | 652 ------------------------
drivers/staging/iio/magnetometer/hmc5843.h | 86 ++++
drivers/staging/iio/magnetometer/hmc5843_core.c | 638 +++++++++++++++++++++++
drivers/staging/iio/magnetometer/hmc5843_i2c.c | 74 +++
drivers/staging/iio/magnetometer/hmc5843_spi.c | 73 +++
7 files changed, 900 insertions(+), 657 deletions(-)
delete mode 100644 drivers/staging/iio/magnetometer/hmc5843.c
create mode 100644 drivers/staging/iio/magnetometer/hmc5843.h
create mode 100644 drivers/staging/iio/magnetometer/hmc5843_core.c
create mode 100644 drivers/staging/iio/magnetometer/hmc5843_i2c.c
create mode 100644 drivers/staging/iio/magnetometer/hmc5843_spi.c

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at