Re: [PATCH 1/9] Add yaffs Kconfig and Makefile

[Charles Manning]

On Thursday 04 November 2010 06:46:07 Greg KH wrote:
> On Thu, Nov 04, 2010 at 05:53:16AM +1300, cdhmanning@xxxxxxxxx wrote:
> > From: Charles Manning <cdhmanning@xxxxxxxxx>
> >
> > Adding yaffs2 file system.
> >
> > Signed-off-by: Charles Manning <cdhmanning@xxxxxxxxx>
> > ---
> >  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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/