| Message ID | 1472737901-26494-1-git-send-email-tomi.valkeinen@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
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.
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,
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?
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
* 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 -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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.
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. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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.
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 -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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,
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 {
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(-)