* [PATCH v4 0/3] OMAP3: beaglexm: GPIO fixes
[not found] <[PATCH v3 0/3] OMAP3: beaglexm: GPIO fixes>
@ 2011-01-07 23:01 ` Nishanth Menon
2011-01-07 23:01 ` Nishanth Menon
` (2 subsequent siblings)
3 siblings, 0 replies; 22+ messages in thread
From: Nishanth Menon @ 2011-01-07 23:01 UTC (permalink / raw)
To: l-o; +Cc: l-a, Kevin, Tony, Koen Kooi, Aaro, Nishanth Menon
Hi,
As discussed in the threads:
http://thread.gmane.org/gmane.linux.ports.arm.omap/47807/
http://marc.info/?t=121540030800004&r=1&w=2
here is the split up series with commit message after discussion:
http://www.beagleboard.org/irclogs/index.php?date=2011-01-06#T19:12:21
Koen Kooi (3):
omap3: beaglexm: fix EHCI power up GPIO dir
omap3: beaglexm: fix DVI reset GPIO
omap3: beaglexm: fix power on of DVI
arch/arm/mach-omap2/board-omap3beagle.c | 37 ++++++++++++++++++++++++++----
1 files changed, 32 insertions(+), 5 deletions(-)
v4:
no functional change.
minor cleanups in commit logs incorporating offline feedback
v3: http://marc.info/?t=129434384700006&r=1&w=2
split up the series, addressed review comments
v2: http://marc.info/?t=129276977900002&r=1&w=2
Reenable the PMU stat LED
v1: http://marc.info/?t=129172571000001&r=1&w=2
Regards,
Nishanth Menon
^ permalink raw reply [flat|nested] 22+ messages in thread* [PATCH v4 0/3] OMAP3: beaglexm: GPIO fixes
@ 2011-01-07 23:01 ` Nishanth Menon
0 siblings, 0 replies; 22+ messages in thread
From: Nishanth Menon @ 2011-01-07 23:01 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
As discussed in the threads:
http://thread.gmane.org/gmane.linux.ports.arm.omap/47807/
http://marc.info/?t=121540030800004&r=1&w=2
here is the split up series with commit message after discussion:
http://www.beagleboard.org/irclogs/index.php?date=2011-01-06#T19:12:21
Koen Kooi (3):
omap3: beaglexm: fix EHCI power up GPIO dir
omap3: beaglexm: fix DVI reset GPIO
omap3: beaglexm: fix power on of DVI
arch/arm/mach-omap2/board-omap3beagle.c | 37 ++++++++++++++++++++++++++----
1 files changed, 32 insertions(+), 5 deletions(-)
v4:
no functional change.
minor cleanups in commit logs incorporating offline feedback
v3: http://marc.info/?t=129434384700006&r=1&w=2
split up the series, addressed review comments
v2: http://marc.info/?t=129276977900002&r=1&w=2
Reenable the PMU stat LED
v1: http://marc.info/?t=129172571000001&r=1&w=2
Regards,
Nishanth Menon
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v4 0/3] OMAP3: beaglexm: GPIO fixes
2011-01-07 23:01 ` Nishanth Menon
@ 2011-01-07 23:17 ` Kevin Hilman
-1 siblings, 0 replies; 22+ messages in thread
From: Kevin Hilman @ 2011-01-07 23:17 UTC (permalink / raw)
To: Nishanth Menon; +Cc: l-o, l-a, Tony, Koen Kooi, Aaro
Nishanth Menon <nm@ti.com> writes:
> As discussed in the threads:
> http://thread.gmane.org/gmane.linux.ports.arm.omap/47807/
> http://marc.info/?t=121540030800004&r=1&w=2
>
> here is the split up series with commit message after discussion:
> http://www.beagleboard.org/irclogs/index.php?date=2011-01-06#T19:12:21
>
> Koen Kooi (3):
> omap3: beaglexm: fix EHCI power up GPIO dir
> omap3: beaglexm: fix DVI reset GPIO
> omap3: beaglexm: fix power on of DVI
>
> arch/arm/mach-omap2/board-omap3beagle.c | 37 ++++++++++++++++++++++++++----
> 1 files changed, 32 insertions(+), 5 deletions(-)
>
> v4:
> no functional change.
> minor cleanups in commit logs incorporating offline feedback
Thanks for the updates. Will queue in my fixes queue.
Kevin
> v3: http://marc.info/?t=129434384700006&r=1&w=2
> split up the series, addressed review comments
>
> v2: http://marc.info/?t=129276977900002&r=1&w=2
> Reenable the PMU stat LED
> v1: http://marc.info/?t=129172571000001&r=1&w=2
>
> Regards,
> Nishanth Menon
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v4 0/3] OMAP3: beaglexm: GPIO fixes
2011-01-07 23:01 ` Nishanth Menon
@ 2011-01-10 19:04 ` Tony Lindgren
-1 siblings, 0 replies; 22+ messages in thread
From: Tony Lindgren @ 2011-01-10 19:04 UTC (permalink / raw)
To: Nishanth Menon; +Cc: l-o, l-a, Kevin, Koen Kooi, Aaro
* Nishanth Menon <nm@ti.com> [110107 15:00]:
> Hi,
> As discussed in the threads:
> http://thread.gmane.org/gmane.linux.ports.arm.omap/47807/
> http://marc.info/?t=121540030800004&r=1&w=2
>
> here is the split up series with commit message after discussion:
> http://www.beagleboard.org/irclogs/index.php?date=2011-01-06#T19:12:21
>
> Koen Kooi (3):
> omap3: beaglexm: fix EHCI power up GPIO dir
> omap3: beaglexm: fix DVI reset GPIO
> omap3: beaglexm: fix power on of DVI
Dropping the earlier versions from omap-fixes, I want to get the
build fixes in ASAP. These can wait a bit until Aaro's comments
are addressed.
Regards,
Tony
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH v4 0/3] OMAP3: beaglexm: GPIO fixes
@ 2011-01-10 19:04 ` Tony Lindgren
0 siblings, 0 replies; 22+ messages in thread
From: Tony Lindgren @ 2011-01-10 19:04 UTC (permalink / raw)
To: linux-arm-kernel
* Nishanth Menon <nm@ti.com> [110107 15:00]:
> Hi,
> As discussed in the threads:
> http://thread.gmane.org/gmane.linux.ports.arm.omap/47807/
> http://marc.info/?t=121540030800004&r=1&w=2
>
> here is the split up series with commit message after discussion:
> http://www.beagleboard.org/irclogs/index.php?date=2011-01-06#T19:12:21
>
> Koen Kooi (3):
> omap3: beaglexm: fix EHCI power up GPIO dir
> omap3: beaglexm: fix DVI reset GPIO
> omap3: beaglexm: fix power on of DVI
Dropping the earlier versions from omap-fixes, I want to get the
build fixes in ASAP. These can wait a bit until Aaro's comments
are addressed.
Regards,
Tony
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v4 0/3] OMAP3: beaglexm: GPIO fixes
2011-01-10 19:04 ` Tony Lindgren
@ 2011-01-10 19:13 ` Nishanth Menon
-1 siblings, 0 replies; 22+ messages in thread
From: Nishanth Menon @ 2011-01-10 19:13 UTC (permalink / raw)
To: Tony Lindgren; +Cc: l-o, l-a, Kevin, Koen Kooi, Aaro
Tony Lindgren had written, on 01/10/2011 01:04 PM, the following:
> * Nishanth Menon <nm@ti.com> [110107 15:00]:
>> Hi,
>> As discussed in the threads:
>> http://thread.gmane.org/gmane.linux.ports.arm.omap/47807/
>> http://marc.info/?t=121540030800004&r=1&w=2
>>
>> here is the split up series with commit message after discussion:
>> http://www.beagleboard.org/irclogs/index.php?date=2011-01-06#T19:12:21
>>
>> Koen Kooi (3):
>> omap3: beaglexm: fix EHCI power up GPIO dir
>> omap3: beaglexm: fix DVI reset GPIO
>> omap3: beaglexm: fix power on of DVI
>
> Dropping the earlier versions from omap-fixes, I want to get the
> build fixes in ASAP. These can wait a bit until Aaro's comments
> are addressed.
Ack. will post a v5 after aligning. build breakages have higher priority
ofcourse ;)
--
Regards,
Nishanth Menon
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH v4 1/3] omap3: beaglexm: fix EHCI power up GPIO dir
[not found] <[PATCH v3 0/3] OMAP3: beaglexm: GPIO fixes>
@ 2011-01-07 23:01 ` Nishanth Menon
2011-01-07 23:01 ` Nishanth Menon
` (2 subsequent siblings)
3 siblings, 0 replies; 22+ messages in thread
From: Nishanth Menon @ 2011-01-07 23:01 UTC (permalink / raw)
To: l-o; +Cc: l-a, Kevin, Tony, Koen Kooi, Aaro, Nishanth Menon
From: Koen Kooi <koen@beagleboard.org>
EHCI enable power pin is inverted (active high) in comparison
to vanilla beagle which is active low. Handle this case conditionally.
Without this fix, Beagle XM 4 port EHCI will not function and no
networking will be available
[nm@ti.com: split up, added descriptive changelogs]
Signed-off-by: Nishanth Menon <nm@ti.com>
Signed-off-by: Koen Kooi <koen@beagleboard.org>
---
arch/arm/mach-omap2/board-omap3beagle.c | 9 ++++++++-
1 files changed, 8 insertions(+), 1 deletions(-)
diff --git a/arch/arm/mach-omap2/board-omap3beagle.c b/arch/arm/mach-omap2/board-omap3beagle.c
index 6c12760..1b5aa7a 100644
--- a/arch/arm/mach-omap2/board-omap3beagle.c
+++ b/arch/arm/mach-omap2/board-omap3beagle.c
@@ -297,9 +297,16 @@ static int beagle_twl_gpio_setup(struct device *dev,
gpio_request(gpio + 1, "EHCI_nOC");
gpio_direction_input(gpio + 1);
- /* TWL4030_GPIO_MAX + 0 == ledA, EHCI nEN_USB_PWR (out, active low) */
+ /*
+ * TWL4030_GPIO_MAX + 0 == ledA, EHCI nEN_USB_PWR (out, XM active
+ * high / others active low)
+ */
gpio_request(gpio + TWL4030_GPIO_MAX, "nEN_USB_PWR");
gpio_direction_output(gpio + TWL4030_GPIO_MAX, 0);
+ if (omap3_beagle_get_rev() == OMAP3BEAGLE_BOARD_XM)
+ gpio_direction_output(gpio + TWL4030_GPIO_MAX, 1);
+ else
+ gpio_direction_output(gpio + TWL4030_GPIO_MAX, 0);
/* TWL4030_GPIO_MAX + 1 == ledB, PMU_STAT (out, active low LED) */
gpio_leds[2].gpio = gpio + TWL4030_GPIO_MAX + 1;
--
1.6.3.3
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH v4 1/3] omap3: beaglexm: fix EHCI power up GPIO dir
@ 2011-01-07 23:01 ` Nishanth Menon
0 siblings, 0 replies; 22+ messages in thread
From: Nishanth Menon @ 2011-01-07 23:01 UTC (permalink / raw)
To: linux-arm-kernel
From: Koen Kooi <koen@beagleboard.org>
EHCI enable power pin is inverted (active high) in comparison
to vanilla beagle which is active low. Handle this case conditionally.
Without this fix, Beagle XM 4 port EHCI will not function and no
networking will be available
[nm at ti.com: split up, added descriptive changelogs]
Signed-off-by: Nishanth Menon <nm@ti.com>
Signed-off-by: Koen Kooi <koen@beagleboard.org>
---
arch/arm/mach-omap2/board-omap3beagle.c | 9 ++++++++-
1 files changed, 8 insertions(+), 1 deletions(-)
diff --git a/arch/arm/mach-omap2/board-omap3beagle.c b/arch/arm/mach-omap2/board-omap3beagle.c
index 6c12760..1b5aa7a 100644
--- a/arch/arm/mach-omap2/board-omap3beagle.c
+++ b/arch/arm/mach-omap2/board-omap3beagle.c
@@ -297,9 +297,16 @@ static int beagle_twl_gpio_setup(struct device *dev,
gpio_request(gpio + 1, "EHCI_nOC");
gpio_direction_input(gpio + 1);
- /* TWL4030_GPIO_MAX + 0 == ledA, EHCI nEN_USB_PWR (out, active low) */
+ /*
+ * TWL4030_GPIO_MAX + 0 == ledA, EHCI nEN_USB_PWR (out, XM active
+ * high / others active low)
+ */
gpio_request(gpio + TWL4030_GPIO_MAX, "nEN_USB_PWR");
gpio_direction_output(gpio + TWL4030_GPIO_MAX, 0);
+ if (omap3_beagle_get_rev() == OMAP3BEAGLE_BOARD_XM)
+ gpio_direction_output(gpio + TWL4030_GPIO_MAX, 1);
+ else
+ gpio_direction_output(gpio + TWL4030_GPIO_MAX, 0);
/* TWL4030_GPIO_MAX + 1 == ledB, PMU_STAT (out, active low LED) */
gpio_leds[2].gpio = gpio + TWL4030_GPIO_MAX + 1;
--
1.6.3.3
^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: [PATCH v4 1/3] omap3: beaglexm: fix EHCI power up GPIO dir
2011-01-07 23:01 ` Nishanth Menon
@ 2011-01-10 12:44 ` Aaro Koskinen
-1 siblings, 0 replies; 22+ messages in thread
From: Aaro Koskinen @ 2011-01-10 12:44 UTC (permalink / raw)
To: Nishanth Menon; +Cc: l-o, l-a, Kevin, Tony, Koen Kooi, Aaro
Hi,
On Fri, 7 Jan 2011, Nishanth Menon wrote:
> From: Koen Kooi <koen@beagleboard.org>
>
> EHCI enable power pin is inverted (active high) in comparison
> to vanilla beagle which is active low. Handle this case conditionally.
>
> Without this fix, Beagle XM 4 port EHCI will not function and no
> networking will be available
>
> [nm@ti.com: split up, added descriptive changelogs]
> Signed-off-by: Nishanth Menon <nm@ti.com>
> Signed-off-by: Koen Kooi <koen@beagleboard.org>
> ---
> arch/arm/mach-omap2/board-omap3beagle.c | 9 ++++++++-
> 1 files changed, 8 insertions(+), 1 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/board-omap3beagle.c b/arch/arm/mach-omap2/board-omap3beagle.c
> index 6c12760..1b5aa7a 100644
> --- a/arch/arm/mach-omap2/board-omap3beagle.c
> +++ b/arch/arm/mach-omap2/board-omap3beagle.c
> @@ -297,9 +297,16 @@ static int beagle_twl_gpio_setup(struct device *dev,
> gpio_request(gpio + 1, "EHCI_nOC");
> gpio_direction_input(gpio + 1);
>
> - /* TWL4030_GPIO_MAX + 0 == ledA, EHCI nEN_USB_PWR (out, active low) */
> + /*
> + * TWL4030_GPIO_MAX + 0 == ledA, EHCI nEN_USB_PWR (out, XM active
> + * high / others active low)
> + */
> gpio_request(gpio + TWL4030_GPIO_MAX, "nEN_USB_PWR");
> gpio_direction_output(gpio + TWL4030_GPIO_MAX, 0);
This call should be deleted.
> + if (omap3_beagle_get_rev() == OMAP3BEAGLE_BOARD_XM)
> + gpio_direction_output(gpio + TWL4030_GPIO_MAX, 1);
> + else
> + gpio_direction_output(gpio + TWL4030_GPIO_MAX, 0);
>
> /* TWL4030_GPIO_MAX + 1 == ledB, PMU_STAT (out, active low LED) */
> gpio_leds[2].gpio = gpio + TWL4030_GPIO_MAX + 1;
> --
> 1.6.3.3
>
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH v4 1/3] omap3: beaglexm: fix EHCI power up GPIO dir
@ 2011-01-10 12:44 ` Aaro Koskinen
0 siblings, 0 replies; 22+ messages in thread
From: Aaro Koskinen @ 2011-01-10 12:44 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
On Fri, 7 Jan 2011, Nishanth Menon wrote:
> From: Koen Kooi <koen@beagleboard.org>
>
> EHCI enable power pin is inverted (active high) in comparison
> to vanilla beagle which is active low. Handle this case conditionally.
>
> Without this fix, Beagle XM 4 port EHCI will not function and no
> networking will be available
>
> [nm at ti.com: split up, added descriptive changelogs]
> Signed-off-by: Nishanth Menon <nm@ti.com>
> Signed-off-by: Koen Kooi <koen@beagleboard.org>
> ---
> arch/arm/mach-omap2/board-omap3beagle.c | 9 ++++++++-
> 1 files changed, 8 insertions(+), 1 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/board-omap3beagle.c b/arch/arm/mach-omap2/board-omap3beagle.c
> index 6c12760..1b5aa7a 100644
> --- a/arch/arm/mach-omap2/board-omap3beagle.c
> +++ b/arch/arm/mach-omap2/board-omap3beagle.c
> @@ -297,9 +297,16 @@ static int beagle_twl_gpio_setup(struct device *dev,
> gpio_request(gpio + 1, "EHCI_nOC");
> gpio_direction_input(gpio + 1);
>
> - /* TWL4030_GPIO_MAX + 0 == ledA, EHCI nEN_USB_PWR (out, active low) */
> + /*
> + * TWL4030_GPIO_MAX + 0 == ledA, EHCI nEN_USB_PWR (out, XM active
> + * high / others active low)
> + */
> gpio_request(gpio + TWL4030_GPIO_MAX, "nEN_USB_PWR");
> gpio_direction_output(gpio + TWL4030_GPIO_MAX, 0);
This call should be deleted.
> + if (omap3_beagle_get_rev() == OMAP3BEAGLE_BOARD_XM)
> + gpio_direction_output(gpio + TWL4030_GPIO_MAX, 1);
> + else
> + gpio_direction_output(gpio + TWL4030_GPIO_MAX, 0);
>
> /* TWL4030_GPIO_MAX + 1 == ledB, PMU_STAT (out, active low LED) */
> gpio_leds[2].gpio = gpio + TWL4030_GPIO_MAX + 1;
> --
> 1.6.3.3
>
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v4 1/3] omap3: beaglexm: fix EHCI power up GPIO dir
2011-01-10 12:44 ` Aaro Koskinen
@ 2011-01-10 14:23 ` Nishanth Menon
-1 siblings, 0 replies; 22+ messages in thread
From: Nishanth Menon @ 2011-01-10 14:23 UTC (permalink / raw)
To: Aaro Koskinen; +Cc: l-o, l-a, Kevin, Tony, Koen Kooi
Aaro Koskinen had written, on 01/10/2011 06:44 AM, the following:
> Hi,
>
> On Fri, 7 Jan 2011, Nishanth Menon wrote:
>> From: Koen Kooi <koen@beagleboard.org>
>>
>> EHCI enable power pin is inverted (active high) in comparison
>> to vanilla beagle which is active low. Handle this case conditionally.
>>
>> Without this fix, Beagle XM 4 port EHCI will not function and no
>> networking will be available
>>
>> [nm@ti.com: split up, added descriptive changelogs]
>> Signed-off-by: Nishanth Menon <nm@ti.com>
>> Signed-off-by: Koen Kooi <koen@beagleboard.org>
>> ---
>> arch/arm/mach-omap2/board-omap3beagle.c | 9 ++++++++-
>> 1 files changed, 8 insertions(+), 1 deletions(-)
>>
>> diff --git a/arch/arm/mach-omap2/board-omap3beagle.c
>> b/arch/arm/mach-omap2/board-omap3beagle.c
>> index 6c12760..1b5aa7a 100644
>> --- a/arch/arm/mach-omap2/board-omap3beagle.c
>> +++ b/arch/arm/mach-omap2/board-omap3beagle.c
>> @@ -297,9 +297,16 @@ static int beagle_twl_gpio_setup(struct device *dev,
>> gpio_request(gpio + 1, "EHCI_nOC");
>> gpio_direction_input(gpio + 1);
>>
>> - /* TWL4030_GPIO_MAX + 0 == ledA, EHCI nEN_USB_PWR (out, active
>> low) */
>> + /*
>> + * TWL4030_GPIO_MAX + 0 == ledA, EHCI nEN_USB_PWR (out, XM active
>> + * high / others active low)
>> + */
>> gpio_request(gpio + TWL4030_GPIO_MAX, "nEN_USB_PWR");
>> gpio_direction_output(gpio + TWL4030_GPIO_MAX, 0);
>
> This call should be deleted.
yikes. yep. thanks for catching.
--
Regards,
Nishanth Menon
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH v4 1/3] omap3: beaglexm: fix EHCI power up GPIO dir
@ 2011-01-10 14:23 ` Nishanth Menon
0 siblings, 0 replies; 22+ messages in thread
From: Nishanth Menon @ 2011-01-10 14:23 UTC (permalink / raw)
To: linux-arm-kernel
Aaro Koskinen had written, on 01/10/2011 06:44 AM, the following:
> Hi,
>
> On Fri, 7 Jan 2011, Nishanth Menon wrote:
>> From: Koen Kooi <koen@beagleboard.org>
>>
>> EHCI enable power pin is inverted (active high) in comparison
>> to vanilla beagle which is active low. Handle this case conditionally.
>>
>> Without this fix, Beagle XM 4 port EHCI will not function and no
>> networking will be available
>>
>> [nm at ti.com: split up, added descriptive changelogs]
>> Signed-off-by: Nishanth Menon <nm@ti.com>
>> Signed-off-by: Koen Kooi <koen@beagleboard.org>
>> ---
>> arch/arm/mach-omap2/board-omap3beagle.c | 9 ++++++++-
>> 1 files changed, 8 insertions(+), 1 deletions(-)
>>
>> diff --git a/arch/arm/mach-omap2/board-omap3beagle.c
>> b/arch/arm/mach-omap2/board-omap3beagle.c
>> index 6c12760..1b5aa7a 100644
>> --- a/arch/arm/mach-omap2/board-omap3beagle.c
>> +++ b/arch/arm/mach-omap2/board-omap3beagle.c
>> @@ -297,9 +297,16 @@ static int beagle_twl_gpio_setup(struct device *dev,
>> gpio_request(gpio + 1, "EHCI_nOC");
>> gpio_direction_input(gpio + 1);
>>
>> - /* TWL4030_GPIO_MAX + 0 == ledA, EHCI nEN_USB_PWR (out, active
>> low) */
>> + /*
>> + * TWL4030_GPIO_MAX + 0 == ledA, EHCI nEN_USB_PWR (out, XM active
>> + * high / others active low)
>> + */
>> gpio_request(gpio + TWL4030_GPIO_MAX, "nEN_USB_PWR");
>> gpio_direction_output(gpio + TWL4030_GPIO_MAX, 0);
>
> This call should be deleted.
yikes. yep. thanks for catching.
--
Regards,
Nishanth Menon
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH v4 2/3] omap3: beaglexm: fix DVI reset GPIO
[not found] <[PATCH v3 0/3] OMAP3: beaglexm: GPIO fixes>
@ 2011-01-07 23:01 ` Nishanth Menon
2011-01-07 23:01 ` Nishanth Menon
` (2 subsequent siblings)
3 siblings, 0 replies; 22+ messages in thread
From: Nishanth Menon @ 2011-01-07 23:01 UTC (permalink / raw)
To: l-o; +Cc: l-a, Kevin, Tony, Koen Kooi, Aaro, Nishanth Menon
From: Koen Kooi <koen@beagleboard.org>
GPIO reset line for Beagle XM is different from vanilla beagle
so we populate it as part of gpio update routine.
This in part fixes the issue of display not functioning on beagle XM
platform.
[nm@ti.com: split up, added descriptive changelogs]
Signed-off-by: Nishanth Menon <nm@ti.com>
Signed-off-by: Koen Kooi <koen@beagleboard.org>
---
arch/arm/mach-omap2/board-omap3beagle.c | 8 +++++++-
1 files changed, 7 insertions(+), 1 deletions(-)
diff --git a/arch/arm/mach-omap2/board-omap3beagle.c b/arch/arm/mach-omap2/board-omap3beagle.c
index 1b5aa7a..d628f5e 100644
--- a/arch/arm/mach-omap2/board-omap3beagle.c
+++ b/arch/arm/mach-omap2/board-omap3beagle.c
@@ -199,7 +199,7 @@ static struct omap_dss_device beagle_dvi_device = {
.name = "dvi",
.driver_name = "generic_panel",
.phy.dpi.data_lines = 24,
- .reset_gpio = 170,
+ .reset_gpio = -EINVAL,
.platform_enable = beagle_enable_dvi,
.platform_disable = beagle_disable_dvi,
};
@@ -308,6 +308,12 @@ static int beagle_twl_gpio_setup(struct device *dev,
else
gpio_direction_output(gpio + TWL4030_GPIO_MAX, 0);
+ /* DVI reset GPIO is different between beagle revisions */
+ if (omap3_beagle_get_rev() == OMAP3BEAGLE_BOARD_XM)
+ beagle_dvi_device.reset_gpio = 129;
+ else
+ beagle_dvi_device.reset_gpio = 170;
+
/* TWL4030_GPIO_MAX + 1 == ledB, PMU_STAT (out, active low LED) */
gpio_leds[2].gpio = gpio + TWL4030_GPIO_MAX + 1;
--
1.6.3.3
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH v4 2/3] omap3: beaglexm: fix DVI reset GPIO
@ 2011-01-07 23:01 ` Nishanth Menon
0 siblings, 0 replies; 22+ messages in thread
From: Nishanth Menon @ 2011-01-07 23:01 UTC (permalink / raw)
To: linux-arm-kernel
From: Koen Kooi <koen@beagleboard.org>
GPIO reset line for Beagle XM is different from vanilla beagle
so we populate it as part of gpio update routine.
This in part fixes the issue of display not functioning on beagle XM
platform.
[nm at ti.com: split up, added descriptive changelogs]
Signed-off-by: Nishanth Menon <nm@ti.com>
Signed-off-by: Koen Kooi <koen@beagleboard.org>
---
arch/arm/mach-omap2/board-omap3beagle.c | 8 +++++++-
1 files changed, 7 insertions(+), 1 deletions(-)
diff --git a/arch/arm/mach-omap2/board-omap3beagle.c b/arch/arm/mach-omap2/board-omap3beagle.c
index 1b5aa7a..d628f5e 100644
--- a/arch/arm/mach-omap2/board-omap3beagle.c
+++ b/arch/arm/mach-omap2/board-omap3beagle.c
@@ -199,7 +199,7 @@ static struct omap_dss_device beagle_dvi_device = {
.name = "dvi",
.driver_name = "generic_panel",
.phy.dpi.data_lines = 24,
- .reset_gpio = 170,
+ .reset_gpio = -EINVAL,
.platform_enable = beagle_enable_dvi,
.platform_disable = beagle_disable_dvi,
};
@@ -308,6 +308,12 @@ static int beagle_twl_gpio_setup(struct device *dev,
else
gpio_direction_output(gpio + TWL4030_GPIO_MAX, 0);
+ /* DVI reset GPIO is different between beagle revisions */
+ if (omap3_beagle_get_rev() == OMAP3BEAGLE_BOARD_XM)
+ beagle_dvi_device.reset_gpio = 129;
+ else
+ beagle_dvi_device.reset_gpio = 170;
+
/* TWL4030_GPIO_MAX + 1 == ledB, PMU_STAT (out, active low LED) */
gpio_leds[2].gpio = gpio + TWL4030_GPIO_MAX + 1;
--
1.6.3.3
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH v4 3/3] omap3: beaglexm: fix power on of DVI
[not found] <[PATCH v3 0/3] OMAP3: beaglexm: GPIO fixes>
@ 2011-01-07 23:01 ` Nishanth Menon
2011-01-07 23:01 ` Nishanth Menon
` (2 subsequent siblings)
3 siblings, 0 replies; 22+ messages in thread
From: Nishanth Menon @ 2011-01-07 23:01 UTC (permalink / raw)
To: l-o; +Cc: l-a, Kevin, Tony, Koen Kooi, Aaro, Nishanth Menon
From: Koen Kooi <koen@beagleboard.org>
TFP410 DVI chip is used to provide display out.
This chip is controlled by 2 lines:
LDO which supplies the power is controlled over gpio + 2
and the enable of the chip itself is done over gpio + 1
NOTE: the LDO is necessary for LED, serial blocks as well.
gpio + 1 was used to sense USB overcurrent in vanilla beagle.
Without this fix, the display would not function as the LDO
remains shut down.
[nm@ti.com: split up, added descriptive changelogs]
Signed-off-by: Nishanth Menon <nm@ti.com>
Signed-off-by: Koen Kooi <koen@beagleboard.org>
---
arch/arm/mach-omap2/board-omap3beagle.c | 20 +++++++++++++++++---
1 files changed, 17 insertions(+), 3 deletions(-)
diff --git a/arch/arm/mach-omap2/board-omap3beagle.c b/arch/arm/mach-omap2/board-omap3beagle.c
index d628f5e..7c82730 100644
--- a/arch/arm/mach-omap2/board-omap3beagle.c
+++ b/arch/arm/mach-omap2/board-omap3beagle.c
@@ -293,9 +293,10 @@ static int beagle_twl_gpio_setup(struct device *dev,
/* REVISIT: need ehci-omap hooks for external VBUS
* power switch and overcurrent detect
*/
-
- gpio_request(gpio + 1, "EHCI_nOC");
- gpio_direction_input(gpio + 1);
+ if (omap3_beagle_get_rev() != OMAP3BEAGLE_BOARD_XM) {
+ gpio_request(gpio + 1, "EHCI_nOC");
+ gpio_direction_input(gpio + 1);
+ }
/*
* TWL4030_GPIO_MAX + 0 == ledA, EHCI nEN_USB_PWR (out, XM active
@@ -317,6 +318,19 @@ static int beagle_twl_gpio_setup(struct device *dev,
/* TWL4030_GPIO_MAX + 1 == ledB, PMU_STAT (out, active low LED) */
gpio_leds[2].gpio = gpio + TWL4030_GPIO_MAX + 1;
+ /*
+ * gpio + 1 on Xm controls the TFP410's enable line (active low)
+ * gpio + 2 control varies depending on the board rev as follows:
+ * P7/P8 revisions(prototype): Camera EN
+ * A2+ revisions (production): LDO (supplies DVI, serial, led blocks)
+ */
+ if (omap3_beagle_get_rev() == OMAP3BEAGLE_BOARD_XM) {
+ gpio_request(gpio + 1, "nDVI_PWR_EN");
+ gpio_direction_output(gpio + 1, 0);
+ gpio_request(gpio + 2, "DVI_LDO_EN");
+ gpio_direction_output(gpio + 2, 1);
+ }
+
return 0;
}
--
1.6.3.3
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH v4 3/3] omap3: beaglexm: fix power on of DVI
@ 2011-01-07 23:01 ` Nishanth Menon
0 siblings, 0 replies; 22+ messages in thread
From: Nishanth Menon @ 2011-01-07 23:01 UTC (permalink / raw)
To: linux-arm-kernel
From: Koen Kooi <koen@beagleboard.org>
TFP410 DVI chip is used to provide display out.
This chip is controlled by 2 lines:
LDO which supplies the power is controlled over gpio + 2
and the enable of the chip itself is done over gpio + 1
NOTE: the LDO is necessary for LED, serial blocks as well.
gpio + 1 was used to sense USB overcurrent in vanilla beagle.
Without this fix, the display would not function as the LDO
remains shut down.
[nm at ti.com: split up, added descriptive changelogs]
Signed-off-by: Nishanth Menon <nm@ti.com>
Signed-off-by: Koen Kooi <koen@beagleboard.org>
---
arch/arm/mach-omap2/board-omap3beagle.c | 20 +++++++++++++++++---
1 files changed, 17 insertions(+), 3 deletions(-)
diff --git a/arch/arm/mach-omap2/board-omap3beagle.c b/arch/arm/mach-omap2/board-omap3beagle.c
index d628f5e..7c82730 100644
--- a/arch/arm/mach-omap2/board-omap3beagle.c
+++ b/arch/arm/mach-omap2/board-omap3beagle.c
@@ -293,9 +293,10 @@ static int beagle_twl_gpio_setup(struct device *dev,
/* REVISIT: need ehci-omap hooks for external VBUS
* power switch and overcurrent detect
*/
-
- gpio_request(gpio + 1, "EHCI_nOC");
- gpio_direction_input(gpio + 1);
+ if (omap3_beagle_get_rev() != OMAP3BEAGLE_BOARD_XM) {
+ gpio_request(gpio + 1, "EHCI_nOC");
+ gpio_direction_input(gpio + 1);
+ }
/*
* TWL4030_GPIO_MAX + 0 == ledA, EHCI nEN_USB_PWR (out, XM active
@@ -317,6 +318,19 @@ static int beagle_twl_gpio_setup(struct device *dev,
/* TWL4030_GPIO_MAX + 1 == ledB, PMU_STAT (out, active low LED) */
gpio_leds[2].gpio = gpio + TWL4030_GPIO_MAX + 1;
+ /*
+ * gpio + 1 on Xm controls the TFP410's enable line (active low)
+ * gpio + 2 control varies depending on the board rev as follows:
+ * P7/P8 revisions(prototype): Camera EN
+ * A2+ revisions (production): LDO (supplies DVI, serial, led blocks)
+ */
+ if (omap3_beagle_get_rev() == OMAP3BEAGLE_BOARD_XM) {
+ gpio_request(gpio + 1, "nDVI_PWR_EN");
+ gpio_direction_output(gpio + 1, 0);
+ gpio_request(gpio + 2, "DVI_LDO_EN");
+ gpio_direction_output(gpio + 2, 1);
+ }
+
return 0;
}
--
1.6.3.3
^ permalink raw reply related [flat|nested] 22+ messages in thread* Re: [PATCH v4 3/3] omap3: beaglexm: fix power on of DVI
2011-01-07 23:01 ` Nishanth Menon
@ 2011-01-10 12:51 ` Aaro Koskinen
-1 siblings, 0 replies; 22+ messages in thread
From: Aaro Koskinen @ 2011-01-10 12:51 UTC (permalink / raw)
To: Nishanth Menon; +Cc: l-o, l-a, Kevin, Tony, Koen Kooi, Aaro
Hi,
On Fri, 7 Jan 2011, Nishanth Menon wrote:
> From: Koen Kooi <koen@beagleboard.org>
>
> TFP410 DVI chip is used to provide display out.
> This chip is controlled by 2 lines:
> LDO which supplies the power is controlled over gpio + 2
> and the enable of the chip itself is done over gpio + 1
> NOTE: the LDO is necessary for LED, serial blocks as well.
>
> gpio + 1 was used to sense USB overcurrent in vanilla beagle.
>
> Without this fix, the display would not function as the LDO
> remains shut down.
>
> [nm@ti.com: split up, added descriptive changelogs]
> Signed-off-by: Nishanth Menon <nm@ti.com>
> Signed-off-by: Koen Kooi <koen@beagleboard.org>
> ---
> arch/arm/mach-omap2/board-omap3beagle.c | 20 +++++++++++++++++---
> 1 files changed, 17 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/board-omap3beagle.c b/arch/arm/mach-omap2/board-omap3beagle.c
> index d628f5e..7c82730 100644
> --- a/arch/arm/mach-omap2/board-omap3beagle.c
> +++ b/arch/arm/mach-omap2/board-omap3beagle.c
> @@ -293,9 +293,10 @@ static int beagle_twl_gpio_setup(struct device *dev,
> /* REVISIT: need ehci-omap hooks for external VBUS
> * power switch and overcurrent detect
> */
> -
> - gpio_request(gpio + 1, "EHCI_nOC");
> - gpio_direction_input(gpio + 1);
> + if (omap3_beagle_get_rev() != OMAP3BEAGLE_BOARD_XM) {
> + gpio_request(gpio + 1, "EHCI_nOC");
> + gpio_direction_input(gpio + 1);
> + }
>
> /*
> * TWL4030_GPIO_MAX + 0 == ledA, EHCI nEN_USB_PWR (out, XM active
> @@ -317,6 +318,19 @@ static int beagle_twl_gpio_setup(struct device *dev,
> /* TWL4030_GPIO_MAX + 1 == ledB, PMU_STAT (out, active low LED) */
> gpio_leds[2].gpio = gpio + TWL4030_GPIO_MAX + 1;
>
> + /*
> + * gpio + 1 on Xm controls the TFP410's enable line (active low)
> + * gpio + 2 control varies depending on the board rev as follows:
> + * P7/P8 revisions(prototype): Camera EN
> + * A2+ revisions (production): LDO (supplies DVI, serial, led blocks)
> + */
> + if (omap3_beagle_get_rev() == OMAP3BEAGLE_BOARD_XM) {
> + gpio_request(gpio + 1, "nDVI_PWR_EN");
> + gpio_direction_output(gpio + 1, 0);
> + gpio_request(gpio + 2, "DVI_LDO_EN");
> + gpio_direction_output(gpio + 2, 1);
FYI, gpiolib provides gpio_request_one() and gpio_request_array() calls
which make code simpler. I think any new code should use those _and_
do proper error checking.
> + }
> +
> return 0;
> }
>
> --
> 1.6.3.3
>
^ permalink raw reply [flat|nested] 22+ messages in thread* [PATCH v4 3/3] omap3: beaglexm: fix power on of DVI
@ 2011-01-10 12:51 ` Aaro Koskinen
0 siblings, 0 replies; 22+ messages in thread
From: Aaro Koskinen @ 2011-01-10 12:51 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
On Fri, 7 Jan 2011, Nishanth Menon wrote:
> From: Koen Kooi <koen@beagleboard.org>
>
> TFP410 DVI chip is used to provide display out.
> This chip is controlled by 2 lines:
> LDO which supplies the power is controlled over gpio + 2
> and the enable of the chip itself is done over gpio + 1
> NOTE: the LDO is necessary for LED, serial blocks as well.
>
> gpio + 1 was used to sense USB overcurrent in vanilla beagle.
>
> Without this fix, the display would not function as the LDO
> remains shut down.
>
> [nm at ti.com: split up, added descriptive changelogs]
> Signed-off-by: Nishanth Menon <nm@ti.com>
> Signed-off-by: Koen Kooi <koen@beagleboard.org>
> ---
> arch/arm/mach-omap2/board-omap3beagle.c | 20 +++++++++++++++++---
> 1 files changed, 17 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/board-omap3beagle.c b/arch/arm/mach-omap2/board-omap3beagle.c
> index d628f5e..7c82730 100644
> --- a/arch/arm/mach-omap2/board-omap3beagle.c
> +++ b/arch/arm/mach-omap2/board-omap3beagle.c
> @@ -293,9 +293,10 @@ static int beagle_twl_gpio_setup(struct device *dev,
> /* REVISIT: need ehci-omap hooks for external VBUS
> * power switch and overcurrent detect
> */
> -
> - gpio_request(gpio + 1, "EHCI_nOC");
> - gpio_direction_input(gpio + 1);
> + if (omap3_beagle_get_rev() != OMAP3BEAGLE_BOARD_XM) {
> + gpio_request(gpio + 1, "EHCI_nOC");
> + gpio_direction_input(gpio + 1);
> + }
>
> /*
> * TWL4030_GPIO_MAX + 0 == ledA, EHCI nEN_USB_PWR (out, XM active
> @@ -317,6 +318,19 @@ static int beagle_twl_gpio_setup(struct device *dev,
> /* TWL4030_GPIO_MAX + 1 == ledB, PMU_STAT (out, active low LED) */
> gpio_leds[2].gpio = gpio + TWL4030_GPIO_MAX + 1;
>
> + /*
> + * gpio + 1 on Xm controls the TFP410's enable line (active low)
> + * gpio + 2 control varies depending on the board rev as follows:
> + * P7/P8 revisions(prototype): Camera EN
> + * A2+ revisions (production): LDO (supplies DVI, serial, led blocks)
> + */
> + if (omap3_beagle_get_rev() == OMAP3BEAGLE_BOARD_XM) {
> + gpio_request(gpio + 1, "nDVI_PWR_EN");
> + gpio_direction_output(gpio + 1, 0);
> + gpio_request(gpio + 2, "DVI_LDO_EN");
> + gpio_direction_output(gpio + 2, 1);
FYI, gpiolib provides gpio_request_one() and gpio_request_array() calls
which make code simpler. I think any new code should use those _and_
do proper error checking.
> + }
> +
> return 0;
> }
>
> --
> 1.6.3.3
>
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH v4 3/3] omap3: beaglexm: fix power on of DVI
2011-01-10 12:51 ` Aaro Koskinen
@ 2011-01-10 14:34 ` Nishanth Menon
-1 siblings, 0 replies; 22+ messages in thread
From: Nishanth Menon @ 2011-01-10 14:34 UTC (permalink / raw)
To: Aaro Koskinen; +Cc: l-o, l-a, Kevin, Tony, Koen Kooi
Aaro Koskinen had written, on 01/10/2011 06:51 AM, the following:
[..]
>> + if (omap3_beagle_get_rev() == OMAP3BEAGLE_BOARD_XM) {
>> + gpio_request(gpio + 1, "nDVI_PWR_EN");
>> + gpio_direction_output(gpio + 1, 0);
>> + gpio_request(gpio + 2, "DVI_LDO_EN");
>> + gpio_direction_output(gpio + 2, 1);
>
> FYI, gpiolib provides gpio_request_one() and gpio_request_array() calls
> which make code simpler. I think any new code should use those _and_
> do proper error checking.
Thanks for pointing this out. Funny why it is in
include/asm-generic/gpio.h instead of in include/linux/gpio.h
considering it implemented in drivers/gpio/gpiolib.c :(
Am I right in understanding that you are mentioning about DVI gpios
alone? in that case, the LDO gpio is used for other functions as well in
addition to driving DVI. I suppose we dont want to loose that function
because dvi pwr_en gpio failed.
--
Regards,
Nishanth Menon
^ permalink raw reply [flat|nested] 22+ messages in thread* [PATCH v4 3/3] omap3: beaglexm: fix power on of DVI
@ 2011-01-10 14:34 ` Nishanth Menon
0 siblings, 0 replies; 22+ messages in thread
From: Nishanth Menon @ 2011-01-10 14:34 UTC (permalink / raw)
To: linux-arm-kernel
Aaro Koskinen had written, on 01/10/2011 06:51 AM, the following:
[..]
>> + if (omap3_beagle_get_rev() == OMAP3BEAGLE_BOARD_XM) {
>> + gpio_request(gpio + 1, "nDVI_PWR_EN");
>> + gpio_direction_output(gpio + 1, 0);
>> + gpio_request(gpio + 2, "DVI_LDO_EN");
>> + gpio_direction_output(gpio + 2, 1);
>
> FYI, gpiolib provides gpio_request_one() and gpio_request_array() calls
> which make code simpler. I think any new code should use those _and_
> do proper error checking.
Thanks for pointing this out. Funny why it is in
include/asm-generic/gpio.h instead of in include/linux/gpio.h
considering it implemented in drivers/gpio/gpiolib.c :(
Am I right in understanding that you are mentioning about DVI gpios
alone? in that case, the LDO gpio is used for other functions as well in
addition to driving DVI. I suppose we dont want to loose that function
because dvi pwr_en gpio failed.
--
Regards,
Nishanth Menon
^ permalink raw reply [flat|nested] 22+ messages in thread