On Mon, Oct 27, 2014 at 01:11:49PM +0100, Krzysztof Kozlowski wrote:

> @@ -247,6 +250,8 @@ static struct regulator_ops max77686_ldo_ops = {
>       .set_voltage_sel        = regulator_set_voltage_sel_regmap,
>       .set_voltage_time_sel   = regulator_set_voltage_time_sel,
>       .set_suspend_mode       = max77686_ldo_set_suspend_mode,
> +     .set_suspend_disable    = max77686_set_suspend_disable,
> +     .set_suspend_enable     = max77686_enable,

This looks wrong, you're using the regular enable operation as suspend
enable.  How does that work without disrupting the current runtime
state?

Attachment: signature.asc
Description: Digital signature

Reply via email to