Re: [PATCH 2/2] regulator: max77686: Implement .set_ramp_delay() callback.

From: Yadwinder Singh Brar
Date: Tue Jun 19 2012 - 04:21:04 EST


On Tue, Jun 19, 2012 at 1:48 PM, Axel Lin <axel.lin@xxxxxxxxx> wrote:
>>  static struct regulator_ops max77686_ops = {
>>        .list_voltage           = regulator_list_voltage_linear,
>>        .map_voltage            = regulator_map_voltage_linear,
>> @@ -90,6 +114,7 @@ static struct regulator_ops max77686_buck_dvs_ops = {
>>        .get_voltage_sel        = regulator_get_voltage_sel_regmap,
>>        .set_voltage_sel        = regulator_set_voltage_sel_regmap,
>>        .set_voltage_time_sel   = regulator_set_voltage_time_sel,
>> +       .set_ramp_delay         = max77686_set_ramp_delay,
>>  };
>
> I think what you want here is to set .set_ramp_delay callback for
> max77686_buck_dvs_ops
> rather than max77686_ops.

Yes, I will correct it.

Thanks,
Yadwinder.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/