Devicetree
 help / color / mirror / Atom feed
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 0/5] Versatile Express DT support, take 2
Date: Wed, 16 Nov 2011 15:33:05 +0000	[thread overview]
Message-ID: <20111116153305.GA2073@localhost.localdomain> (raw)
In-Reply-To: <1321036026-23411-1-git-send-email-pawel.moll-5wv7dgnIgG8@public.gmane.org>

On Fri, Nov 11, 2011 at 06:27:01PM +0000, Pawel Moll wrote:
> Hello again,
> 
> I've manage to clean up the VE DT patches.
> 
> Changes since RFC:
> 
> * I think I did all the changes suggested by Rob (including
>   using "custom" clock lookups instead of auxdata - do
>   comment, please ;-)
> 
> * Merged CA5s and CA9 code into one file.
> 
> * Added documentation for the vexpress bindings.
> 
> * Reorganized the tree so the "flat aliases search" patch
>   is not longer necessary.
> 
> * Moved the alias node from motherboard's to tile's DTS,
>   as this is the place to decide about serial port ordering,
>   timers being used etc.
> 
> * Got the SMSC working (my fault, obviously ;-)
> 
> As usual - feedback more than welcome (my tests suggest
> that the "ATAG version" still works, so feel free to test
> at least that ;-)

I haven't actually tried this code out yet, but I'm broadly in
agreement.

I'm happy for this series to replace my previously posted
series, since it builds on common changes.

I have a few comments on the detail -- see separate mails.

Cheers
---Dave

      parent reply	other threads:[~2011-11-16 15:33 UTC|newest]

Thread overview: 49+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-11 18:27 [PATCH 0/5] Versatile Express DT support, take 2 Pawel Moll
2011-11-11 18:27 ` [PATCH 1/5] ARM: vexpress: Get rid of MMIO_P2V Pawel Moll
     [not found]   ` <1321036026-23411-2-git-send-email-pawel.moll-5wv7dgnIgG8@public.gmane.org>
2011-11-16 15:35     ` Dave Martin
2011-11-16 16:16       ` Pawel Moll
     [not found]         ` <1321460205.3137.352.camel-okZbbLrgpR/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2011-11-16 17:28           ` Dave Martin
2011-11-16 17:30             ` Pawel Moll
2011-11-17  7:02               ` Ryan Harkin
2011-11-17 15:43     ` Russell King - ARM Linux
2011-11-18 12:20       ` Pawel Moll
     [not found]         ` <1321618848.24216.322.camel-okZbbLrgpR/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2011-11-18 17:44           ` Russell King - ARM Linux
2011-11-11 18:27 ` [PATCH 2/5] ARM: vexpress: Remove platform SMP functions from ct_desc Pawel Moll
     [not found]   ` <1321036026-23411-3-git-send-email-pawel.moll-5wv7dgnIgG8@public.gmane.org>
2011-11-17 15:31     ` Russell King - ARM Linux
2011-11-18 12:20       ` Pawel Moll
2011-11-11 18:27 ` [PATCH 3/5] ARM: vexpress: Add DT support in v2m Pawel Moll
     [not found]   ` <1321036026-23411-4-git-send-email-pawel.moll-5wv7dgnIgG8@public.gmane.org>
2011-11-16 15:44     ` Dave Martin
     [not found]       ` <20111116154402.GE2073-bi+AKbBUZKY6gyzm1THtWbp2dZbC/Bob@public.gmane.org>
2011-11-16 16:26         ` Rob Herring
2011-11-16 16:37           ` Pawel Moll
     [not found]             ` <1321461458.3137.374.camel-okZbbLrgpR/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2011-11-16 16:59               ` Rob Herring
2011-11-16 17:07                 ` Pawel Moll
2011-11-16 17:37                   ` Pawel Moll
     [not found]                     ` <1321465054.3137.386.camel-okZbbLrgpR/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2011-11-16 19:14                       ` Dave Martin
     [not found]                   ` <1321463271.3137.380.camel-okZbbLrgpR/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2011-11-16 17:39                     ` Dave Martin
     [not found]                       ` <20111116173945.GG2073-bi+AKbBUZKY6gyzm1THtWbp2dZbC/Bob@public.gmane.org>
2011-11-16 17:50                         ` Dave Martin
2011-11-16 17:55                           ` Pawel Moll
2011-11-17 15:53                     ` Russell King - ARM Linux
2011-11-18 12:20                       ` Pawel Moll
     [not found]                         ` <1321618849.24216.323.camel-okZbbLrgpR/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2011-11-18 17:49                           ` Russell King - ARM Linux
2011-11-16 16:35       ` Pawel Moll
     [not found]         ` <1321461315.3137.371.camel-okZbbLrgpR/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2011-11-16 17:57           ` Dave Martin
2011-11-17 13:50             ` Pawel Moll
     [not found]               ` <1321537813.3137.475.camel-okZbbLrgpR/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2011-11-17 14:41                 ` Dave Martin
2011-11-17 16:05     ` Russell King - ARM Linux
     [not found]       ` <20111117160536.GU9581-l+eeeJia6m9vn6HldHNs0ANdhmdF6hFW@public.gmane.org>
2011-11-17 18:37         ` Dave Martin
     [not found]           ` <20111117183726.GE3044-bi+AKbBUZKY6gyzm1THtWbp2dZbC/Bob@public.gmane.org>
2011-11-18 17:52             ` Russell King - ARM Linux
2011-11-18 12:20       ` Pawel Moll
2011-11-11 18:27 ` [PATCH 4/5] ARM: vexpress: Initial RS1 memory map support Pawel Moll
     [not found]   ` <1321036026-23411-5-git-send-email-pawel.moll-5wv7dgnIgG8@public.gmane.org>
2011-11-16 15:42     ` Dave Martin
2011-11-16 16:28       ` Pawel Moll
     [not found]         ` <1321460897.3137.364.camel-okZbbLrgpR/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2011-11-16 18:03           ` Dave Martin
2011-11-17 15:36     ` Russell King - ARM Linux
2011-11-18 12:20       ` Pawel Moll
     [not found]         ` <1321618850.24216.324.camel-okZbbLrgpR/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2011-11-18 17:56           ` Russell King - ARM Linux
2011-11-11 18:27 ` [PATCH 5/5] ARM: vexpress: DT-based support for CoreTiles Express A5x2 and A9x4 Pawel Moll
     [not found]   ` <1321036026-23411-6-git-send-email-pawel.moll-5wv7dgnIgG8@public.gmane.org>
2011-11-11 22:30     ` Rob Herring
2011-11-11 22:54       ` Pawel Moll
2011-11-16 15:36     ` Dave Martin
2011-11-16 16:22       ` Pawel Moll
     [not found]         ` <1321460559.3137.358.camel-okZbbLrgpR/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2011-11-16 18:17           ` Dave Martin
     [not found] ` <1321036026-23411-1-git-send-email-pawel.moll-5wv7dgnIgG8@public.gmane.org>
2011-11-16 15:33   ` Dave Martin [this message]

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=20111116153305.GA2073@localhost.localdomain \
    --to=dave.martin-qsej5fyqhm4dnm+yrofe0a@public.gmane.org \
    --cc=devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=pawel.moll-5wv7dgnIgG8@public.gmane.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox