linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: robherring2@gmail.com (Rob Herring)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: vexpress: initial device tree support
Date: Wed, 21 Sep 2011 11:37:54 -0500	[thread overview]
Message-ID: <4E7A12E2.9040107@gmail.com> (raw)
In-Reply-To: <CACxGe6sTYYXbpXvPb4C+y4JX0eRxLvwjbFycXA8Mjc-XWS1QUA@mail.gmail.com>

On 09/21/2011 09:57 AM, Grant Likely wrote:
> On Wed, Sep 21, 2011 at 7:24 AM, Rob Herring <robherring2@gmail.com> wrote:
>> On 09/21/2011 04:19 AM, Dave Martin wrote:
>>>       * arm,amba-bus -- widely used by other boards and patchsets, but
>>>         seems not to be documented.
>>>
>>
>> This should be dropped. There's not really any bus component to an amba
>> bus. All the probing info is within the primecell peripherals.
> 
> No, if it is an AMBA bus, then it is entirely appropriate to declare
> it as an amba bus, but to also be compatible with "simple-bus".  In
> fact, it would be better to use a compatible string that specifies the
> specific implementation of AMBA bus since there are several versions
> of the spec.

And type of AMBA bus as the spec includes AXI, AHB, and APB. None of
which have any sort of programmability or software view.

If this is required, then the policy should be simple-bus should never
be allowed alone as every bus has some underlying type. Seems like
overkill for buses like this.

Rob

  parent reply	other threads:[~2011-09-21 16:37 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-21  9:19 [PATCH] ARM: vexpress: initial device tree support Dave Martin
2011-09-21 13:24 ` Rob Herring
2011-09-21 14:24   ` Dave Martin
2011-09-21 14:33     ` Pawel Moll
2011-09-21 15:49       ` Dave Martin
2011-09-21 14:57   ` Grant Likely
2011-09-21 16:01     ` Pawel Moll
2011-09-21 16:17       ` Dave Martin
2011-09-21 16:28         ` Pawel Moll
2011-09-21 16:37     ` Rob Herring [this message]
2011-09-21 17:15       ` Dave Martin
2011-09-21 17:47         ` Mitch Bradley
2011-09-22 12:19           ` Dave Martin
2012-01-09 23:26 ` Tabi Timur-B04825
2012-01-10  0:42   ` Mitch Bradley
2012-01-10  2:24     ` Tabi Timur-B04825
2012-01-10 12:22     ` Jamie Lokier
2012-01-10 21:58       ` Timur Tabi
2012-01-10 22:35         ` Mitch Bradley
2012-01-10 23:55           ` Stephen Warren
2012-01-11  0:02             ` Timur Tabi
2012-01-11  0:28           ` Timur Tabi
2012-01-11  6:43             ` Mitch Bradley
2012-01-11 20:17               ` Timur Tabi
2012-01-11 23:20                 ` Mitch Bradley
2012-01-11 23:32                   ` Timur Tabi
2012-01-11 20:29               ` Stephen Warren
2012-01-11 20:32                 ` Timur Tabi
2012-01-11 20:36                   ` Stephen Warren
2012-01-11 21:37                     ` Timur Tabi
2012-01-11 21:57                       ` Stephen Warren
2012-01-12 12:24                     ` Jamie Lokier
2012-01-12 16:49                       ` Stephen Warren
2012-01-11 23:16                 ` Mitch Bradley
2012-01-12  0:15                   ` Stephen Warren
2012-01-12  0:38                     ` Mitch Bradley
2012-01-12  0:47                       ` Mitch Bradley
2012-01-12 16:45                       ` Stephen Warren
2012-01-12 12:09                     ` Jamie Lokier
2012-01-12 16:52                       ` Stephen Warren
2012-01-10 11:04   ` Dave Martin

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=4E7A12E2.9040107@gmail.com \
    --to=robherring2@gmail.com \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).