From: Nishanth Menon <nm@ti.com> To: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com> Cc: Alim Akhtar <alim.akhtar@samsung.com>, Avri Altman <avri.altman@wdc.com>, Rob Herring <robh+dt@kernel.org>, Wei Xu <xuwei5@hisilicon.com>, Andy Gross <agross@kernel.org>, Bjorn Andersson <bjorn.andersson@linaro.org>, Vignesh Raghavendra <vigneshr@ti.com>, Tero Kristo <kristo@kernel.org>, "James E.J. Bottomley" <jejb@linux.ibm.com>, "Martin K. Petersen" <martin.petersen@oracle.com>, Chanho Park <chanho61.park@samsung.com>, Srinivas Kandagatla <srinivas.kandagatla@linaro.org>, Jan Kotas <jank@cadence.com>, <linux-scsi@vger.kernel.org>, <devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>, <linux-arm-kernel@lists.infradead.org>, <linux-samsung-soc@vger.kernel.org>, <linux-arm-msm@vger.kernel.org> Subject: Re: [RFC PATCH 8/8] arm64: dts: ti: use 'freq-table' in UFS node Date: Mon, 21 Feb 2022 07:13:40 -0600 [thread overview] Message-ID: <20220221131340.q3hjpjevqrfvhggv@specimen> (raw) In-Reply-To: <20220219184554.44887-1-krzysztof.kozlowski@canonical.com> On 19:45-20220219, Krzysztof Kozlowski wrote: > The 'freq-table-hz' property is deprecated by UFS bindings. > The uint32-array requires also element to be passed within one <> block. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com> > --- > arch/arm64/boot/dts/ti/k3-j721e-main.dtsi | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi > index 599861259a30..c3afef0321ae 100644 > --- a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi > +++ b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi > @@ -1257,7 +1257,9 @@ ufs@4e84000 { > compatible = "cdns,ufshc-m31-16nm", "jedec,ufs-2.0"; > reg = <0x0 0x4e84000 0x0 0x10000>; > interrupts = <GIC_SPI 17 IRQ_TYPE_LEVEL_HIGH>; > - freq-table-hz = <250000000 250000000>, <19200000 19200000>, <19200000 19200000>; > + freq-table = <250000000 250000000 <min max> is much more readable and less error prone in case of a large set. > + 19200000 19200000 > + 19200000 19200000>; are you sure the removal of the tuple adds value? > clocks = <&k3_clks 277 0>, <&k3_clks 277 1>, <&k3_clks 277 1>; > clock-names = "core_clk", "phy_clk", "ref_clk"; > dma-coherent; > -- > 2.32.0 > -- Regards, Nishanth Menon Key (0xDDB5849D1736249D)/Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
WARNING: multiple messages have this Message-ID (diff)
From: Nishanth Menon <nm@ti.com> To: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com> Cc: Alim Akhtar <alim.akhtar@samsung.com>, Avri Altman <avri.altman@wdc.com>, Rob Herring <robh+dt@kernel.org>, Wei Xu <xuwei5@hisilicon.com>, Andy Gross <agross@kernel.org>, Bjorn Andersson <bjorn.andersson@linaro.org>, Vignesh Raghavendra <vigneshr@ti.com>, Tero Kristo <kristo@kernel.org>, "James E.J. Bottomley" <jejb@linux.ibm.com>, "Martin K. Petersen" <martin.petersen@oracle.com>, Chanho Park <chanho61.park@samsung.com>, Srinivas Kandagatla <srinivas.kandagatla@linaro.org>, Jan Kotas <jank@cadence.com>, <linux-scsi@vger.kernel.org>, <devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>, <linux-arm-kernel@lists.infradead.org>, <linux-samsung-soc@vger.kernel.org>, <linux-arm-msm@vger.kernel.org> Subject: Re: [RFC PATCH 8/8] arm64: dts: ti: use 'freq-table' in UFS node Date: Mon, 21 Feb 2022 07:13:40 -0600 [thread overview] Message-ID: <20220221131340.q3hjpjevqrfvhggv@specimen> (raw) In-Reply-To: <20220219184554.44887-1-krzysztof.kozlowski@canonical.com> On 19:45-20220219, Krzysztof Kozlowski wrote: > The 'freq-table-hz' property is deprecated by UFS bindings. > The uint32-array requires also element to be passed within one <> block. > > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com> > --- > arch/arm64/boot/dts/ti/k3-j721e-main.dtsi | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi > index 599861259a30..c3afef0321ae 100644 > --- a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi > +++ b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi > @@ -1257,7 +1257,9 @@ ufs@4e84000 { > compatible = "cdns,ufshc-m31-16nm", "jedec,ufs-2.0"; > reg = <0x0 0x4e84000 0x0 0x10000>; > interrupts = <GIC_SPI 17 IRQ_TYPE_LEVEL_HIGH>; > - freq-table-hz = <250000000 250000000>, <19200000 19200000>, <19200000 19200000>; > + freq-table = <250000000 250000000 <min max> is much more readable and less error prone in case of a large set. > + 19200000 19200000 > + 19200000 19200000>; are you sure the removal of the tuple adds value? > clocks = <&k3_clks 277 0>, <&k3_clks 277 1>, <&k3_clks 277 1>; > clock-names = "core_clk", "phy_clk", "ref_clk"; > dma-coherent; > -- > 2.32.0 > -- Regards, Nishanth Menon Key (0xDDB5849D1736249D)/Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D _______________________________________________ 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-02-21 13:14 UTC|newest] Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-02-19 18:42 [RFC PATCH 0/8] dt-bindings: ufs: add common platform bindings + fixes Krzysztof Kozlowski 2022-02-19 18:42 ` Krzysztof Kozlowski 2022-02-19 18:42 ` [RFC PATCH 1/8] dt-bindings: ufs: add common platform bindings Krzysztof Kozlowski 2022-02-19 18:42 ` Krzysztof Kozlowski 2022-02-19 18:42 ` [RFC PATCH 2/8] dt-bindings: ufs: samsung,exynos-ufs: use common bindings Krzysztof Kozlowski 2022-02-19 18:42 ` [RFC PATCH 2/8] dt-bindings: ufs: samsung, exynos-ufs: " Krzysztof Kozlowski 2022-02-19 18:42 ` [RFC PATCH 3/8] dt-bindings: ufs: cdns,ufshc: convert to dtschema Krzysztof Kozlowski 2022-02-19 18:42 ` Krzysztof Kozlowski 2022-02-20 17:06 ` Krzysztof Kozlowski 2022-02-20 17:06 ` [RFC PATCH 3/8] dt-bindings: ufs: cdns, ufshc: " Krzysztof Kozlowski 2022-02-21 2:36 ` [RFC PATCH 3/8] dt-bindings: ufs: cdns,ufshc: " Rob Herring 2022-02-21 2:36 ` [RFC PATCH 3/8] dt-bindings: ufs: cdns, ufshc: " Rob Herring 2022-02-19 18:42 ` [RFC PATCH 4/8] scsi: ufs: deprecate 'freq-table-hz' property Krzysztof Kozlowski 2022-02-19 18:42 ` Krzysztof Kozlowski 2022-02-19 18:42 ` [RFC PATCH 5/8] arm64: dts: hi3670: use 'freq-table' in UFS node Krzysztof Kozlowski 2022-02-19 18:42 ` Krzysztof Kozlowski 2022-02-19 18:42 ` [RFC PATCH 6/8] arm64: dts: exynos: " Krzysztof Kozlowski 2022-02-19 18:42 ` Krzysztof Kozlowski 2022-02-19 18:44 ` [RFC PATCH 7/8] arm64: dts: qcom: " Krzysztof Kozlowski 2022-02-19 18:44 ` Krzysztof Kozlowski 2022-02-19 18:45 ` [RFC PATCH 8/8] arm64: dts: ti: " Krzysztof Kozlowski 2022-02-19 18:45 ` Krzysztof Kozlowski 2022-02-21 13:13 ` Nishanth Menon [this message] 2022-02-21 13:13 ` Nishanth Menon 2022-02-21 13:20 ` Krzysztof Kozlowski 2022-02-21 13:20 ` Krzysztof Kozlowski 2022-02-22 14:07 ` Nishanth Menon 2022-02-22 14:07 ` Nishanth Menon 2022-02-22 14:24 ` Krzysztof Kozlowski 2022-02-22 14:24 ` Krzysztof Kozlowski
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=20220221131340.q3hjpjevqrfvhggv@specimen \ --to=nm@ti.com \ --cc=agross@kernel.org \ --cc=alim.akhtar@samsung.com \ --cc=avri.altman@wdc.com \ --cc=bjorn.andersson@linaro.org \ --cc=chanho61.park@samsung.com \ --cc=devicetree@vger.kernel.org \ --cc=jank@cadence.com \ --cc=jejb@linux.ibm.com \ --cc=kristo@kernel.org \ --cc=krzysztof.kozlowski@canonical.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-arm-msm@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-samsung-soc@vger.kernel.org \ --cc=linux-scsi@vger.kernel.org \ --cc=martin.petersen@oracle.com \ --cc=robh+dt@kernel.org \ --cc=srinivas.kandagatla@linaro.org \ --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: linkBe 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.