<br><br><div class="gmail_quote">2010/10/21 Baruch Siach <span dir="ltr">&lt;<a href="mailto:baruch@tkos.co.il">baruch@tkos.co.il</a>&gt;</span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Hi xiao,<br>
<div class="im"><br>
On Thu, Oct 21, 2010 at 10:08:02AM +0800, xiao jiang wrote:<br>
&gt; 2010/10/21 xiao jiang &lt;<a href="mailto:jgq516@gmail.com">jgq516@gmail.com</a>&gt;<br>
</div><div class="im">&gt; &gt; 2010/10/20 Baruch Siach &lt;<a href="mailto:baruch@tkos.co.il">baruch@tkos.co.il</a>&gt;<br>
</div><div><div></div><div class="h5">&gt; &gt;&gt; On Wed, Oct 20, 2010 at 05:34:43PM +0800, <a href="mailto:jgq516@gmail.com">jgq516@gmail.com</a> wrote:<br>
&gt; &gt;&gt; &gt; From: Xiao Jiang &lt;<a href="mailto:jgq516@gmail.com">jgq516@gmail.com</a>&gt;<br>
&gt; &gt;&gt; &gt;<br>
&gt; &gt;&gt; &gt; Enable debug board support for mx35pdk. As suggested<br>
&gt; &gt;&gt; &gt; by Fabio, add MXC_DEBUG_BOARD config and redefinition<br>
&gt; &gt;&gt; &gt; EXPIO_PARENT_INT.<br>
&gt; &gt;&gt; &gt;<br>
&gt; &gt;&gt; &gt; Signed-off-by: Xiao Jiang &lt;<a href="mailto:jgq516@gmail.com">jgq516@gmail.com</a>&gt;<br>
&gt; &gt;&gt; &gt; Signed-off-by: Fabio Estevam &lt;<a href="mailto:fabio.estevam@freescale.com">fabio.estevam@freescale.com</a>&gt;<br>
&gt; &gt;&gt; &gt; ---<br>
&gt; &gt;&gt; &gt;  arch/arm/mach-mx3/Kconfig         |    1 +<br>
&gt; &gt;&gt; &gt;  arch/arm/mach-mx3/mach-mx35_3ds.c |    8 ++++++++<br>
&gt; &gt;&gt; &gt;  2 files changed, 9 insertions(+), 0 deletions(-)<br>
&gt; &gt;&gt; &gt;<br>
&gt; &gt;&gt; &gt; diff --git a/arch/arm/mach-mx3/Kconfig b/arch/arm/mach-mx3/Kconfig<br>
&gt; &gt;&gt; &gt; index 85beece..cdf6ac5 100644<br>
&gt; &gt;&gt; &gt; --- a/arch/arm/mach-mx3/Kconfig<br>
&gt; &gt;&gt; &gt; +++ b/arch/arm/mach-mx3/Kconfig<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; [snip]<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; &gt; @@ -95,6 +99,10 @@ static void __init mxc_board_init(void)<br>
&gt; &gt;&gt; &gt;       imx35_add_imx_uart0(&amp;uart_pdata);<br>
&gt; &gt;&gt; &gt;<br>
&gt; &gt;&gt; &gt;       mxc_register_device(&amp;mxc_otg_udc_device, &amp;usb_pdata);<br>
&gt; &gt;&gt; &gt; +<br>
&gt; &gt;&gt; &gt; +     if (mxc_expio_init(MX35_CS5_BASE_ADDR, EXPIO_PARENT_INT))<br>
&gt; &gt;&gt; &gt; +             pr_warn(&quot;Init of the debugboard failed, all &quot;<br>
&gt; &gt;&gt; &gt; +                     &quot;devices on the board are unusable.\n&quot;);<br>
&gt; &gt;&gt;                             ^^^^^<br>
&gt; &gt;&gt; You probably mean debugboard here. I&#39;d change the wording.<br>
&gt; &gt;&gt;<br>
&gt; &gt;&gt; There are some devices in the debug board, such as lan9217 ethernet. If<br>
&gt; &gt; mxc_expio_init failed,<br>
&gt; &gt; then the lan9217 can&#39;t be use.<br>
<br>
</div></div>Of course. But this does not make devices on the main board unusable, isn&#39;t<br>
it? The user might think that his whole system is unusable because of this<br>
init failure.<br>
<div class="im"><br></div></blockquote><div>Ok, no relation with main board, to avoid confuse, maybe &quot;all devices on the <b>debug</b> board are unusable&quot; is more make sense, is it ok?<br><br>Thanks,<br>Xiao Jiang<br>
</div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div class="im">
&gt; &gt; BTW: In mach-mx5/board-mx51_3ds.c and mach-mx31_3ds.c, the warn messages<br>
&gt; &gt; are same.<br>
&gt; &gt; if (mxc_expio_init(MX51_CS5_BASE_ADDR,<br>
&gt; &gt; EXPIO_PARENT_INT))<br>
&gt; &gt;<br>
&gt; &gt;                  printk(KERN_WARNING &quot;Init of the debugboard failed, all &quot;<br>
&gt; &gt;<br>
&gt; &gt;                                      &quot;devices on the board are<br>
&gt; &gt; unusable.\n&quot;);<br>
&gt; &gt;<br>
&gt; &gt; Ooo, I pressed  the keyboard neglectfully[?], and you can see below<br>
&gt; sentence in mach-mx31_3ds.c.<br>
&gt; if (!mxc_expio_init(CS5_BASE_ADDR, EXPIO_PARENT_INT))<br>
&gt;          printk(KERN_WARNING &quot;Init of the debugboard failed, all &quot;<br>
&gt;                &quot;devices on the board are unusable.\n&quot;);<br>
<br>
</div>Then this wording should also change there. Adding Jason Wang to Cc.<br>
<br>
baruch<br>
<br>
&gt; Thanks,<br>
&gt; Xiao Jiang<br>
<font color="#888888"><br>
--<br>
</font><div><div></div><div class="h5">                                                     ~. .~   Tk Open Systems<br>
=}------------------------------------------------ooO--U--Ooo------------{=<br>
   - <a href="mailto:baruch@tkos.co.il">baruch@tkos.co.il</a> - tel: +972.2.679.5364, <a href="http://www.tkos.co.il" target="_blank">http://www.tkos.co.il</a> -<br>
</div></div></blockquote></div><br>