From: Olof Johansson <olof@lixom.net> To: Russell King - ARM Linux <linux@arm.linux.org.uk> Cc: Maxime Ripard <maxime.ripard@free-electrons.com>, Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>, Arnd Bergmann <arnd@arndb.de>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, Emilio Lopez <emilio@elopez.com.ar>, torvalds@linux-foundation.org Subject: Re: [PATCH 0/2] ARM: sunxi: Convert DTSI to new CPU bindings Date: Sat, 29 Jun 2013 15:54:26 -0700 [thread overview] Message-ID: <20130629225426.GA12221@quad.lixom.net> (raw) In-Reply-To: <20130629193819.GD3353@n2100.arm.linux.org.uk> On Sat, Jun 29, 2013 at 08:38:19PM +0100, Russell King - ARM Linux wrote: > On Fri, Jun 28, 2013 at 01:05:42PM -0700, Olof Johansson wrote: > > On Fri, Jun 28, 2013 at 1:03 PM, Maxime Ripard > > <maxime.ripard@free-electrons.com> wrote: > > > On Fri, Jun 28, 2013 at 06:15:32PM +0100, Lorenzo Pieralisi wrote: > > >> The patch above should already be queued in next/dt right ? > > > > > > Indeed. > > > > > > Then why the latest patch of your patchset got in 3.10, while the > > > patches actually fixing the DT it would have impacted were delayed to > > > 3.11? > > > > > > (And why was it merged so late in the development cycle?) > > > > This. So now we have to scramble because some device trees will > > produce warnings at boot. > > > > Russell, the alternative is to revert Lorenzo's patch for 3.10 (and > > re-introduce it for 3.11). Do you have a preference? > > Sorry but I really don't understand what all the fuss in this thread > is about. > > This thread seems to be saying that two development patches were > merged, which were 7762/1 and 7763/1, and that 7764/1 is a fix? > Are you sure about that, because that's not how they're described, > and not how they look either. Most of this ruffle seems to be about the fact that booting a kernel with a device tree that doesn't conform to the brand spanking new, and never previously enforced, binding for the cpu nodes will produce a WARN_ON(). Lots of our in-tree device trees fall into this category. And while I think it was a bad idea for Lorenzo to ask for this to be merged as a fix this late (and most in particular for stable), as far as I can tell nothing (new) is broken by it -- just the alarming warning is being printed. I think it probably makes sense to downgrade the WARN to just a printk, and people will be a lot less worried. How about the below? If you're OK with it, Russell, can we get your ack so Linus can apply directly given the imminence of final 3.10? Or, if you prefer, you can of course apply and send it on instead. Thanks, -Olof ----- ARM: dt: Only print warning, not WARN() on bad cpu map in device tree Due to recent changes and expecations of proper cpu bindings, there are now cases for many of the in-tree devicetrees where a WARN() will hit on boot due to badly formatted /cpus nodes. Downgrade this to a pr_warn() to be less alarmist, since it's not a new problem. Tested on Arndale, Cubox, Seaboard and Panda ES. Panda hits the WARN without this, the others do not. Signed-off-by: Olof Johansson <olof@lixom.net> diff --git a/arch/arm/kernel/devtree.c b/arch/arm/kernel/devtree.c index 0905502..707f99e 100644 --- a/arch/arm/kernel/devtree.c +++ b/arch/arm/kernel/devtree.c @@ -152,9 +152,11 @@ void __init arm_dt_init_cpu_maps(void) tmp_map[i] = hwid; } - if (WARN(!bootcpu_valid, "DT missing boot CPU MPIDR[23:0], " - "fall back to default cpu_logical_map\n")) + if (!bootcpu_valid) { + pr_warn("DT missing boot CPU MPIDR[23:0], fall back to " + "default cpu_logical_map\n"); return; + } /* * Since the boot CPU node contains proper data, and all nodes have
WARNING: multiple messages have this Message-ID (diff)
From: olof@lixom.net (Olof Johansson) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 0/2] ARM: sunxi: Convert DTSI to new CPU bindings Date: Sat, 29 Jun 2013 15:54:26 -0700 [thread overview] Message-ID: <20130629225426.GA12221@quad.lixom.net> (raw) In-Reply-To: <20130629193819.GD3353@n2100.arm.linux.org.uk> On Sat, Jun 29, 2013 at 08:38:19PM +0100, Russell King - ARM Linux wrote: > On Fri, Jun 28, 2013 at 01:05:42PM -0700, Olof Johansson wrote: > > On Fri, Jun 28, 2013 at 1:03 PM, Maxime Ripard > > <maxime.ripard@free-electrons.com> wrote: > > > On Fri, Jun 28, 2013 at 06:15:32PM +0100, Lorenzo Pieralisi wrote: > > >> The patch above should already be queued in next/dt right ? > > > > > > Indeed. > > > > > > Then why the latest patch of your patchset got in 3.10, while the > > > patches actually fixing the DT it would have impacted were delayed to > > > 3.11? > > > > > > (And why was it merged so late in the development cycle?) > > > > This. So now we have to scramble because some device trees will > > produce warnings at boot. > > > > Russell, the alternative is to revert Lorenzo's patch for 3.10 (and > > re-introduce it for 3.11). Do you have a preference? > > Sorry but I really don't understand what all the fuss in this thread > is about. > > This thread seems to be saying that two development patches were > merged, which were 7762/1 and 7763/1, and that 7764/1 is a fix? > Are you sure about that, because that's not how they're described, > and not how they look either. Most of this ruffle seems to be about the fact that booting a kernel with a device tree that doesn't conform to the brand spanking new, and never previously enforced, binding for the cpu nodes will produce a WARN_ON(). Lots of our in-tree device trees fall into this category. And while I think it was a bad idea for Lorenzo to ask for this to be merged as a fix this late (and most in particular for stable), as far as I can tell nothing (new) is broken by it -- just the alarming warning is being printed. I think it probably makes sense to downgrade the WARN to just a printk, and people will be a lot less worried. How about the below? If you're OK with it, Russell, can we get your ack so Linus can apply directly given the imminence of final 3.10? Or, if you prefer, you can of course apply and send it on instead. Thanks, -Olof ----- ARM: dt: Only print warning, not WARN() on bad cpu map in device tree Due to recent changes and expecations of proper cpu bindings, there are now cases for many of the in-tree devicetrees where a WARN() will hit on boot due to badly formatted /cpus nodes. Downgrade this to a pr_warn() to be less alarmist, since it's not a new problem. Tested on Arndale, Cubox, Seaboard and Panda ES. Panda hits the WARN without this, the others do not. Signed-off-by: Olof Johansson <olof@lixom.net> diff --git a/arch/arm/kernel/devtree.c b/arch/arm/kernel/devtree.c index 0905502..707f99e 100644 --- a/arch/arm/kernel/devtree.c +++ b/arch/arm/kernel/devtree.c @@ -152,9 +152,11 @@ void __init arm_dt_init_cpu_maps(void) tmp_map[i] = hwid; } - if (WARN(!bootcpu_valid, "DT missing boot CPU MPIDR[23:0], " - "fall back to default cpu_logical_map\n")) + if (!bootcpu_valid) { + pr_warn("DT missing boot CPU MPIDR[23:0], fall back to " + "default cpu_logical_map\n"); return; + } /* * Since the boot CPU node contains proper data, and all nodes have
next prev parent reply other threads:[~2013-06-29 22:54 UTC|newest] Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top 2013-06-28 16:44 [PATCH 0/2] ARM: sunxi: Convert DTSI to new CPU bindings Maxime Ripard 2013-06-28 16:44 ` Maxime Ripard 2013-06-28 16:44 ` [PATCH 1/2] ARM: dts: sunxi: cpus/cpu nodes dts updates Maxime Ripard 2013-06-28 16:44 ` Maxime Ripard 2013-06-28 16:44 ` [PATCH 2/2] sunxi: a10s: dtsi: Convert cpu node to the new cpu bindings Maxime Ripard 2013-06-28 16:44 ` Maxime Ripard 2013-06-28 17:15 ` [PATCH 0/2] ARM: sunxi: Convert DTSI to new CPU bindings Lorenzo Pieralisi 2013-06-28 17:15 ` Lorenzo Pieralisi 2013-06-28 20:03 ` Maxime Ripard 2013-06-28 20:03 ` Maxime Ripard 2013-06-28 20:05 ` Olof Johansson 2013-06-28 20:05 ` Olof Johansson 2013-06-28 21:45 ` Lorenzo Pieralisi 2013-06-28 21:45 ` Lorenzo Pieralisi 2013-06-29 18:07 ` Maxime Ripard 2013-06-29 18:07 ` Maxime Ripard 2013-06-29 19:03 ` Lorenzo Pieralisi 2013-06-29 19:03 ` Lorenzo Pieralisi 2013-06-29 19:38 ` Russell King - ARM Linux 2013-06-29 19:38 ` Russell King - ARM Linux 2013-06-29 20:11 ` Lorenzo Pieralisi 2013-06-29 20:11 ` Lorenzo Pieralisi 2013-06-29 22:54 ` Olof Johansson [this message] 2013-06-29 22:54 ` Olof Johansson 2013-06-29 23:14 ` Russell King - ARM Linux 2013-06-29 23:14 ` Russell King - ARM Linux 2013-06-29 23:20 ` Olof Johansson 2013-06-29 23:20 ` Olof Johansson 2013-06-30 9:48 ` Lorenzo Pieralisi 2013-06-30 9:48 ` Lorenzo Pieralisi 2013-07-05 10:19 ` Lorenzo Pieralisi 2013-07-05 10:19 ` Lorenzo Pieralisi 2013-07-05 10:19 ` Lorenzo Pieralisi 2013-07-12 9:37 ` Maxime Ripard 2013-07-12 9:37 ` Maxime Ripard 2013-07-12 9:37 ` Maxime Ripard 2013-07-02 12:12 ` Nicolas Ferre 2013-07-02 12:12 ` Nicolas Ferre 2013-07-02 12:56 ` Lorenzo Pieralisi 2013-07-02 12:56 ` Lorenzo Pieralisi
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=20130629225426.GA12221@quad.lixom.net \ --to=olof@lixom.net \ --cc=arnd@arndb.de \ --cc=emilio@elopez.com.ar \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux@arm.linux.org.uk \ --cc=lorenzo.pieralisi@arm.com \ --cc=maxime.ripard@free-electrons.com \ --cc=torvalds@linux-foundation.org \ /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.