<div dir="ltr"><div>Hi Sascha,<br><br></div>I've given the patchset a (very) quick once over, but both myself and Arnaud have tested PATA - any chance you could push them to the ata mailing list to get accepted (I believe that's where they belong) - Feel free to add my Tested-By: and Arnaud said his as well.<br>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Dec 19, 2012 at 2:56 AM, Sascha Hauer <span dir="ltr"><<a href="mailto:s.hauer@pengutronix.de" target="_blank">s.hauer@pengutronix.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Tue, Dec 18, 2012 at 11:30:17AM +0100, Arnaud Patard wrote:<br>
> Arnaud Patard (Rtp) <<a href="mailto:arnaud.patard@rtp-net.org">arnaud.patard@rtp-net.org</a>> writes:<br>
> Hi,<br>
><br>
><br>
> >><br>
> >>><br>
> >>> ><br>
> >>> > I'm not posting the patches here since I do not consider them ready, but<br>
> >>> > for everyone interested you can pull the following git tree. I have no<br>
> >>> > idea if the patches work together with the preinstalled U-Boot, I use<br>
> >>> > barebox on the Smartbook.<br>
> >>><br>
> >>> I've (very quickly) tested it yesterday and it didn't even boot. I've<br>
> >>> yet to look at the serial port output to check if it's a problem on my<br>
> >>> side or a bug in the kernel.<br>
> >><br>
> >> Do you mean barebox doesn't boot or the kernel doesn't boot?<br>
> ><br>
> > No idea. All I get is a black screen, so I need to look at the serial<br>
> > output. Oh, it's uboot not barebox btw. Sorry for the confusion.<br>
><br>
> After digging a little bit, turns out that someone failed to make sure<br>
> that clocks are enabled before reading/writing some<br>
> registers. imx51_ipu_mipi_setup() is called very early in the boot and<br>
> it's way before the clocks are enabled. So, as the clock is not enabled<br>
> when booting on my sb, it's freezing hard. A quick hack to move the call<br>
> to the end of mx51_clocks_init() allows to boot.<br>
<br>
</div>Shawn already noticed this. A fix should appear soon in mainline.<br>
<div class="im"><br>
><br>
> Then, I'm getting a oops. i2c doesn't seem to work (maybe related to<br>
> Matt and Steev remarks about pinctrl settings) so the edid is not found<br>
> and when the mtl017 driver is trying to fall back, it falls back the the<br>
> edid which may be provided by the dts. As it's not the case, it's<br>
> crashing at the memcmp() call. With a new test to check that, I can boot<br>
> on my mmc.<br>
><br>
> PATA is detected too but usb doesn't detect anything on the bus.<br>
><br>
> Next steps will probably be something like checking mtl017/lvds,lcd<br>
> gpios/regulators/pinctrl and then look at usb.<br>
<br>
</div>This is probably because I'm using barebox. The i2c pinmux is probably<br>
configured in the bootloader correctly. That should be easy to fix.<br>
USB may be more work though...<br>
<div class="im"><br>
><br>
> Oh, btw, the smartbook have only 512M of ram, not 1G so you need to fix<br>
> that in your dts file.<br>
<br>
</div>Will do. The bootloader fixes that for me currently, so I didn't realize<br>
this.<br>
<div class="HOEnZb"><div class="h5"><br>
Sascha<br>
<br>
--<br>
Pengutronix e.K.                           |                             |<br>
Industrial Linux Solutions                 | <a href="http://www.pengutronix.de/" target="_blank">http://www.pengutronix.de/</a>  |<br>
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: <a href="tel:%2B49-5121-206917-0" value="+4951212069170">+49-5121-206917-0</a>    |<br>
Amtsgericht Hildesheim, HRA 2686           | Fax:   <a href="tel:%2B49-5121-206917-5555" value="+4951212069175555">+49-5121-206917-5555</a> |<br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Best Regards,<br>Steev<br><br>Steev Klimaszewski <<a href="mailto:steev@genesi-usa.com" target="_blank">steev@genesi-usa.com</a>><br>Senior Software Engineer, Genesi USA, Inc.
</div>