lkml.org 
[lkml]   [2010]   [Nov]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH 1/9] Add yaffs Kconfig and Makefile
Date
On Thursday 04 November 2010 06:46:07 Greg KH wrote:
> On Thu, Nov 04, 2010 at 05:53:16AM +1300, cdhmanning@gmail.com wrote:
> > From: Charles Manning <cdhmanning@gmail.com>
> >
> > Adding yaffs2 file system.
> >
> > Signed-off-by: Charles Manning <cdhmanning@gmail.com>
> > ---
> > fs/yaffs2/Kconfig | 190
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++ fs/yaffs2/Makefile |
> > 17 +++++
>
> Don't do this as your first patch, make it your last one. Otherwise the
> build will break when you apply this one to the tree.
> Remember, we need to be able to have a correct build at every patch in
> the whole development series, otherwise tools like 'git bisect' will not
> work properly.

I left the actual hook in to the tree building to the last patch which changes
fs/Kconfig and fs/Makefile. Thus building the whole of Linux would not
break.

I suppose the best is to put all the Kconfigs and Makefile changes in the last
patch.

>
> Also, is your goal to get this cleaned up good enough to get it into .38
> in the "real" place in the kernel, or do you want to see it go through
> staging and take a bit longer to get everything cleaned up properly
> before moving to the "real" location?

My goal it to get this into mainline, .38 preferred.

I am a complete newbie when it comes to this process. What's the best path to
getting there?

My preference is to keep cleaning up code in yaffs.net and patching from there
as this will keep the trees closer together and minimise clerical issues.

It is also a goal to keep the multi-kernel version of the code going in
yaffs.net so that there is always a backport on tap. This is an important
issue for the embedded folk.

From what you have said it sounds like skipping staging might be better, but I
have insufficient understanding of the process to understand why. I'll talk
to some people I know that understand this better.

Thanks

Charles



\
 
 \ /
  Last update: 2010-11-04 22:01    [W:0.193 / U:3.040 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site