> From: Saravana Kannan <skan...@codeaurora.org>
> 
> If the new governor fails to start, switch back to old governor so that the
> devfreq state is not left in some weird limbo.
> 
> Signed-off-by: Sibi Sankar <si...@codeaurora.org>
> Signed-off-by: Saravana Kannan <skan...@codeaurora.org>
> Reviewed-by: Chanwoo Choi <cw00.c...@samsung.com>

Hello,

In overall, the idea and the implementation looks good.

However, I have a question:

What if the following line fails?

+               df->governor->event_handler(df, DEVFREQ_GOV_START,
+                                           NULL);

Don't we still need something to handle for such events?

Cheers,
MyungJoo

Reply via email to