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 E286AC4332F for ; Tue, 22 Feb 2022 14:09:30 +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:In-Reply-To:MIME-Version:References: 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=5yfm32/j5cV1Yl4dKM8SPBx4oN4ZbGFd0Dk76TzG7UM=; b=YHmPKFIf7pZ67a qa84oKSL9m2s0zceisnfdIB0kGYcBk5xACYc7IHtdJUo7RTULEsINDu0HWHFosqEQokR2GlSNdST0 lDmuPvmafaSiDEmxytrUKAeiPxNBFYtU1zBaANbJGamY5Pyq7JkfQfGJl0g0INov41iOBnhxTbmGt Ml/RDQztEx0sAT0wr+Hi80O051WDqTYTwiKaAVs3DspCkgvnkq+LXozFYrXzpg+idKLKG/uky1amY OSR18iKkh5YMNID6ZVdF4/+JhBlcJ1YdXXlTwo6nO7+bOMp/2bpD6/Y6Os/C0/iiw3IIT2GS5Mx05 GXOs1V02eArldf3X7FYw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nMVpm-009oLg-5D; Tue, 22 Feb 2022 14:08:06 +0000 Received: from fllv0015.ext.ti.com ([198.47.19.141]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nMVph-009oKb-5m for linux-arm-kernel@lists.infradead.org; Tue, 22 Feb 2022 14:08:03 +0000 Received: from lelv0265.itg.ti.com ([10.180.67.224]) by fllv0015.ext.ti.com (8.15.2/8.15.2) with ESMTP id 21ME7MkP005019; Tue, 22 Feb 2022 08:07:22 -0600 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1645538842; bh=8HPnjRNTg1rgRuGDrTg7AuCbtaoqyKy2IHjGLaHD7Fs=; h=Date:From:To:CC:Subject:References:In-Reply-To; b=ctwrzo4XA7iB3hzjs0/vCaLdhyfnlm9x8Tdm/vef0FUl6cqTWqKcvKg8na866ipQ3 LkfBXKjRm6agy3zbZ0R+Hv1+oLn5AMKHSzkk/YA4GXqoL7gpruaaFDcenNcJOSZymK u2NjCkYLufIp1G3dAt7RFNAxpFND1thYMo9NUp/8= Received: from DFLE114.ent.ti.com (dfle114.ent.ti.com [10.64.6.35]) by lelv0265.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 21ME7MSc086515 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Tue, 22 Feb 2022 08:07:22 -0600 Received: from DFLE115.ent.ti.com (10.64.6.36) by DFLE114.ent.ti.com (10.64.6.35) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2308.14; Tue, 22 Feb 2022 08:07:21 -0600 Received: from fllv0040.itg.ti.com (10.64.41.20) by DFLE115.ent.ti.com (10.64.6.36) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2308.14 via Frontend Transport; Tue, 22 Feb 2022 08:07:21 -0600 Received: from localhost (ileax41-snat.itg.ti.com [10.172.224.153]) by fllv0040.itg.ti.com (8.15.2/8.15.2) with ESMTP id 21ME7LqE034437; Tue, 22 Feb 2022 08:07:21 -0600 Date: Tue, 22 Feb 2022 08:07:21 -0600 From: Nishanth Menon To: Krzysztof Kozlowski CC: Alim Akhtar , Avri Altman , Rob Herring , Wei Xu , Andy Gross , Bjorn Andersson , Vignesh Raghavendra , Tero Kristo , "James E.J. Bottomley" , "Martin K. Petersen" , Chanho Park , Srinivas Kandagatla , Jan Kotas , , , , , , Subject: Re: [RFC PATCH 8/8] arm64: dts: ti: use 'freq-table' in UFS node Message-ID: <20220222140721.bzbspfcethml2psm@running> References: <20220219184224.44339-1-krzysztof.kozlowski@canonical.com> <20220219184554.44887-1-krzysztof.kozlowski@canonical.com> <20220221131340.q3hjpjevqrfvhggv@specimen> <19705501-2391-14a4-0eac-4b2b647a9735@canonical.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <19705501-2391-14a4-0eac-4b2b647a9735@canonical.com> User-Agent: NeoMutt/20171215 X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220222_060801_364262_E3E1F3D5 X-CRM114-Status: GOOD ( 18.44 ) 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 14:20-20220221, Krzysztof Kozlowski wrote: > On 21/02/2022 14:13, Nishanth Menon wrote: > > 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 > >> --- > >> 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 = ; > >> - freq-table-hz = <250000000 250000000>, <19200000 19200000>, <19200000 19200000>; > >> + freq-table = <250000000 250000000 > > > > is much more readable and less error prone in case of a large > > set. > > Pairs are easily visible if split per line, so readability is similar. > Not much is lost. > > > > >> + 19200000 19200000 > >> + 19200000 19200000>; > > > > are you sure the removal of the tuple adds value? > > DT schema requires it, or rather: I do not know how to express uint32 > tuples in DT schema. All my tries failed. https://www.kernel.org/doc/Documentation/devicetree/bindings/opp/opp-v1.yaml comes to mind.. -- 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