Messages in this thread | | | Date | Tue, 7 Oct 2014 15:34:05 +0200 | From | Frans Klaver <> | Subject | Re: [PATCH v3 1/2] power: reset: add LTC2952 poweroff support |
| |
On Tue, Oct 07, 2014 at 06:16:19AM -0700, Guenter Roeck wrote: > On 10/07/2014 01:15 AM, Frans Klaver wrote: > > On Mon, Oct 06, 2014 at 02:32:10PM -0700, Guenter Roeck wrote: > > >> Any reason for not using devm_gpiod_get() for those functions ? > > > > René said in the first review round: > > > >>>> devm_* functions do not seem to common to me and are sparsely > >>>> documented, therefore I preferred the regular functions. This > >>>> error jump should go to err_io and the clean-up loop must check > >>>> if the gpio entry is actually filled in > > > > I now unrolled these loops, and quite frankly I'm not that sure devm_* > > functions are actually that sparsely used. The way to get around that is > > to use them more. I'll see about using those instead, maybe add some > > documentation as well. > > > > "git grep devm_| wc" on 3.17 returns 7344. > > If that is not enough, nothing ever will.
Heh, precisely. I've switched to the devm_* functions. Code's much cleaner now.
Thanks, Frans -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |