All of lore.kernel.org
 help / color / mirror / Atom feed
From: lee.jones@linaro.org (Lee Jones)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 4/9] ARM: ux500: Move Snowball's thermal DT node into existing PRCMU one
Date: Thu, 4 Apr 2013 08:20:16 +0100	[thread overview]
Message-ID: <20130404072016.GA3004@gmail.com> (raw)
In-Reply-To: <CACRpkda08Joe9aXRgYDUs622GELL+O1vZSnzzCBa++m1zZZi=g@mail.gmail.com>

On Wed, 03 Apr 2013, Linus Walleij wrote:

> On Wed, Apr 3, 2013 at 10:38 PM, Linus Walleij <linus.walleij@linaro.org> wrote:
> > On Tue, Apr 2, 2013 at 3:21 PM, Lee Jones <lee.jones@linaro.org> wrote:
> >
> >> When the thermal DT node was inserted a new PRCMU node was created;
> >> however, one already exists in the Snowball DTS file. Here we
> >> amalgamate the two into a single consolidated node.
> >>
> >> Signed-off-by: Lee Jones <lee.jones@linaro.org>
> >
> > Patch applied.
> 
> After applying this I get this:
> 
>   DTC     arch/arm/boot/dts/snowball.dtb
> Error: /home/elinwal/linux-stericsson/arch/arm/boot/dts/snowball.dts:351.3-352.1
> syntax error
> FATAL ERROR: Unable to parse input tree
> make[3]: *** [arch/arm/boot/dts/snowball.dtb] Error 1
> make[2]: *** [dtbs] Error 2
> make[1]: *** [sub-make] Error 2
> make[1]: Leaving directory `/home/elinwal/linux-stericsson'
> make: *** [build-dtbs] Error 2
> 
> Does not appear prior to applying patch.

That doesn't really tell me anything.

Can you paste the hunk to which the error is referring please?

> It doesn't quiet apply cleanly either, what is this based on?

It's based on 'arm-soc/fixes', which should be appearing in Mainline
rc:s any day now.

-- 
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org ? Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

WARNING: multiple messages have this Message-ID (diff)
From: Lee Jones <lee.jones@linaro.org>
To: Linus Walleij <linus.walleij@linaro.org>
Cc: "linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Arnd Bergmann <arnd@arndb.de>,
	Linus WALLEIJ <linus.walleij@stericsson.com>
Subject: Re: [PATCH 4/9] ARM: ux500: Move Snowball's thermal DT node into existing PRCMU one
Date: Thu, 4 Apr 2013 08:20:16 +0100	[thread overview]
Message-ID: <20130404072016.GA3004@gmail.com> (raw)
In-Reply-To: <CACRpkda08Joe9aXRgYDUs622GELL+O1vZSnzzCBa++m1zZZi=g@mail.gmail.com>

On Wed, 03 Apr 2013, Linus Walleij wrote:

> On Wed, Apr 3, 2013 at 10:38 PM, Linus Walleij <linus.walleij@linaro.org> wrote:
> > On Tue, Apr 2, 2013 at 3:21 PM, Lee Jones <lee.jones@linaro.org> wrote:
> >
> >> When the thermal DT node was inserted a new PRCMU node was created;
> >> however, one already exists in the Snowball DTS file. Here we
> >> amalgamate the two into a single consolidated node.
> >>
> >> Signed-off-by: Lee Jones <lee.jones@linaro.org>
> >
> > Patch applied.
> 
> After applying this I get this:
> 
>   DTC     arch/arm/boot/dts/snowball.dtb
> Error: /home/elinwal/linux-stericsson/arch/arm/boot/dts/snowball.dts:351.3-352.1
> syntax error
> FATAL ERROR: Unable to parse input tree
> make[3]: *** [arch/arm/boot/dts/snowball.dtb] Error 1
> make[2]: *** [dtbs] Error 2
> make[1]: *** [sub-make] Error 2
> make[1]: Leaving directory `/home/elinwal/linux-stericsson'
> make: *** [build-dtbs] Error 2
> 
> Does not appear prior to applying patch.

That doesn't really tell me anything.

Can you paste the hunk to which the error is referring please?

> It doesn't quiet apply cleanly either, what is this based on?

It's based on 'arm-soc/fixes', which should be appearing in Mainline
rc:s any day now.

-- 
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

  parent reply	other threads:[~2013-04-04  7:20 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-02 13:21 [PATCH 1/9] ARM: ux500: Increase the size of the PRCMU's TCPM size Lee Jones
2013-04-02 13:21 ` Lee Jones
2013-04-02 13:21 ` [PATCH 2/9] ARM: ux500: Remove incorrect DB9540 PRCMU TCDM base location Lee Jones
2013-04-02 13:21   ` Lee Jones
2013-04-03 20:33   ` Linus Walleij
2013-04-03 20:33     ` Linus Walleij
2013-04-02 13:21 ` [PATCH 3/9] ARM: ux500: Supply a DTS file for the u8540 platform Lee Jones
2013-04-02 13:21   ` Lee Jones
2013-04-03 20:34   ` Linus Walleij
2013-04-03 20:34     ` Linus Walleij
2013-04-04  7:29     ` [PATCH] " Lee Jones
2013-04-04  7:29       ` Lee Jones
2013-04-23 10:42       ` Linus Walleij
2013-04-23 10:42         ` Linus Walleij
2013-04-23 10:58       ` Linus Walleij
2013-04-23 10:58         ` Linus Walleij
2013-04-24  7:52         ` Lee Jones
2013-04-24  7:52           ` Lee Jones
2013-04-02 13:21 ` [PATCH 4/9] ARM: ux500: Move Snowball's thermal DT node into existing PRCMU one Lee Jones
2013-04-02 13:21   ` Lee Jones
2013-04-03 20:38   ` Linus Walleij
2013-04-03 20:38     ` Linus Walleij
2013-04-03 21:36     ` Linus Walleij
2013-04-03 21:36       ` Linus Walleij
2013-04-03 21:45       ` Linus Walleij
2013-04-03 21:45         ` Linus Walleij
2013-04-04  7:20       ` Lee Jones [this message]
2013-04-04  7:20         ` Lee Jones
2013-04-02 13:21 ` [PATCH 5/9] ARM: ux500: Over-ride TCDM size when booting the DB8540 platform Lee Jones
2013-04-02 13:21   ` Lee Jones
2013-04-03 20:39   ` Linus Walleij
2013-04-03 20:39     ` Linus Walleij
2013-04-04  7:33     ` [PATCH] " Lee Jones
2013-04-04  7:33       ` Lee Jones
2013-04-23 10:59       ` Linus Walleij
2013-04-23 10:59         ` Linus Walleij
2013-04-23 15:09         ` Lee Jones
2013-04-23 15:09           ` Lee Jones
2013-04-02 13:21 ` [PATCH 6/9] ARM: ux500: Create a new of_dev_auxdata structure for u8540 enablement Lee Jones
2013-04-02 13:21   ` Lee Jones
2013-04-03 20:42   ` Linus Walleij
2013-04-03 20:42     ` Linus Walleij
2013-04-02 13:21 ` [PATCH 7/9] ARM: ux500: Add a VQMMC (level-shifting) regulator DT node for Snowball Lee Jones
2013-04-02 13:21   ` Lee Jones
2013-04-03 20:44   ` Linus Walleij
2013-04-03 20:44     ` Linus Walleij
2013-04-02 13:21 ` [PATCH 8/9] mfd: db8500-prcmu: Return early if the TCPM cannot be located Lee Jones
2013-04-02 13:21   ` Lee Jones
2013-04-03 20:46   ` Linus Walleij
2013-04-03 20:46     ` Linus Walleij
2013-04-02 13:21 ` [PATCH 9/9] mfd: db8500-prcmu: Support platform dependant device selection Lee Jones
2013-04-02 13:21   ` Lee Jones
2013-04-03 20:47   ` Linus Walleij
2013-04-03 20:47     ` Linus Walleij
2013-04-04 10:15     ` Lee Jones
2013-04-04 10:15       ` Lee Jones
2013-04-03 20:32 ` [PATCH 1/9] ARM: ux500: Increase the size of the PRCMU's TCPM size Linus Walleij
2013-04-03 20:32   ` Linus Walleij

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=20130404072016.GA3004@gmail.com \
    --to=lee.jones@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.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: link
Be 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.