On Tuesday, August 8, 2023 12:46:23 PM CEST Conor Dooley wrote:

On Tue, Aug 08, 2023 at 12:41:22PM +0200, Duje Mihanović wrote:

quoted

They are used by the clock driver when calling mmp_clk_init which then 

uses

quoted

that as the size of a struct clk array it allocates. In retrospect, 50 for
each block may be too much as from what I can tell by reading the
mmp_register_* functions (number of clocks + 1) for each block should be
enough, anything less than that causes a null pointer dereference sometime
during clock initialization.

I think you might have misread my question, I'm not really interested in
the implementation detail of the driver. If these are not used in
devicetree, remove them - otherwise they are being needlessly added to
the ABI.

Should I also do this in the rest of the MMP clock drivers?

Regards,
Duje