From: Thierry Reding <thierry.reding@gmail.com>
To: Alexandre Courbot <gnurou@gmail.com>
Cc: Kukjin Kim <kgene.kim@samsung.com>,
Jingoo Han <jg1.han@samsung.com>, Ben Dooks <ben-linux@fluff.org>,
Alexandre Courbot <acourbot@nvidia.com>,
Bryan Wu <cooloney@gmail.com>,
"linux-pwm@vger.kernel.org" <linux-pwm@vger.kernel.org>,
"linux-gpio@vger.kernel.org" <linux-gpio@vger.kernel.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 1/2] ARM: SAMSUNG: remove gpio flags in dev-backlight
Date: Tue, 6 May 2014 17:17:39 +0200 [thread overview]
Message-ID: <20140506151736.GA7176@ulmo> (raw)
In-Reply-To: <CAAVeFuJYrez9Ugqerys9Un0jRHnwMcwZ9BxnswbF=2m_LpMLSw@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 1807 bytes --]
On Tue, May 06, 2014 at 11:33:00PM +0900, Alexandre Courbot wrote:
> On Tue, May 6, 2014 at 1:59 PM, Kukjin Kim <kgene.kim@samsung.com> wrote:
> > Alexandre Courbot wrote:
> >>
> >> On Thu, Apr 10, 2014 at 11:14 PM, Alexandre Courbot <gnurou@gmail.com>
> >> wrote:
> >> > On Thu, Apr 10, 2014 at 6:51 PM, Jingoo Han <jg1.han@samsung.com> wrote:
> >> >> On Thursday, April 10, 2014 1:17 PM, Alexandre Courbot wrote:
> >> >>>
> >> >>> Ping, can I have the Samsung folks review and ,aybe even merge this
> >> >>> patch? enable_gpio_flags is never used in any code, is replaced by
> >> >>> gpiod, and we would like to remove it altogether from pwm_bl. Thanks!
> >> >>
> >> >> OK, I see. It looks good.
> >> >>
> >> >> As far as I know, 'enable_gpio_flags' has not been being used
> >> >> for Samsung platform. So, there will be no side effect,
> >> >> if 'enable_gpio_flags' is removed from 'arch/arm/plat-samsung'
> >> >> directory.
> >> >>
> >> >> Reviewed-by: Jingoo Han <jg1.han@samsung.com>
> >> >
> >> > Great, thanks. Ben, Kukjin, could we have your Acked-by?
> >> >
> >> > Thierry, if the Samsung maintainers are ok with it, and 2/2 of this
> >> > series is also ok for you (you merged the same for simple-panel
> >> > already), can you take both into your tree?
> >>
> >> Last call - could we have a Acked-by from Ben or Kukjin and merge this
> >> through Thierry's tree? Otherwise I will just have to drop this
> >> series, which would be sad.
> >
> > Very sorry for late response. I missed... :(
> >
> > Looks good to me, so please go ahead with my ack.
>
> Thanks Kukjin!
>
> Thierry, do you want me to rebase and resend these (now old) patches?
> I haven't heard anything from you about them yet.
No, that's fine. I can handle them.
Thanks,
Thierry
[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
next prev parent reply other threads:[~2014-05-06 15:19 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-02-27 5:53 [PATCH 0/2] pwm-backlight: switch to gpiod interface (part 1) Alexandre Courbot
2014-02-27 5:53 ` Alexandre Courbot
2014-02-27 5:53 ` [PATCH 1/2] ARM: SAMSUNG: remove gpio flags in dev-backlight Alexandre Courbot
2014-02-27 5:53 ` Alexandre Courbot
2014-04-10 4:17 ` Alexandre Courbot
2014-04-10 9:51 ` Jingoo Han
2014-04-10 14:14 ` Alexandre Courbot
2014-04-21 6:07 ` Alexandre Courbot
2014-05-06 4:59 ` Kukjin Kim
2014-05-06 14:33 ` Alexandre Courbot
2014-05-06 15:17 ` Thierry Reding [this message]
2014-05-07 8:14 ` Thierry Reding
2014-02-27 5:53 ` [PATCH 2/2] pwm-backlight: switch to gpiod interface Alexandre Courbot
2014-02-27 5:53 ` Alexandre Courbot
2014-05-07 8:18 ` Thierry Reding
2014-05-07 8:39 ` Alexandre Courbot
2014-03-26 5:01 ` [PATCH 0/2] pwm-backlight: switch to gpiod interface (part 1) Alexandre Courbot
2014-03-27 0:08 ` Bryan Wu
2014-03-27 1:04 ` Jingoo Han
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=20140506151736.GA7176@ulmo \
--to=thierry.reding@gmail.com \
--cc=acourbot@nvidia.com \
--cc=ben-linux@fluff.org \
--cc=cooloney@gmail.com \
--cc=gnurou@gmail.com \
--cc=jg1.han@samsung.com \
--cc=kgene.kim@samsung.com \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pwm@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.