Re: [PATCH 1/5] gpiolib: devres: Introduce the function devm_request_gpio_array

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 10/07/14 10:21, Linus Walleij wrote:
On Wed, Jul 9, 2014 at 1:18 PM, Rob Jones <rob.jones@xxxxxxxxxxxxxxx> wrote:

Please note that I submitted a patch on 02/07/14 to create this
function which was acked by Linus Walleij on 05/07/14.

Great minds think alike, and all that.

My mind certainly isn't any great :-(


I doubt that :-)

My left hand seems to be unaware of what the right hand is doing.

Did I ACK that for merging through some other tree then?


You did: https://lkml.org/lkml/2014/7/4/715

And I guess if Himangi want to submit these ASoC patches, they
will have to go through the same tree as those patches in that case?


I don't know enough to comment.

And would you consider implementing a gpiod version...?


I will certainly consider it. I'm in the middle of something else right
now but I should be available in a day or two and I'll have a look.

However, I think that the version I submitted better replicates the
original (non devm) functionality, see below.

I didn't, however, add it to the documentation. +1 on that.

Can you include that oneliner in your patch then to reduce
problems?


Certainly, with acknowledgement to Himangi Saraogi. Should I resubmit
the patch series with amendment or should I put it in as a separate
patch? Sorry for the noobie question but I *am* new to the kernel
lists.

Yours,
Linus Walleij



--
Rob Jones
Codethink Ltd
mailto:rob.jones@xxxxxxxxxxxxxxx
tel:+44 161 236 5575
--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux