From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Horman Date: Wed, 10 Jul 2013 01:22:34 +0000 Subject: Re: [PATCH v2 0/7] Backlight drivers for SH and ARM SH-Mobile boards Message-Id: <20130710012234.GR32603@verge.net.au> List-Id: References: <1372965210-10024-1-git-send-email-laurent.pinchart+renesas@ideasonboard.com> In-Reply-To: <1372965210-10024-1-git-send-email-laurent.pinchart+renesas@ideasonboard.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-sh@vger.kernel.org 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 >