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 6282EEB64D9 for ; Sat, 17 Jun 2023 13:33:21 +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:MIME-Version:Message-ID:Date: In-reply-to:Subject:Cc:To:From:References:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Owner; bh=vJrq/iysyjawCk6O0dVrhcCwYoA2xoFcCWeESRJF7PE=; b=ow3OLTlThaLeLbxKJy16xmVTA5 a59znSum2V8KOc4KAIA4nHpxkK8kHSZi4a29L/glnQq20NAVUQtGKzMD/60fSy9rQDBg0v6rSuTrb GW9g4eWslb9sqOpe0Qr5LnfoPJhCXE+JpBUBv8Dqx9fWmA6fwO/qL/s15WFN7LxIe5z8wlOKbUzJQ gfd+oXHHd/ocFcQnkkzC1TZRr660BbKNCqBf5hhQlOqpeBnb/JZYnxqIfJxoQwLnUVPeR8aJlNabl B0ZjTKievYi0tnQ5hWMk8Xo6QUz+7elVn525gbliR8uxP4uH3oyAnyrxd7aacw+Fb50r6SZUTY74g c7s8at7A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qAW2y-003fXv-34; Sat, 17 Jun 2023 13:32:56 +0000 Received: from mout-p-201.mailbox.org ([2001:67c:2050:0:465::201]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qAW2w-003fXW-1e for linux-arm-kernel@lists.infradead.org; Sat, 17 Jun 2023 13:32:56 +0000 Received: from smtp102.mailbox.org (smtp102.mailbox.org [10.196.197.102]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-384) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by mout-p-201.mailbox.org (Postfix) with ESMTPS id 4QjxmL3rQ0z9sks; Sat, 17 Jun 2023 15:32:50 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oltmanns.dev; s=MBO0001; t=1687008770; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=AZoI4Bw3sCCyCXPTMLBEPFCqY8Pv58J03l2/6Q77EHA=; b=QZKf+EFTILZO4/YhLIUf7kpElHYaNl+CQynchuZFnmnCaPr44U0e7D9CWo76xCX8YFTRj/ 86v5AC8NrdL2uTOQYjN+9IeH8wAQ3NvWECUS1kPtHdd7EVio0VClDQd0qcwTpRIr5bGQSa la0RL4TCuclqXrpuQTwuxbU6BfEnLB6AKFdbCKOvfQB9BWFaZTkeulgdm7+IU/sNzI4OqC 5/EGA4mVI4GzaBB1/lABi2Ju6ETporuiRiOJ/39n8ZOuRkRTrNtdSpSc005F/aB49eC3W9 qalHOdOTV7Q/snc8VP5i4AX+IncTgTvTUpXCMAm7hpsonAvbJ3Ymz3sTfXFvYw== References: <20230617102919.27564-3-frank@oltmanns.dev> <202306172036.EevtaVp6-lkp@intel.com> From: Frank Oltmanns To: kernel test robot Cc: Michael Turquette , Stephen Boyd , llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev, "A.s. Dong" , Abel Vesa , Fabio Estevam , linux-arm-kernel@lists.infradead.org, linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, NXP Linux Team , Peng Fan , Pengutronix Kernel Team , Sascha Hauer , Shawn Guo , Elaine Zhang Subject: Re: [PATCH v4 2/2] clk: fractional-divider: tests: Add test suite for edge cases In-reply-to: <202306172036.EevtaVp6-lkp@intel.com> Date: Sat, 17 Jun 2023 15:32:31 +0200 Message-ID: <87legi2p00.fsf@oltmanns.dev> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230617_063254_773148_F723390E X-CRM114-Status: GOOD ( 15.38 ) 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 2023-06-17 at 20:15:20 +0800, kernel test robot wrote: > Hi Frank, > > kernel test robot noticed the following build warnings: > > [auto build test WARNING on clk/clk-next] > [also build test WARNING on linus/master v6.4-rc6 next-20230616] > [If your patch is applied to the wrong git tree, kindly drop us a note. > And when submitting patch, we suggest to use '--base' as documented in > https://git-scm.com/docs/git-format-patch#_base_tree_information] > > url: https://github.com/intel-lab-lkp/linux/commits/Frank-Oltmanns/clk-fractional-divider-Improve-approximation-when-zero-based-and-export/20230617-183118 > base: https://git.kernel.org/pub/scm/linux/kernel/git/clk/linux.git clk-next > patch link: https://lore.kernel.org/r/20230617102919.27564-3-frank%40oltmanns.dev > patch subject: [PATCH v4 2/2] clk: fractional-divider: tests: Add test suite for edge cases > config: hexagon-randconfig-r035-20230617 (https://download.01.org/0day-ci/archive/20230617/202306172036.EevtaVp6-lkp@intel.com/config) > compiler: clang version 17.0.0 (https://github.com/llvm/llvm-project.git 4a5ac14ee968ff0ad5d2cc1ffa0299048db4c88a) > reproduce: (https://download.01.org/0day-ci/archive/20230617/202306172036.EevtaVp6-lkp@intel.com/reproduce) > > If you fix the issue in a separate patch/commit (i.e. not just a new version of > the same patch/commit), kindly add following tags > | Reported-by: kernel test robot > | Closes: https://lore.kernel.org/oe-kbuild-all/202306172036.EevtaVp6-lkp@intel.com/ > > All warnings (new ones prefixed by >>): > >>> drivers/clk/clk-fractional-divider_test.c:26:41: warning: variable 'max_m' set but not used [-Wunused-but-set-variable] > 26 | unsigned long rate, parent_rate, m, n, max_m, max_n; > | ^ > drivers/clk/clk-fractional-divider_test.c:88:41: warning: variable 'max_m' set but not used [-Wunused-but-set-variable] > 88 | unsigned long rate, parent_rate, m, n, max_m, max_n; > | ^ > 2 warnings generated. Clang compiler warnings are fixed in version 5: https://lore.kernel.org/all/20230617131041.18313-1-frank@oltmanns.dev/ > > > vim +/max_m +26 drivers/clk/clk-fractional-divider_test.c > > 15 > 16 /* > 17 * Test the maximum denominator case for fd clock without flags. > 18 * > 19 * Expect the highest possible denominator to be used in order to get as close as possible to the > 20 * requested rate. > 21 */ > 22 static void clk_fd_test_approximation_max_denominator(struct kunit *test) > 23 { > 24 struct clk_fractional_divider *fd; > 25 struct clk_hw *hw; > > 26 unsigned long rate, parent_rate, m, n, max_m, max_n; > 27 > 28 fd = kunit_kzalloc(test, sizeof(*fd), GFP_KERNEL); > 29 KUNIT_ASSERT_NOT_NULL(test, fd); > 30 > 31 fd->mwidth = 3; > 32 max_m = 7; > 33 fd->nwidth = 3; > 34 max_n = 7; > 35 > 36 hw = &fd->hw; > 37 > 38 rate = 240000000; > 39 parent_rate = (max_n + 1) * rate; /* so that it exceeds the maximum divisor */ > 40 > 41 clk_fractional_divider_general_approximation(hw, rate, &parent_rate, &m, &n); > 42 KUNIT_EXPECT_EQ(test, parent_rate, (max_n + 1) * rate); /* parent remains unchanged */ > 43 KUNIT_EXPECT_EQ(test, m, 1); > 44 KUNIT_EXPECT_EQ(test, n, max_n); > 45 } > 46 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel