RE: [PATCH v4] regulator: Add S5M8767A regulator driver

From: Sangbeom Kim
Date: Mon Jan 09 2012 - 03:35:47 EST


Hi,
On Mon, Jan 9, 2011 at 04:25PM, Mark Brown wrote:

> I was expecting a new version with these:
>
> > +static int s5m8767_reg_enable_suspend(struct regulator_dev *rdev)
> > +static int s5m8767_reg_disable_suspend(struct regulator_dev *rdev)
>

After delete above functions, Is it right to code like below?

+static struct regulator_ops s5m8767_buck_ops = {
+ .enable = s5m8767_reg_enable,
+ .disable = s5m8767_reg_disable,
+ .set_suspend_enable = s5m8767_reg_enable,
+ .set_suspend_disable = s5m8767_reg_disable,

Thanks,
Sangbeom.

--
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/