Re: [PATCH v5 5/5] iio: humidity: si7020: add No Hold read mode

From: Andrea Galbusera <gizero@gmail.com>
To: Jonathan Cameron <jic23@kernel.org>
Cc: Wolfram Sang <wsa@the-dreams.de>,
	Nicola Corna <nicola@corna.info>,
	Stephen Warren <swarren@wwwdotorg.org>,
	Lee Jones <lee@kernel.org>, Eric Anholt <eric@anholt.net>,
	linux-i2c@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	Hartmut Knaack <knaack.h@gmx.de>,
	Lars-Peter Clausen <lars@metafoo.de>,
	Peter Meerwald <pmeerw@pmeerw.net>,
	linux-iio@vger.kernel.org, linux-rpi-kernel@lists.infradead.org
Subject: Re: [PATCH v5 5/5] iio: humidity: si7020: add No Hold read mode
Date: Wed, 10 May 2017 16:46:09 +0200	[thread overview]
Message-ID: <CAC+thW1ZL_vOLcQ87rbGk7OJfB+T=tGeEDUiLEVsu2jpbM0rrA@mail.gmail.com> (raw)
In-Reply-To: <56632BE7.2050000@kernel.org>

[-- Attachment #1: Type: text/plain, Size: 2048 bytes --]

Hi!

On Sat, Dec 5, 2015 at 7:24 PM, Jonathan Cameron <jic23@kernel.org> wrote:

> On 30/11/15 17:33, Wolfram Sang wrote:
> > On Sat, Oct 31, 2015 at 10:21:56AM +0000, Jonathan Cameron wrote:
> >> On 29/10/15 11:34, Nicola Corna wrote:
> >>> The Si7013/20/21 modules support 2 read modes:
> >>>  * Hold mode (blocking), where the device stretches the clock until
> the end
> >>> of the measurement
> >>>  * No Hold mode (non-blocking), where the device replies NACK for
> every I2C
> >>> call during the measurement
> >>> Here the No Hold mode is implemented, selectable with the blocking_io
> >>> variable within si7020_platform_data. The default mode is Hold, unless
> the
> >>> adapter does not support clock stretching, in which case the No Hold
> mode
> >>> is used.
> >>>
> >>> Signed-off-by: Nicola Corna <nicola@corna.info>
> >> I'm fine with this.  The dependency below should I think show up in the
> >> coming merge window, so this lot might as well go via Wolfram.
> >>
> >> Acked-by: Jonathan Cameron <jic23@kernel.org>
> >
> > Not sure, will think about it...
> >
> Nicola,
>
> If Wolfram doesn't pick this up, please ping me after the next merge
> window.
> Whilst I'll still have it in my queue, it'll be so long ago by then I'll
> probably
> miss it!
>
> Jonathan
> --
> 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
>

Resurrecting this almost two years old thread because I just found myself
in the situation of using Si7020 with an I2C adapter with broken clock
stretching (probably the same that originated this work). Well, AFAIKT it
seems that this already ack-ed patchset never landed in mainline, probably
due to unlucky timing with merging windows. Didn't try the original patches
myself yet, but going to do it soon... Just wanted to ping people who may
already have an updated version at hand...

Regards

P.S: sorry for unintentional double posting of HTML emails... :-(

[-- Attachment #2: Type: text/html, Size: 3153 bytes --]