Message ID | 20231129111041.26782-1-krzysztof.kozlowski@linaro.org |
---|---|
State | New |
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:a5a7:0:b0:403:3b70:6f57 with SMTP id d7csp263688vqn; Wed, 29 Nov 2023 03:14:10 -0800 (PST) X-Google-Smtp-Source: AGHT+IG0vcD+sExhMSgDjnycHCpW1E/v5aeTMwKwBCQwjK4+9AclvJb89jFloFcQnzw4PLRHr4fZ X-Received: by 2002:a17:902:d4c7:b0:1d0:1257:d35b with SMTP id o7-20020a170902d4c700b001d01257d35bmr2862761plg.67.1701256450169; Wed, 29 Nov 2023 03:14:10 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1701256450; cv=none; d=google.com; s=arc-20160816; b=QCY7deCfrqXbD2q1/bdjfLVlzH26E9zq1/PpH2uthecRW9jAu7r886SDiJ3+hsjiDa uM401FADDk88lpfgC7XaIsz/mAfak0fEezne8SC89Z9EBCE99PSNEMjox6YQrqUlWxIo 6OTGmhe/bRcx4XMgpQgWymdwL/IRa0F2vCsZBn0pQ+QhMleNYebj2+zaVR5djUBRPf0T XYtO5Qua8QX5R8uywOtnBndxYunTGJteUFfVjXF5szvGMiTQj46oL781zAYFja3uDVM7 nogCOVIOj8qylEr8Ebj5283AD/djFNLIXk5f29bc+bsyl/MJ/5Dah3mbUzMrjuiRL9L9 OssA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=5n2a/2E5KdJNNYUo3GWmeCj+sTZDQr2kUzeB2+OZTGU=; fh=PlqVqIPDNzEtftefD+UaG9f+PW9XfP+U+TPIs/5R+Fw=; b=EDCgIutWwdrGHzl3Wo5lqKfQEolNrBW4lHubT7YKSTemCQX8IbJMKImzYaF/B5yUZx 8ykIm6T9rnZyRiJzrnNg8i1t5UnpQnfoAk+Z4qb0MTaXrmXEBztJyIY8T61CeZigRY1W W+4SMAVfxlo1p934HhspjUzRVgmOvF7hkvnIa5Sm1CHhV7dsnm5ckemfSvh8S/TgEVKL q4TjwfhTwn8ZxOJx7Tu2tiizGNkKChsQmSK0onwA1F0oUPLd/TTmHiE14ekkp4H5A9B+ nKVSvSrE1CZevsrUlY1hRAuouPGvyi147DQZkMyjJVVF80PSZc/Mab1okchcQDkl84j2 EYLw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=dGQrJDgp; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:3 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from lipwig.vger.email (lipwig.vger.email. [2620:137:e000::3:3]) by mx.google.com with ESMTPS id d10-20020a170902654a00b001cf5d4e776csi13252538pln.287.2023.11.29.03.14.09 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 29 Nov 2023 03:14:10 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:3 as permitted sender) client-ip=2620:137:e000::3:3; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=dGQrJDgp; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:3 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by lipwig.vger.email (Postfix) with ESMTP id 90064803A51C; Wed, 29 Nov 2023 03:14:04 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at lipwig.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231496AbjK2LNz (ORCPT <rfc822;toshivichauhan@gmail.com> + 99 others); Wed, 29 Nov 2023 06:13:55 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36986 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232799AbjK2LNj (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Wed, 29 Nov 2023 06:13:39 -0500 Received: from mail-lf1-x130.google.com (mail-lf1-x130.google.com [IPv6:2a00:1450:4864:20::130]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D493344AA for <linux-kernel@vger.kernel.org>; Wed, 29 Nov 2023 03:10:46 -0800 (PST) Received: by mail-lf1-x130.google.com with SMTP id 2adb3069b0e04-507a29c7eefso8650298e87.1 for <linux-kernel@vger.kernel.org>; Wed, 29 Nov 2023 03:10:46 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1701256245; x=1701861045; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=5n2a/2E5KdJNNYUo3GWmeCj+sTZDQr2kUzeB2+OZTGU=; b=dGQrJDgpaL22J1na3wRLlX1pr8VzAIXggNbC9cJZCd9kzsqLUG8kCNwSPkCOItlVl9 VySV9p76yHHZ2unRrq5zCZZOL6bckBu7ScruYpSC3wx1xKpcOtxdaJ7tpwOPa2IJvkF/ quizStJCPUqy8n/Bhffvqks3jQBD6LUB0ZdJS8IOI7WSDGicG9K6ty/Tbb2c5NYqlcyX peIgBQ3wjel1QLpgB0UV4eo2HDVmprK13Aa7WP5iiJdBC/2xBmVHxXC/3R4FY2NmQg2z yRi7ahvs+rhHh+eXOmrmb2CCqtPeoKbEjV4OBC9Sje1RWO/C9wf3KbxQqPb1f4gRb/gA +zxg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1701256245; x=1701861045; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=5n2a/2E5KdJNNYUo3GWmeCj+sTZDQr2kUzeB2+OZTGU=; b=WLWr9xaCjRdx2zlskGkiWtBdXoEcbSnqpE99pGbNfguO59fcmJ38GuR5cqansd2lVY gqkPEq+0BKNBL/ZsVTAEG5i4hHJiSKl/H7SKd2dWIhwbW0PTxgZFBJl7gVGl6upG7OKq 0FuGgNXN8F+BDS8/415WvMHAaiBhFadpkKUazbNiMvuHtRBppwuLJKQBegaTjj+NMOT0 Xfs51ZQL1QO94h6ZlKi5JUcGXRzNRb4LCHChn/PQFu+x40o1oz/1FbQMuYyMgdnRR1He 42Xi7UPtHHy0dxEHqpRkJq1yUgmBHQkh3/fOSCtjjYW7sQzM3W2i9E0jqtTKLYFtK9GF rKJg== X-Gm-Message-State: AOJu0YypHyy/ZnbAAewNiAs4TaKgHB3Ixa6sD4Ks7tIIJKH2m075vrMb WfmtDUh609ONQF1H4HHjBow5OQ== X-Received: by 2002:a05:6512:2186:b0:508:268b:b087 with SMTP id b6-20020a056512218600b00508268bb087mr9510426lft.26.1701256245015; Wed, 29 Nov 2023 03:10:45 -0800 (PST) Received: from krzk-bin.. ([178.197.223.109]) by smtp.gmail.com with ESMTPSA id x13-20020a170906710d00b009b2ca104988sm7823719ejj.98.2023.11.29.03.10.43 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 29 Nov 2023 03:10:44 -0800 (PST) From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> To: Andreas Klinger <ak@it-klinger.de>, Jonathan Cameron <jic23@kernel.org>, Lars-Peter Clausen <lars@metafoo.de>, Rob Herring <robh+dt@kernel.org>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Conor Dooley <conor+dt@kernel.org>, linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Subject: [PATCH] dt-bindings: iio: honeywell,mprls0025pa: drop ref from pressure properties Date: Wed, 29 Nov 2023 12:10:41 +0100 Message-Id: <20231129111041.26782-1-krzysztof.kozlowski@linaro.org> X-Mailer: git-send-email 2.34.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lipwig.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 (lipwig.vger.email [0.0.0.0]); Wed, 29 Nov 2023 03:14:04 -0800 (PST) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1783896683403632251 X-GMAIL-MSGID: 1783896683403632251 |
Series |
dt-bindings: iio: honeywell,mprls0025pa: drop ref from pressure properties
|
|
Commit Message
Krzysztof Kozlowski
Nov. 29, 2023, 11:10 a.m. UTC
The dtschema treats now properties with '-pascal' suffix as standard one
and already defines $ref for them, thus the $ref should be dropped from
the bindings.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
dtschema change was merged:
https://github.com/devicetree-org/dt-schema/commit/2a1708dcf4ff0b25c4ec46304d6d6cc655c3e635
but not yet released as new dtschema version.
This change should be applied once new dtschema version is released or
Rob says otherwise.
---
.../devicetree/bindings/iio/pressure/honeywell,mprls0025pa.yaml | 2 --
1 file changed, 2 deletions(-)
Comments
On Wed, 29 Nov 2023 12:10:41 +0100, Krzysztof Kozlowski wrote: > The dtschema treats now properties with '-pascal' suffix as standard one > and already defines $ref for them, thus the $ref should be dropped from > the bindings. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > --- > > dtschema change was merged: > https://github.com/devicetree-org/dt-schema/commit/2a1708dcf4ff0b25c4ec46304d6d6cc655c3e635 > but not yet released as new dtschema version. > > This change should be applied once new dtschema version is released or > Rob says otherwise. > --- > .../devicetree/bindings/iio/pressure/honeywell,mprls0025pa.yaml | 2 -- > 1 file changed, 2 deletions(-) > My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check' on your patch (DT_CHECKER_FLAGS is new in v5.13): yamllint warnings/errors: dtschema/dtc warnings/errors: /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/iio/pressure/honeywell,mprls0025pa.yaml: honeywell,pmin-pascal: missing type definition /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/iio/pressure/honeywell,mprls0025pa.yaml: honeywell,pmax-pascal: missing type definition doc reference errors (make refcheckdocs): See https://patchwork.ozlabs.org/project/devicetree-bindings/patch/20231129111041.26782-1-krzysztof.kozlowski@linaro.org The base for the series is generally the latest rc1. A different dependency should be noted in *this* patch. If you already ran 'make dt_binding_check' and didn't see the above error(s), then make sure 'yamllint' is installed and dt-schema is up to date: pip3 install dtschema --upgrade Please check and re-submit after running the above command yourself. Note that DT_SCHEMA_FILES can be set to your schema file to speed up checking your schema. However, it must be unset to test all examples with your schema.
On 29/11/2023 13:27, Rob Herring wrote: > > On Wed, 29 Nov 2023 12:10:41 +0100, Krzysztof Kozlowski wrote: >> The dtschema treats now properties with '-pascal' suffix as standard one >> and already defines $ref for them, thus the $ref should be dropped from >> the bindings. >> >> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >> >> --- >> >> dtschema change was merged: >> https://github.com/devicetree-org/dt-schema/commit/2a1708dcf4ff0b25c4ec46304d6d6cc655c3e635 >> but not yet released as new dtschema version. >> >> This change should be applied once new dtschema version is released or >> Rob says otherwise. >> --- >> .../devicetree/bindings/iio/pressure/honeywell,mprls0025pa.yaml | 2 -- >> 1 file changed, 2 deletions(-) >> > > My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check' > on your patch (DT_CHECKER_FLAGS is new in v5.13): > > yamllint warnings/errors: > > dtschema/dtc warnings/errors: > /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/iio/pressure/honeywell,mprls0025pa.yaml: honeywell,pmin-pascal: missing type definition > /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/iio/pressure/honeywell,mprls0025pa.yaml: honeywell,pmax-pascal: missing type definition > That's expected, depends on the dtschema change. Best regards, Krzysztof
On Wed, Nov 29, 2023 at 6:29 AM Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote: > > On 29/11/2023 13:27, Rob Herring wrote: > > > > On Wed, 29 Nov 2023 12:10:41 +0100, Krzysztof Kozlowski wrote: > >> The dtschema treats now properties with '-pascal' suffix as standard one > >> and already defines $ref for them, thus the $ref should be dropped from > >> the bindings. > >> > >> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > >> > >> --- > >> > >> dtschema change was merged: > >> https://github.com/devicetree-org/dt-schema/commit/2a1708dcf4ff0b25c4ec46304d6d6cc655c3e635 > >> but not yet released as new dtschema version. > >> > >> This change should be applied once new dtschema version is released or > >> Rob says otherwise. > >> --- > >> .../devicetree/bindings/iio/pressure/honeywell,mprls0025pa.yaml | 2 -- > >> 1 file changed, 2 deletions(-) > >> > > > > My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check' > > on your patch (DT_CHECKER_FLAGS is new in v5.13): > > > > yamllint warnings/errors: > > > > dtschema/dtc warnings/errors: > > /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/iio/pressure/honeywell,mprls0025pa.yaml: honeywell,pmin-pascal: missing type definition > > /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/iio/pressure/honeywell,mprls0025pa.yaml: honeywell,pmax-pascal: missing type definition > > > > That's expected, depends on the dtschema change. Well, it wasn't because dtschema version used is git tree. However, the CI job was using master rather than main branch. master falls behind when I use the web interface... Now fixed the CI job to use main. Rob
On Wed, Nov 29, 2023 at 12:10:41PM +0100, Krzysztof Kozlowski wrote: > The dtschema treats now properties with '-pascal' suffix as standard one > and already defines $ref for them, thus the $ref should be dropped from > the bindings. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > --- > > dtschema change was merged: > https://github.com/devicetree-org/dt-schema/commit/2a1708dcf4ff0b25c4ec46304d6d6cc655c3e635 > but not yet released as new dtschema version. > > This change should be applied once new dtschema version is released or > Rob says otherwise. I've made a release, so go ahead and apply this. Acked-by: Rob Herring <robh@kernel.org>
On Fri, 1 Dec 2023 16:35:47 -0600 Rob Herring <robh@kernel.org> wrote: > On Wed, Nov 29, 2023 at 12:10:41PM +0100, Krzysztof Kozlowski wrote: > > The dtschema treats now properties with '-pascal' suffix as standard one > > and already defines $ref for them, thus the $ref should be dropped from > > the bindings. > > > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > > > --- > > > > dtschema change was merged: > > https://github.com/devicetree-org/dt-schema/commit/2a1708dcf4ff0b25c4ec46304d6d6cc655c3e635 > > but not yet released as new dtschema version. > > > > This change should be applied once new dtschema version is released or > > Rob says otherwise. > > I've made a release, so go ahead and apply this. > > Acked-by: Rob Herring <robh@kernel.org> Applied to the togreg branch of iio.git and initially pushed out as testing for 0-day etc to poke at it Thanks, Jonathan
diff --git a/Documentation/devicetree/bindings/iio/pressure/honeywell,mprls0025pa.yaml b/Documentation/devicetree/bindings/iio/pressure/honeywell,mprls0025pa.yaml index b31f8120f14e..d9e903fbfd99 100644 --- a/Documentation/devicetree/bindings/iio/pressure/honeywell,mprls0025pa.yaml +++ b/Documentation/devicetree/bindings/iio/pressure/honeywell,mprls0025pa.yaml @@ -53,12 +53,10 @@ properties: honeywell,pmin-pascal: description: Minimum pressure value the sensor can measure in pascal. - $ref: /schemas/types.yaml#/definitions/uint32 honeywell,pmax-pascal: description: Maximum pressure value the sensor can measure in pascal. - $ref: /schemas/types.yaml#/definitions/uint32 honeywell,transfer-function: description: |