<table cellspacing="0" cellpadding="0" border="0" ><tr><td valign="top" style="font: inherit;">Hi Marek,<br>
 1) Thanks for the advice, I will remember to not top post :-)<br>
<br>
 2) I spent lot of time testing your new additions to the mxs SPI driver and it works fine. Great job.<br>
<br>
Regards,<br>
Alain-Serge<br><br><br>--- On <b>Thu, 6/28/12, Marek Vasut <i><marex@denx.de></i></b> wrote:<br><blockquote style="border-left: 2px solid rgb(16, 16, 255); margin-left: 5px; padding-left: 5px;"><br>From: Marek Vasut <marex@denx.de><br>Subject: Re: [PATCH 7/7] ARM: mxs: Add SPI driver for mx233/mx28<br>To: linux-arm-kernel@lists.infradead.org<br>Cc: "Alain-Serge Nagni" <alainsergenagni@yahoo.com>, "Fabio Estevam" <festevam@gmail.com>, "Shawn Guo" <shawn.guo@linaro.org><br>Date: Thursday, June 28, 2012, 6:09 PM<br><br><div class="plainMail">Dear Alain-Serge Nagni,<br><br>> Hi Fabio and Marek,<br>>  So I did rebuild the Kernel and UBoot to support device tree.  I did<br>> reboot the system and this time I’m not having any kernel panic. Which is<br>> a good signJ. After investigation I notice that the  SSP2 node status is<br>> marked “disabled” (in the file imx28.dtsi).  So there is no
 way based on<br>> the patch that was publish to have the imx-spi driver loaded in the kernel<br>> at boot time. <br>> Mareck,<br>>  I guess that  you have a piece of code that you forgot to publish? If not<br>> how can I have your SPI drive to run? <br>>  Right now I guess that the next step would be to define the SPI node in<br>> the im28-evk.dts file.  I will open a thread to see if anybody as a<br>> solution for that. What is your take on that?<br>>  <br>> Thanks guys,<br>> Alain-Serge         <br><br>[...]<br><br>Fabio already answered your question. Lemme teach you a few more rules:<br>1) Do not top post in the email conversation ;-)<br>2) About DT. You're supposed to include imx28.dtsi in your imx28-yourboard.dts <br>file. Then everything that's already defined in the imx28.dtsi will be present <br>in your imx28-yourboard.dts . But the key thing is, if you
 define something <br>again, it will override the stuff already defined in the imx28.dtsi file. But <br>you don't have to redefine everything, just redefine what you need changed.<br><br>Best regards,<br>Marek Vasut<br></div></blockquote></td></tr></table>