Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.36 as permitted sender) client-ip=23.128.96.36;
From:   "Hennerich, Michael" <Michael.Hennerich@analog.com>
To:     Jonathan Cameron <jic23@kernel.org>,
        David Lechner <dlechner@baylibre.com>
CC:     "linux-iio@vger.kernel.org" <linux-iio@vger.kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-staging@lists.linux.dev" <linux-staging@lists.linux.dev>,
        David Lechner <david@lechnology.com>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Conor Dooley <conor+dt@kernel.org>,
        "Sa, Nuno" <Nuno.Sa@analog.com>,
        Axel Haslam <ahaslam@baylibre.com>,
        Philip Molloy <pmolloy@baylibre.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCH v3 22/27] staging: iio: resolver: ad2s1210: convert LOS
 threshold to event attr
Thread-Topic: [PATCH v3 22/27] staging: iio: resolver: ad2s1210: convert LOS
 threshold to event attr
Thread-Index: AQHZ8voK4kGP7RgaL0+1UUgTmxMToLAzgJiAgAX7huA=
Date:   Wed, 4 Oct 2023 11:01:56 +0000
Message-ID: <SN7PR03MB71329FB51185EAD7BE6C65A78ECBA@SN7PR03MB7132.namprd03.prod.outlook.com>
References: <20230929-ad2s1210-mainline-v3-0-fa4364281745@baylibre.com>
        <20230929-ad2s1210-mainline-v3-22-fa4364281745@baylibre.com>
 <20230930163209.17ee0020@jic23-huawei>
In-Reply-To: <20230930163209.17ee0020@jic23-huawei>
Accept-Language: de-DE, en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?D/84odI/64ykLw2JnbPtXTawNPtAAIxqyniXmVv8a2JloAf3A+VaiS8buVO/?=
 =?us-ascii?Q?dOEw+qN1B01KPGHavbDZDQX/AJhzNYByXkJ4j6qks0I9RU3gyCr/fZ8cFrxM?=
 =?us-ascii?Q?RUmJvAbwdP+qU0XIYObRyI5ehNVbJL/NkXS2Pxsx//NYn1uXa7v4YdQtavAZ?=
 =?us-ascii?Q?DTIvogy6KBQbXq+nXbmKpfjUABeMYmWh0KoEm1TiFLJRbV/Lelnw4mdXVonH?=
 =?us-ascii?Q?RXbp0f0lLo8voKCSkWZs2elkvgRr3hET+67xybjLOULvpbNq/hdnxza3Wc2E?=
 =?us-ascii?Q?dVXpYeEyhTK6Q1Fzn6ZqQQcS5rWlTMeIeGKDxJTvpoHr3/ntRqWxNpkLH/Le?=
 =?us-ascii?Q?8vZoXZsmDC6YSFcrbQGQEO/TT5g9/jLIZAenQYLRoJyrMbaBeXegLPWP0fIs?=
 =?us-ascii?Q?LYTdk3k5gNhDLbvDqR/lyRzy+9p5I3I1mdz6zaQi3RtX+zlBsXA0M0CWLUhY?=
 =?us-ascii?Q?cPae9+4cSJnZ+mLuwhv+jxx3F6HB4tkJnEEFXMK6GxIxq3Bd2+/KosXUsFD+?=
 =?us-ascii?Q?tEwvkJcL8lVTd3Fcdxqk2NcDVmkNLpA8kaUP08/X47bqTzuZQBATEA7+wPuZ?=
 =?us-ascii?Q?W21Xk8cRFb7IsNDkrZUhlow+8SQ6M/87O8jVFYjmopsZznU+R1PGchqWoQsI?=
 =?us-ascii?Q?3JkiYxG+mkpEaeDdkiJ7BnUJKhEegooFcB0o6IsW2O4XDgrIeb39ujHZp13i?=
 =?us-ascii?Q?Vo/iJSd7zd5vyJotj4UNsipK0zzwMvJlo9GINdvRicH7bu3J5aRLMucMgHAE?=
 =?us-ascii?Q?4hmiIX2Xo1H3iM88TM00k7jYCcotyNHgfr5Oab1WCRpvYzWaf2tYkJ4HZ+Sx?=
 =?us-ascii?Q?1pFBID/TzHGdORdvqZg4HHgfd+5WwjVSlorI66kWu2/brHkfK3TSFdU5OIOg?=
 =?us-ascii?Q?N2gIGuxBWk8cReyjePKt+YqQZYqX5g8LG8v0OtVq6b7UgLHQZ4+XrNu22hdA?=
 =?us-ascii?Q?r2sRccHzJ1C9dP6/WYwBjuGZduXrZcF22EFmumLcSNIcBixN40SlImaWG8+J?=
 =?us-ascii?Q?lFpTw0GZtC8H4YGNpUAq4W6h/50TKOk9bD/lrhNg/FgMCILDw01Lk6bK4Bq3?=
 =?us-ascii?Q?GnwPakki5FQ84h4eSjJ8eO5EgccevtnMG31vDz3O6gSYNUXwPJlIUJf7F8cw?=
 =?us-ascii?Q?ElWiAUWTz+Sd1/MakvkfsT0avVAfbuWYu5Xp/RuvSmB1J1ti1/buAocvhhYK?=
 =?us-ascii?Q?haIXJ5dsW9bWC98uWmVZJHAPbRnyeswTKqOfGYGrwlG72+wUQvu2k1FzzkSb?=
 =?us-ascii?Q?1dDvKq5Ctkx/sDr0EtZD7jy6/i4BxZPzaXzHZztpr2a1/aqvHIAowoZO7J7v?=
 =?us-ascii?Q?2LA6uuS6iaPrLYCCQ7JQSzO6usgiGezcDc2Lx2n1k1cu4kTRcM8bHGByh7ZD?=
 =?us-ascii?Q?QQhl3VFhJlzThXZ0KJlPwjWFXSC0y+OWlYP9+PtD5ZiDL9KH3kiY98JPkcHd?=
 =?us-ascii?Q?VAQEczSyo8RbfWoraCYDt4QG3pO7HGYAiKxKgmHAaqj7tLSebTxgGgXmMMYL?=
 =?us-ascii?Q?qZeAgGFtT4m7loeWQVRj9gGYWyc+QqoXRUGh0bPDIRB0Z0J6/VipfPmQFOwD?=
 =?us-ascii?Q?LlvQoYHDMSmvPtoSe975h5lRTayt4aqujaswGDXmR4zcvDdAQmBB87aawW/B?=
 =?us-ascii?Q?pg=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: analog.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: SN7PR03MB7132.namprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: cbc3b264-0684-4b60-d1b7-08dbc4c9536e
X-MS-Exchange-CrossTenant-originalarrivaltime: 04 Oct 2023 11:01:56.9805
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: eaa689b4-8f87-40e0-9c6f-7228de4d754a
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: mbbVJ0FVtJttSLC9LBvCc/36o42wkwTyfh9u0PaP1JfN35GanC6off/EalQgWlv9HaJORGS4dljhtDzbZ027ySYqFJCBPF8G72zICG/pFmg=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY5PR03MB5297
X-Proofpoint-ORIG-GUID: ATNzkWjhXjf-mxwMYmgsY_cMgr8P3fMZ
X-Proofpoint-GUID: ATNzkWjhXjf-mxwMYmgsY_cMgr8P3fMZ
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.267,Aquarius:18.0.980,Hydra:6.0.619,FMLib:17.11.176.26
 definitions=2023-10-04_02,2023-10-02_01,2023-05-22_02
X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 phishscore=0 mlxscore=0
 lowpriorityscore=0 priorityscore=1501 clxscore=1015 spamscore=0
 malwarescore=0 impostorscore=0 bulkscore=0 mlxlogscore=999 adultscore=0
 suspectscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.19.0-2309180000 definitions=main-2310040080
X-Spam-Status: No, score=-0.8 required=5.0 tests=DKIM_SIGNED,DKIM_VALID,
	HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,
	SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on pete.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (pete.vger.email [0.0.0.0]); Wed, 04 Oct 2023 04:03:21 -0700 (PDT)


> -----Original Message-----
> From: Jonathan Cameron <jic23@kernel.org>
> Sent: Samstag, 30. September 2023 17:32
> To: David Lechner <dlechner@baylibre.com>
> Cc: linux-iio@vger.kernel.org; devicetree@vger.kernel.org; linux-
> staging@lists.linux.dev; David Lechner <david@lechnology.com>; Rob Herrin=
g
> <robh+dt@kernel.org>; Krzysztof Kozlowski
> <krzysztof.kozlowski+dt@linaro.org>; Conor Dooley <conor+dt@kernel.org>;
> Hennerich, Michael <Michael.Hennerich@analog.com>; Sa, Nuno
> <Nuno.Sa@analog.com>; Axel Haslam <ahaslam@baylibre.com>; Philip Molloy
> <pmolloy@baylibre.com>; linux-kernel@vger.kernel.org
> Subject: Re: [PATCH v3 22/27] staging: iio: resolver: ad2s1210: convert L=
OS
> threshold to event attr
>=20
>=20
> On Fri, 29 Sep 2023 12:23:27 -0500
> David Lechner <dlechner@baylibre.com> wrote:
>=20
> > From: David Lechner <david@lechnology.com>
> >
> > From: David Lechner <dlechner@baylibre.com>
> >
> > 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 sign=
als.
> >
> > 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 <dlechner@baylibre.com>
>=20
> I think I'm fine with treating these internal signals like this, but I wo=
uld ideally
> like someone from Analog devices to take a look at how these are being do=
ne
> 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.

-Michael

>=20
> Jonathan