[RFT PATCH] iio: magn: Add support for BMM150 magnetometer

[RFT PATCH] iio: magn: Add support for BMM150 magnetometer

[Daniel Baluta]

BMM150 is register compatible with magnetometer part of
BMC156.

Datasheet is at:
http://www.mouser.com/ds/2/783/BST-BMM150-DS001-01-786480.pdf

Signed-off-by: Daniel Baluta <daniel.baluta@intel.com>
---
Lucas let me know if it works for you. I don't have yet the BMM150 evaluation
board. It should be available in few weeks.

I'll just leave this patch here in case anyone steps in to test it.

 drivers/iio/magnetometer/bmc150_magn_i2c.c | 3 +++
 drivers/iio/magnetometer/bmc150_magn_spi.c | 3 +++
 2 files changed, 6 insertions(+)

diff --git a/drivers/iio/magnetometer/bmc150_magn_i2c.c b/drivers/iio/magnetometer/bmc150_magn_i2c.c
index eddc7f0..ee05722 100644
--- a/drivers/iio/magnetometer/bmc150_magn_i2c.c
+++ b/drivers/iio/magnetometer/bmc150_magn_i2c.c
@@ -2,6 +2,7 @@
  * 3-axis magnetometer driver supporting following I2C Bosch-Sensortec chips:
  *  - BMC150
  *  - BMC156
+ *  - BMM150
  *
  * Copyright (c) 2016, Intel Corporation.
  *
@@ -49,6 +50,7 @@ static int bmc150_magn_i2c_remove(struct i2c_client *client)
 static const struct acpi_device_id bmc150_magn_acpi_match[] = {
 	{"BMC150B", 0},
 	{"BMC156B", 0},
+	{"BMM150B", 0},
 	{},
 };
 MODULE_DEVICE_TABLE(acpi, bmc150_magn_acpi_match);
@@ -56,6 +58,7 @@ MODULE_DEVICE_TABLE(acpi, bmc150_magn_acpi_match);
 static const struct i2c_device_id bmc150_magn_i2c_id[] = {
 	{"bmc150_magn",	0},
 	{"bmc156_magn", 0},
+	{"bmm150_magn", 0},
 	{}
 };
 MODULE_DEVICE_TABLE(i2c, bmc150_magn_i2c_id);
diff --git a/drivers/iio/magnetometer/bmc150_magn_spi.c b/drivers/iio/magnetometer/bmc150_magn_spi.c
index c4c738a..7d4152d 100644
--- a/drivers/iio/magnetometer/bmc150_magn_spi.c
+++ b/drivers/iio/magnetometer/bmc150_magn_spi.c
@@ -2,6 +2,7 @@
  * 3-axis magnetometer driver support following SPI Bosch-Sensortec chips:
  *  - BMC150
  *  - BMC156
+ *  - BMM150
  *
  * Copyright (c) 2016, Intel Corporation.
  *
@@ -41,6 +42,7 @@ static int bmc150_magn_spi_remove(struct spi_device *spi)
 static const struct spi_device_id bmc150_magn_spi_id[] = {
 	{"bmc150_magn", 0},
 	{"bmc156_magn", 0},
+	{"bmm150_magn", 0},
 	{}
 };
 MODULE_DEVICE_TABLE(spi, bmc150_magn_spi_id);
@@ -48,6 +50,7 @@ MODULE_DEVICE_TABLE(spi, bmc150_magn_spi_id);
 static const struct acpi_device_id bmc150_magn_acpi_match[] = {
 	{"BMC150B", 0},
 	{"BMC156B", 0},
+	{"BMM150B", 0},
 	{},
 };
 MODULE_DEVICE_TABLE(acpi, bmc150_magn_acpi_match);
-- 
2.5.0

Re: [RFT PATCH] iio: magn: Add support for BMM150 magnetometer

[Lucas De Marchi]

On Tue, Apr 26, 2016 at 9:39 AM, Daniel Baluta <daniel.baluta@intel.com> wrote:
> BMM150 is register compatible with magnetometer part of
> BMC156.
>
> Datasheet is at:
> http://www.mouser.com/ds/2/783/BST-BMM150-DS001-01-786480.pdf
>
> Signed-off-by: Daniel Baluta <daniel.baluta@intel.com>
> ---
> Lucas let me know if it works for you. I don't have yet the BMM150 evaluation
> board. It should be available in few weeks.
>
> I'll just leave this patch here in case anyone steps in to test it.

It will take a while for my test board to arrive here, so expect delay
on testing this.

thanks
Lucas De Marchi

Re: [RFT PATCH] iio: magn: Add support for BMM150 magnetometer

[Jonathan Cameron]

On 27/04/16 15:55, Lucas De Marchi wrote:
> On Tue, Apr 26, 2016 at 9:39 AM, Daniel Baluta <daniel.baluta@intel.com> wrote:
>> BMM150 is register compatible with magnetometer part of
>> BMC156.
>>
>> Datasheet is at:
>> http://www.mouser.com/ds/2/783/BST-BMM150-DS001-01-786480.pdf
>>
>> Signed-off-by: Daniel Baluta <daniel.baluta@intel.com>
>> ---
>> Lucas let me know if it works for you. I don't have yet the BMM150 evaluation
>> board. It should be available in few weeks.
>>
>> I'll just leave this patch here in case anyone steps in to test it.
> 
> It will take a while for my test board to arrive here, so expect delay
> on testing this.
I'm dropping this from my list of outstanding patches for now.
It'll resurface when you guys restart the thread.

Jonathan
> 
> thanks
> Lucas De Marchi
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

Re: [RFT PATCH] iio: magn: Add support for BMM150 magnetometer

[Daniel Baluta]

On Wed, May 4, 2016 at 1:26 PM, Jonathan Cameron <jic23@kernel.org> wrote:
> On 27/04/16 15:55, Lucas De Marchi wrote:
>> On Tue, Apr 26, 2016 at 9:39 AM, Daniel Baluta <daniel.baluta@intel.com> wrote:
>>> BMM150 is register compatible with magnetometer part of
>>> BMC156.
>>>
>>> Datasheet is at:
>>> http://www.mouser.com/ds/2/783/BST-BMM150-DS001-01-786480.pdf
>>>
>>> Signed-off-by: Daniel Baluta <daniel.baluta@intel.com>
>>> ---
>>> Lucas let me know if it works for you. I don't have yet the BMM150 evaluation
>>> board. It should be available in few weeks.
>>>
>>> I'll just leave this patch here in case anyone steps in to test it.
>>
>> It will take a while for my test board to arrive here, so expect delay
>> on testing this.
> I'm dropping this from my list of outstanding patches for now.
> It'll resurface when you guys restart the thread.

Got the shuttle board and basic tests with generic_buffer seem to work fine.

Jonathan can you pick this up if no other comments?

thanks,
Daniel.

Re: [RFT PATCH] iio: magn: Add support for BMM150 magnetometer

[Jonathan Cameron]

On 18/05/16 15:01, Daniel Baluta wrote:
> On Wed, May 4, 2016 at 1:26 PM, Jonathan Cameron <jic23@kernel.org> wrote:
>> On 27/04/16 15:55, Lucas De Marchi wrote:
>>> On Tue, Apr 26, 2016 at 9:39 AM, Daniel Baluta <daniel.baluta@intel.com> wrote:
>>>> BMM150 is register compatible with magnetometer part of
>>>> BMC156.
>>>>
>>>> Datasheet is at:
>>>> http://www.mouser.com/ds/2/783/BST-BMM150-DS001-01-786480.pdf
>>>>
>>>> Signed-off-by: Daniel Baluta <daniel.baluta@intel.com>
>>>> ---
>>>> Lucas let me know if it works for you. I don't have yet the BMM150 evaluation
>>>> board. It should be available in few weeks.
>>>>
>>>> I'll just leave this patch here in case anyone steps in to test it.
>>>
>>> It will take a while for my test board to arrive here, so expect delay
>>> on testing this.
>> I'm dropping this from my list of outstanding patches for now.
>> It'll resurface when you guys restart the thread.
> 
> Got the shuttle board and basic tests with generic_buffer seem to work fine.
> 
> Jonathan can you pick this up if no other comments?
Done.  It might be worth a follow up to list the supported parts in the Kconfig
description.  Given they aren't listed already, I'm not going to add this one
on it's own and it would make little sense to add them all in this patch.
> 
> thanks,
> Daniel.
>