Messages in this thread | | | Date | Thu, 3 Jul 2014 16:37:16 -0400 | Subject | Re: [PATCH 2/2] mach-spear: fixed spear1340.c file | From | Nick Krause <> |
| |
Very well then I will read the documentation on Kconfig in order to understand that and fix up my patches for that. On the other hand I will send a email before the patches to tell in what order to apply them. Cheers Nick
On Thu, Jul 3, 2014 at 2:36 PM, Paul Bolle <pebolle@tiscali.nl> wrote: > [I fixed the top posting.] > > On Thu, 2014-07-03 at 14:08 -0400, Nick Krause wrote: >> On Thu, Jul 3, 2014 at 2:04 PM, Paul Bolle <pebolle@tiscali.nl> wrote: >> > This patch, together with patch 1/2, basically moves a chunk of code >> > into a separate file, didn't it? If so, why did you split that move in >> > two patches? >> >> Yes it is and I did it in two patches in order to be more readable. > > It makes it harder to understand the change (I had to _guess_ it was a > move). Moreover, depending on the order that these two patches would be > merged, we could end up with a chunk of code being either included twice > or not included at all, in some range of commits. Neither would be good. > >> > And how does all this work without any changes to a Makefile? >> >> Furthermore I don't known Kconfig well enough to do the Makefile >> for the file I created. > > Then I think you should, well, study the kernel build system before > submitting a change like this. And you can also ask a question or two to > get things going. (Not sure what the relevant list would be.) > > > Paul Bolle >
| |