From: Magnus D. <mag...@gm...> - 2007-07-25 04:08:21
|
On 7/25/07, David McCullough <Dav...@se...> wrote: > Jivin Magnus Damm lays it down ... > > On 7/25/07, David McCullough <Dav...@se...> wrote: > > > > > >Jivin Paul Mundt lays it down ... > > >> On Wed, Jul 25, 2007 at 09:51:22AM +0900, Paul Mundt wrote: > > >> Since we're in the purging mood, perhaps it's also worth looking at some > > >> others: > > >> > > >> - snapgear boards are also unlikely to be revitalized, so that's > > >> probably another good candidate. > > > > > >How bad is it ? I am running 2.6.22 here on the snapgear boards and > > >it seems ok. I can send you some diffs if you want ? > > > > As of 2.6.23-rc1: > > > > The snapgear_defconfig seems to be out of sync with the rest of the > > Kconfig - both SH7751 and SH7751R are defined in the defconfig - this > > leads to that only SH7751 is selected which in turn leads to that > > CONFIG_SH_SECUREEDGE5410 never gets set. So the defconfig doesn't > > select the right board. I sent patches fixing a similar issue on r2d > > this a while ago. > > The CONFIG_SH_SECUREEDGE5410 define can (well used to) handle both > 7751 and 7751R CPU's and the kernel adjusted according to whatever HW it > booted on. > > Is this still possible ? According to arch/sh/Kconfig, only 7751R is supported as of 2.6.23-rc1. So if the board can be populated with 7751 or 7751R, then the Kconfig needs to be updated. And maybe we should have two defconfigs to make sure that both combinations build... > > Also, the cpu-specific interrupt code has been rewritten for > > 2.6.23-rc1. The new code seems to work just fine on the r2d board, but > > it wouldn't hurt with some testing on other 7750-ish boards. > > Ok, then I guess I need to be on 2.6.23-rc1, where is the git tree I > should clone to make patches against (yes I should check for the doc :-) I built my stuff on top of 2.6.23-rc1 yesterday, but since then a few patches have been accepted by Paul. So use the sh-2.6 tree: http://git.kernel.org/?p=linux/kernel/git/lethal/sh-2.6.git;a=summary / magnus |