| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <20091216090224.GB19749@nokia.com> Date: Wed, 16 Dec 2009 11:02:24 +0200 From: Felipe Balbi <felipe.balbi@...ia.com> To: ext Stephen Rothwell <sfr@...b.auug.org.au> Cc: Liam Girdwood <lrg@...mlogic.co.uk>, "linux-next@...r.kernel.org" <linux-next@...r.kernel.org>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, Balaji T K <balajitk@...com>, Samuel Ortiz <sameo@...ux.intel.com> Subject: Re: linux-next: tree build failure Hi, On Wed, Dec 16, 2009 at 08:21:32AM +0100, ext Stephen Rothwell wrote: >Hi Liam, > >Today's linux-next build (powerpc allyesconfig) failed like this: > >drivers/regulator/twl-regulator.c: In function 'twlreg_disable': >drivers/regulator/twl-regulator.c:158: error: 'P1_GRP' undeclared (first use in this function) >drivers/regulator/twl-regulator.c:158: error: 'P2_GRP' undeclared (first use in this function) >drivers/regulator/twl-regulator.c:158: error: 'P3_GRP' undeclared (first use in this function) >drivers/regulator/twl-regulator.c: At top level: >drivers/regulator/twl-regulator.c:358: error: 'twlldo_list_voltage' undeclared here (not in a function) >drivers/regulator/twl-regulator.c:450:35: error: macro "TWL_ADJUSTABLE_LDO" requires 4 arguments, but only 3 given >drivers/regulator/twl-regulator.c:450: error: 'TWL_ADJUSTABLE_LDO' undeclared here (not in a function) >drivers/regulator/twl-regulator.c:451:40: error: macro "TWL_ADJUSTABLE_LDO" requires 4 arguments, but only 3 given >drivers/regulator/twl-regulator.c:452:35: error: macro "TWL_ADJUSTABLE_LDO" requires 4 arguments, but only 3 given >drivers/regulator/twl-regulator.c:453:35: error: macro "TWL_ADJUSTABLE_LDO" requires 4 arguments, but only 3 given >drivers/regulator/twl-regulator.c:454:35: error: macro "TWL_ADJUSTABLE_LDO" requires 4 arguments, but only 3 given >drivers/regulator/twl-regulator.c:455:35: error: macro "TWL_ADJUSTABLE_LDO" requires 4 arguments, but only 3 given >drivers/regulator/twl-regulator.c:456:35: error: macro "TWL_ADJUSTABLE_LDO" requires 4 arguments, but only 3 given >drivers/regulator/twl-regulator.c:457:35: error: macro "TWL_ADJUSTABLE_LDO" requires 4 arguments, but only 3 given >drivers/regulator/twl-regulator.c:458:35: error: macro "TWL_ADJUSTABLE_LDO" requires 4 arguments, but only 3 given >drivers/regulator/twl-regulator.c:459:34: error: macro "TWL_ADJUSTABLE_LDO" requires 4 arguments, but only 3 given >drivers/regulator/twl-regulator.c:460:35: error: macro "TWL_ADJUSTABLE_LDO" requires 4 arguments, but only 3 given >drivers/regulator/twl-regulator.c:461:40: error: macro "TWL_FIXED_LDO" requires 5 arguments, but only 4 given >drivers/regulator/twl-regulator.c:461: error: 'TWL_FIXED_LDO' undeclared here (not in a function) >drivers/regulator/twl-regulator.c:462:39: error: macro "TWL_ADJUSTABLE_LDO" requires 4 arguments, but only 3 given >drivers/regulator/twl-regulator.c:463:39: error: macro "TWL_FIXED_LDO" requires 5 arguments, but only 4 given >drivers/regulator/twl-regulator.c:464:34: error: macro "TWL_ADJUSTABLE_LDO" requires 4 arguments, but only 3 given >drivers/regulator/twl-regulator.c:465:35: error: macro "TWL_ADJUSTABLE_LDO" requires 4 arguments, but only 3 given >drivers/regulator/twl-regulator.c:466:35: error: macro "TWL_ADJUSTABLE_LDO" requires 4 arguments, but only 3 given >drivers/regulator/twl-regulator.c:467:39: error: macro "TWL_FIXED_LDO" requires 5 arguments, but only 4 given >drivers/regulator/twl-regulator.c:468:39: error: macro "TWL_FIXED_LDO" requires 5 arguments, but only 4 given >drivers/regulator/twl-regulator.c:469:39: error: macro "TWL_FIXED_LDO" requires 5 arguments, but only 4 given >drivers/regulator/twl-regulator.c:474: error: 'VAUX1_6030_VSEL_table' undeclared here (not in a function) >drivers/regulator/twl-regulator.c:474: warning: type defaults to 'int' in declaration of 'type name' >drivers/regulator/twl-regulator.c:474: warning: type defaults to 'int' in declaration of 'type name' >drivers/regulator/twl-regulator.c:474: error: negative width in bit-field '<anonymous>' >drivers/regulator/twl-regulator.c:474: warning: type defaults to 'int' in declaration of 'type name' >drivers/regulator/twl-regulator.c:474: warning: type defaults to 'int' in declaration of 'type name' >drivers/regulator/twl-regulator.c:474: error: negative width in bit-field '<anonymous>' >drivers/regulator/twl-regulator.c:475: error: 'VAUX2_6030_VSEL_table' undeclared here (not in a function) >drivers/regulator/twl-regulator.c:475: warning: type defaults to 'int' in declaration of 'type name' >drivers/regulator/twl-regulator.c:475: warning: type defaults to 'int' in declaration of 'type name' >drivers/regulator/twl-regulator.c:475: error: negative width in bit-field '<anonymous>' >drivers/regulator/twl-regulator.c:475: warning: type defaults to 'int' in declaration of 'type name' >drivers/regulator/twl-regulator.c:475: warning: type defaults to 'int' in declaration of 'type name' >drivers/regulator/twl-regulator.c:475: error: negative width in bit-field '<anonymous>' >drivers/regulator/twl-regulator.c:476: error: 'VAUX3_6030_VSEL_table' undeclared here (not in a function) >drivers/regulator/twl-regulator.c:476: warning: type defaults to 'int' in declaration of 'type name' >drivers/regulator/twl-regulator.c:476: warning: type defaults to 'int' in declaration of 'type name' >drivers/regulator/twl-regulator.c:476: error: negative width in bit-field '<anonymous>' >drivers/regulator/twl-regulator.c:476: warning: type defaults to 'int' in declaration of 'type name' >drivers/regulator/twl-regulator.c:476: warning: type defaults to 'int' in declaration of 'type name' >drivers/regulator/twl-regulator.c:476: error: negative width in bit-field '<anonymous>' >drivers/regulator/twl-regulator.c:477: error: 'VMMC_VSEL_table' undeclared here (not in a function) >drivers/regulator/twl-regulator.c:477: warning: type defaults to 'int' in declaration of 'type name' >drivers/regulator/twl-regulator.c:477: warning: type defaults to 'int' in declaration of 'type name' >drivers/regulator/twl-regulator.c:477: error: negative width in bit-field '<anonymous>' >drivers/regulator/twl-regulator.c:477: warning: type defaults to 'int' in declaration of 'type name' >drivers/regulator/twl-regulator.c:477: warning: type defaults to 'int' in declaration of 'type name' >drivers/regulator/twl-regulator.c:477: error: negative width in bit-field '<anonymous>' >drivers/regulator/twl-regulator.c:478: error: 'VPP_VSEL_table' undeclared here (not in a function) >drivers/regulator/twl-regulator.c:478: warning: type defaults to 'int' in declaration of 'type name' >drivers/regulator/twl-regulator.c:478: warning: type defaults to 'int' in declaration of 'type name' >drivers/regulator/twl-regulator.c:478: error: negative width in bit-field '<anonymous>' >drivers/regulator/twl-regulator.c:478: warning: type defaults to 'int' in declaration of 'type name' >drivers/regulator/twl-regulator.c:478: warning: type defaults to 'int' in declaration of 'type name' >drivers/regulator/twl-regulator.c:478: error: negative width in bit-field '<anonymous>' >drivers/regulator/twl-regulator.c:479: error: 'VUSIM_VSEL_table' undeclared here (not in a function) >drivers/regulator/twl-regulator.c:479: warning: type defaults to 'int' in declaration of 'type name' >drivers/regulator/twl-regulator.c:479: warning: type defaults to 'int' in declaration of 'type name' >drivers/regulator/twl-regulator.c:479: error: negative width in bit-field '<anonymous>' >drivers/regulator/twl-regulator.c:479: warning: type defaults to 'int' in declaration of 'type name' >drivers/regulator/twl-regulator.c:479: warning: type defaults to 'int' in declaration of 'type name' >drivers/regulator/twl-regulator.c:479: error: negative width in bit-field '<anonymous>' >drivers/regulator/twl-regulator.c: In function 'twlreg_probe': >drivers/regulator/twl-regulator.c:544: error: implicit declaration of function 'twl4030reg_write' > >I guess we knew this was coming :-( > >Caused by a bad rebase of the voltage tree onto Linus' tree (in order to >do fixups for commit fc7b92fca4e546184557f1c53f84ad57c66b7695 "mfd: >Rename all twl4030_i2c*" and others). > >I have reverted all the commits in the voltage tree that touched >drivers/regulator/twl-regulator.c for today: > >6051d7a2786c57d0c5c4227dd9ddfc348a86ce91 "twl4030-regulator: Fixes VAUX1-3 exclusion introduced" >2677c78184a52edcc228c20a1c05c14925118293 "Reset REMAP configuration in regulator probe" >de449258306274739ed3a747f49b9ac8f91f9e4a "Define critical regulators as always_on" >66d65d352f6fd61dd5695925906b140f1e0ac9c3 "twl4030-regulator: Add all TWL regulators to twreg_info" >f33a0d47bd44dbf44a45c33b32e1e3bbbf8d06e6 "twl4030-regulator: Remove regulator from all groups when disabling" > >Please fix up this tree properly. You may have been better off merging >Linus' tree into yours and fixing up the merge commit than trying to do >the rebase. All this crap would not have come if we weren't to rename twl4030 into twl. The argument that we need to rename files to make the code reusable is plain STUPID. I made my point several times that we don't need to rename the files to make the code reusable, twl4030-*.c were already supporting twl4030, twl5030, twl5031 and some of the tps versions. Now several trees are broken because of the stupid mess created by one person who can't even run make when creating a patch. Now we have to pay the price and have a broken bisect point, great. -- balbi -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists