All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andrew Lunn <andrew@lunn.ch>
To: Simon Guinot <simon.guinot@sequanux.org>
Cc: Gregory CLEMENT <gregory.clement@free-electrons.com>,
	Bryan Wu <cooloney@gmail.com>, Richard Purdie <rpurdie@rpsys.net>,
	Jason Cooper <jason@lakedaemon.net>,
	Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
	linux-arm-kernel@lists.infradead.org, linux-leds@vger.kernel.org,
	Vincent Donnefort <vdonnefort@gmail.com>
Subject: Re: [PATCH 0/3] Add DT support for netxbig LEDs
Date: Mon, 30 Mar 2015 14:30:08 +0200	[thread overview]
Message-ID: <20150330123008.GC17650@lunn.ch> (raw)
In-Reply-To: <20150330083040.GY1509@kw.sim.vm.gnt>

On Mon, Mar 30, 2015 at 10:30:40AM +0200, Simon Guinot wrote:
> On Mon, Mar 16, 2015 at 05:01:10PM +0100, Gregory CLEMENT wrote:
> > Hi all,
> > 
> > On 09/03/2015 10:41, Simon Guinot wrote:
> > > On Mon, Feb 23, 2015 at 04:04:18PM +0100, Simon Guinot wrote:
> > >> Hello,
> > >>
> > >> This patch series adds DT support for the LEDs found on the Kirkwood-based
> > >> LaCie boards 2Big and 5Big Network v2.
> > >>
> > >> Simon
> > >>
> > >> Simon Guinot (3):
> > >>   leds: netxbig: add device tree binding
> > >>   ARM: Kirkwood: add LED DT entries for netxbig boards
> > >>   ARM: mvebu: remove static LED setup for netxbig boards
> > > 
> > > Hi Brian,
> > > 
> > > Please, consider merging the first patch of this serie.
> > 
> > Brian,
> > 
> > here again let me know when this patch will be merged, and I will
> > also need a stable tag as patch 2 depends on this one.
> > 
> > Andrew, Jason,
> > 
> > patch 2 and 3 looks OK for me. What is your opinion about it?
> 
> Hi Gregory, Andrew and Jason,
> 
> This patch set seems stuck. I believe that Bryan is probably too busy to
> handle it...
> 
> Is that possible to merge it via the mvebu tree ? Since only some LaCie
> boards are impacted, I think it could make sense.

Hi Simon

Hard one. I also have a driver stuck in led limbo.

Last time we took a fix via mvebu, it all went messy at the last
minute. We made it very clear to Bryon we planned to take a patch via
mvebu, we had queued it via mvebu, we had sent a pull request to
arm-soc, all with no reply from Bryon. Only at the last minute did he
jump in, pull it himself and send it to Linus. That caused Olof all
sorts of problems with his tree.

To stop that happening again, i think we need an Acked-by from Bryan.

   Andrew

WARNING: multiple messages have this Message-ID (diff)
From: andrew@lunn.ch (Andrew Lunn)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/3] Add DT support for netxbig LEDs
Date: Mon, 30 Mar 2015 14:30:08 +0200	[thread overview]
Message-ID: <20150330123008.GC17650@lunn.ch> (raw)
In-Reply-To: <20150330083040.GY1509@kw.sim.vm.gnt>

On Mon, Mar 30, 2015 at 10:30:40AM +0200, Simon Guinot wrote:
> On Mon, Mar 16, 2015 at 05:01:10PM +0100, Gregory CLEMENT wrote:
> > Hi all,
> > 
> > On 09/03/2015 10:41, Simon Guinot wrote:
> > > On Mon, Feb 23, 2015 at 04:04:18PM +0100, Simon Guinot wrote:
> > >> Hello,
> > >>
> > >> This patch series adds DT support for the LEDs found on the Kirkwood-based
> > >> LaCie boards 2Big and 5Big Network v2.
> > >>
> > >> Simon
> > >>
> > >> Simon Guinot (3):
> > >>   leds: netxbig: add device tree binding
> > >>   ARM: Kirkwood: add LED DT entries for netxbig boards
> > >>   ARM: mvebu: remove static LED setup for netxbig boards
> > > 
> > > Hi Brian,
> > > 
> > > Please, consider merging the first patch of this serie.
> > 
> > Brian,
> > 
> > here again let me know when this patch will be merged, and I will
> > also need a stable tag as patch 2 depends on this one.
> > 
> > Andrew, Jason,
> > 
> > patch 2 and 3 looks OK for me. What is your opinion about it?
> 
> Hi Gregory, Andrew and Jason,
> 
> This patch set seems stuck. I believe that Bryan is probably too busy to
> handle it...
> 
> Is that possible to merge it via the mvebu tree ? Since only some LaCie
> boards are impacted, I think it could make sense.

Hi Simon

Hard one. I also have a driver stuck in led limbo.

Last time we took a fix via mvebu, it all went messy at the last
minute. We made it very clear to Bryon we planned to take a patch via
mvebu, we had queued it via mvebu, we had sent a pull request to
arm-soc, all with no reply from Bryon. Only at the last minute did he
jump in, pull it himself and send it to Linus. That caused Olof all
sorts of problems with his tree.

To stop that happening again, i think we need an Acked-by from Bryan.

   Andrew

  reply	other threads:[~2015-03-30 12:34 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-23 15:04 [PATCH 0/3] Add DT support for netxbig LEDs Simon Guinot
2015-02-23 15:04 ` Simon Guinot
2015-02-23 15:04 ` [PATCH 1/3] leds: netxbig: add device tree binding Simon Guinot
2015-02-23 15:04   ` Simon Guinot
2015-02-23 15:04 ` [PATCH 2/3] ARM: Kirkwood: add LED DT entries for netxbig boards Simon Guinot
2015-02-23 15:04   ` Simon Guinot
2015-02-23 15:04 ` [PATCH 3/3] ARM: mvebu: remove static LED setup " Simon Guinot
2015-02-23 15:04   ` Simon Guinot
2015-03-09  9:41 ` [PATCH 0/3] Add DT support for netxbig LEDs Simon Guinot
2015-03-09  9:41   ` Simon Guinot
2015-03-16 16:01   ` Gregory CLEMENT
2015-03-16 16:01     ` Gregory CLEMENT
2015-03-30  8:30     ` Simon Guinot
2015-03-30  8:30       ` Simon Guinot
2015-03-30 12:30       ` Andrew Lunn [this message]
2015-03-30 12:30         ` Andrew Lunn
2015-03-30 13:22         ` Jason Cooper
2015-03-30 13:22           ` Jason Cooper
2015-03-30 13:29           ` Gregory CLEMENT
2015-03-30 13:29             ` Gregory CLEMENT
2015-03-30 13:46             ` Jason Cooper
2015-03-30 13:46               ` Jason Cooper
2015-03-30 14:20               ` Gregory CLEMENT
2015-03-30 14:20                 ` Gregory CLEMENT
2015-03-23 10:31   ` Simon Guinot
2015-03-23 10:31     ` Simon Guinot

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=20150330123008.GC17650@lunn.ch \
    --to=andrew@lunn.ch \
    --cc=cooloney@gmail.com \
    --cc=gregory.clement@free-electrons.com \
    --cc=jason@lakedaemon.net \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-leds@vger.kernel.org \
    --cc=rpurdie@rpsys.net \
    --cc=sebastian.hesselbarth@gmail.com \
    --cc=simon.guinot@sequanux.org \
    --cc=vdonnefort@gmail.com \
    /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.