From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.3 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_2 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id C8549C43461 for ; Sun, 13 Sep 2020 09:39:40 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 7AF91207EA for ; Sun, 13 Sep 2020 09:39:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1599989980; bh=LDOuPsnG3YrpXDfserIQ0Ztfuc5KAHndkk/KwBoM/JY=; h=Date:From:To:Cc:Subject:In-Reply-To:References:List-ID:From; b=vA1koGDPU0F+cXF75y0jeiT0BvoEURrELjNjg8aPFLy4I6nYSvdxO4dC6tzrHAEpb 1cKoXaDqQ0QvfFRx69t/UPRoWgwdcdctwDqQuWGTnNiOVTLS/p7I6s1xa5OYiglUz2 nOj5f2nXUQsFfRz2GfNO5ChKlKOwpCFsdWU3heWw= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1725949AbgIMJji (ORCPT ); Sun, 13 Sep 2020 05:39:38 -0400 Received: from mail.kernel.org ([198.145.29.99]:43572 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725916AbgIMJjg (ORCPT ); Sun, 13 Sep 2020 05:39:36 -0400 Received: from archlinux (cpc149474-cmbg20-2-0-cust94.5-4.cable.virginm.net [82.4.196.95]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id EDB872076C; Sun, 13 Sep 2020 09:39:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1599989976; bh=LDOuPsnG3YrpXDfserIQ0Ztfuc5KAHndkk/KwBoM/JY=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=hoQHanextgqLEMj3cbQvPcpM4R0u2unqari+FSr8C9XnyBCauE+L4sLLhjyI1ChBG PElubQgSF0R8uiJBB2y1m9BIBmbp6fq+2zmR7am10Dn41r1uB49hjSjZ+jaE/VmMH+ 3QhM8T7ZNy3woveHJpDhCCaXyj22fc6XONSXb0eE= Date: Sun, 13 Sep 2020 10:39:30 +0100 From: Jonathan Cameron To: Christian Eggers Cc: Andy Shevchenko , Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald-Stadler , linux-iio , Linux Kernel Mailing List Subject: Re: [PATCH] iio: light: as73211: Increase measurement timeout Message-ID: <20200913103930.7ba15df0@archlinux> In-Reply-To: <6187855.5mNlXCv1x4@n95hx1g2> References: <20200909154439.10308-1-ceggers@arri.de> <6187855.5mNlXCv1x4@n95hx1g2> X-Mailer: Claws Mail 3.17.6 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 9 Sep 2020 20:04:24 +0200 Christian Eggers wrote: > On Wednesday, 9 September 2020, 18:58:39 CEST, Andy Shevchenko wrote: > > On Wed, Sep 9, 2020 at 6:45 PM Christian Eggers wrote= : =20 > > > We found some sensors which are much slower (20% at room temperature) > > > than nominal. According to the data sheet, up to 27% is possible. Now= I > > > add 33% to the nominal time out, hopefully this is enough. =20 > > > > I'm wondering if this is a function of temperature. > > And if that has an actual thermistor inside... =20 > I haven't designed the hardware, but I guess that clock generation is do= ne > using an RC oscillator which depends on temperature. Temperature measurem= ent > is possible, but I think the recommended way is to externally control the > integration time using the SYN pin and to get the actual number of RC cyc= les > from a register. But generating the required waveform for the SYN pin usi= ng a > uC general purpose timer seems challenging. >=20 > I wouldn't be surprised if the measurement results also depend on the exa= ct > value of the supply voltage. Applied to the togreg branch of iio.git and added a fixes tag to the patch that introduced the driver. Note, please try to provide me with one of tho= se even if the patch hasn't yet gotten into mainline. It makes life a little easier for the stable maintainers etc. I don't always remember to add one at point of applying the patch. Thanks, Jonathan >=20 >=20 > ________________________________ > [http://assets.arri.com/media/sign/2020-04-03-E-mail-signature-Stellar2_= V1.jpg] >=20 > Get all the latest information from www.arri.com, = Facebook, Twitter, Instagram and YouTube. >=20 > Arnold & Richter Cine Technik GmbH & Co. Betriebs KG > Sitz: M=C3=BCnchen - Registergericht: Amtsgericht M=C3=BCnchen - Handelsr= egisternummer: HRA 57918 > Pers=C3=B6nlich haftender Gesellschafter: Arnold & Richter Cine Technik G= mbH > Sitz: M=C3=BCnchen - Registergericht: Amtsgericht M=C3=BCnchen - Handelsr= egisternummer: HRB 54477 > Gesch=C3=A4ftsf=C3=BChrer: Dr. Michael Neuh=C3=A4user; Stephan Schenk; Wa= lter Trauninger; Markus Zeiler