Received-SPF: pass (google.com: domain of linux-kernel+bounces-175496-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
From: "Peng Fan (OSS)" <peng.fan@oss.nxp.com>
Date: Fri, 10 May 2024 17:19:00 +0800
Subject: [PATCH v2 05/17] clk: imx: pll14xx: Add constraint for fvco
 frequency
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
Message-Id: <20240510-imx-clk-v2-5-c998f315d29c@nxp.com>
References: <20240510-imx-clk-v2-0-c998f315d29c@nxp.com>
In-Reply-To: <20240510-imx-clk-v2-0-c998f315d29c@nxp.com>
To: Abel Vesa <abelvesa@kernel.org>, 
 Michael Turquette <mturquette@baylibre.com>, 
 Stephen Boyd <sboyd@kernel.org>, Shawn Guo <shawnguo@kernel.org>, 
 Sascha Hauer <s.hauer@pengutronix.de>, 
 Pengutronix Kernel Team <kernel@pengutronix.de>, 
 Fabio Estevam <festevam@gmail.com>, Jacky Bai <ping.bai@nxp.com>, 
 Ye Li <ye.li@nxp.com>, Dong Aisheng <aisheng.dong@nxp.com>
Cc: linux-clk@vger.kernel.org, imx@lists.linux.dev, 
 linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, 
 Abel Vesa <abel.vesa@linaro.org>, Peng Fan <peng.fan@nxp.com>, 
 Shengjiu Wang <shengjiu.wang@nxp.com>, Chancel Liu <chancel.liu@nxp.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
X-MS-PublicTrafficType: Email
X-MS-TrafficTypeDiagnostic: DU0PR04MB9417:EE_|AM9PR04MB8065:EE_
X-MS-Office365-Filtering-Correlation-Id: f4c0d3e2-1016-48a7-5901-08dc70d12f24
X-MS-Exchange-SharedMailbox-RoutingAgent-Processed: True
X-MS-Exchange-SenderADCheck: 1
X-MS-Exchange-AntiSpam-Relay: 0
X-Microsoft-Antispam:
	BCL:0;ARA:13230031|52116005|366007|1800799015|376005|7416005|921011|38350700005;
X-Microsoft-Antispam-Message-Info:
	=?utf-8?B?aWhFeUltK29Bd3V5cmZXQmdQZ3FLZW1YdkxGVFBUb0M5QnF5Z1A3N3NpQisv?=
 =?utf-8?B?Y1N4ai9KMkNHeW40Y0VQMlZJLzRxMURsdkZ4Mkg2RnNOYzF3YkhCSjMxRlhm?=
 =?utf-8?B?ZTRncHk0QUM2Y3NjRUtGWXB4ZkJHR28zMHpXNFcyVTFVZ1R5Nm9nVWl6WmNB?=
 =?utf-8?B?OVFvRElWZnN0YTlpM0l0anlaZjhYVk93dzhvTkRDc2QrZldQSjA0enROeGpm?=
 =?utf-8?B?SDhPTEgvWU9XYlpPWEkxckpQNzI1RUkwenMwZEZiRjhzTTVJdzRGbXk5Wmt3?=
 =?utf-8?B?MUoreTlZd0c4MncrZThFNnlpZDhENTQxRXNFUkdWME1McllMM1hOZnkzTDJM?=
 =?utf-8?B?Rm00VUh0YzJKcmhmS0JOUVluS0R0REpYNStvV2FQUlZKb21JRFc1THV3WUE5?=
 =?utf-8?B?ZkZMOWE2bkRpcDRISHZxdUNjWlRzZWhuT256SFkyaGxvTXRtT21kNmNEdlc3?=
 =?utf-8?B?ZFRodXd4cjZ3ZEFkMTRFck56QkIvOTkwYmN3d0k2MXdUb3BvMFJHSlFueDVu?=
 =?utf-8?B?VWNnZ3d2cGlrZEdKMmRtY2s4OGJOdXZVU0ZCNEJibDlqZU1WOEtuTmhLd0xT?=
 =?utf-8?B?QmVBMW8rMHFoWFN2WXQvLzg5dlhVV3Q1V2JUYzlqWUdwVU1ielJLd2F3VGsw?=
 =?utf-8?B?QVVIVEpGMG4wemtIdDFPdVRCcTV1dWF1WWpSRngrRlBLOUV5SUkzN1ZxQVFB?=
 =?utf-8?B?d0h0K3NOQWtobVNqam5QaVh1L2JQMGJURUFrV0gxT0VYeHVwSVF6QWZYbmsy?=
 =?utf-8?B?VEJuMzlyRUZyNTVNVCt4T3ZYSDRtclF5c0wwSHY3QjBNRmhyTzc4QTNKamRK?=
 =?utf-8?B?a2ZuY1hHTzBNckoxMG9raUhreUpOemFoUTg3dHFNSmVQWWYxdnU2QWVtdnor?=
 =?utf-8?B?d3FPWkthd0ZWZ0ZSU09XL1BvWFlzcHZONjdoUVBtY1JJT0tSS1kwREhWTVNR?=
 =?utf-8?B?TENZTnhxdWsyQlhycVJqY1JEeTdCajA0eW9hUW41MUl4RXFBNSt5SVJ1WGJz?=
 =?utf-8?B?ays3WmJyWjVIK0J5TGU3clZ5bTZlMU9lYUxUVW5EcHFxQ3hVOHk2V3NBR3dE?=
 =?utf-8?B?Yml4eVJFcStsMzRGaFBRS1ZLSmRKMEV3YW1zNjhJbCtkK1NndGRvUk04VkRU?=
 =?utf-8?B?d3lXNUZCUWtSSzc3UXJGSXdZL1kwYTJ3K3U0MzVMR3lEVTRzejJBL2ZxRDRZ?=
 =?utf-8?B?amZ2ZUJvOWdhNkpYS0s1OG96TVlxR3VnSDV6QnZ3anVBN2Y1NFBVZElzd2ow?=
 =?utf-8?B?Z01PQllCcXpteklTSmlkelRuQ3FpQnFXZklqUnhsRUw2ZVFRbGRZNVRXQVU3?=
 =?utf-8?B?RytCbnRRejl4WTFPOWwyQXRjUmpFZ2liWHpwb2U5NGJZeHk1Rmlsbis2Rks5?=
 =?utf-8?B?dUhLT2J5SGZSMlkzUGdhd0JPMDRLVGNkWnNtTFNWbjhsWXZZTCtTeWsvWWd3?=
 =?utf-8?B?UGQzclFSNG9Vak9ha3Z3K3REODg0dE1tQi9iRmxJR1hYTXVzakVjcXUzSXBO?=
 =?utf-8?B?eWxyUFFEYUxSZVh5Qk4zVGtiOWFhQ0JKNmh1SHA0UHJVbUNVUVZZSUxCakFp?=
 =?utf-8?B?Y1VqVC9aRVVkTVp1VFJEclYvUEY2NEViVUJWVGdrTG9tNVM4cUlYd2hmVVJh?=
 =?utf-8?B?LzFvM1BRd0t4Z2VONHVhdG1laU1hSS95c3ZHY0swdkRZZllZOTllU0RXVUhJ?=
 =?utf-8?B?Zis5SjlEQk5RcG1pMktlTXo4SVhmaVVXU3IxdGZHenM1dEt2TXQ2dlJYNkRY?=
 =?utf-8?B?ekhjbTFWME9HcTFvNGxRb0VkWGdlQmloMkRJTHFJZFJNRThuV0ZINjdhOTdW?=
 =?utf-8?Q?gW90EJcu6io5tFoGdH+f4kfsfGIEb0tmpwPxY=3D?=
X-Forefront-Antispam-Report:
	CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DU0PR04MB9417.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(52116005)(366007)(1800799015)(376005)(7416005)(921011)(38350700005);DIR:OUT;SFP:1101;
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?SnlkVXF5d2hUNGZLZXh5ZWwxRHZPVklOTXRRdUpPLzl0Tkx1NERWanNibVVl?=
 =?utf-8?B?NzZUNmtOQVR3bGlZSFliVzhNRmtESXg4NDBhSnR3YWFDTUdJczFiRmNxWkJC?=
 =?utf-8?B?ZDBsVXVidUFDZEN2M0l4MFF6akZaS2prbi9RZjlvbzV0cXQ2TU8rZXpjTlZs?=
 =?utf-8?B?ektkZUhMWHNzT3BQVnB6bEFHclNaWWk4VzVvbVhWTGY2T2V3TUVtR0FGS2hh?=
 =?utf-8?B?RDVvM05RWDV6TGErZ096dC82dGhCaVpUbzVsWXBQc05KcStRNHhseWNFUGpr?=
 =?utf-8?B?aFF5a0g4UUFXUFRvOEJKRGlsaDZsYU1tbU53YzNvWUVQb1kzTHQ5N2JIN1Bm?=
 =?utf-8?B?WE53aVJhaDVyR0ZXb3hrVldyc2F4VVorL0hCRS9XeVlFK2xuRitNN2hBdHU0?=
 =?utf-8?B?WVVCRTJBckNZeDRqaUNNRy9RRER3ZllIOVpZRGkzMjd4UllRY1d2UzFORk5z?=
 =?utf-8?B?dkc3cThBWEhsaGlCUFROdGZsK3BQRXlmRVQyUnVMSmxBS24yVlJjSkxZZjQ5?=
 =?utf-8?B?eWE2bnpJQjEyY1dYSFdweU94MkVWMUtQWURBRlZFcG50ajVhakliL3FUbDJz?=
 =?utf-8?B?WTg3eEhYSUlHMmI0SnorOTJRb1ZKZWZHWWhIeXJqbWFhd3A1SUlJR3I0QllM?=
 =?utf-8?B?MG4yN3BaYWZDVmFjb1NkaXNzeEc2U3FYdzgvQjJxSFJEL1pvWENCOTE4V29z?=
 =?utf-8?B?dmVlenM5M0c3YnlKTDZheExjY2lJVktCa0ZKbjJYRkdZSkVuZlVTVUsweWFT?=
 =?utf-8?B?d3NxZFlaNlRmeW9LTjROMlhaNnRxRFljU3FIaFROTTlhRzNDNlJ0dm1yWTdG?=
 =?utf-8?B?aEhYakVoNC9RUUZtbUoycmJkcjBRdU04VjBEOVU3dGxXallkUFZhNCswOXVY?=
 =?utf-8?B?bjhIU0VDNjdHTWRYU09IWGd2eWRkSXlCcEZaTFJwSjkxRE95RkoremN5cjA4?=
 =?utf-8?B?cjArK1Y0czkxR1VRd2FHNFRHYkdOQndWWEVPUkJ2Tit0U2x3d0NtT3l1dG1L?=
 =?utf-8?B?ZDE3a3pOdFRLbExZdWtlMEtmNFQ3aGFoLzM5a1RXdG51VGJDS1R1clVUcEZr?=
 =?utf-8?B?bFVLaVVyTHRqNStsMktmYUhzM1JqOS9rS2N2V0VnWm5PdFNRdnpieFpXQ09a?=
 =?utf-8?B?QzNDQXNFRWNEdW5mNjBiVmZiaWo0N1hwOSs3eEhhRHlVc0xDdXA1d2VYdW11?=
 =?utf-8?B?alR0a0NsdCttYVhuc3ArZ3hrUlQ0cnNtM3d3ZDh2emVpRWZGRUFVVUI5bVdG?=
 =?utf-8?B?VTVWVXRJbi9kYndEejg5SEdPNzFiQ3VmbkxiTVFuTDluMzhUNGVUanhBakJa?=
 =?utf-8?B?KzhTUnQ3VE9qeXExbE54aVhqOXJHVnVJVnl5TklQUFpiTWxaTGNXVVFXeUpG?=
 =?utf-8?B?QlEwQytXMWsyNTZFYTNyTEpieVhMcC84T3phNjArNTRNL2ZVbXpPT3ROK1Bq?=
 =?utf-8?B?QWZDUWJUWmtBTUoxemcrV2ZjZG1FZjI3YzVXMU5XZElaeTRGM3NkcnlmUCtU?=
 =?utf-8?B?ckpnR0xSb1RPYU9ualJobkwyVFEzc01wLzU5aWxPeElzNVFicFRyUFhEcnJY?=
 =?utf-8?B?anZ0TTRhRWhCLzFmcWltY1poZHVZTkpRNE9KNm02OGxXTnJYOExmMjlKSXZO?=
 =?utf-8?B?UTNuNUpSU3FpTkNxdTM0cWgyRVNwcWZqcFRKMnpoRUZFSzZHWEg0UGI3R2dr?=
 =?utf-8?B?NGVZNVFpVjZqY2hGSk5xOGNIM2h2N1pua1d0Q3FaYzBNdGVWWkwyMGQwY09S?=
 =?utf-8?B?Z0dWcTlYZFBXYUZJRXNFUFBRNHg3OWxEK3JtYTJlSEhoVTNqVEhrRnVSU3VF?=
 =?utf-8?B?aGt6R05Da002MjB4ZjRvdmZMZlFpZXVtYWtHZzBwQ2svaHpIeFJQRTY2SXVK?=
 =?utf-8?B?M1BRNUF4Z3ZBQU9md2Eya0xIeWZjY3VjZGs1aDEwS0hNNDVUODJDeGR0UmFm?=
 =?utf-8?B?TytXNnRwS3dWMFMxaVlOM2ZZUXE3UzhsU2w4bWVTK3VFbmpEdmNaZUp2NzZ4?=
 =?utf-8?B?U0FYTTg0cEJCcFk0cCtZeEhveEdROUF5WENWQjFqM1FvZGc3VHNhOE9pSFh1?=
 =?utf-8?B?dnNyK3hRcVFGQys2Wjh1bjZtRHplU2JHbG1qNEU1L3FTY1FYd3kvMWZaQVR5?=
 =?utf-8?Q?kEQr4ZChngdaUI8KDhBgAxaEL?=
X-OriginatorOrg: oss.nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f4c0d3e2-1016-48a7-5901-08dc70d12f24
X-MS-Exchange-CrossTenant-AuthSource: DU0PR04MB9417.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 May 2024 09:11:32.2188
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: n3oqMpmuTyhYmYbYPw61Pr0hcA3ZO1X7JiyRiwGHABPIvLTskRBmTwb54GrUWUvxgZ+A0xAbRTBVyZM6wmGn7Q==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM9PR04MB8065

From: Shengjiu Wang <shengjiu.wang@nxp.com>

The fvco frequency range is between 1600MHz and 3200MHz, without
this constraint the fvco may out of range, the real output
frequency is no accurate.

Aslo correct the name for fvco and fout clock.

Fixes: b09c68dc57c9 ("clk: imx: pll14xx: Support dynamic rates")
Signed-off-by: Shengjiu Wang <shengjiu.wang@nxp.com>
Acked-by: Jacky Bai <ping.bai@nxp.com>
Tested-by: Chancel Liu <chancel.liu@nxp.com>
Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
 drivers/clk/imx/clk-pll14xx.c | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/drivers/clk/imx/clk-pll14xx.c b/drivers/clk/imx/clk-pll14xx.c
index d63564dbb12c..55812bfb9ec2 100644
--- a/drivers/clk/imx/clk-pll14xx.c
+++ b/drivers/clk/imx/clk-pll14xx.c
@@ -131,7 +131,7 @@ static void imx_pll14xx_calc_settings(struct clk_pll14xx *pll, unsigned long rat
 {
 	u32 pll_div_ctl0, pll_div_ctl1;
 	int mdiv, pdiv, sdiv, kdiv;
-	long fout, rate_min, rate_max, dist, best = LONG_MAX;
+	long fvco, fout, rate_min, rate_max, dist, best = LONG_MAX;
 	const struct imx_pll14xx_rate_table *tt;
 
 	/*
@@ -144,6 +144,8 @@ static void imx_pll14xx_calc_settings(struct clk_pll14xx *pll, unsigned long rat
 	 *
 	 * fvco = (m * 65536 + k) * prate / (p * 65536)
 	 * fout = (m * 65536 + k) * prate / (p * 65536) / (1 << sdiv)
+	 *
+	 * e) 1600MHz <= fvco <= 3200MHz
 	 */
 
 	/* First try if we can get the desired rate from one of the static entries */
@@ -193,6 +195,10 @@ static void imx_pll14xx_calc_settings(struct clk_pll14xx *pll, unsigned long rat
 			kdiv = pll1443x_calc_kdiv(mdiv, pdiv, sdiv, rate, prate);
 			fout = pll14xx_calc_rate(pll, mdiv, pdiv, sdiv, kdiv, prate);
 
+			fvco = fout << sdiv;
+
+			if (fvco < 1600000000 || fvco > 3200000000)
+				continue;
 			/* best match */
 			dist = abs((long)rate - (long)fout);
 			if (dist < best) {

-- 
2.37.1