* [PATCH] ARM: tegra: harmony: Register and configure WM8903 IRQ GPIO
@ 2011-03-11 20:25 Stephen Warren
2011-04-05 21:12 ` Stephen Warren
0 siblings, 1 reply; 2+ messages in thread
From: Stephen Warren @ 2011-03-11 20:25 UTC (permalink / raw)
To: linux-arm-kernel
Technically, we need to request and configure the GPIO used as the WM8903
interrupt. This prevents conflicting registrations, and assures that the
GPIO is correctly configured in all cases, e.g. if the bootloader left the
GPIO in some unexpected state.
In practice, the previous code works as-is, at least when using ChromeOS's
U-Boot as the boot-loader.
Signed-off-by: Stephen Warren <swarren@nvidia.com>
---
arch/arm/mach-tegra/board-harmony.c | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/arch/arm/mach-tegra/board-harmony.c b/arch/arm/mach-tegra/board-harmony.c
index e7c2fea..91178a6 100644
--- a/arch/arm/mach-tegra/board-harmony.c
+++ b/arch/arm/mach-tegra/board-harmony.c
@@ -130,6 +130,9 @@ static void __init harmony_i2c_init(void)
platform_device_register(&tegra_i2c_device3);
platform_device_register(&tegra_i2c_device4);
+ gpio_request(TEGRA_GPIO_CDC_IRQ, "wm8903");
+ gpio_direction_input(TEGRA_GPIO_CDC_IRQ);
+
i2c_register_board_info(0, &wm8903_board_info, 1);
}
--
1.7.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [PATCH] ARM: tegra: harmony: Register and configure WM8903 IRQ GPIO
2011-03-11 20:25 [PATCH] ARM: tegra: harmony: Register and configure WM8903 IRQ GPIO Stephen Warren
@ 2011-04-05 21:12 ` Stephen Warren
0 siblings, 0 replies; 2+ messages in thread
From: Stephen Warren @ 2011-04-05 21:12 UTC (permalink / raw)
To: linux-arm-kernel
Stephen Warren wrote at Friday, March 11, 2011 1:26 PM:
> Technically, we need to request and configure the GPIO used as the WM8903
> interrupt. This prevents conflicting registrations, and assures that the
> GPIO is correctly configured in all cases, e.g. if the bootloader left the
> GPIO in some unexpected state.
>
> In practice, the previous code works as-is, at least when using ChromeOS's
> U-Boot as the boot-loader.
>
> Signed-off-by: Stephen Warren <swarren@nvidia.com>
Olof,
I think this patch got missed. It's not especially important in practice,
but the code may as well be complete.
Thanks.
> ---
> arch/arm/mach-tegra/board-harmony.c | 3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/mach-tegra/board-harmony.c b/arch/arm/mach-
> tegra/board-harmony.c
> index e7c2fea..91178a6 100644
> --- a/arch/arm/mach-tegra/board-harmony.c
> +++ b/arch/arm/mach-tegra/board-harmony.c
> @@ -130,6 +130,9 @@ static void __init harmony_i2c_init(void)
> platform_device_register(&tegra_i2c_device3);
> platform_device_register(&tegra_i2c_device4);
>
> + gpio_request(TEGRA_GPIO_CDC_IRQ, "wm8903");
> + gpio_direction_input(TEGRA_GPIO_CDC_IRQ);
> +
> i2c_register_board_info(0, &wm8903_board_info, 1);
> }
>
> --
> 1.7.1
--
nvpublic
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2011-04-05 21:12 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-03-11 20:25 [PATCH] ARM: tegra: harmony: Register and configure WM8903 IRQ GPIO Stephen Warren
2011-04-05 21:12 ` Stephen Warren
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).