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=-15.2 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham 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 0139DC6377D for ; Thu, 22 Jul 2021 09:37:10 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id BC0506120C for ; Thu, 22 Jul 2021 09:37:09 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org BC0506120C Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=cerno.tech Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=kYfoAAasJc1RhyaryCJEYueyT7S1T4K5EsCQwFov17Q=; b=v117zX7/Al6FfR vb1g2Z9aclIjQuNRXNlZA2YNudcPCXE9CfF6e/2m/GoIud8YLaY7o723in38R5VnQkJ7QRb5lfNaD JMbsUl+86Sy9fxF/ptUaiW0IjMrODZNPvTfi0fg+aWgV1RuE7IQpvqyENN6+1wtRNw9wQVXBBEoFc nUQewZ2UH6Ms1PPJWUCt58BzvH0R2gTacVSywT57HHRWcWoBQs/Ia1hlzC5YZqA7Yu/ObeZCVPSYu mqc3NIejTFpkvm0Bm5/ro0yl2iC3ozitrByXM4iRZBdCfijOBDnZgcELdAgHM0j/q6t4zAfTaDBfp LinQ9E9jlGVEbnleUb5A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1m6V6v-000vzd-VP; Thu, 22 Jul 2021 09:35:22 +0000 Received: from new2-smtp.messagingengine.com ([66.111.4.224]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1m6V6b-000vuD-FD for linux-arm-kernel@lists.infradead.org; Thu, 22 Jul 2021 09:35:03 +0000 Received: from compute5.internal (compute5.nyi.internal [10.202.2.45]) by mailnew.nyi.internal (Postfix) with ESMTP id 934A25816EA; Thu, 22 Jul 2021 05:34:57 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute5.internal (MEProxy); Thu, 22 Jul 2021 05:34:57 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cerno.tech; h= date:from:to:cc:subject:message-id:references:mime-version :content-type:content-transfer-encoding:in-reply-to; s=fm3; bh=8 y0zRQbruxumYDBEGoO0So5HhtW4nXa/vBkFW73KMCs=; b=NS3kdV8NH28cZH6TL qcMWJWI/pvX6CQRqFEeFyrQ171Nmqz56GZxC9iG6rSFNXsL++BjJl5kWcqVyVKAr z/4c7w2ah84USoTeTHQJg47D2GdpSztZwtXE/XcJWcUKbxwb1KWrYYOje+IxlEo1 IPgSHMlXpZDWRWeYytLpLoEq4dsmJkpPvOriJvCI9BT8yYPaLZ//a2SoxHojxH5o XseBFsaELm6qTPoYjW0evokPaBIEjJhKSBVu/TgzL8LnzdnFPF98m8vWKz27jAe/ 2Wzv165j/1ppVQOIGGSKTgMnF8XyCJxsvj7ePvMKTLVWwa1EMdVYd4ubQLbWKvrC u6YhQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm3; bh=8y0zRQbruxumYDBEGoO0So5HhtW4nXa/vBkFW73KM Cs=; b=CypnoWG+pHCaku9z9mgZSO9LrvyY7WmJ/wu5QZ1YH12iKJ8itFb1BXMzb VYxWmwYYW2OoQzDNMrlvZ5bZiLVk3LwnOILWH8o8rejr3Iz2DLasa7gYs6vi9qqF MvZSAZKHWlsTy2QmDYzgbmGJKSHhJusBG9ydUyD5OobHtEkrcwpU6gPLBdWJIfS1 5PEb3JT/mIvYGZmaKXrPbdNYJjm9AllgK/0sQi0+jm6Q1P1mnc/FJcoWJgC5ulCK mvFmDmajNaG+xNupebmnRIyTyZdnzAXlIRABXZGfTT9gRwAxSTOI4N+BZgdeZYd3 Vp0Umg1xCQLrTSD2tvOXG73wdsapw== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvtddrfeeigdduhecutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfghnecu uegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenuc fjughrpeffhffvuffkfhggtggugfgjsehtqhertddttddvnecuhfhrohhmpeforgigihhm vgcutfhiphgrrhguuceomhgrgihimhgvsegtvghrnhhordhtvggthheqnecuggftrfgrth htvghrnhepgfekheefieeitddtudegteeftdekudelieefffeljeegfeffvdegiedvtedt tdejnecuffhomhgrihhnpeguvghvihgtvghtrhgvvgdrohhrghdpkhgvrhhnvghlrdhorh hgnecuvehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehmrghilhhfrhhomhepmhgr gihimhgvsegtvghrnhhordhtvggthh X-ME-Proxy: Received: by mail.messagingengine.com (Postfix) with ESMTPA; Thu, 22 Jul 2021 05:34:56 -0400 (EDT) Date: Thu, 22 Jul 2021 11:34:53 +0200 From: Maxime Ripard To: Jonathan Cameron Cc: Chen-Yu Tsai , Jernej Skrabec , devicetree@vger.kernel.org, Rob Herring , Frank Rowand , linux-arm-kernel@lists.infradead.org, linux-sunxi@googlegroups.com, Guenter Roeck , Jean Delvare , Jonathan Cameron , linux-hwmon@vger.kernel.org Subject: Re: [PATCH 14/54] dt-bindings: hwmon: Add IIO HWMON binding Message-ID: <20210722093453.2jltinepdphrajed@gilmour> References: <20210721140424.725744-1-maxime@cerno.tech> <20210721140424.725744-15-maxime@cerno.tech> <20210722102027.000035a5@Huawei.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20210722102027.000035a5@Huawei.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210722_023501_670223_8939EEA3 X-CRM114-Status: GOOD ( 26.70 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Jonathan, On Thu, Jul 22, 2021 at 10:20:27AM +0100, Jonathan Cameron wrote: > On Wed, 21 Jul 2021 16:03:44 +0200 > Maxime Ripard wrote: > > > Even though we had the iio-hwmon driver for some time and a number of > > boards using it already, we never had a binding for it. Let's add it > > based on what the driver expects and the boards are providing. > > > > Cc: Guenter Roeck > > Cc: Jean Delvare > > Cc: Jonathan Cameron > > Cc: linux-hwmon@vger.kernel.org > > Signed-off-by: Maxime Ripard > > --- > > .../devicetree/bindings/hwmon/iio-hwmon.yaml | 37 +++++++++++++++++++ > > 1 file changed, 37 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml > > > > diff --git a/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml b/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml > > new file mode 100644 > > index 000000000000..49dd40f663a6 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml > > @@ -0,0 +1,37 @@ > > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > > +%YAML 1.2 > > +--- > > +$id: "http://devicetree.org/schemas/hwmon/iio-hwmon.yaml#" > > +$schema: "http://devicetree.org/meta-schemas/core.yaml#" > > + > > +title: ADC-attached Hardware Sensor Device Tree Bindings > > + > > +maintainers: > > + - Jonathan Cameron > > I'd like to share the blame for this one. Binding was originally added > by Guenter. > > https://lore.kernel.org/linux-iio/1359668588-13678-4-git-send-email-linux@roeck-us.net/ > https://lore.kernel.org/linux-iio/1359668588-13678-11-git-send-email-linux@roeck-us.net/ > :) > > As bindings go this one has always been controversial because > it's mapping between linux subsystems rather than really describing > hardware. We've had various discussions over the years on how to do > this differently, but perhaps just documenting the current state is the > way to go. That doesn't stop us moving to something better in the > future. There's DT using it already, so I guess it's a bit too late to have that discussion indeed :) > > + > > +description: > > > + Bindings for hardware monitoring devices connected to ADC controllers > > + supporting the Industrial I/O bindings. > > + > > +properties: > > + compatible: > > + const: iio-hwmon > > + > > + io-channels: > > + minItems: 1 > > + maxItems: 1024 > > Why 1024? Ought to be enough for anyone :) We have to set a maximum and I couldn't find any actual maximum documented, if you prefer another value let me know Maxime _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel