RE: [PATCH v2] regulator: add new regulator driver for lp872x

From: Kim, Milo
Date: Tue Jun 19 2012 - 02:42:05 EST


>
> On Fri, Jun 15, 2012 at 07:11:48PM +0800, Axel Lin wrote:
>
> > > +       ret = gpio_request(gpio, "LP872X DVS");
>
> > devm_gpio_request
>
> Better yet...
>
> > > +       gpio_direction_output(gpio, dvs->init_state);
>
> ...devm_gpio_request_one().
>

Thanks for the update.
In patch v3, devm_gpio_request_one() will be used in replacement of gpio_request() and gpio_direction_output().

Thanks & BR
Milo -

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