[PATCH 1/4]new driver initial patch

[Aaron.Chen 陈俊杰]

Hi,

From 0766145c2e2134325b45072e8e52b9bef7ff463b Mon Sep 17 00:00:00 2001
From: Aaron Chen <aaron.chen at siliconmotion.com>
Date: Wed, 1 Aug 2012 16:26:24 +0800
Subject: [PATCH 1/4] new driver initial patch 1

modified man folder, which is suitable for the new driver.

Signed-off-by: Aaron.Chen <aaron.chen at siliconmotion.com>
---
 man/Makefile.am |   21 ++++++++++++++++++---
 1 files changed, 18 insertions(+), 3 deletions(-)

diff --git a/man/Makefile.am b/man/Makefile.am
index e1182ee..c6c85ab 100644
--- a/man/Makefile.am
+++ b/man/Makefile.am
@@ -31,10 +31,25 @@ EXTRA_DIST = @DRIVER_NAME at .man

 CLEANFILES = $(driverman_DATA)

-# String replacements in MAN_SUBSTS now come from xorg-macros.m4 via configure
-
+SED = sed
+
+# Strings to replace in man pages
+XORGRELSTRING = @PACKAGE_STRING@
+  XORGMANNAME = X Version 11
+
+MAN_SUBSTS = \
+       -e 's|__vendorversion__|"$(XORGRELSTRING)" "$(XORGMANNAME)"|' \
+       -e 's|__xorgversion__|"$(XORGRELSTRING)" "$(XORGMANNAME)"|' \
+       -e 's|__xservername__|Xorg|g' \
+       -e 's|__xconfigfile__|xorg.conf|g' \
+       -e 's|__projectroot__|$(prefix)|g' \
+       -e 's|__appmansuffix__|$(APP_MAN_SUFFIX)|g' \
+       -e 's|__drivermansuffix__|$(DRIVER_MAN_SUFFIX)|g' \
+       -e 's|__adminmansuffix__|$(ADMIN_MAN_SUFFIX)|g' \
+       -e 's|__miscmansuffix__|$(MISC_MAN_SUFFIX)|g' \
+       -e 's|__filemansuffix__|$(FILE_MAN_SUFFIX)|g'

 SUFFIXES = .$(DRIVER_MAN_SUFFIX) .man

 .man.$(DRIVER_MAN_SUFFIX):
-       $(AM_V_GEN)$(SED) $(MAN_SUBSTS) < $< > $@
+       sed $(MAN_SUBSTS) < $< > $@
-- 
1.7.5.4





-----邮件原件-----
发件人: Michal Suchanek [mailto:hramrach at gmail.com] 
发送时间: 2012年7月20日 18:43
收件人: Aaron.Chen 陈俊杰
抄送: Matt Turner; xorg-devel at lists.x.org; caesar.qiu 裘赛海; Ilena.Zhou周菁华; Paul.Chen 陈波; mill.chen 陈军
主题: Re: 答复: 答复: [PATCH]new driver for siliconmotion

On 20 July 2012 11:56, Aaron.Chen  陈俊杰 <aaron.chen at siliconmotion.com> wrote:
> Hi Matt&Michal,
>
> Thank you for your advice. We are updating our codes in order to submit the source code successfully.
>
> We are doing the following changes. Next patch will be in the end of 
> this month. (by the way, shall I make a new patch based on older 
> source code or based on last patch?)

The patch(es) should be based on the current upstream smi driver git tree ( http://cgit.freedesktop.org/xorg/driver/xf86-video-siliconmotion/ in this case).

This is somewhat implied but not explicitly written on http://wiki.x.org/wiki/Development/Documentation/SubmittingPatches


> 1. > configure.ac                    |   81 +-
>>There are a lot of useless changes in configure.ac, like lowering the AC_PREREQ number and changing the bugzilla link. Also, the version number in configure.ac is wrong.
>
> We are updating the file.
>
> 2. > man/Makefile.am                 |   64 +-
>>You replaced the Oracle copyright notice with the old Sun copyright notice. Clearly wrong, and it makes it clear that this wasn't well reviewed or rebased.
>
> We've replaced old Sun copyright notice with the Oracle copyright notice. Is it OK?
>
> 3. > src/CALLMAP                     |   22 +
>>This is a useless file that was probably removed from git a long time ago. Don't add it back.
>
> The file has been deleted.
>
> 4. > src/Imakefile                   |  116 +
>>WTF? No. Just no.
>>I see that the driver has RANDR support. No version of the X server ever had an Imake build system and also RANDR support. Remove this Imakefile.
>
> The file has been deleted.
>

All the above issues discovered by the OP show that you have not merged your code properly with upstream git code. That is,  you add back obsolete files that have been removed previously or go back to old revisions of files that have been updated in the git repository.
You need to fix not only the issues explicitly mentioned above but review the patch for possible other similar problems.

> 5.
> +*         Copyright (c) 2007 by Silicon Motion, Inc. (SMI)
> +*
> +*  All rights are reserved. Reproduction or in part is prohibited
> +*  without the written consent of the copyright owner.
>>That doesn't look good.
>
> We will use the same old format:    Copyright (C) YYYY Silicon Motion, Inc.  All Rights Reserved.
> Is this OK?

The part that is definitely not OK is "Reproduction or in part is prohibited
+*  without the written consent of the copyright owner."

Thanks

Michal
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-new-driver-initial-patch-1.patch
Type: application/octet-stream
Size: 1496 bytes
Desc: 0001-new-driver-initial-patch-1.patch
URL: <http://lists.x.org/archives/xorg-devel/attachments/20120801/2a4b3f77/attachment-0001.obj>