All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rob Jones <rob.jones@codethink.co.uk>
To: Mark Brown <broonie@kernel.org>
Cc: linus.walleij@linaro.org, gnurou@gmail.com, lgirdwood@gmail.com,
	linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-kernel@codethink.co.uk, ian.molton@codethink.co.uk,
	ben.dooks@codethink.co.uk, heiko@sntech.de
Subject: Re: Change gpio-reulator-probe() to use managed resources.
Date: Fri, 20 Jun 2014 09:38:29 +0100	[thread overview]
Message-ID: <53A3F305.9070903@codethink.co.uk> (raw)
In-Reply-To: <20140619175745.GS5099@sirena.org.uk>


On 19/06/14 18:57, Mark Brown wrote:
> On Thu, Jun 19, 2014 at 04:46:01PM +0100, Rob Jones wrote:
>> Add new managed resource functions as needed to achieve this.
>>
>> Amend the following functions:
> You appear to have omitted the actual patch, I'm quoting the entire
> mail here.  Please see Documentation/SubmittingPatches for some tips
> on how to do things.

I omitted to put "[Patch 0/4][v2]" on the front of the subject line.The 
actual patches followed.

It was a consequence of lack of familiarity with the tools; I'm 
learning, so it shouldn't happen again. Would you like me to resubmit?

>
>> devm_kstrdup()
>> gpio_regulator_probe()
>>
>> Add the following functions:
>>
>> devm_kmemdup()
>> devm_gpio_request_array()
>> devm_gpio_free_array()
>>
>> Remove the following functions:
>>
>> gpio_regulator_remove()
>>
>>
>>  From Rob Jones <rob.jones@codethink.co.uk> # This line is ignored.
>> From: Rob Jones <rob.jones@codethink.co.uk>
>> Subject:
>> In-Reply-To:
>>
>>

-- 
Rob Jones
Project Manager
Codethink Ltd
mailto:rob.jones@codethink.co.uk
tel:+44 161 236 5575


  reply	other threads:[~2014-06-20  8:38 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-06-19 15:46 Change gpio-reulator-probe() to use managed resources Rob Jones
2014-06-19 15:46 ` [PATCH 1/4] drivers/gpio: devres.c: allow gpio array requests for managed devices Rob Jones
2014-06-20 22:29   ` [Linux-kernel] " Ben Hutchings
2014-06-19 15:46 ` [PATCH 2/4] drivers/base: devres.c: Add block copy func. " Rob Jones
2014-06-20 22:31   ` Ben Hutchings
2014-06-20 22:31     ` [Linux-kernel] " Ben Hutchings
2014-06-19 15:46 ` [PATCH 3/4] drivers/base: devres.c: use devm_kmemdup() from with devm_kstrdup() Rob Jones
2014-06-19 15:59   ` Joe Perches
2014-06-20  9:20     ` Rob Jones
2014-06-19 15:46 ` [PATCH 4/4] drivers/regulator: gpio-regulator.c: use managed resources for probe Rob Jones
2014-06-19 17:57 ` Change gpio-reulator-probe() to use managed resources Mark Brown
2014-06-20  8:38   ` Rob Jones [this message]
2014-06-20 10:20     ` Mark Brown

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=53A3F305.9070903@codethink.co.uk \
    --to=rob.jones@codethink.co.uk \
    --cc=ben.dooks@codethink.co.uk \
    --cc=broonie@kernel.org \
    --cc=gnurou@gmail.com \
    --cc=heiko@sntech.de \
    --cc=ian.molton@codethink.co.uk \
    --cc=lgirdwood@gmail.com \
    --cc=linus.walleij@linaro.org \
    --cc=linux-gpio@vger.kernel.org \
    --cc=linux-kernel@codethink.co.uk \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.