linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Grant Likely <grant.likely@secretlab.ca>
To: stephan@gatzka.org
Cc: microblaze-uclinux@itee.uq.edu.au,
	devicetree-discuss <devicetree-discuss@lists.ozlabs.org>,
	linuxppc-dev <linuxppc-dev@ozlabs.org>
Subject: Re: Request review of device tree documentation
Date: Sun, 13 Jun 2010 23:54:04 -0600	[thread overview]
Message-ID: <AANLkTim4d2Red_abGwrSbOQ-9FrM89pArFvo2ISbkWP2@mail.gmail.com> (raw)
In-Reply-To: <4C13C4D9.2030108@gatzka.org>

On Sat, Jun 12, 2010 at 11:33 AM, Stephan Gatzka <stephan@gatzka.org> wrote=
:
> Hi Grant,
>
>> I've been doing a bit of work on some introductory level documentation
>> of the flattened device tree. =A0I've got a rough copy up on the
>> devicetree.org wiki, and I could use some feedback. =A0If anyone has
>> some time to look at it, you can find it here:
>>
>> http://devicetree.org/Device_Tree_Usage
>>
>> Thanks,
>> g.
>>
>
> this looks good. Maybe an example of a complete host/PCI bridge might be
> helpful. Probably I can write something during the next week.

Hi Stephan,

I see that you started drafting some of this on the wiki.  Thanks for
the draft you've done so far.  Some comments:

- Instead, of using an MPC5200 example, add a pci bus to the sample
Coyote's Revenge system used in the rest of the page and describe
that.  The goal of this document is to lead a user step-by-step how
each part of the device tree works.  So, instead of plopping down the
complete PCI bus node, the document should gradually build it up, and
talk about each element as it is added.  Focus on how it all works
together.

- It would be userful to also show a PCI-to-PCI bridge, and maybe a
fixed PCI device as children of the host bridge node.

- The current PCI nodes on all powerpc boards depend on a device_type
=3D "PCI" property, but I'd like to look at moving away from that for
new PCI controllers (device_type describes facilities in real open
firmware.  It shouldn't have any meaning in the flattened device
tree).  I need to look into the details though to see if it is
feasible or not.

- Describing the interrupt-map property will be particularly fiddly.
It could have a section all to itself before you even get to talking
about PCI irq swizzling.

Thanks again for the help!
g.

  parent reply	other threads:[~2010-06-14  5:54 UTC|newest]

Thread overview: 75+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-06-11 22:59 Request review of device tree documentation Grant Likely
2010-06-11 23:47 ` Dan Malek
2010-06-12  2:58   ` Benjamin Herrenschmidt
2010-06-12  4:48     ` Mitch Bradley
2010-06-12  6:53   ` Grant Likely
2010-06-12  8:19     ` Mitch Bradley
2010-06-12 10:45       ` Benjamin Herrenschmidt
2010-06-12 10:48         ` Benjamin Herrenschmidt
2010-06-12 16:30           ` Mitch Bradley
2010-06-12 22:52             ` Benjamin Herrenschmidt
2010-06-13  5:07               ` Grant Likely
2010-06-13  5:39                 ` Mitch Bradley
2010-06-13  5:59                   ` Benjamin Herrenschmidt
2010-06-13  6:45                     ` Mitch Bradley
2010-06-13  8:29                       ` Benjamin Herrenschmidt
2010-06-14  5:36                         ` Grant Likely
2010-06-14 20:00                           ` Ben Dooks
2010-06-13  8:57                       ` Benjamin Herrenschmidt
2010-06-14  5:23                     ` Grant Likely
2010-06-14  7:38                       ` Russell King - ARM Linux
2010-06-14  7:45                         ` Mitch Bradley
2010-06-14  9:25                           ` Russell King - ARM Linux
2010-06-14  9:36                             ` Benjamin Herrenschmidt
2010-06-14  9:47                               ` Russell King - ARM Linux
2010-06-14 14:29                                 ` Jamie Lokier
2010-06-14 13:51                       ` Nicolas Pitre
2010-06-14 15:35                         ` Grant Likely
2010-06-14 15:58                           ` Nicolas Pitre
2010-06-14 16:16                             ` Grant Likely
2010-06-14  5:02                   ` Grant Likely
2010-06-14 12:44                     ` David Gibson
2010-06-14 14:59                       ` Nicolas Pitre
2010-06-14 15:08                         ` Grant Likely
2010-06-14 16:02                         ` Jamie Lokier
2010-06-14 16:23                           ` Nicolas Pitre
2010-06-14 16:29                             ` Grant Likely
2010-06-14 16:28                           ` Grant Likely
2010-06-14 16:33                             ` Jamie Lokier
2010-06-14 16:58                           ` Mitch Bradley
2010-06-14 17:26                             ` Nicolas Pitre
2010-06-14 18:20                               ` Mitch Bradley
2010-06-14 19:40                                 ` Nicolas Pitre
2010-06-14 20:08                                   ` Mark Brown
2010-06-16  6:09                             ` Mike Rapoport
2010-06-16  6:13                               ` Mitch Bradley
2010-06-16  6:17                                 ` Mike Rapoport
2010-06-16  6:32                                   ` Mitch Bradley
2010-06-16  6:47                                     ` Mike Rapoport
2010-06-16  7:40                                       ` Mitch Bradley
2010-06-16  9:45                                         ` Vladimir Pantelic
2010-06-16 10:39                                         ` Mike Rapoport
2010-06-16 11:41                                           ` Jamie Lokier
2010-06-16 13:48                                             ` Jamie Bennett
2010-06-16 14:39                                           ` Nicolas Pitre
2010-06-16 17:43                                             ` Tim Bird
2010-06-16  6:52                                     ` M. Warner Losh
2010-06-18 22:12                                       ` Frank Rowand
2010-06-15  2:02                         ` David Gibson
2010-06-14 15:51                       ` M. Warner Losh
2010-06-13  5:48                 ` Benjamin Herrenschmidt
2010-06-14  5:13                   ` Grant Likely
2010-06-14  6:09                     ` Benjamin Herrenschmidt
2010-06-14  6:17                       ` Mitch Bradley
2010-06-12 22:15     ` Olof Johansson
2010-06-12 23:09       ` Grant Likely
2010-06-13  6:47         ` [microblaze-uclinux] " Edgar E. Iglesias
2010-06-12  3:00 ` Benjamin Herrenschmidt
2010-06-12  3:07   ` Benjamin Herrenschmidt
2010-06-13 13:12     ` Jeremy Kerr
2010-06-14  5:40       ` Grant Likely
2010-06-12 17:33 ` Stephan Gatzka
2010-06-12 18:19   ` Grant Likely
2010-06-14  5:54   ` Grant Likely [this message]
2010-08-05  4:43 ` David Gibson
2010-09-01 16:19   ` Grant Likely

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=AANLkTim4d2Red_abGwrSbOQ-9FrM89pArFvo2ISbkWP2@mail.gmail.com \
    --to=grant.likely@secretlab.ca \
    --cc=devicetree-discuss@lists.ozlabs.org \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=microblaze-uclinux@itee.uq.edu.au \
    --cc=stephan@gatzka.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).