All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pavel Machek <pavel@ucw.cz>
To: Bryan Wu <cooloney@gmail.com>
Cc: Jacek Anaszewski <j.anaszewski@samsung.com>,
	Sakari Ailus <sakari.ailus@iki.fi>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	Linux LED Subsystem <linux-leds@vger.kernel.org>,
	"linux-doc@vger.kernel.org" <linux-doc@vger.kernel.org>,
	"rpurdie@rpsys.net" <rpurdie@rpsys.net>,
	Rob Herring <robh+dt@kernel.org>, Pawel Moll <pawel.moll@arm.com>,
	Mark Rutland <mark.rutland@arm.com>,
	Ian Campbell <ijc+devicetree@hellion.org.uk>,
	Kumar Gala <galak@codeaurora.org>, Rob Landley <rob@landley.net>
Subject: Re: [PATCH] Add attributes neccessary for LED flashes to devicetree/bindings/leds/common.txt
Date: Wed, 3 Dec 2014 19:58:22 +0100	[thread overview]
Message-ID: <20141203185822.GA25306@amd> (raw)
In-Reply-To: <CAK5ve-+FBoHD+CPy8GmNDOd3BxvTn_6UB0ZOmjCrEJCpWGyAww@mail.gmail.com>

On Tue 2014-11-25 22:09:49, Bryan Wu wrote:
> On Mon, Nov 24, 2014 at 2:53 PM, Pavel Machek <pavel@ucw.cz> wrote:
> > On Fri 2014-11-21 09:15:31, Jacek Anaszewski wrote:
> >> Hi Pavel,
> >>
> >> On 11/20/2014 09:52 PM, Pavel Machek wrote:
> >> >On Thu 2014-11-20 15:48:26, Jacek Anaszewski wrote:
> >> >>On 11/20/2014 02:38 PM, Jacek Anaszewski wrote:
> >> >>>Hi Pavel,
> >> >>>
> >> >>>On 11/20/2014 02:17 PM, Pavel Machek wrote:
> >> >>>>
> >> >>>>Add attributes neccessary for LED flashes to
> >> >>>>devicetree/bindings/leds/common.txt .
> >> >>>>
> >> >>>>This will allow me to add device tree support for adp1653 i2c flash
> >> >>>>LED driver, and allow Jacek Anaszewski to add support for more LED
> >> >>>>drivers..
> >> >..
> >> >>
> >> >>Actually, we've agreed with Sakari that we can handle
> >> >>indicator-pattern later.
> >> >
> >> >Good.
> >> >
> >> >>I would remove references to you, me and adp1653 driver from the commit
> >> >>message and mention that this modifications adjust the led common
> >> >>bindings to the LED Flash class that is to be added.
> >> >
> >> >Ok, who can take this patch? Can you edit the changelog, or should I
> >> >do it? Anything else that needs changing?
> >>
> >> It's your patch :) For me the contents of the patch are ok.
> >> It requires device tree maintainer approval anyway.
> >
> > Yeah, I was hoping relevant maintainers would speak up. I was not
> > asking you to edit the changelog, sorry I was unclear.
> >
> > Bryan Wu, Richard Purdie: You are the maintainers. Can you take the
> > patch?
> 
> I'm OK for this patch, but since Jacek and Sakari are working on
> pushing LED Flash class and driver into our LED subsystem. I need you
> guys' Ack for this. Then I can take it in my tree.

Ok, patch with the 2 acks is now waiting in your inbox :-).
									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

  parent reply	other threads:[~2014-12-03 18:58 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-20 13:17 [PATCH] Add attributes neccessary for LED flashes to devicetree/bindings/leds/common.txt Pavel Machek
2014-11-20 13:38 ` Jacek Anaszewski
2014-11-20 14:48   ` Jacek Anaszewski
2014-11-20 20:52     ` Pavel Machek
2014-11-21  8:15       ` Jacek Anaszewski
2014-11-24 22:53         ` Pavel Machek
2014-11-26  6:09           ` Bryan Wu
2014-12-03 15:51             ` Pavel Machek
2014-12-03 16:10               ` Sakari Ailus
2014-12-03 18:58             ` Pavel Machek [this message]
2014-11-20 19:19   ` Pavel Machek
2014-11-20 15:03 ` Sakari Ailus
2014-11-20 20:53   ` Pavel Machek
2014-11-21  7:39     ` Sakari Ailus
2014-11-26 12:47       ` Pavel Machek
2014-11-21  7:38   ` Jacek Anaszewski
2014-11-21  7:40     ` Sakari Ailus
2014-12-03 16:03     ` Sakari Ailus
2014-11-26  7:57 ` Jacek Anaszewski
2014-12-03 18:57 ` [PATCH w/ acks] DT: leds: Add flash LED devices related properties Pavel Machek
2014-12-03 19:03   ` Bryan Wu
2014-12-20 15:02     ` Pavel Machek
2014-12-22  2:27   ` Rob Herring
2014-12-22 11:45     ` Pavel Machek

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=20141203185822.GA25306@amd \
    --to=pavel@ucw.cz \
    --cc=cooloney@gmail.com \
    --cc=devicetree@vger.kernel.org \
    --cc=galak@codeaurora.org \
    --cc=ijc+devicetree@hellion.org.uk \
    --cc=j.anaszewski@samsung.com \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-leds@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=pawel.moll@arm.com \
    --cc=rob@landley.net \
    --cc=robh+dt@kernel.org \
    --cc=rpurdie@rpsys.net \
    --cc=sakari.ailus@iki.fi \
    /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.