All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: am57xx-beagle-x15: fix HDMI LS_OE GPIO
@ 2016-09-01 13:51 ` Tomi Valkeinen
  0 siblings, 0 replies; 22+ messages in thread
From: Tomi Valkeinen @ 2016-09-01 13:51 UTC (permalink / raw)
  To: Tony Lindgren, linux-omap, linux-arm-kernel
  Cc: Nishanth Menon, Tomi Valkeinen

LS_OE GPIO was changed from gpio6_28 to gpio2_30 for the Rev B1 board,
and the current .dts file has the old GPIO.

Rev B1 is the earliest board supported, so we can just change the .dts.

This fixes "omapdss error: HDMI I2C Master Error" which causes failure
to read EDID from the monitor.

Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
---
 arch/arm/boot/dts/am57xx-beagle-x15.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/am57xx-beagle-x15.dts b/arch/arm/boot/dts/am57xx-beagle-x15.dts
index c4d04c5293b9..07603d9e2da7 100644
--- a/arch/arm/boot/dts/am57xx-beagle-x15.dts
+++ b/arch/arm/boot/dts/am57xx-beagle-x15.dts
@@ -120,7 +120,7 @@
 		pinctrl-0 = <&tpd12s015_pins>;
 
 		gpios = <&gpio7 10 GPIO_ACTIVE_HIGH>,	/* gpio7_10, CT CP HPD */
-			<&gpio6 28 GPIO_ACTIVE_HIGH>,	/* gpio6_28, LS OE */
+			<&gpio2 30 GPIO_ACTIVE_HIGH>,	/* gpio2_30, LS OE */
 			<&gpio7 12 GPIO_ACTIVE_HIGH>;	/* gpio7_12/sp1_cs2, HPD */
 
 		ports {
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 22+ messages in thread

* [PATCH] ARM: dts: am57xx-beagle-x15: fix HDMI LS_OE GPIO
@ 2016-09-01 13:51 ` Tomi Valkeinen
  0 siblings, 0 replies; 22+ messages in thread
From: Tomi Valkeinen @ 2016-09-01 13:51 UTC (permalink / raw)
  To: linux-arm-kernel

LS_OE GPIO was changed from gpio6_28 to gpio2_30 for the Rev B1 board,
and the current .dts file has the old GPIO.

Rev B1 is the earliest board supported, so we can just change the .dts.

This fixes "omapdss error: HDMI I2C Master Error" which causes failure
to read EDID from the monitor.

Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
---
 arch/arm/boot/dts/am57xx-beagle-x15.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/am57xx-beagle-x15.dts b/arch/arm/boot/dts/am57xx-beagle-x15.dts
index c4d04c5293b9..07603d9e2da7 100644
--- a/arch/arm/boot/dts/am57xx-beagle-x15.dts
+++ b/arch/arm/boot/dts/am57xx-beagle-x15.dts
@@ -120,7 +120,7 @@
 		pinctrl-0 = <&tpd12s015_pins>;
 
 		gpios = <&gpio7 10 GPIO_ACTIVE_HIGH>,	/* gpio7_10, CT CP HPD */
-			<&gpio6 28 GPIO_ACTIVE_HIGH>,	/* gpio6_28, LS OE */
+			<&gpio2 30 GPIO_ACTIVE_HIGH>,	/* gpio2_30, LS OE */
 			<&gpio7 12 GPIO_ACTIVE_HIGH>;	/* gpio7_12/sp1_cs2, HPD */
 
 		ports {
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 22+ messages in thread

* Re: [PATCH] ARM: dts: am57xx-beagle-x15: fix HDMI LS_OE GPIO
  2016-09-01 13:51 ` Tomi Valkeinen
@ 2016-09-01 14:09   ` Nishanth Menon
  -1 siblings, 0 replies; 22+ messages in thread
From: Nishanth Menon @ 2016-09-01 14:09 UTC (permalink / raw)
  To: Tomi Valkeinen, Tony Lindgren, linux-omap, linux-arm-kernel,
	beagleboard-x15@googlegroups.com

On 09/01/2016 08:51 AM, Tomi Valkeinen wrote:
> LS_OE GPIO was changed from gpio6_28 to gpio2_30 for the Rev B1 board,
> and the current .dts file has the old GPIO.
>
> Rev B1 is the earliest board supported, so we can just change the .dts.
>
> This fixes "omapdss error: HDMI I2C Master Error" which causes failure
> to read EDID from the monitor.
>
> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
> ---
>  arch/arm/boot/dts/am57xx-beagle-x15.dts | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/am57xx-beagle-x15.dts b/arch/arm/boot/dts/am57xx-beagle-x15.dts
> index c4d04c5293b9..07603d9e2da7 100644
> --- a/arch/arm/boot/dts/am57xx-beagle-x15.dts
> +++ b/arch/arm/boot/dts/am57xx-beagle-x15.dts
> @@ -120,7 +120,7 @@
>  		pinctrl-0 = <&tpd12s015_pins>;
>
>  		gpios = <&gpio7 10 GPIO_ACTIVE_HIGH>,	/* gpio7_10, CT CP HPD */
> -			<&gpio6 28 GPIO_ACTIVE_HIGH>,	/* gpio6_28, LS OE */
> +			<&gpio2 30 GPIO_ACTIVE_HIGH>,	/* gpio2_30, LS OE */
>  			<&gpio7 12 GPIO_ACTIVE_HIGH>;	/* gpio7_12/sp1_cs2, HPD */
>
>  		ports {
>

I believe we have A2s being shipped as well. we dont want to break A2s 
either -> I propose we refactor the dts to introduce revb1 X15.

-- 
Regards,
Nishanth Menon

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [PATCH] ARM: dts: am57xx-beagle-x15: fix HDMI LS_OE GPIO
@ 2016-09-01 14:09   ` Nishanth Menon
  0 siblings, 0 replies; 22+ messages in thread
From: Nishanth Menon @ 2016-09-01 14:09 UTC (permalink / raw)
  To: linux-arm-kernel

On 09/01/2016 08:51 AM, Tomi Valkeinen wrote:
> LS_OE GPIO was changed from gpio6_28 to gpio2_30 for the Rev B1 board,
> and the current .dts file has the old GPIO.
>
> Rev B1 is the earliest board supported, so we can just change the .dts.
>
> This fixes "omapdss error: HDMI I2C Master Error" which causes failure
> to read EDID from the monitor.
>
> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
> ---
>  arch/arm/boot/dts/am57xx-beagle-x15.dts | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/am57xx-beagle-x15.dts b/arch/arm/boot/dts/am57xx-beagle-x15.dts
> index c4d04c5293b9..07603d9e2da7 100644
> --- a/arch/arm/boot/dts/am57xx-beagle-x15.dts
> +++ b/arch/arm/boot/dts/am57xx-beagle-x15.dts
> @@ -120,7 +120,7 @@
>  		pinctrl-0 = <&tpd12s015_pins>;
>
>  		gpios = <&gpio7 10 GPIO_ACTIVE_HIGH>,	/* gpio7_10, CT CP HPD */
> -			<&gpio6 28 GPIO_ACTIVE_HIGH>,	/* gpio6_28, LS OE */
> +			<&gpio2 30 GPIO_ACTIVE_HIGH>,	/* gpio2_30, LS OE */
>  			<&gpio7 12 GPIO_ACTIVE_HIGH>;	/* gpio7_12/sp1_cs2, HPD */
>
>  		ports {
>

I believe we have A2s being shipped as well. we dont want to break A2s 
either -> I propose we refactor the dts to introduce revb1 X15.

-- 
Regards,
Nishanth Menon

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH] ARM: dts: am57xx-beagle-x15: fix HDMI LS_OE GPIO
  2016-09-01 14:09   ` Nishanth Menon
@ 2016-09-01 14:12     ` Robert Nelson
  -1 siblings, 0 replies; 22+ messages in thread
From: Robert Nelson @ 2016-09-01 14:12 UTC (permalink / raw)
  To: Nishanth Menon
  Cc: Tony Lindgren, beagleboard-x15@googlegroups.com,
	linux-omap@vger.kernel.org, Tomi Valkeinen,
	linux-arm-kernel@lists.infradead.org

On Thu, Sep 1, 2016 at 9:09 AM, Nishanth Menon <nm@ti.com> wrote:
> On 09/01/2016 08:51 AM, Tomi Valkeinen wrote:
>>
>> LS_OE GPIO was changed from gpio6_28 to gpio2_30 for the Rev B1 board,
>> and the current .dts file has the old GPIO.
>>
>> Rev B1 is the earliest board supported, so we can just change the .dts.
>>
>> This fixes "omapdss error: HDMI I2C Master Error" which causes failure
>> to read EDID from the monitor.
>>
>> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
>> ---
>>  arch/arm/boot/dts/am57xx-beagle-x15.dts | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/boot/dts/am57xx-beagle-x15.dts
>> b/arch/arm/boot/dts/am57xx-beagle-x15.dts
>> index c4d04c5293b9..07603d9e2da7 100644
>> --- a/arch/arm/boot/dts/am57xx-beagle-x15.dts
>> +++ b/arch/arm/boot/dts/am57xx-beagle-x15.dts
>> @@ -120,7 +120,7 @@
>>                 pinctrl-0 = <&tpd12s015_pins>;
>>
>>                 gpios = <&gpio7 10 GPIO_ACTIVE_HIGH>,   /* gpio7_10, CT CP
>> HPD */
>> -                       <&gpio6 28 GPIO_ACTIVE_HIGH>,   /* gpio6_28, LS OE
>> */
>> +                       <&gpio2 30 GPIO_ACTIVE_HIGH>,   /* gpio2_30, LS OE
>> */
>>                         <&gpio7 12 GPIO_ACTIVE_HIGH>;   /*
>> gpio7_12/sp1_cs2, HPD */
>>
>>                 ports {
>>
>
> I believe we have A2s being shipped as well. we dont want to break A2s
> either -> I propose we refactor the dts to introduce revb1 X15.

There are at-least 200 A2's that have been shipped to end users.

http://uk.rs-online.com/web/p/processor-microcontroller-development-kits/8874764/

Regards,

-- 
Robert Nelson
https://rcn-ee.com/

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [PATCH] ARM: dts: am57xx-beagle-x15: fix HDMI LS_OE GPIO
@ 2016-09-01 14:12     ` Robert Nelson
  0 siblings, 0 replies; 22+ messages in thread
From: Robert Nelson @ 2016-09-01 14:12 UTC (permalink / raw)
  To: linux-arm-kernel

On Thu, Sep 1, 2016 at 9:09 AM, Nishanth Menon <nm@ti.com> wrote:
> On 09/01/2016 08:51 AM, Tomi Valkeinen wrote:
>>
>> LS_OE GPIO was changed from gpio6_28 to gpio2_30 for the Rev B1 board,
>> and the current .dts file has the old GPIO.
>>
>> Rev B1 is the earliest board supported, so we can just change the .dts.
>>
>> This fixes "omapdss error: HDMI I2C Master Error" which causes failure
>> to read EDID from the monitor.
>>
>> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
>> ---
>>  arch/arm/boot/dts/am57xx-beagle-x15.dts | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/boot/dts/am57xx-beagle-x15.dts
>> b/arch/arm/boot/dts/am57xx-beagle-x15.dts
>> index c4d04c5293b9..07603d9e2da7 100644
>> --- a/arch/arm/boot/dts/am57xx-beagle-x15.dts
>> +++ b/arch/arm/boot/dts/am57xx-beagle-x15.dts
>> @@ -120,7 +120,7 @@
>>                 pinctrl-0 = <&tpd12s015_pins>;
>>
>>                 gpios = <&gpio7 10 GPIO_ACTIVE_HIGH>,   /* gpio7_10, CT CP
>> HPD */
>> -                       <&gpio6 28 GPIO_ACTIVE_HIGH>,   /* gpio6_28, LS OE
>> */
>> +                       <&gpio2 30 GPIO_ACTIVE_HIGH>,   /* gpio2_30, LS OE
>> */
>>                         <&gpio7 12 GPIO_ACTIVE_HIGH>;   /*
>> gpio7_12/sp1_cs2, HPD */
>>
>>                 ports {
>>
>
> I believe we have A2s being shipped as well. we dont want to break A2s
> either -> I propose we refactor the dts to introduce revb1 X15.

There are at-least 200 A2's that have been shipped to end users.

http://uk.rs-online.com/web/p/processor-microcontroller-development-kits/8874764/

Regards,

-- 
Robert Nelson
https://rcn-ee.com/

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH] ARM: dts: am57xx-beagle-x15: fix HDMI LS_OE GPIO
  2016-09-01 14:12     ` Robert Nelson
@ 2016-09-01 14:14       ` Nishanth Menon
  -1 siblings, 0 replies; 22+ messages in thread
From: Nishanth Menon @ 2016-09-01 14:14 UTC (permalink / raw)
  To: Robert Nelson
  Cc: Tony Lindgren, beagleboard-x15@googlegroups.com,
	linux-omap@vger.kernel.org, Tomi Valkeinen,
	linux-arm-kernel@lists.infradead.org

On 09/01/2016 09:12 AM, Robert Nelson wrote:
> On Thu, Sep 1, 2016 at 9:09 AM, Nishanth Menon <nm@ti.com> wrote:
>> On 09/01/2016 08:51 AM, Tomi Valkeinen wrote:
>>>
>>> LS_OE GPIO was changed from gpio6_28 to gpio2_30 for the Rev B1 board,
>>> and the current .dts file has the old GPIO.
>>>
>>> Rev B1 is the earliest board supported, so we can just change the .dts.
>>>
>>> This fixes "omapdss error: HDMI I2C Master Error" which causes failure
>>> to read EDID from the monitor.
>>>
>>> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
>>> ---
>>>  arch/arm/boot/dts/am57xx-beagle-x15.dts | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/arch/arm/boot/dts/am57xx-beagle-x15.dts
>>> b/arch/arm/boot/dts/am57xx-beagle-x15.dts
>>> index c4d04c5293b9..07603d9e2da7 100644
>>> --- a/arch/arm/boot/dts/am57xx-beagle-x15.dts
>>> +++ b/arch/arm/boot/dts/am57xx-beagle-x15.dts
>>> @@ -120,7 +120,7 @@
>>>                 pinctrl-0 = <&tpd12s015_pins>;
>>>
>>>                 gpios = <&gpio7 10 GPIO_ACTIVE_HIGH>,   /* gpio7_10, CT CP
>>> HPD */
>>> -                       <&gpio6 28 GPIO_ACTIVE_HIGH>,   /* gpio6_28, LS OE
>>> */
>>> +                       <&gpio2 30 GPIO_ACTIVE_HIGH>,   /* gpio2_30, LS OE
>>> */
>>>                         <&gpio7 12 GPIO_ACTIVE_HIGH>;   /*
>>> gpio7_12/sp1_cs2, HPD */
>>>
>>>                 ports {
>>>
>>
>> I believe we have A2s being shipped as well. we dont want to break A2s
>> either -> I propose we refactor the dts to introduce revb1 X15.
>
> There are at-least 200 A2's that have been shipped to end users.
>
> http://uk.rs-online.com/web/p/processor-microcontroller-development-kits/8874764/


Thanks. Tomi: if you are ok - I already do have a refactoring code 
from older kernel. I will rebase to latest and repost.

Tony: is there a branch youd' like me to base the changes on?


-- 
Regards,
Nishanth Menon

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [PATCH] ARM: dts: am57xx-beagle-x15: fix HDMI LS_OE GPIO
@ 2016-09-01 14:14       ` Nishanth Menon
  0 siblings, 0 replies; 22+ messages in thread
From: Nishanth Menon @ 2016-09-01 14:14 UTC (permalink / raw)
  To: linux-arm-kernel

On 09/01/2016 09:12 AM, Robert Nelson wrote:
> On Thu, Sep 1, 2016 at 9:09 AM, Nishanth Menon <nm@ti.com> wrote:
>> On 09/01/2016 08:51 AM, Tomi Valkeinen wrote:
>>>
>>> LS_OE GPIO was changed from gpio6_28 to gpio2_30 for the Rev B1 board,
>>> and the current .dts file has the old GPIO.
>>>
>>> Rev B1 is the earliest board supported, so we can just change the .dts.
>>>
>>> This fixes "omapdss error: HDMI I2C Master Error" which causes failure
>>> to read EDID from the monitor.
>>>
>>> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
>>> ---
>>>  arch/arm/boot/dts/am57xx-beagle-x15.dts | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/arch/arm/boot/dts/am57xx-beagle-x15.dts
>>> b/arch/arm/boot/dts/am57xx-beagle-x15.dts
>>> index c4d04c5293b9..07603d9e2da7 100644
>>> --- a/arch/arm/boot/dts/am57xx-beagle-x15.dts
>>> +++ b/arch/arm/boot/dts/am57xx-beagle-x15.dts
>>> @@ -120,7 +120,7 @@
>>>                 pinctrl-0 = <&tpd12s015_pins>;
>>>
>>>                 gpios = <&gpio7 10 GPIO_ACTIVE_HIGH>,   /* gpio7_10, CT CP
>>> HPD */
>>> -                       <&gpio6 28 GPIO_ACTIVE_HIGH>,   /* gpio6_28, LS OE
>>> */
>>> +                       <&gpio2 30 GPIO_ACTIVE_HIGH>,   /* gpio2_30, LS OE
>>> */
>>>                         <&gpio7 12 GPIO_ACTIVE_HIGH>;   /*
>>> gpio7_12/sp1_cs2, HPD */
>>>
>>>                 ports {
>>>
>>
>> I believe we have A2s being shipped as well. we dont want to break A2s
>> either -> I propose we refactor the dts to introduce revb1 X15.
>
> There are at-least 200 A2's that have been shipped to end users.
>
> http://uk.rs-online.com/web/p/processor-microcontroller-development-kits/8874764/


Thanks. Tomi: if you are ok - I already do have a refactoring code 
from older kernel. I will rebase to latest and repost.

Tony: is there a branch youd' like me to base the changes on?


-- 
Regards,
Nishanth Menon

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH] ARM: dts: am57xx-beagle-x15: fix HDMI LS_OE GPIO
  2016-09-01 14:14       ` Nishanth Menon
@ 2016-09-01 14:57         ` Tomi Valkeinen
  -1 siblings, 0 replies; 22+ messages in thread
From: Tomi Valkeinen @ 2016-09-01 14:57 UTC (permalink / raw)
  To: Nishanth Menon, Robert Nelson
  Cc: Tony Lindgren, beagleboard-x15@googlegroups.com,
	linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org


[-- Attachment #1.1.1: Type: text/plain, Size: 525 bytes --]

On 01/09/16 17:14, Nishanth Menon wrote:

>>> I believe we have A2s being shipped as well. we dont want to break A2s
>>> either -> I propose we refactor the dts to introduce revb1 X15.
>>
>> There are at-least 200 A2's that have been shipped to end users.
>>
>> http://uk.rs-online.com/web/p/processor-microcontroller-development-kits/8874764/
>>
> 
> 
> Thanks. Tomi: if you are ok - I already do have a refactoring code from
> older kernel. I will rebase to latest and repost.

Sounds good to me.

 Tomi


[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

[-- Attachment #2: Type: text/plain, Size: 176 bytes --]

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [PATCH] ARM: dts: am57xx-beagle-x15: fix HDMI LS_OE GPIO
@ 2016-09-01 14:57         ` Tomi Valkeinen
  0 siblings, 0 replies; 22+ messages in thread
From: Tomi Valkeinen @ 2016-09-01 14:57 UTC (permalink / raw)
  To: linux-arm-kernel

On 01/09/16 17:14, Nishanth Menon wrote:

>>> I believe we have A2s being shipped as well. we dont want to break A2s
>>> either -> I propose we refactor the dts to introduce revb1 X15.
>>
>> There are at-least 200 A2's that have been shipped to end users.
>>
>> http://uk.rs-online.com/web/p/processor-microcontroller-development-kits/8874764/
>>
> 
> 
> Thanks. Tomi: if you are ok - I already do have a refactoring code from
> older kernel. I will rebase to latest and repost.

Sounds good to me.

 Tomi

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20160901/a81fda17/attachment.sig>

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH] ARM: dts: am57xx-beagle-x15: fix HDMI LS_OE GPIO
  2016-09-01 14:14       ` Nishanth Menon
@ 2016-09-01 14:57         ` Tony Lindgren
  -1 siblings, 0 replies; 22+ messages in thread
From: Tony Lindgren @ 2016-09-01 14:57 UTC (permalink / raw)
  To: Nishanth Menon
  Cc: beagleboard-x15@googlegroups.com, linux-omap@vger.kernel.org,
	Tomi Valkeinen, Robert Nelson,
	linux-arm-kernel@lists.infradead.org

* Nishanth Menon <nm@ti.com> [160901 07:15]:
> On 09/01/2016 09:12 AM, Robert Nelson wrote:
> > On Thu, Sep 1, 2016 at 9:09 AM, Nishanth Menon <nm@ti.com> wrote:
> > > On 09/01/2016 08:51 AM, Tomi Valkeinen wrote:
> > > > 
> > > > LS_OE GPIO was changed from gpio6_28 to gpio2_30 for the Rev B1 board,
> > > > and the current .dts file has the old GPIO.
> > > > 
> > > > Rev B1 is the earliest board supported, so we can just change the .dts.
> > > > 
> > > > This fixes "omapdss error: HDMI I2C Master Error" which causes failure
> > > > to read EDID from the monitor.
...
> > > I believe we have A2s being shipped as well. we dont want to break A2s
> > > either -> I propose we refactor the dts to introduce revb1 X15.
> > 
> > There are at-least 200 A2's that have been shipped to end users.
> > 
> > http://uk.rs-online.com/web/p/processor-microcontroller-development-kits/8874764/
> 
> 
> Thanks. Tomi: if you are ok - I already do have a refactoring code from
> older kernel. I will rebase to latest and repost.
> 
> Tony: is there a branch youd' like me to base the changes on?

Probably best to do it against current omapf-for-v4.9/dt as it has all
the skeleton.dtsi removal now applied.

Regards,

Tony

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [PATCH] ARM: dts: am57xx-beagle-x15: fix HDMI LS_OE GPIO
@ 2016-09-01 14:57         ` Tony Lindgren
  0 siblings, 0 replies; 22+ messages in thread
From: Tony Lindgren @ 2016-09-01 14:57 UTC (permalink / raw)
  To: linux-arm-kernel

* Nishanth Menon <nm@ti.com> [160901 07:15]:
> On 09/01/2016 09:12 AM, Robert Nelson wrote:
> > On Thu, Sep 1, 2016 at 9:09 AM, Nishanth Menon <nm@ti.com> wrote:
> > > On 09/01/2016 08:51 AM, Tomi Valkeinen wrote:
> > > > 
> > > > LS_OE GPIO was changed from gpio6_28 to gpio2_30 for the Rev B1 board,
> > > > and the current .dts file has the old GPIO.
> > > > 
> > > > Rev B1 is the earliest board supported, so we can just change the .dts.
> > > > 
> > > > This fixes "omapdss error: HDMI I2C Master Error" which causes failure
> > > > to read EDID from the monitor.
...
> > > I believe we have A2s being shipped as well. we dont want to break A2s
> > > either -> I propose we refactor the dts to introduce revb1 X15.
> > 
> > There are at-least 200 A2's that have been shipped to end users.
> > 
> > http://uk.rs-online.com/web/p/processor-microcontroller-development-kits/8874764/
> 
> 
> Thanks. Tomi: if you are ok - I already do have a refactoring code from
> older kernel. I will rebase to latest and repost.
> 
> Tony: is there a branch youd' like me to base the changes on?

Probably best to do it against current omapf-for-v4.9/dt as it has all
the skeleton.dtsi removal now applied.

Regards,

Tony

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH] ARM: dts: am57xx-beagle-x15: fix HDMI LS_OE GPIO
  2016-09-01 14:57         ` Tony Lindgren
@ 2016-09-01 15:00           ` Nishanth Menon
  -1 siblings, 0 replies; 22+ messages in thread
From: Nishanth Menon @ 2016-09-01 15:00 UTC (permalink / raw)
  To: Tony Lindgren
  Cc: beagleboard-x15@googlegroups.com, linux-omap@vger.kernel.org,
	Tomi Valkeinen, Robert Nelson,
	linux-arm-kernel@lists.infradead.org

On 09/01/2016 09:57 AM, Tony Lindgren wrote:
> * Nishanth Menon <nm@ti.com> [160901 07:15]:
>> On 09/01/2016 09:12 AM, Robert Nelson wrote:
>>> On Thu, Sep 1, 2016 at 9:09 AM, Nishanth Menon <nm@ti.com> wrote:
>>>> On 09/01/2016 08:51 AM, Tomi Valkeinen wrote:
>>>>>
>>>>> LS_OE GPIO was changed from gpio6_28 to gpio2_30 for the Rev B1 board,
>>>>> and the current .dts file has the old GPIO.
>>>>>
>>>>> Rev B1 is the earliest board supported, so we can just change the .dts.
>>>>>
>>>>> This fixes "omapdss error: HDMI I2C Master Error" which causes failure
>>>>> to read EDID from the monitor.
> ...
>>>> I believe we have A2s being shipped as well. we dont want to break A2s
>>>> either -> I propose we refactor the dts to introduce revb1 X15.
>>>
>>> There are at-least 200 A2's that have been shipped to end users.
>>>
>>> http://uk.rs-online.com/web/p/processor-microcontroller-development-kits/8874764/
>>
>>
>> Thanks. Tomi: if you are ok - I already do have a refactoring code from
>> older kernel. I will rebase to latest and repost.
>>
>> Tony: is there a branch youd' like me to base the changes on?
>
> Probably best to do it against current omapf-for-v4.9/dt as it has all
> the skeleton.dtsi removal now applied.

Ok will do, will try to post this later today or early tomorrow.


-- 
Regards,
Nishanth Menon

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [PATCH] ARM: dts: am57xx-beagle-x15: fix HDMI LS_OE GPIO
@ 2016-09-01 15:00           ` Nishanth Menon
  0 siblings, 0 replies; 22+ messages in thread
From: Nishanth Menon @ 2016-09-01 15:00 UTC (permalink / raw)
  To: linux-arm-kernel

On 09/01/2016 09:57 AM, Tony Lindgren wrote:
> * Nishanth Menon <nm@ti.com> [160901 07:15]:
>> On 09/01/2016 09:12 AM, Robert Nelson wrote:
>>> On Thu, Sep 1, 2016 at 9:09 AM, Nishanth Menon <nm@ti.com> wrote:
>>>> On 09/01/2016 08:51 AM, Tomi Valkeinen wrote:
>>>>>
>>>>> LS_OE GPIO was changed from gpio6_28 to gpio2_30 for the Rev B1 board,
>>>>> and the current .dts file has the old GPIO.
>>>>>
>>>>> Rev B1 is the earliest board supported, so we can just change the .dts.
>>>>>
>>>>> This fixes "omapdss error: HDMI I2C Master Error" which causes failure
>>>>> to read EDID from the monitor.
> ...
>>>> I believe we have A2s being shipped as well. we dont want to break A2s
>>>> either -> I propose we refactor the dts to introduce revb1 X15.
>>>
>>> There are at-least 200 A2's that have been shipped to end users.
>>>
>>> http://uk.rs-online.com/web/p/processor-microcontroller-development-kits/8874764/
>>
>>
>> Thanks. Tomi: if you are ok - I already do have a refactoring code from
>> older kernel. I will rebase to latest and repost.
>>
>> Tony: is there a branch youd' like me to base the changes on?
>
> Probably best to do it against current omapf-for-v4.9/dt as it has all
> the skeleton.dtsi removal now applied.

Ok will do, will try to post this later today or early tomorrow.


-- 
Regards,
Nishanth Menon

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH] ARM: dts: am57xx-beagle-x15: fix HDMI LS_OE GPIO
  2016-09-01 15:00           ` Nishanth Menon
@ 2017-11-14  0:27             ` Bill Traynor
  -1 siblings, 0 replies; 22+ messages in thread
From: Bill Traynor @ 2017-11-14  0:27 UTC (permalink / raw)
  To: Nishanth Menon
  Cc: Tony Lindgren, beagleboard-x15@googlegroups.com, Tomi Valkeinen,
	linux-omap@vger.kernel.org, Robert Nelson,
	linux-arm-kernel@lists.infradead.org

On Thu, Sep 1, 2016 at 11:00 AM, Nishanth Menon <nm@ti.com> wrote:
> On 09/01/2016 09:57 AM, Tony Lindgren wrote:
>>
>> * Nishanth Menon <nm@ti.com> [160901 07:15]:
>>>
>>> On 09/01/2016 09:12 AM, Robert Nelson wrote:
>>>>
>>>> On Thu, Sep 1, 2016 at 9:09 AM, Nishanth Menon <nm@ti.com> wrote:
>>>>>
>>>>> On 09/01/2016 08:51 AM, Tomi Valkeinen wrote:
>>>>>>
>>>>>>
>>>>>> LS_OE GPIO was changed from gpio6_28 to gpio2_30 for the Rev B1 board,
>>>>>> and the current .dts file has the old GPIO.
>>>>>>
>>>>>> Rev B1 is the earliest board supported, so we can just change the
>>>>>> .dts.
>>>>>>
>>>>>> This fixes "omapdss error: HDMI I2C Master Error" which causes failure
>>>>>> to read EDID from the monitor.
>>
>> ...
>>>>>
>>>>> I believe we have A2s being shipped as well. we dont want to break A2s
>>>>> either -> I propose we refactor the dts to introduce revb1 X15.
>>>>
>>>>
>>>> There are at-least 200 A2's that have been shipped to end users.
>>>>
>>>>
>>>> http://uk.rs-online.com/web/p/processor-microcontroller-development-kits/8874764/
>>>
>>>
>>>
>>> Thanks. Tomi: if you are ok - I already do have a refactoring code from
>>> older kernel. I will rebase to latest and repost.
>>>
>>> Tony: is there a branch youd' like me to base the changes on?
>>
>>
>> Probably best to do it against current omapf-for-v4.9/dt as it has all
>> the skeleton.dtsi removal now applied.
>
>
> Ok will do, will try to post this later today or early tomorrow.

I'm getting this error again with the latest X15 image on my A2
version board.  I don't see this patch in the mainline kernel.  Did it
never get applied?

Thanks
Bill

>
>
> --
> Regards,
> Nishanth Menon
>
> --
> You received this message because you are subscribed to the Google Groups
> "beagleboard-x15" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to beagleboard-x15+unsubscribe@googlegroups.com.
> To post to this group, send email to beagleboard-x15@googlegroups.com.
> To view this discussion on the web visit
> https://groups.google.com/d/msgid/beagleboard-x15/de3bf6ab-a07a-b82d-1e45-5f96a6e1039e%40ti.com.
>
> For more options, visit https://groups.google.com/d/optout.

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [PATCH] ARM: dts: am57xx-beagle-x15: fix HDMI LS_OE GPIO
@ 2017-11-14  0:27             ` Bill Traynor
  0 siblings, 0 replies; 22+ messages in thread
From: Bill Traynor @ 2017-11-14  0:27 UTC (permalink / raw)
  To: linux-arm-kernel

On Thu, Sep 1, 2016 at 11:00 AM, Nishanth Menon <nm@ti.com> wrote:
> On 09/01/2016 09:57 AM, Tony Lindgren wrote:
>>
>> * Nishanth Menon <nm@ti.com> [160901 07:15]:
>>>
>>> On 09/01/2016 09:12 AM, Robert Nelson wrote:
>>>>
>>>> On Thu, Sep 1, 2016 at 9:09 AM, Nishanth Menon <nm@ti.com> wrote:
>>>>>
>>>>> On 09/01/2016 08:51 AM, Tomi Valkeinen wrote:
>>>>>>
>>>>>>
>>>>>> LS_OE GPIO was changed from gpio6_28 to gpio2_30 for the Rev B1 board,
>>>>>> and the current .dts file has the old GPIO.
>>>>>>
>>>>>> Rev B1 is the earliest board supported, so we can just change the
>>>>>> .dts.
>>>>>>
>>>>>> This fixes "omapdss error: HDMI I2C Master Error" which causes failure
>>>>>> to read EDID from the monitor.
>>
>> ...
>>>>>
>>>>> I believe we have A2s being shipped as well. we dont want to break A2s
>>>>> either -> I propose we refactor the dts to introduce revb1 X15.
>>>>
>>>>
>>>> There are at-least 200 A2's that have been shipped to end users.
>>>>
>>>>
>>>> http://uk.rs-online.com/web/p/processor-microcontroller-development-kits/8874764/
>>>
>>>
>>>
>>> Thanks. Tomi: if you are ok - I already do have a refactoring code from
>>> older kernel. I will rebase to latest and repost.
>>>
>>> Tony: is there a branch youd' like me to base the changes on?
>>
>>
>> Probably best to do it against current omapf-for-v4.9/dt as it has all
>> the skeleton.dtsi removal now applied.
>
>
> Ok will do, will try to post this later today or early tomorrow.

I'm getting this error again with the latest X15 image on my A2
version board.  I don't see this patch in the mainline kernel.  Did it
never get applied?

Thanks
Bill

>
>
> --
> Regards,
> Nishanth Menon
>
> --
> You received this message because you are subscribed to the Google Groups
> "beagleboard-x15" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to beagleboard-x15+unsubscribe at googlegroups.com.
> To post to this group, send email to beagleboard-x15 at googlegroups.com.
> To view this discussion on the web visit
> https://groups.google.com/d/msgid/beagleboard-x15/de3bf6ab-a07a-b82d-1e45-5f96a6e1039e%40ti.com.
>
> For more options, visit https://groups.google.com/d/optout.

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH] ARM: dts: am57xx-beagle-x15: fix HDMI LS_OE GPIO
  2017-11-14  0:27             ` Bill Traynor
@ 2017-11-14  7:33               ` Tomi Valkeinen
  -1 siblings, 0 replies; 22+ messages in thread
From: Tomi Valkeinen @ 2017-11-14  7:33 UTC (permalink / raw)
  To: Bill Traynor, Nishanth Menon
  Cc: Tony Lindgren, beagleboard-x15@googlegroups.com,
	linux-omap@vger.kernel.org, Robert Nelson,
	linux-arm-kernel@lists.infradead.org

On 14/11/17 02:27, Bill Traynor wrote:

> I'm getting this error again with the latest X15 image on my A2
> version board.  I don't see this patch in the mainline kernel.  Did it
> never get applied?

With am57xx-beagle-x15.dts? That seems be the A2 version.

-- 
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [PATCH] ARM: dts: am57xx-beagle-x15: fix HDMI LS_OE GPIO
@ 2017-11-14  7:33               ` Tomi Valkeinen
  0 siblings, 0 replies; 22+ messages in thread
From: Tomi Valkeinen @ 2017-11-14  7:33 UTC (permalink / raw)
  To: linux-arm-kernel

On 14/11/17 02:27, Bill Traynor wrote:

> I'm getting this error again with the latest X15 image on my A2
> version board.  I don't see this patch in the mainline kernel.  Did it
> never get applied?

With am57xx-beagle-x15.dts? That seems be the A2 version.

-- 
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH] ARM: dts: am57xx-beagle-x15: fix HDMI LS_OE GPIO
  2017-11-14  7:33               ` Tomi Valkeinen
@ 2017-11-14 14:14                 ` Bill Traynor
  -1 siblings, 0 replies; 22+ messages in thread
From: Bill Traynor @ 2017-11-14 14:14 UTC (permalink / raw)
  To: Tomi Valkeinen
  Cc: Nishanth Menon, Tony Lindgren, beagleboard-x15@googlegroups.com,
	linux-omap@vger.kernel.org, Robert Nelson,
	linux-arm-kernel@lists.infradead.org

On Tue, Nov 14, 2017 at 2:33 AM, Tomi Valkeinen <tomi.valkeinen@ti.com> wrote:
> On 14/11/17 02:27, Bill Traynor wrote:
>
>> I'm getting this error again with the latest X15 image on my A2
>> version board.  I don't see this patch in the mainline kernel.  Did it
>> never get applied?
>
> With am57xx-beagle-x15.dts? That seems be the A2 version.

Yes.
I see the following in the boot message

loading /boot/dtbs/4.9.35-ti-r44/am57xx-beagle-x15.dtb ...

I can get around this by inserting/uninserting an sdcard once the
kernel starts booting.

This is with the Debian 9.0 2017-07-02 4GB SD LXQT image.

Thanks
Bill

>
> --
> Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
> Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [PATCH] ARM: dts: am57xx-beagle-x15: fix HDMI LS_OE GPIO
@ 2017-11-14 14:14                 ` Bill Traynor
  0 siblings, 0 replies; 22+ messages in thread
From: Bill Traynor @ 2017-11-14 14:14 UTC (permalink / raw)
  To: linux-arm-kernel

On Tue, Nov 14, 2017 at 2:33 AM, Tomi Valkeinen <tomi.valkeinen@ti.com> wrote:
> On 14/11/17 02:27, Bill Traynor wrote:
>
>> I'm getting this error again with the latest X15 image on my A2
>> version board.  I don't see this patch in the mainline kernel.  Did it
>> never get applied?
>
> With am57xx-beagle-x15.dts? That seems be the A2 version.

Yes.
I see the following in the boot message

loading /boot/dtbs/4.9.35-ti-r44/am57xx-beagle-x15.dtb ...

I can get around this by inserting/uninserting an sdcard once the
kernel starts booting.

This is with the Debian 9.0 2017-07-02 4GB SD LXQT image.

Thanks
Bill

>
> --
> Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
> Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

^ permalink raw reply	[flat|nested] 22+ messages in thread

* Re: [PATCH] ARM: dts: am57xx-beagle-x15: fix HDMI LS_OE GPIO
  2017-11-14 14:14                 ` Bill Traynor
@ 2017-11-14 16:07                   ` Robert Nelson
  -1 siblings, 0 replies; 22+ messages in thread
From: Robert Nelson @ 2017-11-14 16:07 UTC (permalink / raw)
  To: Bill Traynor
  Cc: Nishanth Menon, Tony Lindgren, beagleboard-x15@googlegroups.com,
	Tomi Valkeinen, linux-omap@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org

On Tue, Nov 14, 2017 at 8:14 AM, Bill Traynor <btraynor@gmail.com> wrote:
> On Tue, Nov 14, 2017 at 2:33 AM, Tomi Valkeinen <tomi.valkeinen@ti.com> wrote:
>> On 14/11/17 02:27, Bill Traynor wrote:
>>
>>> I'm getting this error again with the latest X15 image on my A2
>>> version board.  I don't see this patch in the mainline kernel.  Did it
>>> never get applied?
>>
>> With am57xx-beagle-x15.dts? That seems be the A2 version.
>
> Yes.
> I see the following in the boot message
>
> loading /boot/dtbs/4.9.35-ti-r44/am57xx-beagle-x15.dtb ...
>
> I can get around this by inserting/uninserting an sdcard once the
> kernel starts booting.
>
> This is with the Debian 9.0 2017-07-02 4GB SD LXQT image.

Weird, ldo1 is set to always on:

https://github.com/RobertCNelson/linux-stable-rcn-ee/blob/4.9.35-ti-r44/arch/arm/boot/dts/am57xx-beagle-x15-common.dtsi#L276-L283

Something else is going on..

Regards,

-- 
Robert Nelson
https://rcn-ee.com/

^ permalink raw reply	[flat|nested] 22+ messages in thread

* [PATCH] ARM: dts: am57xx-beagle-x15: fix HDMI LS_OE GPIO
@ 2017-11-14 16:07                   ` Robert Nelson
  0 siblings, 0 replies; 22+ messages in thread
From: Robert Nelson @ 2017-11-14 16:07 UTC (permalink / raw)
  To: linux-arm-kernel

On Tue, Nov 14, 2017 at 8:14 AM, Bill Traynor <btraynor@gmail.com> wrote:
> On Tue, Nov 14, 2017 at 2:33 AM, Tomi Valkeinen <tomi.valkeinen@ti.com> wrote:
>> On 14/11/17 02:27, Bill Traynor wrote:
>>
>>> I'm getting this error again with the latest X15 image on my A2
>>> version board.  I don't see this patch in the mainline kernel.  Did it
>>> never get applied?
>>
>> With am57xx-beagle-x15.dts? That seems be the A2 version.
>
> Yes.
> I see the following in the boot message
>
> loading /boot/dtbs/4.9.35-ti-r44/am57xx-beagle-x15.dtb ...
>
> I can get around this by inserting/uninserting an sdcard once the
> kernel starts booting.
>
> This is with the Debian 9.0 2017-07-02 4GB SD LXQT image.

Weird, ldo1 is set to always on:

https://github.com/RobertCNelson/linux-stable-rcn-ee/blob/4.9.35-ti-r44/arch/arm/boot/dts/am57xx-beagle-x15-common.dtsi#L276-L283

Something else is going on..

Regards,

-- 
Robert Nelson
https://rcn-ee.com/

^ permalink raw reply	[flat|nested] 22+ messages in thread

end of thread, other threads:[~2017-11-14 16:07 UTC | newest]

Thread overview: 22+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-01 13:51 [PATCH] ARM: dts: am57xx-beagle-x15: fix HDMI LS_OE GPIO Tomi Valkeinen
2016-09-01 13:51 ` Tomi Valkeinen
2016-09-01 14:09 ` Nishanth Menon
2016-09-01 14:09   ` Nishanth Menon
2016-09-01 14:12   ` Robert Nelson
2016-09-01 14:12     ` Robert Nelson
2016-09-01 14:14     ` Nishanth Menon
2016-09-01 14:14       ` Nishanth Menon
2016-09-01 14:57       ` Tomi Valkeinen
2016-09-01 14:57         ` Tomi Valkeinen
2016-09-01 14:57       ` Tony Lindgren
2016-09-01 14:57         ` Tony Lindgren
2016-09-01 15:00         ` Nishanth Menon
2016-09-01 15:00           ` Nishanth Menon
2017-11-14  0:27           ` Bill Traynor
2017-11-14  0:27             ` Bill Traynor
2017-11-14  7:33             ` Tomi Valkeinen
2017-11-14  7:33               ` Tomi Valkeinen
2017-11-14 14:14               ` Bill Traynor
2017-11-14 14:14                 ` Bill Traynor
2017-11-14 16:07                 ` Robert Nelson
2017-11-14 16:07                   ` Robert Nelson

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.