[PATCH] clk: scmi: add is_prepared hook
Cristian Marussi
cristian.marussi at arm.com
Fri Jul 26 04:14:54 PDT 2024
On Thu, Jul 25, 2024 at 05:07:41PM +0800, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan at nxp.com>
>
> Some clks maybe default enabled by hardware, so add is_prepared hook
> to get the status of the clk. Then when disabling unused clks, those
> unused clks but default hardware on clks could be in off state to save
> power.
>
Hi Peng,
seems a good addition to me, I forgot to add supporrt for non atomic
scenarios like for clk enable/disable....
...having said that, though, you basically copied the original ATOMIC
version of this function and changed only the ATOMIC -> NON_ATOMIC param
for the state_get() call...
...unless I am missing something, this sounds to me as needless duplication
of code...please rework the original existing function to be an internal helper
acceppting an additinal parameter (ATOMIC, NON_ATOMIC) and then build 2 wrappers
omn top of it for the original and your new function...
Thanks,
Cristian
More information about the linux-arm-kernel
mailing list