Received: by 2002:a05:7412:8d09:b0:fa:4c10:6cad with SMTP id bj9csp594808rdb; Tue, 16 Jan 2024 09:36:16 -0800 (PST) X-Google-Smtp-Source: AGHT+IGfnKDcNhsDp8zXwpvQ0KjeJo6u2qvLPDWlTwpBM82U/8IyTVtAZfY+9CueJM1aNrzG5OT0 X-Received: by 2002:a17:90a:8c8:b0:28c:f473:5e23 with SMTP id 8-20020a17090a08c800b0028cf4735e23mr3895599pjn.46.1705426576580; Tue, 16 Jan 2024 09:36:16 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1705426576; cv=pass; d=google.com; s=arc-20160816; b=b8ZOeCu6ugYOAITKlyP5DOY2Jhf8K1c1MJmvJ1sA9djA6wrmFqNw/+iwp3e6ze7XFT dcWPOr2/1EnUB2DCnvvYGpCyQZRZsddVTyqUGOjxazhyDkqgMfy8Eg49jiGtkeSYQPid NnZ4ahGAA91xfvR7J5A3jvFj6sGifpgvDTLeXWV8xfdgwaWK6vB1h2wiN/0/inlYtjsb /ZXVBdREvWTcX9Sc2IpwCii3hESXBF+wwHvGAL6WWU398TINA9M/QfinuxY+dq+V9LBI Xwe8gW135JeGtaaBUBmP9tvj3ZDJi2owohVHOKSoafYFZ47yC79/PEJXvmkmnxWlv9cE r+ag== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=in-reply-to:content-disposition:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:message-id:subject:cc :to:from:date:dkim-signature; bh=QvghzboVnKddg18PmVKoXVcd4p8TB6k40BhKbsem3nU=; fh=BWnAjYfdlmb/sAx8Uvpgk9jVfIGX6yDVGiHr1S1pzXo=; b=XPv71YGWe3AS2Ca0WEYVUD58ZX4CIZJDUv/4ldZI2skb85qvNxptjycgX+8StUa+KW wR53xTLR8gWnVztkPLU08t+TWbnSm4YRHeXQFZ9qJY8ofZ0piz4oPmI2RYFrfC4Npkd/ tqeJju1W5L4fag6RRVFk86Y+n0DF/0dFGYfDSNTb1l8v5YpWHvZ2xu/FzTMOZmT6VpPh jOIPSIT5ynrE2d1gDv+2Z2nhbXt6jrdDWM+Tn6e0c+GjSus3Cn+ySw1wDw83IPNgXCGj pqbazvKrHTMq+wnsG4uyTnXY06QjE+FQr3imFAMXnGaz3mvLWeJRyoKXjvtjrrcJtGUm EKFg== ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=n8mMRdRy; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-kernel+bounces-27627-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-27627-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [139.178.88.99]) by mx.google.com with ESMTPS id sb16-20020a17090b50d000b0028db5e93702si12221945pjb.21.2024.01.16.09.36.16 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 16 Jan 2024 09:36:16 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-27627-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=n8mMRdRy; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-kernel+bounces-27627-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-27627-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sv.mirrors.kernel.org (Postfix) with ESMTPS id 20315286BDA for ; Tue, 16 Jan 2024 17:30:35 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id C12B51CABA; Tue, 16 Jan 2024 17:30:26 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="n8mMRdRy" 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 E1BD61CA87; Tue, 16 Jan 2024 17:30:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1705426226; cv=none; b=VilWETcTcbsE5uCaUwqhh6YBULLUH9gMCBiBgQ/OGqZdACGix6IUsojpq8ZyKma+5YV8NWO65QaLjzwujtpECSHX/dyYQqzzsnRHqVV0KAOhbJnF22AbX4idIlEkis1oG2+KgRtZnAit5Izo7gp3QVCmJkLLz4ndIZRmc6w8a8U= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1705426226; c=relaxed/simple; bh=MHbGCgDXoLO32ycO/d98J+MN0cEFtEnVDmEKDb8wMKM=; h=Received:DKIM-Signature:Date:From:To:Cc:Subject:Message-ID: References:MIME-Version:Content-Type:Content-Disposition: In-Reply-To; b=tZJJ/0rUXOdKz2Wm0ZgIznEHSIsG0iRZj0TEwz+gvLgVZ9q4U9aNfLjPSkp5Oa2ZnWfAtDNqOiR9tzjucr+AYGM3E4AkkQIGea1QiW6sFjATIs757y1fb7MQ2slMgDga+k4sLQOlLCjHFCmPjCQhfvifl+dCm5nE0arzCunl4Dg= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=n8mMRdRy; arc=none smtp.client-ip=10.30.226.201 Received: by smtp.kernel.org (Postfix) with ESMTPSA id 429EFC43399; Tue, 16 Jan 2024 17:30:25 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1705426225; bh=MHbGCgDXoLO32ycO/d98J+MN0cEFtEnVDmEKDb8wMKM=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=n8mMRdRyuQK3x1YDlCIYh4rOrj4/+pme/Xl+Oz+G404d7dUqv++TTe9baz3z89B1p m1jNEdf7miWbDQ2Jk1lqN2ebccMcFj+zptjgjwnpBfWz2uGLbhfYv1XmtdDvWHJKjJ n2g3jyYfqTso8XYrVe9Gu67LtKhfyFncXfB1FgtawtMMOpFMIPYc+sW7WyZN4SrLy2 RwaXDubX4IFBdk/9r/aFqWPlGu0Yxtrm5mWm9j8/oMEJbgf72OpBEvudQAWMTlxiqV eg5JFZr33fDOodbi9vaRudll9qep9HfEsiw2NeXtf5wZfiK93RlGoi/b0RmhDANJwj 5wWjTppvXIO+A== Date: Tue, 16 Jan 2024 11:30:23 -0600 From: Rob Herring To: Petre Rodan Cc: Krzysztof Kozlowski , linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Jonathan Cameron , Lars-Peter Clausen , Krzysztof Kozlowski , Conor Dooley Subject: Re: [PATCH 2/6] dt-bindings: iio: pressure: honeywell,hsc030pa.yaml add sleep-mode Message-ID: <20240116173023.GA139792-robh@kernel.org> References: <20240110172306.31273-1-petre.rodan@subdimension.ro> <20240110172306.31273-3-petre.rodan@subdimension.ro> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Fri, Jan 12, 2024 at 09:30:30AM +0200, Petre Rodan wrote: > > Hello Krzysztof, > > On Wed, Jan 10, 2024 at 09:48:34PM +0100, Krzysztof Kozlowski wrote: > > On 10/01/2024 18:22, Petre Rodan wrote: > > > Add sleep-mode property present in some custom chips. > > > > > > This flag activates a special wakeup sequence prior to conversion. > > > > > > Signed-off-by: Petre Rodan > > > --- > > > .../bindings/iio/pressure/honeywell,hsc030pa.yaml | 10 ++++++++++ > > > 1 file changed, 10 insertions(+) > > > > > > diff --git a/Documentation/devicetree/bindings/iio/pressure/honeywell,hsc030pa.yaml b/Documentation/devicetree/bindings/iio/pressure/honeywell,hsc030pa.yaml > > > index 89977b9f01cf..350da1d6991b 100644 > > > --- a/Documentation/devicetree/bindings/iio/pressure/honeywell,hsc030pa.yaml > > > +++ b/Documentation/devicetree/bindings/iio/pressure/honeywell,hsc030pa.yaml > > > @@ -86,6 +86,15 @@ properties: > > > Maximum pressure value the sensor can measure in pascal. > > > To be specified only if honeywell,pressure-triplet is set to "NA". > > > > > > + honeywell,sleep-mode: > > > > "Sleep mode" naming suggests there are choices, like mode foo and mode > > bar. Probably you want something like "sleep-between-measurements" or > > something matching how does it work. > > "sleep mode" is the terminology used by Honeywell and it defines a chip capability. > it is present in the HSC/SSC and ABP series of ICs. > > other such options (capabilities) include temperature output in the ABP series. > > the action the driver needs to perform if this option is present is to provide a > wake-up sequence before reading out the conversions. > > now regarding a rename of this property, I would vote to leave it as is - for the > users to have a 1:1 equivalence of terms between the driver and the datasheet. > > I say that because for instance in circuit design when a part symbol and > footprint is drawn based on a datasheet it is recommended to keep the same pin > notations and the same block diagram as in the datasheet, precisely for this 1:1 > equivalence, so there is no uncertainty for the end-user. At least add a '-en' suffix so it is clear this property enables the mode. We have both flavors (enables and disables). Low power modes between samples is pretty common on these devices. We should consider if this should be a common property. Jonathan? Rob