linux-mips
[Top] [All Lists]

Re: [PATCH v6 3/8] Loongson: YeeLoong: add backlight driver

To: Wu Zhangjin <wuzhangjin@gmail.com>
Subject: Re: [PATCH v6 3/8] Loongson: YeeLoong: add backlight driver
From: Ralf Baechle <ralf@linux-mips.org>
Date: Tue, 1 Dec 2009 14:57:45 +0000
Cc: linux-mips@linux-mips.org, zhangfx@lemote.com, Richard Purdie <rpurdie@rpsys.net>
In-reply-to: <1259679137.12571.4.camel@falcon>
Original-recipient: rfc822;linux-mips@linux-mips.org
References: <cover.1259664573.git.wuzhangjin@gmail.com> <4328ee6b15dce3cb600dddf4e7151532ddc77f17.1259664573.git.wuzhangjin@gmail.com> <20091201140643.GC14064@linux-mips.org> <1259679137.12571.4.camel@falcon>
Sender: linux-mips-bounce@linux-mips.org
User-agent: Mutt/1.5.20 (2009-08-17)
On Tue, Dec 01, 2009 at 10:52:17PM +0800, Wu Zhangjin wrote:

> On Tue, 2009-12-01 at 14:06 +0000, Ralf Baechle wrote:
> > On Tue, Dec 01, 2009 at 07:08:42PM +0800, Wu Zhangin wrote:
> > 
> > > From: Wu Zhangjin <wuzhangjin@gmail.com>
> > > 
> > > This patch adds YeeLoong Backlight Driver, which provides standard
> > > interface for user-space applications to control the brightness of the
> > > backlight.
> > > 
> > > Signed-off-by: Wu Zhangjin <wuzhangjin@gmail.com>
> > 
> > You split old, big driver into several individual drivers - good.
> > 
> > Now we can actually move things to their rightf place and for a backlight
> > drivers that should be drivers/video/backlight/.  Convert it to a platform
> > driver.
> 
> Okay, but for these drivers need the ec_kb3310b.h, so, we also need to
> move it from arch/mips/loongson/lemote-2f/yeeloong_laptop/ec_kb3310b.h
> to arch/mips/include/asm/mach-loongson/ec_kb3310b.h.
> 
> and seems some subsystem have no maintainer, such as the hwmon driver:

Which is true and sometimes a bit of a nuisance but one we can live with.

> HARDWARE MONITORING
> L:      lm-sensors@lm-sensors.org
> W:      http://www.lm-sensors.org/
> S:      Orphan
> F:      drivers/hwmon/
> 
> So, who should I send this patch to?

I suggest you add Andrew Morton <akpm@linux-foundation.org> and linux-kernel
to the To: list.

  Ralf

<Prev in Thread] Current Thread [Next in Thread>