答复: [PATCH]siliconmotion new driver initial patch

Aaron.Chen 陈俊杰 aaron.chen at siliconmotion.com
Mon Aug 13 19:44:39 PDT 2012


Hi Alan,

Thank you for your reply. We will change the configure.ac/Makefile.am. But one more thing I need your advice. This is a initial patch. There are many code has been changed. In order to make the patch smaller shall I separate the patch into small parts? What kind of small parts shall I made?

Aaron

-----邮件原件-----
发件人: Alan Coopersmith [mailto:alan.coopersmith at oracle.com] 
发送时间: 2012年8月13日 22:35
收件人: Aaron.Chen 陈俊杰
抄送: xorg-devel at lists.x.org
主题: Re: [PATCH]siliconmotion new driver initial patch

On 08/12/12 11:07 PM, Aaron.Chen  陈俊杰 wrote:
> I didn't found this mail show up on the xorg-devel mail list and there is no response. Maybe it didn't sent successfully, so I sent it again.

2.2 MB mails require moderator approval to be sent to the mailing list.
Please don't send multiple copies of multi-megabyte mails.

Though really, you can also assume sending a single 2.2MB patch is a waste of time that no one will ever review, especially since the first few files shown continue to repeat mistakes we already told you we would not accept, such as undoing past cleanups of configure.ac/Makefile.am to revert them to older, less functional states.

-- 
	-Alan Coopersmith-              alan.coopersmith at oracle.com
	 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


More information about the xorg-devel mailing list