[PATCH] clk: Dont return error when assigned-clocks is empty or missing
Michal Simek
michal.simek at amd.com
Tue Jul 11 16:20:58 CEST 2023
Hi Sean,
On 7/11/23 15:40, Sean Anderson wrote:
> On 7/11/23 05:51, Ashok Reddy Soma wrote:
>> There is a chance that assigned-clock-rates is given and assigned-clocks
>> could be empty. Dont return error in that case, because the probe of the
>> corresponding driver will not be called at all if this fails.
>> Better to continue to look for it and return 0.
>
> No, this is an error in the device tree. assigned-clock-rates depends on
> assigned-clocks, so you must provide the latter if the former is present.
We were also checking it and in the Linux kernel it is handle like this.
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/clk/clk-conf.c#n95
It means you can have rate assigned but not assigned-clocks property.
And yes in working case both should be present to work properly.
Thanks,
Michal
More information about the U-Boot
mailing list