On Tue, Dec 17, 2013 at 07:29:34PM +0200, Tomi Valkeinen wrote: > >> I added N900 display DT support on top of my v2 series, including > >> pinmuxing. Can you check if it looks right and works? > >> > >> git://git.kernel.org/pub/scm/linux/kernel/git/tomba/linux.git work/dss-dt > > > > I just tried it and it does not work. On a first look the pinmuxing > > looks fishy: 0x0d4 is muxed two times. > > Hmm, so it is. > > I'm not really familiar with SDI, I just muxed all the SDI pins, except > datapair3. I previously thought that there's only the data and clock > pairs for SDI, but the TRM revealed more sdi pins, so I included them. > It is well possible that these can be removed: > > 0x0d0 (PIN_OUTPUT | MUX_MODE1) /* dss_data18.sdi_vsync */ > 0x0d2 (PIN_OUTPUT | MUX_MODE1) /* dss_data19.sdi_hsync */ > 0x0d4 (PIN_OUTPUT | MUX_MODE1) /* dss_data20.sdi_den */ > 0x0d6 (PIN_OUTPUT | MUX_MODE1) /* dss_data21.sdi_stp */
Just removing the dss_data20.sdi_den pin was enough to get a working display. I
don't know if the other pins are needed, because the display pins are already
muxed correctly by the bootloader.
-- Sebastian
diff --git a/arch/arm/boot/dts/omap3-n900.dts b/arch/arm/boot/dts/omap3-n900.dts
index 39e5e50..33f29ac 100644
--- a/arch/arm/boot/dts/omap3-n900.dts
+++ b/arch/arm/boot/dts/omap3-n900.dts
@@ -163,7 +163,7 @@
0x0d0 (PIN_OUTPUT | MUX_MODE1) /*
dss_data18.sdi_vsync */
0x0d2 (PIN_OUTPUT | MUX_MODE1) /*
dss_data19.sdi_hsync */
- 0x0d4 (PIN_OUTPUT | MUX_MODE1) /* dss_data20.sdi_den
*/
+ //0x0d4 (PIN_OUTPUT | MUX_MODE1) /*
dss_data20.sdi_den */
0x0d6 (PIN_OUTPUT | MUX_MODE1) /* dss_data21.sdi_stp
*/
0x0d8 (PIN_OUTPUT | MUX_MODE1) /* dss_data22.sdi_clkp
*/
0x0da (PIN_OUTPUT | MUX_MODE1) /* dss_data23.sdi_clkn
*/
signature.asc
Description: Digital signature
