Re: [PATCH 3/15] qe_lib: Add common files
From: Paul Mackerras <hidden>
Date: 2006-10-02 07:37:20
Kim Phillips writes:
qe_lib: Add common files
quoted hunk ↗ jump to hunk
diff --git a/arch/powerpc/sysdev/qe_lib/Makefile b/arch/powerpc/sysdev/qe_lib/Makefile new file mode 100644 index 0000000..9a54a81 --- /dev/null +++ b/arch/powerpc/sysdev/qe_lib/Makefile@@ -0,0 +1,8 @@ +# +# Makefile for the linux ppc-specific parts of QE +# +obj-$(CONFIG_QUICC_ENGINE)+= qe_common.o qe.o qe_ic.o qe_io.o
But qe.c, qe_ic.c and qe_io.c don't exist as of this patch, so the kernel won't build. Also I notice that qe_common.c gets added by this patch and then modified by subsequent patches. Why don't we just add the final version instead? It looks to me like most of this series needs to be committed as one commit so that git bisect still works. Please re-send the patches (except the rheap fix) as a series where the kernel will build and run after each patch in the series. Also, when you are sending on a patch that someone else created, please make the first non-blank line of the message say "From: " followed by the author's name and email address, so we get the attribution right in the git repository. Paul.