From: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
To: Rob Herring <robh@kernel.org>
Cc: Matthias Brugger <matthias.bgg@gmail.com>,
Andy Gross <agross@kernel.org>, Jan Kotas <jank@cadence.com>,
linux-arm-msm@vger.kernel.org, linux-scsi@vger.kernel.org,
Rob Herring <robh+dt@kernel.org>,
Avri Altman <avri.altman@wdc.com>, Wei Xu <xuwei5@hisilicon.com>,
devicetree@vger.kernel.org,
Bjorn Andersson <bjorn.andersson@linaro.org>,
Vignesh Raghavendra <vigneshr@ti.com>,
linux-arm-kernel@lists.infradead.org,
linux-mediatek@lists.infradead.org,
linux-samsung-soc@vger.kernel.org,
Alim Akhtar <alim.akhtar@samsung.com>,
Li Wei <liwei213@huawei.com>,
linux-kernel@vger.kernel.org,
Stanley Chu <stanley.chu@mediatek.com>
Subject: Re: [PATCH v3 03/12] dt-bindings: ufs: cdns,ufshc: convert to dtschema
Date: Mon, 7 Mar 2022 10:10:55 +0100 [thread overview]
Message-ID: <02bd539c-f704-dbe5-4d58-fc62314025a7@canonical.com> (raw)
In-Reply-To: <1646623480.209864.1496443.nullmailer@robh.at.kernel.org>
On 07/03/2022 04:24, Rob Herring wrote:
> On Sun, 06 Mar 2022 12:11:16 +0100, Krzysztof Kozlowski wrote:
>> Convert the Cadence Universal Flash Storage (UFS) Controlle to DT schema
>> format.
>>
>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
>> Reviewed-by: Rob Herring <robh@kernel.org>
>> ---
>> .../devicetree/bindings/ufs/cdns,ufshc.txt | 32 ---------
>> .../devicetree/bindings/ufs/cdns,ufshc.yaml | 68 +++++++++++++++++++
>> .../devicetree/bindings/ufs/ti,j721e-ufs.yaml | 7 +-
>> 3 files changed, 71 insertions(+), 36 deletions(-)
>> delete mode 100644 Documentation/devicetree/bindings/ufs/cdns,ufshc.txt
>> create mode 100644 Documentation/devicetree/bindings/ufs/cdns,ufshc.yaml
>>
>
> 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/linux-dt-review/Documentation/devicetree/bindings/ufs/cdns,ufshc.example.dt.yaml: ufs@fd030000: freq-table-hz: 'anyOf' conditional failed, one must be fixed:
> [[0, 0], [0, 0]] is too long
> [0, 0] is too long
> From schema: /usr/local/lib/python3.8/dist-packages/dtschema/schemas/property-units.yaml
>
This will be fixed with my dtschema patch:
https://github.com/devicetree-org/dt-schema/pull/69
Best regards,
Krzysztof
WARNING: multiple messages have this Message-ID (diff)
From: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
To: Rob Herring <robh@kernel.org>
Cc: Matthias Brugger <matthias.bgg@gmail.com>,
Andy Gross <agross@kernel.org>, Jan Kotas <jank@cadence.com>,
linux-arm-msm@vger.kernel.org, linux-scsi@vger.kernel.org,
Rob Herring <robh+dt@kernel.org>,
Avri Altman <avri.altman@wdc.com>, Wei Xu <xuwei5@hisilicon.com>,
devicetree@vger.kernel.org,
Bjorn Andersson <bjorn.andersson@linaro.org>,
Vignesh Raghavendra <vigneshr@ti.com>,
linux-arm-kernel@lists.infradead.org,
linux-mediatek@lists.infradead.org,
linux-samsung-soc@vger.kernel.org,
Alim Akhtar <alim.akhtar@samsung.com>,
Li Wei <liwei213@huawei.com>,
linux-kernel@vger.kernel.org,
Stanley Chu <stanley.chu@mediatek.com>
Subject: Re: [PATCH v3 03/12] dt-bindings: ufs: cdns,ufshc: convert to dtschema
Date: Mon, 7 Mar 2022 10:10:55 +0100 [thread overview]
Message-ID: <02bd539c-f704-dbe5-4d58-fc62314025a7@canonical.com> (raw)
In-Reply-To: <1646623480.209864.1496443.nullmailer@robh.at.kernel.org>
On 07/03/2022 04:24, Rob Herring wrote:
> On Sun, 06 Mar 2022 12:11:16 +0100, Krzysztof Kozlowski wrote:
>> Convert the Cadence Universal Flash Storage (UFS) Controlle to DT schema
>> format.
>>
>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
>> Reviewed-by: Rob Herring <robh@kernel.org>
>> ---
>> .../devicetree/bindings/ufs/cdns,ufshc.txt | 32 ---------
>> .../devicetree/bindings/ufs/cdns,ufshc.yaml | 68 +++++++++++++++++++
>> .../devicetree/bindings/ufs/ti,j721e-ufs.yaml | 7 +-
>> 3 files changed, 71 insertions(+), 36 deletions(-)
>> delete mode 100644 Documentation/devicetree/bindings/ufs/cdns,ufshc.txt
>> create mode 100644 Documentation/devicetree/bindings/ufs/cdns,ufshc.yaml
>>
>
> 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/linux-dt-review/Documentation/devicetree/bindings/ufs/cdns,ufshc.example.dt.yaml: ufs@fd030000: freq-table-hz: 'anyOf' conditional failed, one must be fixed:
> [[0, 0], [0, 0]] is too long
> [0, 0] is too long
> From schema: /usr/local/lib/python3.8/dist-packages/dtschema/schemas/property-units.yaml
>
This will be fixed with my dtschema patch:
https://github.com/devicetree-org/dt-schema/pull/69
Best regards,
Krzysztof
_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek
WARNING: multiple messages have this Message-ID (diff)
From: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
To: Rob Herring <robh@kernel.org>
Cc: Matthias Brugger <matthias.bgg@gmail.com>,
Andy Gross <agross@kernel.org>, Jan Kotas <jank@cadence.com>,
linux-arm-msm@vger.kernel.org, linux-scsi@vger.kernel.org,
Rob Herring <robh+dt@kernel.org>,
Avri Altman <avri.altman@wdc.com>, Wei Xu <xuwei5@hisilicon.com>,
devicetree@vger.kernel.org,
Bjorn Andersson <bjorn.andersson@linaro.org>,
Vignesh Raghavendra <vigneshr@ti.com>,
linux-arm-kernel@lists.infradead.org,
linux-mediatek@lists.infradead.org,
linux-samsung-soc@vger.kernel.org,
Alim Akhtar <alim.akhtar@samsung.com>,
Li Wei <liwei213@huawei.com>,
linux-kernel@vger.kernel.org,
Stanley Chu <stanley.chu@mediatek.com>
Subject: Re: [PATCH v3 03/12] dt-bindings: ufs: cdns,ufshc: convert to dtschema
Date: Mon, 7 Mar 2022 10:10:55 +0100 [thread overview]
Message-ID: <02bd539c-f704-dbe5-4d58-fc62314025a7@canonical.com> (raw)
In-Reply-To: <1646623480.209864.1496443.nullmailer@robh.at.kernel.org>
On 07/03/2022 04:24, Rob Herring wrote:
> On Sun, 06 Mar 2022 12:11:16 +0100, Krzysztof Kozlowski wrote:
>> Convert the Cadence Universal Flash Storage (UFS) Controlle to DT schema
>> format.
>>
>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
>> Reviewed-by: Rob Herring <robh@kernel.org>
>> ---
>> .../devicetree/bindings/ufs/cdns,ufshc.txt | 32 ---------
>> .../devicetree/bindings/ufs/cdns,ufshc.yaml | 68 +++++++++++++++++++
>> .../devicetree/bindings/ufs/ti,j721e-ufs.yaml | 7 +-
>> 3 files changed, 71 insertions(+), 36 deletions(-)
>> delete mode 100644 Documentation/devicetree/bindings/ufs/cdns,ufshc.txt
>> create mode 100644 Documentation/devicetree/bindings/ufs/cdns,ufshc.yaml
>>
>
> 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/linux-dt-review/Documentation/devicetree/bindings/ufs/cdns,ufshc.example.dt.yaml: ufs@fd030000: freq-table-hz: 'anyOf' conditional failed, one must be fixed:
> [[0, 0], [0, 0]] is too long
> [0, 0] is too long
> From schema: /usr/local/lib/python3.8/dist-packages/dtschema/schemas/property-units.yaml
>
This will be fixed with my dtschema patch:
https://github.com/devicetree-org/dt-schema/pull/69
Best regards,
Krzysztof
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2022-03-07 9:11 UTC|newest]
Thread overview: 61+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-03-06 11:11 [PATCH v3 00/12] dt-bindings: ufs: add common platform bindings + fixes Krzysztof Kozlowski
2022-03-06 11:11 ` Krzysztof Kozlowski
2022-03-06 11:11 ` Krzysztof Kozlowski
2022-03-06 11:11 ` [PATCH v3 01/12] dt-bindings: ufs: add common platform bindings Krzysztof Kozlowski
2022-03-06 11:11 ` Krzysztof Kozlowski
2022-03-06 11:11 ` Krzysztof Kozlowski
2022-03-07 18:36 ` Alim Akhtar
2022-03-07 18:36 ` Alim Akhtar
2022-03-07 18:36 ` Alim Akhtar
2022-03-06 11:11 ` [PATCH v3 02/12] dt-bindings: ufs: samsung,exynos-ufs: use common bindings Krzysztof Kozlowski
2022-03-06 11:11 ` [PATCH v3 02/12] dt-bindings: ufs: samsung, exynos-ufs: " Krzysztof Kozlowski
2022-03-06 11:11 ` Krzysztof Kozlowski
2022-03-07 18:39 ` [PATCH v3 02/12] dt-bindings: ufs: samsung,exynos-ufs: " Alim Akhtar
2022-03-07 18:39 ` Alim Akhtar
2022-03-07 18:39 ` Alim Akhtar
2022-03-06 11:11 ` [PATCH v3 03/12] dt-bindings: ufs: cdns,ufshc: convert to dtschema Krzysztof Kozlowski
2022-03-06 11:11 ` Krzysztof Kozlowski
2022-03-06 11:11 ` Krzysztof Kozlowski
2022-03-07 3:24 ` Rob Herring
2022-03-07 3:24 ` [PATCH v3 03/12] dt-bindings: ufs: cdns, ufshc: " Rob Herring
2022-03-07 3:24 ` Rob Herring
2022-03-07 9:10 ` Krzysztof Kozlowski [this message]
2022-03-07 9:10 ` [PATCH v3 03/12] dt-bindings: ufs: cdns,ufshc: " Krzysztof Kozlowski
2022-03-07 9:10 ` Krzysztof Kozlowski
2022-03-06 11:11 ` [PATCH v3 04/12] dt-bindings: ufs: drop unused/old ufs-qcom PHY bindings Krzysztof Kozlowski
2022-03-06 11:11 ` Krzysztof Kozlowski
2022-03-06 11:11 ` Krzysztof Kozlowski
2022-03-06 11:11 ` [PATCH v3 05/12] dt-bindings: ufs: qcom,ufs: convert to dtschema Krzysztof Kozlowski
2022-03-06 11:11 ` Krzysztof Kozlowski
2022-03-06 11:11 ` Krzysztof Kozlowski
2022-03-06 11:11 ` [PATCH v3 06/12] dt-bindings: ufs: hisilicon,ufs: " Krzysztof Kozlowski
2022-03-06 11:11 ` [PATCH v3 06/12] dt-bindings: ufs: hisilicon, ufs: " Krzysztof Kozlowski
2022-03-06 11:11 ` Krzysztof Kozlowski
2022-03-06 11:11 ` [PATCH v3 07/12] dt-bindings: ufs: mediatek,ufs: " Krzysztof Kozlowski
2022-03-06 11:11 ` Krzysztof Kozlowski
2022-03-06 11:11 ` Krzysztof Kozlowski
2022-03-06 11:11 ` [PATCH v3 08/12] dt-bindings: ufs: snps,tc-dwc-g210: " Krzysztof Kozlowski
2022-03-06 11:11 ` [PATCH v3 08/12] dt-bindings: ufs: snps, tc-dwc-g210: " Krzysztof Kozlowski
2022-03-06 11:11 ` Krzysztof Kozlowski
2022-03-06 11:11 ` [PATCH v3 09/12] arm64: dts: hisilicon: align 'freq-table-hz' with dtschema in UFS Krzysztof Kozlowski
2022-03-06 11:11 ` Krzysztof Kozlowski
2022-03-06 11:11 ` Krzysztof Kozlowski
2022-04-19 10:12 ` (subset) " Krzysztof Kozlowski
2022-04-19 10:12 ` Krzysztof Kozlowski
2022-04-19 10:12 ` Krzysztof Kozlowski
2022-04-19 11:35 ` Wei Xu
2022-04-19 11:35 ` Wei Xu
2022-04-19 11:35 ` Wei Xu
2022-03-06 11:11 ` [PATCH v3 10/12] arm64: dts: qcom: msm8996: drop unsupported UFS vddp-ref-clk-max-microamp Krzysztof Kozlowski
2022-03-06 11:11 ` Krzysztof Kozlowski
2022-03-06 11:11 ` Krzysztof Kozlowski
2022-03-06 11:11 ` [PATCH v3 11/12] arm64: dts: qcom: msm8996: correct UFS compatible Krzysztof Kozlowski
2022-03-06 11:11 ` Krzysztof Kozlowski
2022-03-06 11:11 ` Krzysztof Kozlowski
2022-03-06 11:11 ` [PATCH v3 12/12] arm64: dts: qcom: sm8350: drop duplicated ref_clk in UFS Krzysztof Kozlowski
2022-03-06 11:11 ` Krzysztof Kozlowski
2022-03-06 11:11 ` Krzysztof Kozlowski
2022-03-09 2:00 ` [PATCH v3 00/12] dt-bindings: ufs: add common platform bindings + fixes Rob Herring
2022-03-09 2:00 ` Rob Herring
2022-03-09 2:00 ` Rob Herring
2022-04-12 14:30 ` patchwork-bot+linux-arm-msm
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=02bd539c-f704-dbe5-4d58-fc62314025a7@canonical.com \
--to=krzysztof.kozlowski@canonical.com \
--cc=agross@kernel.org \
--cc=alim.akhtar@samsung.com \
--cc=avri.altman@wdc.com \
--cc=bjorn.andersson@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=jank@cadence.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=liwei213@huawei.com \
--cc=matthias.bgg@gmail.com \
--cc=robh+dt@kernel.org \
--cc=robh@kernel.org \
--cc=stanley.chu@mediatek.com \
--cc=vigneshr@ti.com \
--cc=xuwei5@hisilicon.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.