On Wed, 2024-10-23 at 17:30 +0100, Conor Dooley wrote: > On Wed, Oct 23, 2024 at 04:56:54PM +0200, Nuno Sa wrote: > > In order to access the registers of the HW, we need to make sure that > > the AXI bus clock is enabled. Hence let's increase the number of clocks > > by one. > > > > In order to keep backward compatibility, the new axi clock must be the > > last phandle in the array. To make the intent clear, a non mandatory > > clock-names property is also being added. > > Hmm, I'm not sure. I think clock-names actually may need to be mandatory > here, as otherwise you'll not what the second clock is. The driver would > have to interpret no clock-names meaning clock 2 was clkin2. > > So the way things are now is that we just get the parents count with of_clk_get_parent_count() and then get the names with of_clk_get_parent_name() and this is given into 'struct clk_init_data'. So they are effectively clk_parents of the clock we're registering and as you can see clock-names does not really matter. What I'm trying to do is to keep this and still allow to get the AXI bus clock which is something we should get and enable and not rely on others to do it. The idea is then to add the axi bus clock as the last one in the clocks property and I will get it by index with of_clk_get(). The rest pretty much remains the same and we just need to decrement by one the number of parent clocks as the axi clock is not really a parent of our output clock. All that said, and FWIW, clock-names are not even being used in the driver. I just added it to the bindings to make the intent clear. I could have it in the driver but I'm not sure the extra complexity would be worth it... - Nuno Sá