<br><br><div class="gmail_quote">On Thu, Apr 9, 2009 at 2:46 PM, Josh Boyer <span dir="ltr">&lt;<a href="mailto:jwboyer@linux.vnet.ibm.com">jwboyer@linux.vnet.ibm.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div><div></div><div class="h5">On Wed, Apr 08, 2009 at 03:11:25PM -0600, John Linn wrote:<br>
&gt;From: Suneel &lt;[mailto:<a href="mailto:suneel.garapati@xilinx.com">suneel.garapati@xilinx.com</a>]&gt;<br>
&gt;<br>
&gt;Added support for the new xps tft controller.<br>
&gt;<br>
&gt;The new core has PLB interface support in addition to existing<br>
&gt;DCR interface.<br>
&gt;<br>
&gt;The driver has been modified to support this new core which<br>
&gt;can be connected on PLB or DCR bus.<br>
&gt;<br>
&gt;Signed-off-by: Suneel &lt;<a href="mailto:suneelg@xilinx.com">suneelg@xilinx.com</a>&gt;<br>
&gt;Signed-off-by: John Linn &lt;<a href="mailto:john.linn@xilinx.com">john.linn@xilinx.com</a>&gt;<br>
&gt;---<br>
&gt; drivers/video/xilinxfb.c |  227 ++++++++++++++++++++++++++++++++--------------<br>
&gt; 1 files changed, 160 insertions(+), 67 deletions(-)<br>
&gt;<br>
&gt;diff --git a/drivers/video/xilinxfb.c b/drivers/video/xilinxfb.c<br>
&gt;index a82c530..a28a834 100644<br>
&gt;--- a/drivers/video/xilinxfb.c<br>
&gt;+++ b/drivers/video/xilinxfb.c<br>
&gt;@@ -1,17 +1,24 @@<br>
&gt; /*<br>
&gt;- * xilinxfb.c<br>
&gt;  *<br>
&gt;- * Xilinx TFT LCD frame buffer driver<br>
&gt;+ * Xilinx TFT frame buffer driver<br>
&gt;  *<br>
&gt;  * Author: MontaVista Software, Inc.<br>
&gt;  *         <a href="mailto:source@mvista.com">source@mvista.com</a><br>
&gt;  *<br>
&gt;  * 2002-2007 (c) MontaVista Software, Inc.<br>
&gt;  * 2007 (c) Secret Lab Technologies, Ltd.<br>
&gt;+ * 2009 (c) Xilinx Inc.<br>
&gt;  *<br>
&gt;- * This file is licensed under the terms of the GNU General Public License<br>
&gt;- * version 2.  This program is licensed &quot;as is&quot; without any warranty of any<br>
&gt;- * kind, whether express or implied.<br>
&gt;+ * This program is free software; you can redistribute it<br>
&gt;+ * and/or modify it under the terms of the GNU General Public<br>
&gt;+ * License as published by the Free Software Foundation;<br>
&gt;+ * either version 2 of the License, or (at your option) any<br>
&gt;+ * later version.<br>
&gt;+ *<br>
&gt;+ * You should have received a copy of the GNU General Public<br>
&gt;+ * License along with this program; if not, write to the Free<br>
&gt;+ * Software Foundation, Inc., 675 Mass Ave, Cambridge, MA<br>
&gt;+ * 02139, USA.<br>
&gt;  */<br>
<br>
</div></div>What Stephen said.<br>
<div class="im"><br>
&gt; #define NUM_REGS      2<br>
&gt; #define REG_FB_ADDR   0<br>
&gt;@@ -112,6 +123,11 @@ struct xilinxfb_drvdata {<br>
&gt;<br>
&gt;       struct fb_info  info;           /* FB driver info record */<br>
&gt;<br>
&gt;+      u32             regs_phys;      /* phys. address of the control<br>
&gt;+                                              registers */<br>
<br>
</div>Is this driver usable on the 440 based Xilinx devices?  If so, is it possible<br>
to have the physical address of the registers above 4GiB, so is common with<br>
almost all the I/O on the other 440 boards?<br>
<div class="im"><br></div></blockquote></div><br>The driver works fine on 440 based Xilinx boards (the ML510 I use has a 440 core). It might be nice to move physical addresses above 4GB for devices but in all Xilinx tools and reference designs addresses below 4GB are used for periperhals and I think even below 2GB (or even below 1GB). It depends on the design.<br>
<br>Roderick Colenbrander<br>