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 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 smtp.lore.kernel.org (Postfix) with ESMTPS id B9E74C74A5B for ; Sat, 18 Mar 2023 15:42:01 +0000 (UTC) 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:MIME-Version:References:In-Reply-To: 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=LcNh3h7HnIjKlfp1YzQQ7+h+FobR6oeIj2p9gf6zOGk=; b=27V5dTS7cQFrYD Zy5GqH0K7zt2K7+ezuzRAAzntw/PvYxr6L5Hi3LwR4A7G/MijNcb7mkwZx9NAJ3QLkXGFSwBZJRWT oj/ihBkbDnLDo8Ja6Lj0a1Rft5LM4Js1NiK+mv9cKx1J9PzgaD5Oo7cVffrCXYihCZDZYzlfYiYIF YjFOiHjJK43R2GQHWM9NH3TglVkZE9onvhkWuYqEmwL0UDDEonbM7UNZe/hwHjG6hdWriDOlXDCtr Oyccxqtrxx9YcWc6uPCMtc+93BhP9a/HyQ3sgbUs3juDv0wt6E2MGFRff+oU4QQF62QdAbRyJ0wYU dO1c+B+cbNKtvgpDLUOw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1pdYg2-004xVn-2g; Sat, 18 Mar 2023 15:41:02 +0000 Received: from ams.source.kernel.org ([145.40.68.75]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1pdYfz-004xUr-2D for linux-arm-kernel@lists.infradead.org; Sat, 18 Mar 2023 15:41:01 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 94C32B80064; Sat, 18 Mar 2023 15:40:55 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 3EB3EC433EF; Sat, 18 Mar 2023 15:40:51 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1679154054; bh=PBSGuVhbQD6x3zrLgN8oNDlkbuAgVPgYiQwgc1Br7MM=; h=Date:From:To:List-Id:Cc:Subject:In-Reply-To:References:From; b=ZZNeBEpEHnd1ErxTSJhxX6fIqBDK04GsUoLn4sfF7sWgGdCQUexcQwy3vhVleEusX Gq31lKJcZb19d6YapdZPM5JnywRJ8YnozJtrfXl6GYPKWkHOtlage2eTjpeFfHUHjf 7R/sLc06fZk7D9x2M0WiiG2+n4u7O6KbL+U/LhgZzdr3Am36b4+n/rybms6tVEo7GW BBXBr+xWYX/H0l7uCeVHd9xeaEIhvs59yFx5rTZLOAnBXdYzbJnAY1WId0gDh0LhXZ /xdgvsTElxsaHFuAQ/IbSMmgqYMDKM+C+lZ6nzdWHfwO0zlm1Cjjown19Up5yoW/ju 4xnYtJtjMf3UQ== Date: Sat, 18 Mar 2023 15:55:45 +0000 From: Jonathan Cameron To: Stefan Wahren Cc: Rob Herring , Krzysztof Kozlowski , Lars-Peter Clausen , Arnd Bergmann , Olof Johansson , Evgeniy Polyakov , Shawn Guo , Sascha Hauer , Fabio Estevam , linux-imx@nxp.com, Li Yang , Denis Ciocca , soc@kernel.org, linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, stefan.wahren@i2se.com Subject: Re: [PATCH V2 4/8] dt-bindings: iio: st-sensors: Add IIS328DQ accelerometer Message-ID: <20230318155534.16b47583@jic23-huawei> In-Reply-To: <20230317103323.7741-5-stefan.wahren@chargebyte.com> References: <20230317103323.7741-1-stefan.wahren@chargebyte.com> <20230317103323.7741-5-stefan.wahren@chargebyte.com> X-Mailer: Claws Mail 4.1.1 (GTK 3.24.37; x86_64-pc-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230318_084059_894375_A6962C8C X-CRM114-Status: GOOD ( 18.93 ) 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 On Fri, 17 Mar 2023 11:33:19 +0100 Stefan Wahren wrote: > The ST IIS328DQ is an accelerometer sensor, which is compatible with > the already supported ST H3LIS331DL. So add the new compatible > with a fallback to the ST sensor binding. > > Link: https://lore.kernel.org/linux-iio/2bac9ecf-9d2e-967e-9020-1c950487d781@i2se.com/ > Suggested-by: Jonathan Cameron > Signed-off-by: Stefan Wahren > --- > Documentation/devicetree/bindings/iio/st,st-sensors.yaml | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/Documentation/devicetree/bindings/iio/st,st-sensors.yaml b/Documentation/devicetree/bindings/iio/st,st-sensors.yaml > index c6201976378f..5eb71b24a7cb 100644 > --- a/Documentation/devicetree/bindings/iio/st,st-sensors.yaml > +++ b/Documentation/devicetree/bindings/iio/st,st-sensors.yaml > @@ -48,6 +48,9 @@ properties: > - st,lsm330d-accel > - st,lsm330dl-accel > - st,lsm330dlc-accel > + - items: > + - const: st,iis328dq > + - const: st,h3lis331dl-accel Hmm. So this separates the particular 'oneOf' group from the description associated with the ST accelerometers. DT experts, is there a better way to do this? Ideally we'd also associate other similar groups going forwards so it might become less and less obvious how the groupings work. (bit late for the many groups that exist already in that list above this point). If not I'm fine with it being fairly obvious what is going on in the binding file current and we can rethink it later. > - description: Silan Accelerometers > enum: > - silan,sc7a20 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel