All of lore.kernel.org
 help / color / mirror / Atom feed
From: Simon Horman <horms@verge.net.au>
To: linux-sh@vger.kernel.org
Subject: Re: [PATCH v2 0/7] Backlight drivers for SH and ARM SH-Mobile boards
Date: Wed, 10 Jul 2013 01:22:34 +0000	[thread overview]
Message-ID: <20130710012234.GR32603@verge.net.au> (raw)
In-Reply-To: <1372965210-10024-1-git-send-email-laurent.pinchart+renesas@ideasonboard.com>

On Wed, Jul 10, 2013 at 10:07:45AM +0900, Jingoo Han wrote:
> On Friday, July 05, 2013 4:13 AM, Laurent Pinchart wrote:
> > 
> > Hello,
> > 
> > Here are three backlight drivers used by SH and ARM SH-Mobile boards.
> > 
> > The first driver supports simple GPIO-controlled backlights that are turned on
> > or off by a single GPIO (either active high or active low).
> > 
> > The second driver supports the Sanyo LV5207LP LED driver
> > (http://www.onsemi.com/pub_link/Collateral/ENA0943-D.PDF) and exposes the main
> > LED only as a backlight device, as that's all the board I need to driver for
> > uses.
> > 
> > The third driver supports the ROHM Semiconductor BD6107 LED driver (no
> > datasheet publicly available I'm afraid). Similarly to the LV5207LP driver, it
> > exposes the main LED only as a backlight device.
> > 
> > The last four patches port the ARM SH-Mobile Mackerel and AG5EVM and the SH
> > Ecovec24 and KFR2R09 boards to use the new backlight drivers. The code has been
> > compile tested on all four platforms and runtime tested on Mackerel only as I
> > don't have access to the other boards.
> > 
> > As the board code patches depend on the backlight drivers, the drivers should
> > get merged first. Richard, Jingoo, would you like to take them through your
> > tree, or would it be easier to merge them with the ARM and SH patches ?
> 
> Hi Laurent,
> 
> Currently, I cannot use my git tree for backlight subsystem.
> Also, your patches cannot affect other backlight drivers.
> 
> I think that these 7 patches would go through SH tree with my Acked-by.
> I will review your patches.

That is fine by me.

Laurent, can you please let me know what base - e.g. v3.11-rc1 -
is required for the non-shmobile patches?

> 
> Thank you.
> 
> Best regards,
> Jingoo Han
> 
> > 
> > Laurent Pinchart (7):
> >   backlight: Add GPIO-based backlight driver
> >   backlight: Add Sanyo LV5207LP backlight driver
> >   backlight: Add ROHM BD6107 backlight driver
> >   ARM: mach-shmobile: mackerel: Use gpio-backlight
> >   sh: ecovec24: Use gpio-backlight
> >   sh: kfr2r09: Use lv5207lp backlight
> >   ARM: mach-shmobile: ag5evm: Use bd6107 backlight driver
> > 
> >  arch/arm/mach-shmobile/board-ag5evm.c        |  65 ++------
> >  arch/arm/mach-shmobile/board-mackerel.c      |  34 ++---
> >  arch/sh/boards/mach-ecovec24/setup.c         |  37 ++---
> >  arch/sh/boards/mach-kfr2r09/lcd_wqvga.c      |  48 ------
> >  arch/sh/boards/mach-kfr2r09/setup.c          |  19 ++-
> >  arch/sh/include/mach-kfr2r09/mach/kfr2r09.h  |   2 -
> >  drivers/video/backlight/Kconfig              |  19 +++
> >  drivers/video/backlight/Makefile             |   3 +
> >  drivers/video/backlight/bd6107.c             | 213 +++++++++++++++++++++++++++
> >  drivers/video/backlight/gpio_backlight.c     | 133 +++++++++++++++++
> >  drivers/video/backlight/lv5207lp.c           | 171 +++++++++++++++++++++
> >  include/linux/platform_data/bd6107.h         |  19 +++
> >  include/linux/platform_data/gpio_backlight.h |  21 +++
> >  include/linux/platform_data/lv5207lp.h       |  19 +++
> >  14 files changed, 665 insertions(+), 138 deletions(-)
> >  create mode 100644 drivers/video/backlight/bd6107.c
> >  create mode 100644 drivers/video/backlight/gpio_backlight.c
> >  create mode 100644 drivers/video/backlight/lv5207lp.c
> >  create mode 100644 include/linux/platform_data/bd6107.h
> >  create mode 100644 include/linux/platform_data/gpio_backlight.h
> >  create mode 100644 include/linux/platform_data/lv5207lp.h
> > 
> > --
> > Regards,
> > 
> > Laurent Pinchart
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sh" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

  parent reply	other threads:[~2013-07-10  1:22 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-04 19:13 [PATCH v2 0/7] Backlight drivers for SH and ARM SH-Mobile boards Laurent Pinchart
2013-07-10  1:07 ` Jingoo Han
2013-07-10  1:22 ` Simon Horman [this message]
2013-07-10  1:55 ` Jingoo Han
2013-07-10  9:17 ` Laurent Pinchart
2013-07-10 12:21 ` Simon Horman
2013-07-16  5:08 ` Simon Horman

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=20130710012234.GR32603@verge.net.au \
    --to=horms@verge.net.au \
    --cc=linux-sh@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.