[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <aWA7JkZAIGFpAeVZ@redhat.com>
Date: Thu, 8 Jan 2026 18:17:58 -0500
From: Brian Masney <bmasney@...hat.com>
To: kernel test robot <lkp@...el.com>
Cc: oe-kbuild-all@...ts.linux.dev, linux-kernel@...r.kernel.org,
Stephen Boyd <sboyd@...nel.org>, linux-clk@...r.kernel.org
Subject: Re: drivers/clk/microchip/clk-core.c:915:10: sparse: sparse:
Initializer entry defined twice
On Wed, Jan 07, 2026 at 05:52:23AM +0800, kernel test robot wrote:
> Hi Brian,
>
> First bad commit (maybe != root cause):
>
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head: f0b9d8eb98dfee8d00419aa07543bdc2c1a44fb1
> commit: 7d85cd8730ab7701a1cafd0db0d7b6b1f6cfbb91 clk: microchip: core: remove duplicate roclk_determine_rate()
> date: 3 months ago
> config: mips-randconfig-r113-20260106 (https://download.01.org/0day-ci/archive/20260107/202601070531.yJ4x70yl-lkp@intel.com/config)
> compiler: clang version 22.0.0git (https://github.com/llvm/llvm-project 9b8addffa70cee5b2acc5454712d9cf78ce45710)
> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260107/202601070531.yJ4x70yl-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 <lkp@...el.com>
> | Closes: https://lore.kernel.org/oe-kbuild-all/202601070531.yJ4x70yl-lkp@intel.com/
>
> sparse warnings: (new ones prefixed by >>)
> >> drivers/clk/microchip/clk-core.c:915:10: sparse: sparse: Initializer entry defined twice
> drivers/clk/microchip/clk-core.c:919:10: sparse: also defined here
>
> vim +915 drivers/clk/microchip/clk-core.c
>
> ce6e1188465998 Purna Chandra Mandal 2016-05-13 910
> ce6e1188465998 Purna Chandra Mandal 2016-05-13 911 /* sclk with post-divider */
> ce6e1188465998 Purna Chandra Mandal 2016-05-13 912 const struct clk_ops pic32_sclk_ops = {
> ce6e1188465998 Purna Chandra Mandal 2016-05-13 913 .get_parent = sclk_get_parent,
> ce6e1188465998 Purna Chandra Mandal 2016-05-13 914 .set_parent = sclk_set_parent,
> e9f039c08cdc9b Brian Masney 2025-08-11 @915 .determine_rate = sclk_determine_rate,
> ce6e1188465998 Purna Chandra Mandal 2016-05-13 916 .set_rate = sclk_set_rate,
> ce6e1188465998 Purna Chandra Mandal 2016-05-13 917 .recalc_rate = sclk_get_rate,
> ce6e1188465998 Purna Chandra Mandal 2016-05-13 918 .init = sclk_init,
> ce6e1188465998 Purna Chandra Mandal 2016-05-13 919 .determine_rate = __clk_mux_determine_rate,
> ce6e1188465998 Purna Chandra Mandal 2016-05-13 920 };
> ce6e1188465998 Purna Chandra Mandal 2016-05-13 921
>
> :::::: The code at line 915 was first introduced by commit
> :::::: e9f039c08cdc9b38665aee9a88ae21f59c09ba8c clk: microchip: core: convert from round_rate() to determine_rate()
>
> :::::: TO: Brian Masney <bmasney@...hat.com>
> :::::: CC: Brian Masney <bmasney@...hat.com>
I posted a series last month to fix this issue.
https://lore.kernel.org/linux-clk/aWA6rNLNqYfJKTzf@redhat.com/T/#t
Brian
Powered by blists - more mailing lists