[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <50FCA346.2070608@gmail.com>
Date: Mon, 21 Jan 2013 10:09:10 +0800
From: Mark Zhang <nvmarkzhang@...il.com>
To: Alexandre Courbot <acourbot@...dia.com>
CC: Thierry Reding <thierry.reding@...onic-design.de>,
Stephen Warren <swarren@...dotorg.org>,
linux-fbdev@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-tegra@...r.kernel.org, Mark Zhang <markz@...dia.com>,
gnurou@...il.com
Subject: Re: [PATCH 0/3] pwm-backlight: add subdrivers & Tegra support
Hi Alex,
This patch set applies failed on tot linux-next(0118). Here is the log:
markz@...kz-hp6200:~/tegradrm/official-upstream-kernel$ git am
~/Desktop/*.eml
Applying: pwm-backlight: add subdriver mechanism
error: patch failed: drivers/video/backlight/pwm_bl.c:35
error: drivers/video/backlight/pwm_bl.c: patch does not apply
Patch failed at 0001 pwm-backlight: add subdriver mechanism
When you have resolved this problem run "git am --resolved".
If you would prefer to skip this patch, instead run "git am --skip".
To restore the original branch and stop patching run "git am --abort".
Anyway, I'll try to apply this on 3.8-rc4.
Mark
On 01/19/2013 06:30 PM, Alexandre Courbot wrote:
> This series introduces a way to use pwm-backlight hooks with platforms
> that use the device tree through a subdriver system. It also adds support
> for the Tegra-based Ventana board, adding the last missing block to enable
> its panel. Support for other Tegra board can thus be easily added.
>
> I have something else in mind to properly support this (power
> sequences), but this work relies on the GPIO subsystem redesign which will
> take some time. The pwm-backlight subdrivers can do the job by the meantime.
>
> There are a few design points that might need to be discussed:
> 1) Link order is important: subdrivers register themselves in their
> module_init function, which must be called before pwm-backlight's probe.
> This forbids linking subdrivers as separate modules from pwm-backlight.
> 2) The subdriver's data is temporarily passed through the backlight
> device's driver data. This should not hurt, but maybe there is a better way
> to do this.
> 3) Subdrivers must add themselves into pwm-backlight's own of_device_id
> table. It would be cleaner to not have to list subdrivers into
> pwm-backlight's main file, but I cannot think of a way to do otherwise.
>
> Suggestions for the 3 points listed above are very welcome - in any case,
> I hope to make this converge into something mergeable quickly.
>
> Note that these patches are the last missing block to get a functional
> panel on Tegra boards. Using 3.8rc4 and these patches, the internal panel
> on Ventana is usable out-of-the-box. Yay.
>
> Alexandre Courbot (3):
> pwm-backlight: add subdriver mechanism
> tegra: pwm-backlight: add tegra pwm-bl driver
> tegra: ventana: of: add host1x device to DT
>
> arch/arm/boot/dts/tegra20-ventana.dts | 29 +++++-
> arch/arm/configs/tegra_defconfig | 1 +
> drivers/video/backlight/Kconfig | 7 ++
> drivers/video/backlight/Makefile | 4 +
> drivers/video/backlight/pwm_bl.c | 70 ++++++++++++++-
> drivers/video/backlight/pwm_bl_tegra.c | 159 +++++++++++++++++++++++++++++++++
> include/linux/pwm_backlight.h | 15 ++++
> 7 files changed, 281 insertions(+), 4 deletions(-)
> create mode 100644 drivers/video/backlight/pwm_bl_tegra.c
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists