On Wed, Apr 22, 2009 at 12:58:12PM -0500, Ambrose, Martin wrote: > tatus: O > Content-Length: 2889 > Lines: 78 > > Hi Mark,
Hi Martin. > > -----Original Message----- > > From: davinci-linux-open-source-boun...@linux.davincidsp.com > > [mailto:davinci-linux-open-source-boun...@linux.davincidsp.com] On Behalf > > Of Mark A. Greer > > Sent: Tuesday, April 21, 2009 7:13 PM > > Subject: Re: [PATCH 2/3] ARM: da830 - Add base DA830/OMAP-L137 SoC support > > > > On Tue, Apr 21, 2009 at 05:05:14PM -0700, Mark A. Greer wrote: > > > From: Mark A. Greer <mgr...@mvista.com> > > > > > > The da830/omap l137 is a new SoC from TI that is similar > > > to the davinci line. Since its so similar to davinci, > > > put the support for the da830 in the same directory as > > > the davinci code. > > > > I meant to add a couple notes under the '---': > > > > 1) The writethrough dcache patch is required for this to run > > on a da830 but not required for the patch to apply cleanly. > > I tried the latest one from the linux-davinci list but it had fuzz: > http://linux.omap.com/pipermail/davinci-linux-open-source/2009-April/012557.html > > This one, from the ARM subsystem list and which I am using below, applied > without problems: > http://lists.arm.linux.org.uk/lurker/message/20090402.170938.515dcaa2.en.html Yes, I'm still having issues with l-a-k so I can't send email to that list. > > 2) There will be some "defined but not used" warnings from > > mach-davinci/devices-da830.c because the platform data > > struct hasn't been added to the source yet so I didn't make > > a da830_register_<x>() routine for it. > > > > 3) Applies to ed400fa274a55d88a320ba752a94837b02defc41 > > Thanks for this. It builds for me but hangs after uncompressing the kernel. > Do you see anything wrong with this sequence? > > $ git fetch > $ git branch base ed400fa274a55d88a320ba752a94837b02defc41 > > fm $ git checkout base > $ patch -s -p1 < > ../patches/PATCH-ARM-Add-writethrough-dcache-support-for-ARM926EJS-processor.patch > $ patch -s -p1 < > ../patches/PATCH-1-3-ARM-DaVinci-Remove-unused-compare-reg-defines.patch > $ patch -s -p1 < > ../patches/PATCH-2-3-ARM-da830-Add-base-DA830-OMAP-L137-SoC-support.patch > $ patch -s -p1 < > ../patches/PATCH-3-3-ARM-da830-Add-support-for-DA830-OMAP-L137-EVM.patch > $ make ARCH=arm CROSS_COMPILE=arm-none-linux-gnueabi- da830_omapl137_defconfig > $ make ARCH=arm CROSS_COMPILE=arm-none-linux-gnueabi- uImage Looks like you're fetching but not merging. Try this: - 'git checkout master' - 'git pull' - 'git checkout -b da830_test master' (you'll be put on a branch called da830_test) - 'git am <the patches>' - Build & test to see what happens. Its always a good idea to make a new branch before you change anything (especially if you in the master branch). Branches are cheap in git, very useful, and easy to manipulate. > Also do I still need an RMII patch as before? > http://linux.omap.com/pipermail/davinci-linux-open-source/2009-March/012198.html No. You only need to add the writethru patch and it should compile cleanly and run. > I see that the latest patches have "EMAC_VERSION_2" but no rmii_en setting. > Although it doesn't look like my boot sequence is getting far enough for this > to matter. It isn't. You'll have console output before you run into network issues. Mark -- _______________________________________________ Davinci-linux-open-source mailing list Davinci-linux-open-source@linux.davincidsp.com http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
