On Friday 27 March 2009, liqin.chen@xxxxxxxxxxxxx wrote: > linux/score lastest patch place at > http://www.sunplusct.com/images/linux-score-patch/linux-score-20090324.patch > > diff -uprN -x linux-2.6-git.ori/Documentation/dontdiff First of all, welcome here and thanks for your first posting of patches to the Linux kernel. I'll start looking through your code soon, but would like to give you some generic advices first. It may be a lot of things to learn at the beginning but you've already taken the biggest step of coming out to the public with your patches. It seems that you are not using a specific tool to post your patches, while we have git and quilt (amongst others) that help you prepare the patch files for submission. Most of the points should be listed in Documentation/SubmittingPatches, including * specific subject lines in each mail * threaded mails so that all of them show up together * line wrapping * a multiline patch description in each mail Since the kernel development cycle has entered the merge window for 2.6.30, you should not expect the architecture to go into 2.6.30 but you have enough time for addressing all review comments before 2.6.31, as long as you keep posting updates. Specific comments about code that should be done differently don't mean that you made a mistake, because any way you do it, you will get complaints from somebody ;-) Ideally, you should provide a git tree or a patch set so it can be included in linux-next at first. Please just ask if you need help setting up a git server. Do you have a cross-compiler tool chain for x86 hosts somewhere for download? > --- linux-2.6-git.ori/arch/score/include/asm/a.out.h 1970-01-01 > 08:00:00.000000000 +0800 > +++ linux-2.6-git.new/arch/score/include/asm/a.out.h 2009-03-13 > 14:26:33.000000000 +0800 > @@ -0,0 +1,23 @@ > +#ifndef _ASM_A_OUT_H > +#define _ASM_A_OUT_H > + > +struct exec > +{ > + unsigned long a_info; /* Use macros N_MAGIC, etc for access */ > + unsigned a_text; /* length of text, in bytes */ > + unsigned a_data; /* length of data, in bytes */ > + unsigned a_bss; /* length of uninitialized data area for > + file, in bytes */ > + unsigned a_syms; /* length of symbol table data in file, > + in bytes */ > + unsigned a_entry; /* start address */ > + unsigned a_trsize; /* length of relocation info for text, in > + bytes */ > + unsigned a_drsize; /* length of relocation info for data, in > bytes */ > +}; > + > +#define N_TRSIZE(a) ((a).a_trsize) > +#define N_DRSIZE(a) ((a).a_drsize) > +#define N_SYMSIZE(a) ((a).a_syms) > + > +#endif /* _ASM_A_OUT_H */ New architectures normally don't need a.out support any more, so this could be left out. > linux-2.6-git.ori/arch/score/include/asm/atomic.h > linux-2.6-git.new/arch/score/include/asm/bitops.h For these files (and many more), I have done a generic version that I guess I should really post now so that you and the microblaze guys can just use those files instead of copying new ones. Arnd <>< -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
