Re: [PATCH 1/3] dt-bindings: clock: rk3128: Add HCLK_SFC

From: Alex Bee
Date: Wed Jun 05 2024 - 15:35:37 EST



Am 05.06.24 um 19:58 schrieb Conor Dooley:
On Wed, Jun 05, 2024 at 07:21:52PM +0200, Alex Bee wrote:
Add a clock id for SFC's AHB clock.

Signed-off-by: Alex Bee <knaerzche@xxxxxxxxx>
---
NB: I've added the new id at the top of existing HCLK's, since adding it at
the bottom would change CLK_NR_CLKS and therefore the ABI and there is
plenty of "free" ids between PCLK_MIPIPHY and HCLK_SPDIF.
We've removed a bunch of things like "CLK_NR_CLKS" from bindings before
and advocate against adding them. Patches like this being the reason
why..

Yeah, sure ... I thought removing it would be little out of scope for this
series. But as we are already having a helper for finding the "max id"  for
Rockchip clocks, I'll resend and remove that annoying macro and add the new
id at the bottom of the existing (where it belongs).

Alex