On 17.11.2023 21:50, Bryan O'Donoghue wrote: > On 17/11/2023 09:27, Sibi Sankar wrote: >> * Use shared ops in the x1e80100 gcc driver [Bryan]. > > This looks better to me now / more consistent with what we have in sc8280xp - where we do try to hit suspend and => retention/parking matters. Parking the clock is separate from putting the system to sleep. IIUC we usually use shared ops on clocks that may have different users (e.g. not only controlled by Linux) and/or that are crucial to the functioning of hardware (like AXI clocks, which if gated would make the system crash on any access attempt, from any subsystem, unless turned on beforehand) Perhaps Dmitry can use some clearer wording than me.. Konrad
