From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Subject: Re: linux-next: origin tree build failure From: Michael Ellerman To: Ingo Molnar In-Reply-To: <20090112090552.GD26750@elte.hu> References: <20090112104837.69feedec.sfr@canb.auug.org.au> <1231719015.22571.4.camel@pasglop> <20090112090552.GD26750@elte.hu> Content-Type: multipart/signed; micalg="pgp-sha1"; protocol="application/pgp-signature"; boundary="=-fMSAHM2yExMWsU+/H95c" Date: Mon, 12 Jan 2009 20:49:51 +1100 Message-Id: <1231753791.8959.7.camel@localhost> Mime-Version: 1.0 Cc: Stephen Rothwell , Rusty Russell , LKML , Mike Travis , linuxppc-dev@ozlabs.org, linux-next@vger.kernel.org, Paul Mackerras , Andrew Morton , Linus Torvalds , "David S. Miller" Reply-To: michael@ellerman.id.au List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , --=-fMSAHM2yExMWsU+/H95c Content-Type: text/plain Content-Transfer-Encoding: quoted-printable On Mon, 2009-01-12 at 10:05 +0100, Ingo Molnar wrote: > * Benjamin Herrenschmidt wrote: >=20 > > On Mon, 2009-01-12 at 10:48 +1100, Stephen Rothwell wrote: > > > Hi Linus, > > >=20 > > > Today's linux-next build (powerpc ppc64_defconfig) failed like this: > > >=20 > > > arch/powerpc/platforms/pasemi/cpufreq.c: In function 'pas_cpufreq_cpu= _init': > > > arch/powerpc/platforms/pasemi/cpufreq.c:216: error: incompatible type= s in assignment > > > arch/powerpc/platforms/powermac/cpufreq_64.c: In function 'g5_cpufreq= _cpu_init': > > > arch/powerpc/platforms/powermac/cpufreq_64.c:365: error: incompatible= types in assignment > > > arch/powerpc/platforms/cell/cbe_cpufreq.c: In function 'cbe_cpufreq_c= pu_init': > > > arch/powerpc/platforms/cell/cbe_cpufreq.c:121: error: incompatible ty= pes in assignment > > >=20 > > > Caused by commit 835481d9bcd65720b473db6b38746a74a3964218 ("cpumask: > > > convert struct cpufreq_policy to cpumask_var_t") which missed updatin= g > > > all the powerpc (at least) cpufreq drivers. > >=20 > > Yeah, it only updates x86 it seems ...=20 >=20 > Yeah - and that build bug was stupid too - when touching a generic file=20 > that is called include/linux/cpufreq.h and changing a key data field one=20 > should at minimum get the idea that it's generic for a reason and should=20 > start grepping the tree ... >=20 > It slipped through because it didnt get caught in build tests because=20 > cpufreq isnt enabled in the powerpc defconfig. Which defconfig? powerpc(master) $ git grep CPU_FREQ=3Dy arch/powerpc/configs/ppc64_defconfi= g arch/powerpc/configs/ppc64_defconfig:CONFIG_CPU_FREQ=3Dy cheers --=20 Michael Ellerman OzLabs, IBM Australia Development Lab wwweb: http://michael.ellerman.id.au phone: +61 2 6212 1183 (tie line 70 21183) We do not inherit the earth from our ancestors, we borrow it from our children. - S.M.A.R.T Person --=-fMSAHM2yExMWsU+/H95c Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.9 (GNU/Linux) iEYEABECAAYFAklrEj8ACgkQdSjSd0sB4dI1sQCeNse/NW8O16+C2QED0qb2owYq fLQAnjZ8C9w39aprGODx0/e2TVMvIZPB =B0P8 -----END PGP SIGNATURE----- --=-fMSAHM2yExMWsU+/H95c--