<!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: ASCII C program text, with CRLF line terminators
vb_ext.c: ASCII C program text
vb_init.c: ASCII C program text, with CRLF line terminators
vb_setmode.c: ASCII C program text, with CRLF line terminators
xgi_accel.c: ASCII C program text, with CRLF line terminators
xgi_cursor.c: ASCII C program text
xgi_dac.c: ASCII C program text
xgi_dga.c: ASCII C program text
xgi_dri.c: ASCII C program text
xgi_driver.c: ASCII C program text, with CRLF, LF line terminators
xgi_opt.c: ASCII C program text
xgi_setup.c: ASCII C program text, with CRLF, LF line terminators
xgi_vb.c: ASCII C program text
xgi_vga.c: ASCII C program text, with CRLF line terminators
xgi_video.c: 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>