<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 TRANSITIONAL//EN">
<HTML>
<HEAD>
  <META HTTP-EQUIV="Content-Type" CONTENT="text/html; CHARSET=UTF-8">
  <META NAME="GENERATOR" CONTENT="GtkHTML/3.26.0">
</HEAD>
<BODY LINK="#0000ff">
On Mon, 2010-01-25 at 16:29 +0800, Jong Lin wrote:
<BLOCKQUOTE TYPE=CITE>
    <FONT COLOR="#000080">This patch includes the following enhancements and some bug fixings.</FONT><BR>
    <BR>
    <FONT COLOR="#000080">(1). Support ARM</FONT><BR>
    <BR>
    <FONT COLOR="#000080">(2). Support EXA acceleration</FONT><BR>
    <BR>
    <FONT COLOR="#000080">(3). Support EDID with I2C function</FONT><BR>
    <BR>
    <FONT COLOR="#000080">(4). Support Custom Modes of modeline</FONT><BR>
    <BR>
    <FONT COLOR="#000080">(5). Unify code for x86, PowerPC, ARM</FONT><BR>
    <BR>
    <BR>
</BLOCKQUOTE>
<BR>
I used 'git apply --check' to verify the patch would apply but it doesn't I get errors like:<BR>
<BLOCKQUOTE>
    error: patch failed: src/init.c:1
    <BLOCKQUOTE>
<PRE>
error: src/init.c: patch does not apply
</PRE>
    </BLOCKQUOTE>
</BLOCKQUOTE>
The patch shows that the whole content of the init.c file has changed, but I can't see any real code difference. Then I remembered about code page issues when translators worked from different platforms (Unix, Windows...). I checked the file with the 'file' command:<BR>
<BR>
<BLOCKQUOTE>
<PRE>
init.c:&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; ASCII C program text, with CRLF line terminators
vb_ext.c:&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; ASCII C program text
vb_init.c:&nbsp;&nbsp;&nbsp;&nbsp; ASCII C program text, with CRLF line terminators
vb_setmode.c:&nbsp; ASCII C program text, with CRLF line terminators
xgi_accel.c:&nbsp;&nbsp; ASCII C program text, with CRLF line terminators
xgi_cursor.c:&nbsp; ASCII C program text
xgi_dac.c:&nbsp;&nbsp;&nbsp;&nbsp; ASCII C program text
xgi_dga.c:&nbsp;&nbsp;&nbsp;&nbsp; ASCII C program text
xgi_dri.c:&nbsp;&nbsp;&nbsp;&nbsp; ASCII C program text
xgi_driver.c:&nbsp; ASCII C program text, with CRLF, LF line terminators
xgi_opt.c:&nbsp;&nbsp;&nbsp;&nbsp; ASCII C program text
xgi_setup.c:&nbsp;&nbsp; ASCII C program text, with CRLF, LF line terminators
xgi_vb.c:&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; ASCII C program text
xgi_vga.c:&nbsp;&nbsp;&nbsp;&nbsp; ASCII C program text, with CRLF line terminators
xgi_video.c:&nbsp;&nbsp; ASCII C program text
xgi_videohw.c: ASCII C program text

</PRE>
</BLOCKQUOTE>
It just so happens that the files giving errors are the one with CRLF terminators. Most likely, xgi developers edit code on platforms having different file formats. I did a quick scan on other drivers and I have not seen files with CRLF terminators.<BR>
<BR>
It's just a theory for now, most likely others have seen this before and know of to fix it.<BR>
<BR>
<BR>
</BODY>
</HTML>