<html>
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<div class="moz-cite-prefix">On 13-02-04 10:11 PM, Aaron.Chen 陈俊杰
wrote:<br>
</div>
<blockquote
cite="mid:4CE6A5494DEECD498EBCD4C5488B1AFCEAC6E4@CNDC08.cn.smi.ad"
type="cite">
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
<meta name="Generator" content="Microsoft Word 12 (filtered
medium)">
<style><!--
/* Font Definitions */
@font-face
{font-family:宋体;
panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
{font-family:"\@宋体";
panose-1:2 1 6 0 3 1 1 1 1 1;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0cm;
margin-bottom:.0001pt;
text-align:justify;
text-justify:inter-ideograph;
font-size:10.5pt;
font-family:"Calibri","sans-serif";}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:purple;
text-decoration:underline;}
span.EmailStyle17
{mso-style-type:personal-compose;
font-family:"Calibri","sans-serif";
color:windowtext;}
.MsoChpDefault
{mso-style-type:export-only;}
/* Page Definitions */
@page WordSection1
{size:612.0pt 792.0pt;
margin:72.0pt 90.0pt 72.0pt 90.0pt;}
div.WordSection1
{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
<div class="WordSection1">
<p class="MsoNormal"><span lang="EN-US">This is an initial
patch. We add SM712/502 support. Both Xrandr and none-Xrandr
feature is supported in this source code. You can get two
kinds of binary from different build procedure.<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"><o:p> </o:p></span></p>
<p class="MsoNormal"><span lang="EN-US">This is a completely new
driver from the last one. So the patch is big. Is it OK for
review?<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"><o:p> </o:p></span></p>
<p class="MsoNormal"><span lang="EN-US">Here is what we changed:<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US">COPYING
| 20 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> Makefile.am
| 1 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> Release.txt
| 1 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> configure.ac
| 57 +-<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> man/Makefile.am
| 21 +-</span></p>
</div>
</blockquote>
You have been reverting to a very very old version of the makefile,
removing two years of maintenance. I have seen this a while ago, not
sure if it was for the same driver, but it's déjà vu.<br>
<br>
I don know the code, but if it was all blasted with new code without
any regards for the changes that were done, this patch will not be
accepted.<br>
<blockquote
cite="mid:4CE6A5494DEECD498EBCD4C5488B1AFCEAC6E4@CNDC08.cn.smi.ad"
type="cite">
<div class="WordSection1">
<p class="MsoNormal"><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/Makefile.am
| 89 +-<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_chip.c
| 324 +++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_chip.h
| 136 ++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_clock.c
| 615 +++++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_clock.h
| 132 ++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_ddkdebug.c
| 253 +++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_ddkdebug.h
| 166 ++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_display.c
| 426 ++++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_display.h
| 108 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_hardware.c
| 469 ++++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_hardware.h
| 105 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_help.c
| 69 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_help.h
| 47 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_linux.c
| 417 ++++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_mode.c
| 756 +++++++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_mode.h
| 169 ++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_os.c
| 40 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_os.h
| 377 ++++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_power.c
| 434 ++++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_power.h
| 126 ++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_regdc.h
| 781 +++++++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_regdma.h
| 81 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_reggpio.h
| 329 +++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_regsc.h |
1245 +++++++++++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_regzv.h
| 287 +++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_swi2c.c
| 563 +++++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_swi2c.h
| 47 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk502_voyager.h
| 106 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk712.h
| 34 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk712_chip.c
| 178 ++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk712_chip.h
| 76 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk712_help.c
| 52 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk712_help.h
| 123 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk712_mode.c
| 283 +++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk712_mode.h
| 45 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/ddk712_reg.h
| 37 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_502_crtc.c
| 721 ++++++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_502_driver.c
| 792 +++++++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_502_driver.h
| 345 +++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_502_hw.c
| 162 ++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_502_hw.h
| 52 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_502_output.c
| 483 ++++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_712_crtc.c |
1476 ++++++++++++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_712_driver.c
| 672 ++++++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_712_driver.h
| 123 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_712_hw.c
| 582 +++++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_712_hw.h
| 167 ++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_712_output.c
| 700 ++++++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_accel.c |
1404 +++++++++++-<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_accel.h
| 237 ++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_common.c
| 33 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_common.h
| 700 ++++++<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_crtc.c
| 271 ++-<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_crtc.h
| 37 +-<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_dbg.h
| 52 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_driver.c |
4014 ++++++++++++++++-----------------<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_driver.h
| 99 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_output.c
| 180 +-<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_output.h
| 45 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_ver.h
| 43 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_video.c |
5939 ++++++++++++++++++++++++++++---------------------<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/smi_video.h
| 309 ++-<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> src/version.h
| 39 +<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> 68 files changed, 24116
insertions(+), 5186 deletions(-)<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> mode change 100755
=> 100644 autogen.sh</span></p>
</div>
</blockquote>
The script should remain executable.<br>
<blockquote
cite="mid:4CE6A5494DEECD498EBCD4C5488B1AFCEAC6E4@CNDC08.cn.smi.ad"
type="cite">
<div class="WordSection1">
<p class="MsoNormal"><span lang="EN-US"><o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_chip.c<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_chip.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_clock.c<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_clock.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_ddkdebug.c<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_ddkdebug.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_display.c<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_display.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_hardware.c<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_hardware.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_help.c<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_help.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_linux.c<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_mode.c<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_mode.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_os.c<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_os.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_power.c<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_power.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_regdc.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_regdma.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_reggpio.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_regsc.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_regzv.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_swi2c.c<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_swi2c.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk502_voyager.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk712.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk712_chip.c<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk712_chip.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk712_help.c<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk712_help.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk712_mode.c<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk712_mode.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/ddk712_reg.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/smi_502_crtc.c<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/smi_502_driver.c<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/smi_502_driver.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/smi_502_hw.c<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/smi_502_hw.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/smi_502_output.c<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/smi_712_crtc.c<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/smi_712_driver.c<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/smi_712_driver.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/smi_712_hw.c<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/smi_712_hw.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/smi_712_output.c<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/smi_accel.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/smi_common.c<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/smi_common.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/smi_dbg.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/smi_driver.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/smi_output.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/smi_ver.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"> create mode 100644
src/version.h<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US"><o:p> </o:p></span></p>
<p class="MsoNormal"><span lang="EN-US">diff --git a/COPYING
b/COPYING<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US">index 2101aa1..ada7a72
100644<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US">--- a/COPYING<o:p></o:p></span></p>
<p class="MsoNormal"><span lang="EN-US">+++ b/COPYING<o:p></o:p></span></p>
</div>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
<a class="moz-txt-link-abbreviated" href="mailto:xorg-devel@lists.x.org">xorg-devel@lists.x.org</a>: X.Org development
Archives: <a class="moz-txt-link-freetext" href="http://lists.x.org/archives/xorg-devel">http://lists.x.org/archives/xorg-devel</a>
Info: <a class="moz-txt-link-freetext" href="http://lists.x.org/mailman/listinfo/xorg-devel">http://lists.x.org/mailman/listinfo/xorg-devel</a>
</pre>
</blockquote>
<br>
</body>
</html>