Message ID | 20230609140745.65046-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:994d:0:b0:3d9:f83d:47d9 with SMTP id k13csp976253vqr; Fri, 9 Jun 2023 07:10:42 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ6XSA7+Z3sjkadEg332Jw3kEEJXn7zi/Ly0iEdEv/XQpkBkAI5m614VUx6yGhSy7f+6cz5i X-Received: by 2002:a05:6a21:6217:b0:111:366:eeb2 with SMTP id wm23-20020a056a21621700b001110366eeb2mr973718pzb.9.1686319841752; Fri, 09 Jun 2023 07:10:41 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1686319841; cv=none; d=google.com; s=arc-20160816; b=zojZQ0CBRVLX8CtpO0H1ufcowj1/wmB5W8eZ+GAX4QZ7xQAk9PRDwO0T0jIiWrO046 aGX2o5RXQ+ClHfGmifJ42CUgqV5sccaSD141jx9/gSmEu+pOWhap1LlyjZ5G+/Cs+qsm 7LbVITEjLcLTAlvqOoiJO1SQDKaPGN7KQKJ8rxB+eOO9AoZtgARXkTOYIVjsW73ngiLq FnZK5yi0bC4WqC5M3vidZ97B6l5UjGJ9kBCnZYzrLCRTaTXl6L6vSKFdwqAyFTNjHMLQ 6YKfNyzxlrsmssuus0VCFlKl/FcWnO9MXtzr1WTAtXJ/CWpt7WGCl02OJq0wvwwPV9Qj zJ0A== 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=rFrh4wlGw8fqzifmvwiSox8R6fD/3BKItknrEjhKA6k=; b=VzA5daSpWZoGWWr4f45h4EOaLH4QbDFXTvFrZnVsZlQsk0CbqA4bE/LdnQFYET2V67 92aCqkS0IeFP60/cR8mi+pIT4rHrDvrWzgT2QF8LIhj+RzDmmGgZA48cKsinDyKlc0Qn n5Ax2IncMOQAvTj5GXsvrBzPVF2XY4OQ7edNFl9dHz4VkpyGfJGxsguqk6thaer0gcQq 94JZWdTGaToI+0Itu1zNHfcKZ0IosydSs7JkpgymyeU7lQ7FTW0wXa2brZQ9h1XpxwYg uKUS45C/PkV1+m11hzqkgI7q2yiCc3I8s7DZAYQNjliynt30bcJr+FVEc3NyeSDmWUmG sTWg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=qc3PcBph; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 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 (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id w22-20020a639356000000b0054293d20c38si2509078pgm.729.2023.06.09.07.10.26; Fri, 09 Jun 2023 07:10:41 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=qc3PcBph; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S241264AbjFIOI5 (ORCPT <rfc822;liningstudo@gmail.com> + 99 others); Fri, 9 Jun 2023 10:08:57 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36750 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S240665AbjFIOIg (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Fri, 9 Jun 2023 10:08:36 -0400 Received: from mail-lf1-x12a.google.com (mail-lf1-x12a.google.com [IPv6:2a00:1450:4864:20::12a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id BDDA63C31 for <linux-kernel@vger.kernel.org>; Fri, 9 Jun 2023 07:08:09 -0700 (PDT) Received: by mail-lf1-x12a.google.com with SMTP id 2adb3069b0e04-4f654d713c0so1923426e87.3 for <linux-kernel@vger.kernel.org>; Fri, 09 Jun 2023 07:08:09 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1686319667; x=1688911667; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=rFrh4wlGw8fqzifmvwiSox8R6fD/3BKItknrEjhKA6k=; b=qc3PcBphPle0Qnl6wCqXAXmVXeMAjQl/D+j76Bw45bh6endmQBuzw23h7+w/7Ded6N gCe7Rexac7RF61MDwEJ4fEcS6mGwcNU/HSKA8yZ/AXEjoZesSx5efawrEvjwEojzZ/tO 1LQL0wEZIpVRS3YSH0Ry7ES0ifWYLp57TZUEseM6r1TSjfAXrNn5GxlKkXBdDFbsn4h0 /SX/AykOkhlr2kxpTrBuGrf0VtzwaMundsJhI4/wLMwaS8IbLYzh4eG7DZt42nPENwg2 JMnnagp9Iwncxyea39ImYEH3IfwMBAl2dMlsobXeyZJEOk3cFxvJaWPAD3tuSdvpB0rJ LMgA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1686319667; x=1688911667; 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=rFrh4wlGw8fqzifmvwiSox8R6fD/3BKItknrEjhKA6k=; b=UNMdkr3wWGcvkLD9m7xxbiZBWoJi8aSBLz+45jhFppQ0fAgbSN1ty5TIhVbOOmqspn OnzUjJak5XxsD7yR2YhbV4i8T1R0rl3F1JC82bqwDLJWTcc4EBDxcmOD3tiCiQX12LkN YhnnQaP8Ir7nJmkkN6cSpmFdh5YzxZ097xOCGLNgqhGw7VvIm5fcC71E3eu+98jXvu2E FgRDCXBxWVYItsMFXZEbMygLSfwXWiR0bvfOoWgm+Fd+C1F1KAf9qTSJSQjChvfdMahX Cl75ej9Wuk+eVyqCg819RBUYXqRP3HQKb+upc4PzoFU+qyMrGqZqYZ3KwGtjYAI7lKBA miCg== X-Gm-Message-State: AC+VfDx1LEKTrK7F5XjbcLfAiHRr7Wu1J/bRDT0XsjRd2/dEPwSccNk0 muVMC6Ctshk9Evg6Z4TEWWK60A== X-Received: by 2002:a19:f20e:0:b0:4f6:54f1:ea36 with SMTP id q14-20020a19f20e000000b004f654f1ea36mr927952lfh.43.1686319667168; Fri, 09 Jun 2023 07:07:47 -0700 (PDT) Received: from krzk-bin.. ([178.197.219.26]) by smtp.gmail.com with ESMTPSA id p4-20020aa7d304000000b005169ffc81absm1813885edq.51.2023.06.09.07.07.45 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 09 Jun 2023 07:07:46 -0700 (PDT) From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> To: Herbert Xu <herbert@gondor.apana.org.au>, "David S. Miller" <davem@davemloft.net>, Rob Herring <robh+dt@kernel.org>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Conor Dooley <conor+dt@kernel.org>, Linus Walleij <linus.walleij@linaro.org>, linux-crypto@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Subject: [PATCH] dt-bindings: crypto: intel,ixp4xx: drop unneeded quotes Date: Fri, 9 Jun 2023 16:07:45 +0200 Message-Id: <20230609140745.65046-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=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1768234514264463747?= X-GMAIL-MSGID: =?utf-8?q?1768234514264463747?= |
Series |
dt-bindings: crypto: intel,ixp4xx: drop unneeded quotes
|
|
Commit Message
Krzysztof Kozlowski
June 9, 2023, 2:07 p.m. UTC
Cleanup bindings dropping unneeded quotes. Once all these are fixed,
checking for this can be enabled in yamllint.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
.../devicetree/bindings/crypto/intel,ixp4xx-crypto.yaml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Comments
On Fri, Jun 9, 2023 at 4:07 PM Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote: > Cleanup bindings dropping unneeded quotes. Once all these are fixed, > checking for this can be enabled in yamllint. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Reviewed-by: Linus Walleij <linus.walleij@linaro.org> Yours, Linus Walleij
On Fri, Jun 09, 2023 at 04:07:45PM +0200, Krzysztof Kozlowski wrote: > Cleanup bindings dropping unneeded quotes. Once all these are fixed, > checking for this can be enabled in yamllint. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > --- > .../devicetree/bindings/crypto/intel,ixp4xx-crypto.yaml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) You missed $id and $schema values. Within drivers/crypto, there's also: diff --git a/Documentation/devicetree/bindings/crypto/amlogic,gxl-crypto.yaml b/Documentation/devicetree/bi ndings/crypto/amlogic,gxl-crypto.yaml index ecf98a9e72b2..948e11ebe4ee 100644 --- a/Documentation/devicetree/bindings/crypto/amlogic,gxl-crypto.yaml +++ b/Documentation/devicetree/bindings/crypto/amlogic,gxl-crypto.yaml @@ -19,8 +19,8 @@ properties: interrupts: items: - - description: "Interrupt for flow 0" - - description: "Interrupt for flow 1" + - description: Interrupt for flow 0 + - description: Interrupt for flow 1 clocks: maxItems: 1 Can you add this in here. No reason to split this up more that 1 patch per subsystem really. As I mentioned, I have a WIP tree[1] with yamllint checks enabled and I scripted many of the changes. It was clean, but there's some new ones since rebasing to v6.4-rc1: ../Documentation/devicetree/bindings/hwmon/starfive,jh71x0-temp.yaml:30:16: [error] string value is redundantly quoted with any quotes (quoted-strings) ../Documentation/devicetree/bindings/hwmon/starfive,jh71x0-temp.yaml:31:16: [error] string value is redundantly quoted with any quotes (quoted-strings) ../Documentation/devicetree/bindings/hwmon/starfive,jh71x0-temp.yaml:42:16: [error] string value is redundantly quoted with any quotes (quoted-strings) ../Documentation/devicetree/bindings/hwmon/starfive,jh71x0-temp.yaml:43:16: [error] string value is redundantly quoted with any quotes (quoted-strings) ../Documentation/devicetree/bindings/sound/nvidia,tegra-audio-max9808x.yaml:38:11: [error] string value is redundantly quoted with any quotes (quoted-strings) ../Documentation/devicetree/bindings/sound/nvidia,tegra-audio-max9808x.yaml:39:11: [error] string value is redundantly quoted with any quotes (quoted-strings) ../Documentation/devicetree/bindings/sound/nvidia,tegra-audio-max9808x.yaml:40:11: [error] string value is redundantly quoted with any quotes (quoted-strings) ../Documentation/devicetree/bindings/sound/nvidia,tegra-audio-max9808x.yaml:41:11: [error] string value is redundantly quoted with any quotes (quoted-strings) ../Documentation/devicetree/bindings/sound/nvidia,tegra-audio-max9808x.yaml:42:11: [error] string value is redundantly quoted with any quotes (quoted-strings) ../Documentation/devicetree/bindings/sound/nvidia,tegra-audio-max9808x.yaml:43:11: [error] string value is redundantly quoted with any quotes (quoted-strings) ../Documentation/devicetree/bindings/sound/nvidia,tegra-audio-rt5631.yaml:34:11: [error] string value is redundantly quoted with any quotes (quoted-strings) ../Documentation/devicetree/bindings/sound/nvidia,tegra-audio-rt5631.yaml:35:11: [error] string value is redundantly quoted with any quotes (quoted-strings) ../Documentation/devicetree/bindings/sound/nvidia,tegra-audio-rt5631.yaml:36:11: [error] string value is redundantly quoted with any quotes (quoted-strings) ../Documentation/devicetree/bindings/sound/nvidia,tegra-audio-rt5631.yaml:37:11: [error] string value is redundantly quoted with any quotes (quoted-strings) ../Documentation/devicetree/bindings/sound/cirrus,cs35l45.yaml:65:15: [error] string value is redundantly quoted with any quotes (quoted-strings) ../Documentation/devicetree/bindings/sound/cirrus,cs35l45.yaml:74:15: [error] string value is redundantly quoted with any quotes (quoted-strings) ../Documentation/devicetree/bindings/sound/cirrus,cs35l45.yaml:83:15: [error] string value is redundantly quoted with any quotes (quoted-strings) ../Documentation/devicetree/bindings/sound/cirrus,cs35l45.yaml:93:15: [error] string value is redundantly quoted with any quotes (quoted-strings) ../Documentation/devicetree/bindings/sound/cirrus,cs35l45.yaml:117:15: [error] string value is redundantly quoted with any quotes (quoted-strings) Rob [1] https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git/log/?h=dt/yamllint
On 10/06/2023 00:54, Rob Herring wrote: > On Fri, Jun 09, 2023 at 04:07:45PM +0200, Krzysztof Kozlowski wrote: >> Cleanup bindings dropping unneeded quotes. Once all these are fixed, >> checking for this can be enabled in yamllint. >> >> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >> --- >> .../devicetree/bindings/crypto/intel,ixp4xx-crypto.yaml | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) > > You missed $id and $schema values. > > Within drivers/crypto, there's also: > > diff --git a/Documentation/devicetree/bindings/crypto/amlogic,gxl-crypto.yaml b/Documentation/devicetree/bi > ndings/crypto/amlogic,gxl-crypto.yaml > index ecf98a9e72b2..948e11ebe4ee 100644 > --- a/Documentation/devicetree/bindings/crypto/amlogic,gxl-crypto.yaml > +++ b/Documentation/devicetree/bindings/crypto/amlogic,gxl-crypto.yaml > @@ -19,8 +19,8 @@ properties: > > interrupts: > items: > - - description: "Interrupt for flow 0" > - - description: "Interrupt for flow 1" > + - description: Interrupt for flow 0 > + - description: Interrupt for flow 1 > > clocks: > maxItems: 1 > > Can you add this in here. No reason to split this up more that 1 patch > per subsystem really. I'll add it. I am also fine for you to squash it or take your scripted changes/commit if you have it ready. Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/crypto/intel,ixp4xx-crypto.yaml b/Documentation/devicetree/bindings/crypto/intel,ixp4xx-crypto.yaml index e0fe63957888..8e2d95d732ba 100644 --- a/Documentation/devicetree/bindings/crypto/intel,ixp4xx-crypto.yaml +++ b/Documentation/devicetree/bindings/crypto/intel,ixp4xx-crypto.yaml @@ -21,7 +21,7 @@ properties: const: intel,ixp4xx-crypto intel,npe-handle: - $ref: '/schemas/types.yaml#/definitions/phandle-array' + $ref: /schemas/types.yaml#/definitions/phandle-array items: - items: - description: phandle to the NPE this crypto engine