From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id DCCE821105; Thu, 5 Oct 2023 14:16:34 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="uY2BlzqQ" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 604BCC43397; Thu, 5 Oct 2023 14:16:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1696515394; bh=fWQldp1MXyl2dHxGT2BPOTVVly5guzHelUpsTf07C30=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=uY2BlzqQ8OTEFm7DFcjakRUdD34dpyyVkKd74Wjh7zMxYtV15MwqyAvGPk0hxcV6B gkhjqaPypFX1avgFjSTOEZeTVH8xtL5zR5iKdozju80d6HWrX6od/6/7ZhfG4AA7o7 13tCjNmg7ah2rl71e9YZP5Nd3Lhx0rqD7nAA9bqXPAczQwG6xqxqNlEzoW0dlP1dPF fFu40icf6dmWCTUFupXrHJ9QnITf7nrf5Qw6UkgUsi4cXgrGyQMtSNsEAkoQ8y6jki LXjtxWlSHYp289tf6XfWpAWoK4ZvJfDJA7BQSvF+2IpckkUEc/++tVuhncut6UOLF9 YV5vbbCAxXg1w== Date: Thu, 5 Oct 2023 15:16:37 +0100 From: Jonathan Cameron To: "Hennerich, Michael" Cc: David Lechner , "linux-iio@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-staging@lists.linux.dev" , David Lechner , Rob Herring , Krzysztof Kozlowski , Conor Dooley , "Sa, Nuno" , Axel Haslam , Philip Molloy , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH v3 22/27] staging: iio: resolver: ad2s1210: convert LOS threshold to event attr Message-ID: <20231005151637.4dc990f8@jic23-huawei> In-Reply-To: References: <20230929-ad2s1210-mainline-v3-0-fa4364281745@baylibre.com> <20230929-ad2s1210-mainline-v3-22-fa4364281745@baylibre.com> <20230930163209.17ee0020@jic23-huawei> X-Mailer: Claws Mail 4.1.1 (GTK 3.24.38; x86_64-pc-linux-gnu) Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Wed, 4 Oct 2023 11:01:56 +0000 "Hennerich, Michael" wrote: > > -----Original Message----- > > From: Jonathan Cameron > > Sent: Samstag, 30. September 2023 17:32 > > To: David Lechner > > Cc: linux-iio@vger.kernel.org; devicetree@vger.kernel.org; linux- > > staging@lists.linux.dev; David Lechner ; Rob Herring > > ; Krzysztof Kozlowski > > ; Conor Dooley ; > > Hennerich, Michael ; Sa, Nuno > > ; Axel Haslam ; Philip Molloy > > ; linux-kernel@vger.kernel.org > > Subject: Re: [PATCH v3 22/27] staging: iio: resolver: ad2s1210: convert LOS > > threshold to event attr > > > > > > On Fri, 29 Sep 2023 12:23:27 -0500 > > David Lechner wrote: > > > > > From: David Lechner > > > > > > From: David Lechner > > > > > > The AD2S1210 has a programmable threshold for the loss of signal (LOS) > > > fault. This fault is triggered when either the sine or cosine input > > > falls below the threshold voltage. > > > > > > This patch converts the custom device LOS threshold attribute to an > > > event falling edge threshold attribute on a new monitor signal channel. > > > The monitor signal is an internal signal that combines the amplitudes > > > of the sine and cosine inputs as well as the current angle and > > > position output. This signal is used to detect faults in the input signals. > > > > > > The attribute now uses millivolts instead of the raw register value in > > > accordance with the IIO ABI. > > > > > > Emitting the event will be implemented in a later patch. > > > > > > Signed-off-by: David Lechner > > > > I think I'm fine with treating these internal signals like this, but I would ideally > > like someone from Analog devices to take a look at how these are being done > > and make sure our interpretations of the signals make sense to them. We are > > pushing the boundaries a little here (though we have done similar before for > > fault events I think.) > > Hi Jonathan, > David and I we also had some internal discussion related to this. > I'm sure these fault events and thresholds are understood correctly. > Doing it this or the other way, it needs to be properly documented in order to make sense. > So from my perspective whatever makes the most sense from a IIO ABI > perspective, is the way to forward. Great - as long as keep to a logical mapping I quite like the events approach. Most of these faults are real thresholds on things being measured (even if those 'things' are signals from which stuff is derived for the main measurements the device is making.) Jonathan > > -Michael > > > > > Jonathan >