From: Felipe Balbi <balbi@ti.com> To: Marek Belisko <marek@goldelico.com>, Sebastian Reichel <sre@kernel.org> Cc: tony@atomide.com, linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: Linus HEAD build break Date: Fri, 6 Nov 2015 08:24:07 -0600 [thread overview] Message-ID: <87y4ebyznc.fsf@saruman.tx.rr.com> (raw) In-Reply-To: <871tc31atk.fsf@saruman.tx.rr.com> [-- Attachment #1.1: Type: text/plain, Size: 1739 bytes --] Hi again, Felipe Balbi <balbi@ti.com> writes: > Hi Marek, > > your commit af19161aaed7 ("ARM: dts: twl4030: Add iio properties for bci > subnode") breaks build on current linus/master (which current sits in this commit cannot be found in next. How come it's in linus/master ? If it had been in next, we'd find out about it a lot sooner and avoid the build regression. It seems author didn't build his own patch, considering the error is clear. Here's a patch, please merge it or provide a better version. Below has been build tested (omap2plus_defconfig) and boot tested (am437x SK, am437x IDK, beaglebone black) (yeah, I don't have boards which sport MADC around). 8<--------------------------------------------------------------- From 7af26f7123bad3e9c9dcfe9dfe9f4ad58e56f7cb Mon Sep 17 00:00:00 2001 From: Felipe Balbi <balbi@ti.com> Date: Fri, 6 Nov 2015 08:14:45 -0600 Subject: [PATCH] ARM: dts: twl4030: fix Monitoring ADC label commit af19161aaed7 ("ARM: dts: twl4030: Add iio properties for bci subnode") breaks build of several DTS files by referring to a label that doesn't exist. Fix it. Signed-off-by: Felipe Balbi <balbi@ti.com> --- arch/arm/boot/dts/twl4030.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/twl4030.dtsi b/arch/arm/boot/dts/twl4030.dtsi index 482b7aa37808..c8197f209efc 100644 --- a/arch/arm/boot/dts/twl4030.dtsi +++ b/arch/arm/boot/dts/twl4030.dtsi @@ -22,7 +22,7 @@ charger: bci { compatible = "ti,twl4030-bci"; interrupts = <9>, <2>; - io-channels = <&twl4030_madc 11>; + io-channels = <&twl_madc 11>; io-channel-name = "vac"; bci3v1-supply = <&vusb3v1>; }; -- 2.6.2 -- balbi [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 818 bytes --] [-- Attachment #2: Type: text/plain, Size: 176 bytes --] _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
WARNING: multiple messages have this Message-ID (diff)
From: balbi@ti.com (Felipe Balbi) To: linux-arm-kernel@lists.infradead.org Subject: Linus HEAD build break Date: Fri, 6 Nov 2015 08:24:07 -0600 [thread overview] Message-ID: <87y4ebyznc.fsf@saruman.tx.rr.com> (raw) In-Reply-To: <871tc31atk.fsf@saruman.tx.rr.com> Hi again, Felipe Balbi <balbi@ti.com> writes: > Hi Marek, > > your commit af19161aaed7 ("ARM: dts: twl4030: Add iio properties for bci > subnode") breaks build on current linus/master (which current sits in this commit cannot be found in next. How come it's in linus/master ? If it had been in next, we'd find out about it a lot sooner and avoid the build regression. It seems author didn't build his own patch, considering the error is clear. Here's a patch, please merge it or provide a better version. Below has been build tested (omap2plus_defconfig) and boot tested (am437x SK, am437x IDK, beaglebone black) (yeah, I don't have boards which sport MADC around). 8<--------------------------------------------------------------- >From 7af26f7123bad3e9c9dcfe9dfe9f4ad58e56f7cb Mon Sep 17 00:00:00 2001 From: Felipe Balbi <balbi@ti.com> Date: Fri, 6 Nov 2015 08:14:45 -0600 Subject: [PATCH] ARM: dts: twl4030: fix Monitoring ADC label commit af19161aaed7 ("ARM: dts: twl4030: Add iio properties for bci subnode") breaks build of several DTS files by referring to a label that doesn't exist. Fix it. Signed-off-by: Felipe Balbi <balbi@ti.com> --- arch/arm/boot/dts/twl4030.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/twl4030.dtsi b/arch/arm/boot/dts/twl4030.dtsi index 482b7aa37808..c8197f209efc 100644 --- a/arch/arm/boot/dts/twl4030.dtsi +++ b/arch/arm/boot/dts/twl4030.dtsi @@ -22,7 +22,7 @@ charger: bci { compatible = "ti,twl4030-bci"; interrupts = <9>, <2>; - io-channels = <&twl4030_madc 11>; + io-channels = <&twl_madc 11>; io-channel-name = "vac"; bci3v1-supply = <&vusb3v1>; }; -- 2.6.2 -- balbi -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 818 bytes Desc: not available URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20151106/2385362c/attachment.sig>
next prev parent reply other threads:[~2015-11-06 14:24 UTC|newest] Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-11-06 14:06 Linus HEAD build break Felipe Balbi 2015-11-06 14:06 ` Felipe Balbi 2015-11-06 14:24 ` Felipe Balbi [this message] 2015-11-06 14:24 ` Felipe Balbi 2015-11-06 14:31 ` Belisko Marek 2015-11-06 14:31 ` Belisko Marek 2015-11-06 14:40 ` Felipe Balbi 2015-11-06 14:40 ` Felipe Balbi 2015-11-06 14:53 ` Tony Lindgren 2015-11-06 14:53 ` Tony Lindgren 2015-11-06 15:34 ` Sebastian Reichel 2015-11-06 15:34 ` Sebastian Reichel 2015-11-06 16:09 ` Sebastian Reichel 2015-11-06 16:09 ` Sebastian Reichel 2015-11-06 19:03 ` Tony Lindgren 2015-11-06 19:03 ` Tony Lindgren
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=87y4ebyznc.fsf@saruman.tx.rr.com \ --to=balbi@ti.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-omap@vger.kernel.org \ --cc=marek@goldelico.com \ --cc=sre@kernel.org \ --cc=tony@atomide.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.