From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752360AbcEUQjU (ORCPT ); Sat, 21 May 2016 12:39:20 -0400 Received: from saturn.retrosnub.co.uk ([178.18.118.26]:46550 "EHLO saturn.retrosnub.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752090AbcEUQjS (ORCPT ); Sat, 21 May 2016 12:39:18 -0400 Subject: Re: [RFT PATCH] iio: magn: Add support for BMM150 magnetometer To: Daniel Baluta References: <1461674398-18374-1-git-send-email-daniel.baluta@intel.com> Cc: Lucas De Marchi , Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald-Stadler , "linux-iio@vger.kernel.org" , lkml , Lucas De Marchi From: Jonathan Cameron Message-ID: Date: Sat, 21 May 2016 17:39:14 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.1.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 18/05/16 15:01, Daniel Baluta wrote: > On Wed, May 4, 2016 at 1:26 PM, Jonathan Cameron wrote: >> On 27/04/16 15:55, Lucas De Marchi wrote: >>> On Tue, Apr 26, 2016 at 9:39 AM, Daniel Baluta 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 >>>> --- >>>> 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. >