From patchwork Tue Apr 25 07:46:39 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Yang Xiwen via B4 Relay X-Patchwork-Id: 87279 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:b0ea:0:b0:3b6:4342:cba0 with SMTP id b10csp3237728vqo; Tue, 25 Apr 2023 00:50:23 -0700 (PDT) X-Google-Smtp-Source: AKy350ZVRLmEpkLxawObh5E4ic+cfZcwXxrNyW1IvAniEC2mPe8mAwGX9kRBAT3QWn1qkNfXBO+B X-Received: by 2002:a17:903:41cf:b0:1a6:d9a6:a9b4 with SMTP id u15-20020a17090341cf00b001a6d9a6a9b4mr20255861ple.3.1682409022753; Tue, 25 Apr 2023 00:50:22 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1682409022; cv=none; d=google.com; s=arc-20160816; b=H71HFOMigtpCCeEYQPXNVEO1ye6DlCkHCWoCVcy43WFMj8LJpyG2llenK922VLyCsD GsU4lILllDPHwK81mXKFHMlPPu1gS73tVTpaS6Bz9JARgiwnvYtfeEXFv5rUIm80zH2L osG0qbVFENlfdrHYnXAnWATOJpU7RsxsHAbhB/e8Z+ne4MxWy8/PxsQLtCNit5ax08EM b6y2CHRYDzSnx7/g/zUEhidXJh/a/VbBTyH56QAfT2duJbp3gT8RxjiVreiF5a9GMO0n GrOLfkejFyz2T1aehMyw87fvaMP98l7jvUU8OXOYRPIg+Rf+Xx69Bqs5LhQAwb72MUaF Zf/g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:reply-to:cc:to:in-reply-to:references:message-id :content-transfer-encoding:mime-version:subject:date:from :dkim-signature; bh=wG3IR6ypaITZqgUGz1Coq8PlWaPwmfoU7lUOMvlbpVc=; b=xJOWDkVALq6ET/bqoIOryZjklVgIKg7Uo2sjJod3AYkACurBFuu9Dhswbextz7gh55 d67lOOKgrCunw/pNenkbkcifIhpkApSozT8guXNgGyCymJPYHEwH6OxCUgjKqtA1RRFw QFUl9JGaTccosmb+IB19RWhPVlu0WP8f2MBUjxLYy+vpiSUTsHuzHtdVLi9Y1LjocPAX UIeB3eZXqvrG17D/G5vT5XZZEpW85Grgq0UQYE+74i26j+g83onhWtPPgVIcJERSuE91 S40vFaOO46xn2aonE7hfNpV07qAppu/MXRE6+2rg70AznLK5IzRL8pEcgkqYwnHNi0d1 cUwQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=iChgVoSb; 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=kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id u1-20020a1709026e0100b001a6783cf26dsi10893959plk.255.2023.04.25.00.50.08; Tue, 25 Apr 2023 00:50:22 -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=@kernel.org header.s=k20201202 header.b=iChgVoSb; 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=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232815AbjDYHrO (ORCPT + 99 others); Tue, 25 Apr 2023 03:47:14 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54152 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233404AbjDYHqz (ORCPT ); Tue, 25 Apr 2023 03:46:55 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D268D4C13; Tue, 25 Apr 2023 00:46:48 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 6D1D362C2B; Tue, 25 Apr 2023 07:46:48 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPS id C3510C4339C; Tue, 25 Apr 2023 07:46:47 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1682408807; bh=n0uLaqzrXaL0Pzdh4MV4LgrtNsHLsll3avYApfkg/5k=; h=From:Date:Subject:References:In-Reply-To:To:Cc:Reply-To:From; b=iChgVoSbidf99KrJZuNo5w8j0kCaiXSkHWMCDQyI0zo+4ZCIKpsLADS1/St3upxf7 Gjo7oru1SBs1lPBEZKhNyycsXgQ/nI7ZQ1XlSr5rT4f/K0PV0f84gljX//2fNfX5ku o53rqgpS+X4ASsB3CGmi5sn9xwIbGfGVxNM9RDU8fLOS2mNgK+npnhxbmPXjq6256a rB1Wfl3ht32RQ7qioUtCrJ+NWSUhXyTdS+XvCS+HC+NsE3+4AOFYNp3lEs6yUrxUWu Aq8s5xk2B31e5uRivlaLyqCjtD+VtYQkpu+npb8ntLaXwWDPSUBsvT5uMCfDU8tXY6 5bmL8ikuYzdGA== Received: from aws-us-west-2-korg-lkml-1.web.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9D894C6FD18; Tue, 25 Apr 2023 07:46:47 +0000 (UTC) From: Yang Xiwen via B4 Relay Date: Tue, 25 Apr 2023 15:46:39 +0800 Subject: [PATCH v2 1/2] clk: use ULONG_MAX as the initial value for the iteration in clk_mux_determine_rate_flags() MIME-Version: 1.0 Message-Id: <20230421-clk-v2-1-74c6cc2214d1@outlook.com> References: <20230421-clk-v2-0-74c6cc2214d1@outlook.com> In-Reply-To: <20230421-clk-v2-0-74c6cc2214d1@outlook.com> To: Michael Turquette , Stephen Boyd Cc: linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, Yang Xiwen X-Mailer: b4 0.12.0 X-Developer-Signature: v=1; a=ed25519-sha256; t=1682408804; l=1301; i=forbidden405@outlook.com; s=20230415; h=from:subject:message-id; bh=RtgdZyRNyL73Uk7aVZnLxu+b/Md7l1lfROZ/ueMsp+s=; b=awpu1dTnuZNRuDSJV4+H8qKX39whCHYqk53e5OCPtlAUqeIpYXfNYqPv0SypQYFzbZO/jG8bk RjNzFGo8ueOCDeKFIkfFr4iaoFitMCcPL38Cd/k5tI6NqhpryiJ3zFY X-Developer-Key: i=forbidden405@outlook.com; a=ed25519; pk=hfdpPU3AXR+t7fdv58tXCD4UzRNq+fop2TMJezFlAhM= X-Endpoint-Received: by B4 Relay for forbidden405@outlook.com/20230415 with auth_id=44 X-Original-From: Yang Xiwen Reply-To: X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FORGED_REPLYTO, FREEMAIL_REPLYTO_END_DIGIT,RCVD_IN_DNSWL_MED,SPF_HELO_NONE,SPF_PASS, T_SCC_BODY_TEXT_LINE autolearn=no 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: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1764133723667066974?= X-GMAIL-MSGID: =?utf-8?q?1764133723667066974?= From: Yang Xiwen Currently, clk_mux_determine_rate_flags() use 0 as the initial value for selecting the best matching parent. However, this will choose a non-existant rate(0) if the requested rate is closer to 0 than the minimum rate the parents have. Fix that by initializing the initial value to ULONG_MAX and treat it as a magic number. Signed-off-by: Yang Xiwen --- drivers/clk/clk.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c index ae07685c7588b..ab8a2acfac8f3 100644 --- a/drivers/clk/clk.c +++ b/drivers/clk/clk.c @@ -541,6 +541,9 @@ EXPORT_SYMBOL_GPL(__clk_is_enabled); static bool mux_is_better_rate(unsigned long rate, unsigned long now, unsigned long best, unsigned long flags) { + if (best == ULONG_MAX) + return true; + if (flags & CLK_MUX_ROUND_CLOSEST) return abs(now - rate) < abs(best - rate); @@ -600,7 +603,7 @@ int clk_mux_determine_rate_flags(struct clk_hw *hw, { struct clk_core *core = hw->core, *parent, *best_parent = NULL; int i, num_parents, ret; - unsigned long best = 0; + unsigned long best = ULONG_MAX; /* if NO_REPARENT flag set, pass through to current parent */ if (core->flags & CLK_SET_RATE_NO_REPARENT) { From patchwork Tue Apr 25 07:46:40 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Yang Xiwen via B4 Relay X-Patchwork-Id: 87281 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:b0ea:0:b0:3b6:4342:cba0 with SMTP id b10csp3240323vqo; Tue, 25 Apr 2023 00:57:38 -0700 (PDT) X-Google-Smtp-Source: AKy350ZH5mip/oJGnbuHZvdm6I6kepUwPqbEGl/+TYfa9TsnHVLgmg+hxzPLnzEhiw2uhtUvEueE X-Received: by 2002:a17:90b:344:b0:246:c097:6a17 with SMTP id fh4-20020a17090b034400b00246c0976a17mr17363889pjb.24.1682409458215; Tue, 25 Apr 2023 00:57:38 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1682409458; cv=none; d=google.com; s=arc-20160816; b=n/Kpr8psQ5xzM4FM8JvIKPqST47cLHymNpjqb6aUF8XTxmejOJAm5oo7K2KtZ+NbN7 4+bdUN4VubW3fJ7PTz3sRDW18G0FQOq/eBY0P0qRhc76QicvYA1m1IOj146ESCVTXzXa ptnFxG0ux/VnvlLkjcaISs08QyuZ91rXsYS7fPjfF61iJN20Tx2hvXgAAQzZiTAYFlbp rLdEQ8ifLxGcbDrBZ6jBvwDsojAJmXUlqRS3WVWTRLtoPyEmq9zSXFbTQP/Oe9QOqIUT t2IhuGi2d7Zv85u8ed/kc+ROe1q9nTj0wMRjID7HyMeGKnbQc+z+NiqwKdlvymXDR7Op /Z+w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:reply-to:cc:to:in-reply-to:references:message-id :content-transfer-encoding:mime-version:subject:date:from :dkim-signature; bh=MywHUI+4I5gFqXx5j/hPTfwOvTlo4R1DaLu1FJtM+5g=; b=vidHVsV9yZzlPfQm9UNQ/gnEhsclMVXNR3vro9kaTmK35UPRigEhWGoeUhSVC3LWQE 4NgBckyrNR3BVHf7trDveo0vqB/TIk/CDcMhnPJcAwe0E3m+uutv+1bBxkYOi6OGlAPy 8DFxjJ7icaRSCGfu2ZEaXauTn+4EubUTsUbnGsLmEskWsJ6xedkxoiKrqdqidT77B0DH cmRGfhFCJyJ7TgpH/gJxrT8kgF+owFmQSdXaVx3oYJDEEA/5N1pxrJlw5ag2aMn0jzaR ojvrUdREKs7z0y9wl0FC4s+D105cGrdUbj0KRIf2fnEaeVtgVC6WUNutwDOPVJQ1r89w WQeA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=rw97KsIW; 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=kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id 132-20020a63018a000000b0050f89421524si12150523pgb.319.2023.04.25.00.57.23; Tue, 25 Apr 2023 00:57:38 -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=@kernel.org header.s=k20201202 header.b=rw97KsIW; 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=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233434AbjDYHrJ (ORCPT + 99 others); Tue, 25 Apr 2023 03:47:09 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54058 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233330AbjDYHqz (ORCPT ); Tue, 25 Apr 2023 03:46:55 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D29E94C1B; Tue, 25 Apr 2023 00:46:48 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 6EFA662C2E; Tue, 25 Apr 2023 07:46:48 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPS id CA484C433EF; Tue, 25 Apr 2023 07:46:47 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1682408807; bh=6Pnug/27whDytkmeiic0CYbvDPeiXSI2PUOdTUfzKUc=; h=From:Date:Subject:References:In-Reply-To:To:Cc:Reply-To:From; b=rw97KsIWbn3SzxXCuHcnoGrLTib62W+Qs0kkcmNIZhd5JZmsaYBISB3W01kKqQ6th xoVKvhk+/fhHGQNmMWrL5HxKEagDGZGPwuyhBF4mKFv6p+LvFblQS4uQrUec6gqbRZ kROIyXFCVkgUHxZmjjGuYXZoncaXFEl4FBn8OqilrA37ZtIh5VW+sc+NWKcXx0U1di joyCE1U6RegKyF98/0mb/bI6a5zU9baVwWcft9ZPKEYlDrhxFoQR1hybgDmk03+j9U D0OEOoun+sUGBMxRgl4LNGotleUoCVxKypUyEI7ewYedhePDx4q7XPjK1PXvhlOKhW NY5maXrLYljDQ== Received: from aws-us-west-2-korg-lkml-1.web.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.lore.kernel.org (Postfix) with ESMTP id AD4B8C77B76; Tue, 25 Apr 2023 07:46:47 +0000 (UTC) From: Yang Xiwen via B4 Relay Date: Tue, 25 Apr 2023 15:46:40 +0800 Subject: [PATCH v2 2/2] clk: tests: Add missing test case for mux determine_rate MIME-Version: 1.0 Message-Id: <20230421-clk-v2-2-74c6cc2214d1@outlook.com> References: <20230421-clk-v2-0-74c6cc2214d1@outlook.com> In-Reply-To: <20230421-clk-v2-0-74c6cc2214d1@outlook.com> To: Michael Turquette , Stephen Boyd Cc: linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, Yang Xiwen X-Mailer: b4 0.12.0 X-Developer-Signature: v=1; a=ed25519-sha256; t=1682408804; l=1148; i=forbidden405@outlook.com; s=20230415; h=from:subject:message-id; bh=9OLnZFueJynRZ3/SvtdA921aQdEuKJhzY8cmJ1y5r2Y=; b=pdXVz8zXO1kXbFfAdJ/F15BBHF/1kAn2fhFbi8+/zSxB0PPgyE7tRm3SSeQEjMvI/zH4HDqAQ /LWxPZfpgvADWcbrp4mevnApifX4nH2eR+d3HP1k0vnov5Tg2gcRW9J X-Developer-Key: i=forbidden405@outlook.com; a=ed25519; pk=hfdpPU3AXR+t7fdv58tXCD4UzRNq+fop2TMJezFlAhM= X-Endpoint-Received: by B4 Relay for forbidden405@outlook.com/20230415 with auth_id=44 X-Original-From: Yang Xiwen Reply-To: X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FORGED_REPLYTO, FREEMAIL_REPLYTO_END_DIGIT,RCVD_IN_DNSWL_MED,SPF_HELO_NONE,SPF_PASS, T_SCC_BODY_TEXT_LINE autolearn=no 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: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1764134180428258023?= X-GMAIL-MSGID: =?utf-8?q?1764134180428258023?= From: Yang Xiwen Add a missing test case for determine_rate implemented by mux. It tests the behavior of determine_rate when requesting a rate that none of the parents could give. Signed-off-by: Yang Xiwen --- drivers/clk/clk_test.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/drivers/clk/clk_test.c b/drivers/clk/clk_test.c index f9a5c2964c65d..9b1c90de90454 100644 --- a/drivers/clk/clk_test.c +++ b/drivers/clk/clk_test.c @@ -2215,6 +2215,15 @@ static void clk_leaf_mux_set_rate_parent_determine_rate(struct kunit *test) KUNIT_EXPECT_EQ(test, req.best_parent_rate, DUMMY_CLOCK_RATE_2); KUNIT_EXPECT_PTR_EQ(test, req.best_parent_hw, &ctx->mux_ctx.hw); + // Test a non-existant rate which none of the parents could give + clk_hw_init_rate_request(hw, &req, DUMMY_CLOCK_INIT_RATE); + + ret = __clk_determine_rate(hw, &req); + KUNIT_ASSERT_EQ(test, ret, 0); + + KUNIT_EXPECT_EQ(test, req.rate, DUMMY_CLOCK_RATE_1); + KUNIT_EXPECT_EQ(test, req.best_parent_rate, DUMMY_CLOCK_RATE_1); + KUNIT_EXPECT_PTR_EQ(test, req.best_parent_hw, &ctx->mux_ctx.hw); clk_put(clk); }