Message ID | alpine.DEB.2.00.1211011216130.26661@utopia.booyaka.com |
---|---|
State | New |
Headers | show |
* Paul Walmsley <paul@pwsan.com> [121101 05:18]: > Hi Tony > > The following changes since commit 7fc54fd3084457c7f11b9e2e1e3fcd19a3badc33: > > Merge branch 'omap-for-v3.8/cleanup-headers' into omap-for-v3.8/cleanup-prcm (2012-10-26 13:32:22 -0700) > > are available in the git repository at: > > > git://git.kernel.org/pub/scm/linux/kernel/git/pjw/omap-pending.git tags/omap-cleanup-b-for-3.8 > > for you to fetch changes up to f17d60d20eb8e679cdd1e9d507394237e58ce0d8: Tried pulling but.. > These patches remove the use of omap_prcm_get_reset_sources() from the > OMAP watchdog driver, and remove mach-omap2/prcm.c and > plat-omap/include/plat/prcm.h. > > Basic test logs for this branch on top of Tony's cleanup-prcm branch > at commit 7fc54fd3084457c7f11b9e2e1e3fcd19a3badc33 are here: > > http://www.pwsan.com/omap/testlogs/prcm_cleanup_b_3.8/20121101044824/ > > However, cleanup-prcm at 7fc54fd3 does not build for several > Kconfigs here and doesn't include some fixes that are needed for > a successful test. With several reverts, fixes, and workarounds > applied, the following test logs were obtained: ..now wondering which configs do not build for you at 7fc54fd3? It builds at least all my test configs. Well at least the ones I have recovered after accidentally deleting some files yesterday, did not have a back up copy of two of them. However if I pull your branch in, build breaks for me for with the rmk-omap3430-ldp-noconfig and rmk-omap4430-sdp-noconfig: arch/arm/mach-omap2/built-in.o:(.arch.info.init+0x48): undefined reference to `omap44xx_restart' Regards, Tony
On Tue, 6 Nov 2012, Tony Lindgren wrote: > ..now wondering which configs do not build for you at 7fc54fd3? > > It builds at least all my test configs. Well at least the ones > I have recovered after accidentally deleting some files yesterday, > did not have a back up copy of two of them. > > However if I pull your branch in, build breaks for me for > with the rmk-omap3430-ldp-noconfig and rmk-omap4430-sdp-noconfig: > > arch/arm/mach-omap2/built-in.o:(.arch.info.init+0x48): undefined reference to `omap44xx_restart' They all build successfully here. Here are the rmk config results: rmk-omap3430-ldp: http://www.pwsan.com/omap/testlogs/prcm_cleanup_b_3.8/20121101044824/build/rmk_omap3430_ldp_oldconfig/ rmk-omap4430-sdp: http://www.pwsan.com/omap/testlogs/prcm_cleanup_b_3.8/20121101044824/build/rmk_omap4430_sdp_oldconfig/ Maybe the versions of the rmk configs you're using are different than mine? The ones here were updated during the v3.6 -> v3.7-rc1 transition. You can find mine at: http://www.pwsan.com/omap/testlogs/prcm_cleanup_b_3.8/20121101044824/build/rmk_omap3430_ldp_oldconfig/rmk_omap3430_ldp_oldconfig http://www.pwsan.com/omap/testlogs/prcm_cleanup_b_3.8/20121101044824/build/rmk_omap4430_sdp_oldconfig/rmk_omap4430_sdp_oldconfig - Paul
* Paul Walmsley <paul@pwsan.com> [121106 17:54]: > On Tue, 6 Nov 2012, Tony Lindgren wrote: > > > ..now wondering which configs do not build for you at 7fc54fd3? > > > > It builds at least all my test configs. Well at least the ones > > I have recovered after accidentally deleting some files yesterday, > > did not have a back up copy of two of them. The build errors you are talking about are the ones I already pulled into one of the cleanup branches, right? If so, that's fine. > > However if I pull your branch in, build breaks for me for > > with the rmk-omap3430-ldp-noconfig and rmk-omap4430-sdp-noconfig: > > > > arch/arm/mach-omap2/built-in.o:(.arch.info.init+0x48): undefined reference to `omap44xx_restart' > > They all build successfully here. Here are the rmk config results: > > rmk-omap3430-ldp: > http://www.pwsan.com/omap/testlogs/prcm_cleanup_b_3.8/20121101044824/build/rmk_omap3430_ldp_oldconfig/ > > rmk-omap4430-sdp: > http://www.pwsan.com/omap/testlogs/prcm_cleanup_b_3.8/20121101044824/build/rmk_omap4430_sdp_oldconfig/ > > Maybe the versions of the rmk configs you're using are different than > mine? The ones here were updated during the v3.6 -> v3.7-rc1 transition. > You can find mine at: > > http://www.pwsan.com/omap/testlogs/prcm_cleanup_b_3.8/20121101044824/build/rmk_omap3430_ldp_oldconfig/rmk_omap3430_ldp_oldconfig > > http://www.pwsan.com/omap/testlogs/prcm_cleanup_b_3.8/20121101044824/build/rmk_omap4430_sdp_oldconfig/rmk_omap4430_sdp_oldconfig I'm getting errors with the allnoconfig ones, there are total four omap defconfigs there not counting the randconfigs. Regards, Tony
On Tue, 6 Nov 2012, Tony Lindgren wrote: > * Paul Walmsley <paul@pwsan.com> [121106 17:54]: > > On Tue, 6 Nov 2012, Tony Lindgren wrote: > > > > > ..now wondering which configs do not build for you at 7fc54fd3? > > > > > > It builds at least all my test configs. Well at least the ones > > > I have recovered after accidentally deleting some files yesterday, > > > did not have a back up copy of two of them. > > The build errors you are talking about are the ones I already pulled > into one of the cleanup branches, right? If so, that's fine. The build logs I sent were for the "ARM: OMAP: second set of PRCM cleanups for 3.8" branch. The last commit in that branch is f17d60d20eb8e679cdd1e9d507394237e58ce0d8 and it is based on 7fc54fd3084457c7f11b9e2e1e3fcd19a3badc33. That one builds cleanly. If you're asking for a test build of the base commit, 7fc54fd3084457c7f11b9e2e1e3fcd19a3badc33, I just kicked off a test build of that; will post when done. > > Maybe the versions of the rmk configs you're using are different than > > mine? The ones here were updated during the v3.6 -> v3.7-rc1 transition. > > You can find mine at: > > > > http://www.pwsan.com/omap/testlogs/prcm_cleanup_b_3.8/20121101044824/build/rmk_omap3430_ldp_oldconfig/rmk_omap3430_ldp_oldconfig > > > > http://www.pwsan.com/omap/testlogs/prcm_cleanup_b_3.8/20121101044824/build/rmk_omap4430_sdp_oldconfig/rmk_omap4430_sdp_oldconfig > > I'm getting errors with the allnoconfig ones, there are total four > omap defconfigs there not counting the randconfigs. That might indeed explain the discrepancy; so far only have been building his oldconfig seeds here. Will add his allnoconfig seeds and test again. - Paul
On Wed, 7 Nov 2012, Paul Walmsley wrote: > On Tue, 6 Nov 2012, Tony Lindgren wrote: > > > I'm getting errors with the allnoconfig ones, there are total four > > omap defconfigs there not counting the randconfigs. > > That might indeed explain the discrepancy; so far only have been building > his oldconfig seeds here. Will add his allnoconfig seeds and test again. Can reproduce this here. Thanks for catching this. Will fix and send a new pull request today and ensure that the two allnoconfigs are added to the build tests from now on. - Paul
* Paul Walmsley <paul@pwsan.com> [121107 08:57]: > On Wed, 7 Nov 2012, Paul Walmsley wrote: > > > On Tue, 6 Nov 2012, Tony Lindgren wrote: > > > > > I'm getting errors with the allnoconfig ones, there are total four > > > omap defconfigs there not counting the randconfigs. > > > > That might indeed explain the discrepancy; so far only have been building > > his oldconfig seeds here. Will add his allnoconfig seeds and test again. > > Can reproduce this here. Thanks for catching this. Will fix and send a > new pull request today and ensure that the two allnoconfigs are added to > the build tests from now on. OK thanks! Tony
On Wed, 7 Nov 2012, Paul Walmsley wrote: > If you're asking for a test build of the base commit, > 7fc54fd3084457c7f11b9e2e1e3fcd19a3badc33, I just kicked off a test build > of that; will post when done. Just confirmed that this one passes all build tests here. - Paul
* Paul Walmsley <paul@pwsan.com> [121107 10:21]: > On Wed, 7 Nov 2012, Paul Walmsley wrote: > > > If you're asking for a test build of the base commit, > > 7fc54fd3084457c7f11b9e2e1e3fcd19a3badc33, I just kicked off a test build > > of that; will post when done. > > Just confirmed that this one passes all build tests here. OK good to know so no need to merge in anything into cleanup-prcm before pulling in your part2 then. Thanks, Tony
On Wed, 7 Nov 2012, Paul Walmsley wrote: > On Wed, 7 Nov 2012, Paul Walmsley wrote: > > > On Tue, 6 Nov 2012, Tony Lindgren wrote: > > > > > I'm getting errors with the allnoconfig ones, there are total four > > > omap defconfigs there not counting the randconfigs. > > > > That might indeed explain the discrepancy; so far only have been building > > his oldconfig seeds here. Will add his allnoconfig seeds and test again. > > Can reproduce this here. Thanks for catching this. Will fix and send a > new pull request today and ensure that the two allnoconfigs are added to > the build tests from now on. Just a quick followup on this one: I observed a couple of minor regressions during the retest, so, will deal with those before sending out a new pull request. - Paul