linux-kernel - Re: Re: [PATCH v9 2/3] clock: eswin: Add eic7700 clock driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <480d671d.1db1.19b8707eb98.Coremail.dongxuyang@eswincomputing.com>
Date: Sun, 4 Jan 2026 11:23:21 +0800 (GMT+08:00)
From: "Xuyang Dong" <dongxuyang@...incomputing.com>
To: "Marcel Ziswiler" <marcel@...wiler.com>, mturquette@...libre.com,
	sboyd@...nel.org, robh@...nel.org, krzk+dt@...nel.org,
	conor+dt@...nel.org, linux-clk@...r.kernel.org,
	devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
	troy.mitchell@...ux.dev, bmasney@...hat.com
Cc: ningyu@...incomputing.com, linmin@...incomputing.com,
	huangyifeng@...incomputing.com, pinkesh.vaghela@...fochips.com,
	ganboing@...il.com
Subject: Re: Re: [PATCH v9 2/3] clock: eswin: Add eic7700 clock driver

> > +
> > +/* divider clocks */
> > +static struct eswin_divider_clock eic7700_div_clks[] = {
> > +	EIC7700_DIV(EIC7700_CLK_DIV_SYS_CFG_DYNM, "divider_sys_cfg_div_dynm",
> > +		    "fixed_rate_clk_spll0_fout3", 0,
> > +		    EIC7700_REG_OFFSET_SYSCFG_CTRL, 4, 3,
> > +		    CLK_DIVIDER_ONE_BASED),
> > +	EIC7700_DIV(EIC7700_CLK_DIV_NOC_NSP_DYNM, "divider_noc_nsp_div_dynm",
> > +		    "fixed_rate_clk_spll2_fout1", 0,
> > +		    EIC7700_REG_OFFSET_NOC_CTRL, 0, 3, CLK_DIVIDER_ONE_BASED),
> > +	EIC7700_DIV(EIC7700_CLK_DIV_BOOTSPI_DYNM, "divider_bootspi_div_dynm",
> > +		    "gate_clk_spll0_fout2", 0, EIC7700_REG_OFFSET_BOOTSPI_CTRL,
> > +		    4, 6, CLK_DIVIDER_ONE_BASED),
> > +	EIC7700_DIV(EIC7700_CLK_DIV_SCPU_CORE_DYNM,
> > +		    "divider_scpu_core_div_dynm", "fixed_rate_clk_spll0_fout1",
> > +		    0, EIC7700_REG_OFFSET_SCPU_CORE_CTRL, 4, 4,
> > +		    CLK_DIVIDER_ONE_BASED),
> > +	EIC7700_DIV(EIC7700_CLK_DIV_LPCPU_CORE_DYNM,
> > +		    "divider_lpcpu_core_div_dynm", "fixed_rate_clk_spll0_fout1",
> > +		    0, EIC7700_REG_OFFSET_LPCPU_CORE_CTRL, 4, 4,
> > +		    CLK_DIVIDER_ONE_BASED),
> 
> During testing on top of next-20251219 I got the following:
> 
> [    0.126102] divider_lpcpu_core_div_dynm: Zero divisor and CLK_DIVIDER_ALLOW_ZERO not set
> [    0.126194] WARNING: drivers/clk/clk-divider.c:145 at divider_recalc_rate+0x8a/0x9c, CPU#0: swapper/0/1
> 
> Therefore I suggest for above line needing to be:
> 
> +                   CLK_DIVIDER_ONE_BASED | CLK_DIVIDER_ALLOW_ZERO),

Hi Marcel,

Thank you! This is indeed a bug and will be fixed in the next version. 
Could you please share the detailed testing methods with me so that I 
can reproduce this issue on my side?

Best regards,
Xuyang Dong