On 23/08/2024 11:47, Gokul Sriram P wrote: > > On 8/22/2024 5:00 PM, Krzysztof Kozlowski wrote: >>> IM_SLEEP_CLK - Internal Module sleep clock needed for Q6 reset. >>> >>> SLEEP is not an acronym here. >> Then probably you mean "Internal sleep", although "internal" is also >> confusing. Devices do not receive as input something which is internal >> to them. >> >>>>> + >>>>> + clock-names: >>>>> + items: >>>>> + - const: im_sleep >>>> sleep? Are there different sleep clocks here? >>> We have different branches of sleep clk each enabled separately. >>> >>> im_sleep is one of those branches that q6 uses. >> So this device misses other branches? Then provide them. Otherwise it is >> just "sleep". > > ok, I shall keep it as simply "sleep" and move on? Other branches of > sleep clk are irrelevant to remoteproc. Then it is "sleep". The name here describes the clock input in this device, not the clock in other places. Best regards, Krzysztof