All of lore.kernel.org
 help / color / mirror / Atom feed
From: dave.martin@linaro.org (Dave Martin)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3 0/5] Versatile Express DT support
Date: Tue, 29 Nov 2011 17:00:13 +0000	[thread overview]
Message-ID: <20111129170013.GD2829@localhost.localdomain> (raw)
In-Reply-To: <1322579473-8804-1-git-send-email-pawel.moll@arm.com>

On Tue, Nov 29, 2011 at 03:11:08PM +0000, Pawel Moll wrote:
> Hi All,
> 
> Another, hopefully close to the final, version of the series...
> 
> Changes from v2:
> 
> * new patch adding ENDPROC at the end of arch/arm/plat-versatile/headsmp.S
>   and removing use of BSYM() realview and vexpress platform SMP calls
> 
> * updated binding documentation according to Dave's comments
> 
> * changed motherboard labels prefix from mb_ to v2m_ (to be
>   consistent across code and DT files)
> 
> * reworked v2m timer initialization (again) so no #ifdefs are required
>   in non-DT code
> 
> * renamed (to vendor- and platform-specific "arm,v2m-timer") and moved
>   (to vexpress-v2m*.dtsi files) timer DT alias, as it is actually
>   motherboard specific and the tile DTS shouldn't have to care;
>   documentation updated accordingly
> 
> Tested on V2P-CA9 with ATAGs and DT and on V2P-CA5s with DT.
> 
> Previous version has been also tested by Ryan Harkin and
> provisionally acked by Rob Herring.
> 
> Thanks for all your help, especially to Dave for his continuous support!

I'm trying to build... what tree are you basing on?

With v3.2-rc3 or next, I get OF header failures like the following:

arch/arm/kernel/devtree.c: In function ?arm_dt_memblock_reserve?:
arch/arm/kernel/devtree.c:41:7: error: ?initial_boot_params? undeclared (first use in this function)
arch/arm/kernel/devtree.c:41:7: note: each undeclared identifier is reported only once for each function it appears in

[...]

arch/arm/mach-vexpress/v2m.c: In function ?v2m_dt_scan_memory_map?:
arch/arm/mach-vexpress/v2m.c:529:2: error: implicit declaration of function ?of_get_flat_dt_prop?
arch/arm/mach-vexpress/v2m.c:529:7: warning: assignment makes pointer from integer without a cast

arch/arm/mach-vexpress/v2m.c: In function ?v2m_dt_map_io?:
arch/arm/mach-vexpress/v2m.c:538:2: error: implicit declaration of function ?of_scan_flat_dt?

Cheers
---Dave

WARNING: multiple messages have this Message-ID (diff)
From: Dave Martin <dave.martin-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
To: Pawel Moll <pawel.moll-5wv7dgnIgG8@public.gmane.org>
Cc: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [PATCH v3 0/5] Versatile Express DT support
Date: Tue, 29 Nov 2011 17:00:13 +0000	[thread overview]
Message-ID: <20111129170013.GD2829@localhost.localdomain> (raw)
In-Reply-To: <1322579473-8804-1-git-send-email-pawel.moll-5wv7dgnIgG8@public.gmane.org>

On Tue, Nov 29, 2011 at 03:11:08PM +0000, Pawel Moll wrote:
> Hi All,
> 
> Another, hopefully close to the final, version of the series...
> 
> Changes from v2:
> 
> * new patch adding ENDPROC at the end of arch/arm/plat-versatile/headsmp.S
>   and removing use of BSYM() realview and vexpress platform SMP calls
> 
> * updated binding documentation according to Dave's comments
> 
> * changed motherboard labels prefix from mb_ to v2m_ (to be
>   consistent across code and DT files)
> 
> * reworked v2m timer initialization (again) so no #ifdefs are required
>   in non-DT code
> 
> * renamed (to vendor- and platform-specific "arm,v2m-timer") and moved
>   (to vexpress-v2m*.dtsi files) timer DT alias, as it is actually
>   motherboard specific and the tile DTS shouldn't have to care;
>   documentation updated accordingly
> 
> Tested on V2P-CA9 with ATAGs and DT and on V2P-CA5s with DT.
> 
> Previous version has been also tested by Ryan Harkin and
> provisionally acked by Rob Herring.
> 
> Thanks for all your help, especially to Dave for his continuous support!

I'm trying to build... what tree are you basing on?

With v3.2-rc3 or next, I get OF header failures like the following:

arch/arm/kernel/devtree.c: In function ‘arm_dt_memblock_reserve’:
arch/arm/kernel/devtree.c:41:7: error: ‘initial_boot_params’ undeclared (first use in this function)
arch/arm/kernel/devtree.c:41:7: note: each undeclared identifier is reported only once for each function it appears in

[...]

arch/arm/mach-vexpress/v2m.c: In function ‘v2m_dt_scan_memory_map’:
arch/arm/mach-vexpress/v2m.c:529:2: error: implicit declaration of function ‘of_get_flat_dt_prop’
arch/arm/mach-vexpress/v2m.c:529:7: warning: assignment makes pointer from integer without a cast

arch/arm/mach-vexpress/v2m.c: In function ‘v2m_dt_map_io’:
arch/arm/mach-vexpress/v2m.c:538:2: error: implicit declaration of function ‘of_scan_flat_dt’

Cheers
---Dave
_______________________________________________
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/devicetree-discuss

  parent reply	other threads:[~2011-11-29 17:00 UTC|newest]

Thread overview: 106+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-29 15:11 [PATCH v3 0/5] Versatile Express DT support Pawel Moll
2011-11-29 15:11 ` Pawel Moll
2011-11-29 15:11 ` [PATCH v3 1/5] ARM: versatile: Add missing ENDPROC to headsmp.S Pawel Moll
2011-11-29 15:11   ` Pawel Moll
2011-11-29 15:11 ` [PATCH v3 2/5] ARM: vexpress: Get rid of MMIO_P2V Pawel Moll
2011-11-29 15:11   ` Pawel Moll
2011-11-29 21:15   ` Arnd Bergmann
2011-11-29 21:15     ` Arnd Bergmann
2011-11-30 11:34     ` Pawel Moll
2011-11-30 11:34       ` Pawel Moll
2011-11-30 13:48       ` Arnd Bergmann
2011-11-30 13:48         ` Arnd Bergmann
2011-11-30 13:52         ` Pawel Moll
2011-11-30 13:52           ` Pawel Moll
2011-11-29 15:11 ` [PATCH v3 3/5] ARM: vexpress: Add DT support in v2m Pawel Moll
2011-11-29 15:11   ` Pawel Moll
2011-11-29 21:19   ` Arnd Bergmann
2011-11-29 21:19     ` Arnd Bergmann
2011-11-30 11:46     ` Pawel Moll
2011-11-30 11:46       ` Pawel Moll
2011-11-30 13:34       ` Arnd Bergmann
2011-11-30 13:34         ` Arnd Bergmann
2011-11-30 13:38         ` Pawel Moll
2011-11-30 13:38           ` Pawel Moll
2011-11-30 15:10           ` Arnd Bergmann
2011-11-30 15:10             ` Arnd Bergmann
2011-11-30 15:58   ` Dave Martin
2011-11-30 15:58     ` Dave Martin
2011-11-30 18:46     ` Pawel Moll
2011-11-30 18:46       ` Pawel Moll
2011-12-01 10:57       ` Dave Martin
2011-12-01 10:57         ` Dave Martin
2011-12-05 16:25   ` Dave Martin
2011-12-05 16:25     ` Dave Martin
2011-12-05 16:30     ` Pawel Moll
2011-12-05 16:30       ` Pawel Moll
2011-11-29 15:11 ` [PATCH v3 4/5] ARM: vexpress: Initial RS1 memory map support Pawel Moll
2011-11-29 15:11   ` Pawel Moll
2011-11-30 15:37   ` Dave Martin
2011-11-30 15:37     ` Dave Martin
2011-11-30 17:15     ` Pawel Moll
2011-11-30 17:15       ` Pawel Moll
2011-11-30 17:54       ` Dave Martin
2011-11-30 17:54         ` Dave Martin
2011-11-30 18:31         ` Pawel Moll
2011-11-30 18:31           ` Pawel Moll
2011-11-30 20:43         ` Nicolas Pitre
2011-11-30 20:43           ` Nicolas Pitre
2011-11-30 20:48           ` Mark Brown
2011-11-30 20:48             ` Mark Brown
2011-11-30 21:21             ` Stephen Warren
2011-11-30 21:21               ` Stephen Warren
2011-11-30 21:38             ` Nicolas Pitre
2011-11-30 21:38               ` Nicolas Pitre
2011-11-30 21:50               ` Russell King - ARM Linux
2011-11-30 21:50                 ` Russell King - ARM Linux
2011-12-01  3:36                 ` Nicolas Pitre
2011-12-01  3:36                   ` Nicolas Pitre
2011-12-01 11:10               ` Mark Brown
2011-12-01 11:10                 ` Mark Brown
2011-12-01 12:14                 ` Dave Martin
2011-12-01 12:14                   ` Dave Martin
2011-12-01 16:19                   ` Stephen Warren
2011-12-01 16:19                     ` Stephen Warren
2011-12-02 17:41                     ` Stephen Warren
2011-12-02 17:41                       ` Stephen Warren
2011-12-02 19:06                       ` Marek Vasut
2011-12-02 19:06                         ` Marek Vasut
2011-12-02 19:28                       ` Dave Martin
2011-12-02 19:28                         ` Dave Martin
2011-12-02 22:52                       ` Nicolas Pitre
2011-12-02 22:52                         ` Nicolas Pitre
2011-12-02 23:03                         ` Marek Vasut
2011-12-02 23:03                           ` Marek Vasut
2011-12-01 18:37                 ` Russell King - ARM Linux
2011-12-01 18:37                   ` Russell King - ARM Linux
2011-11-29 15:11 ` [PATCH v3 5/5] ARM: vexpress: DT-based support for CoreTiles Express A5x2 and A9x4 Pawel Moll
2011-11-29 15:11   ` Pawel Moll
2011-11-29 16:40   ` Dave Martin
2011-11-29 16:40     ` Dave Martin
2011-11-30 18:39     ` Pawel Moll
2011-11-30 18:39       ` Pawel Moll
2011-12-01 12:21       ` Dave Martin
2011-12-01 12:21         ` Dave Martin
2011-12-05 17:24         ` Pawel Moll
2011-12-05 17:24           ` Pawel Moll
2011-12-05 17:37           ` Dave Martin
2011-12-05 17:37             ` Dave Martin
2011-12-05 17:47             ` Pawel Moll
2011-12-05 17:47               ` Pawel Moll
2011-12-05 18:06               ` Dave Martin
2011-12-05 18:06                 ` Dave Martin
2011-11-29 17:00 ` Dave Martin [this message]
2011-11-29 17:00   ` [PATCH v3 0/5] Versatile Express DT support Dave Martin
2011-11-29 17:16   ` Pawel Moll
2011-11-29 17:16     ` Pawel Moll
2011-11-30 10:48     ` Dave Martin
2011-11-30 10:48       ` Dave Martin
2011-11-30 11:14       ` Pawel Moll
2011-11-30 11:14         ` Pawel Moll
2011-11-30 12:08         ` Dave Martin
2011-11-30 12:08           ` Dave Martin
2011-11-30 12:15           ` Pawel Moll
2011-11-30 12:15             ` Pawel Moll
2011-11-29 21:24 ` Arnd Bergmann
2011-11-29 21:24   ` Arnd Bergmann

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=20111129170013.GD2829@localhost.localdomain \
    --to=dave.martin@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.