[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAOMZO5C2Hg06wb08cRDLCXGDS_rmUAq3fC7Jm++kSumjukyuXA@mail.gmail.com>
Date: Mon, 26 Jan 2026 16:47:45 -0300
From: Fabio Estevam <festevam@...il.com>
To: Łukasz Majewski <lukma@...ladev.com>
Cc: Andrew Lunn <andrew@...n.ch>, Conor Dooley <conor+dt@...nel.org>, Abel Vesa <abelvesa@...nel.org>,
Peng Fan <peng.fan@....com>, Michael Turquette <mturquette@...libre.com>,
Stephen Boyd <sboyd@...nel.org>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>, Shawn Guo <shawnguo@...nel.org>,
Sascha Hauer <s.hauer@...gutronix.de>, Pengutronix Kernel Team <kernel@...gutronix.de>,
linux-clk@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, imx@...ts.linux.dev,
linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH v2] clk: vf610: Add support for the Ethernet switch clocks
Hi Łukasz,
On Mon, Jan 26, 2026 at 4:33 PM Łukasz Majewski <lukma@...ladev.com> wrote:
> Conor (on CC) explicitly asked to remove VF610_CLK_END:
> "[PATCH] dt-bindings: clock: vf610: Add definitions for MTIP L2 switch"
>
> ------>8------------
> > -#define VF610_CLK_END 191
>
> If end is modifiable, it is removable. If you need the define for the
> driver, please move it there.
> ------8<------------
>
> I just follow what he asked.
You should still use VF610_CLK_END. Just define it inside the clk
driver instead of in the dt-bindings.
Please check these two i.MX93 patches that explain the rationale:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v6.19-rc7&id=c0813ce2e5b0d1174782aff30d366509377abc7b
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v6.19-rc7&id=0af18ba60752e8a4ba34404c1d9a4a799da690f5
Powered by blists - more mailing lists