> From: Saravana Kannan <[email protected]>
> 
> 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 <[email protected]>
> Signed-off-by: Saravana Kannan <[email protected]>
> Reviewed-by: Chanwoo Choi <[email protected]>

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