Re: MTD patches for 2.0 / 2.2 kernels

[David Woodhouse <dwmw2@infradead.org>]

eauth@softsys.co.at said:
>  Wouldn' t it be much nicer to have a patch script (like patchin.sh),
> that  1. makes symlinks from the kernel source into the mtd tree, and
> 2. applies a very small patch for the kernel source  (init/main.c,
> drivers/Makefile, ...)

Yes, I think so too. Steven Hill, who generated the patches you're looking
at, had people asking for 'complete' patches. So that's what he produced.
But the 'infrastructure' patch would also be useful. It's just that the
existing ones were out of date, so we removed them.


eauth@softsys.co.at said:
>  I'd like to see all the patches inside the CVS distribution, instead
> of having them to download via ftp seperately. 

If they're only for the infrastructure, that's fine.

The complete patches were removed because they were too big.
The infrastructure patches were removed because they were out of date.

eauth@softsys.co.at said:
> Is there a special reason why there is a seperate link for  each mtd
> source file in patchin.sh, instead of having just one symbolic link
> for the mtd directory?

It's so that the CVS subdirectory of drivers/mtd/ can be different.

I tend to work with my main kernel tree for the board I'm working on ATM in 
CVS. Obviously I also have the MTD code in CVS. So I want to update the 
code in my build tree, I update the _original_ MTD CVS, go to the build 
tree, and 'cvs commit'. Sometimes I actually check it still works between 
those last two steps :)

It also means that when I have map drivers and strange partitioning schemes
for random boards that are under NDA, I can keep them in the internal CVS
tree without accidentally committing them to the public repository.

It does mean that you don't get new files automatically appearing in your 
build tree, but in general, you don't really need the new files unless 
you're starting to use new features, which isn't usually the case once your 
development is under way for a particular target.

But if you're volunteering to keep the 'infrastructure' patches and the 
patchin script up to date, you get to choose how it's done. I hardly touch 
2.2 kernel nowadays anyway, except when I make an effort to check my code 
still works on them.

--
dwmw2




To unsubscribe, send "unsubscribe mtd" to majordomo@infradead.org