All of lore.kernel.org
 help / color / mirror / Atom feed
From: Segher Boessenkool <segher@kernel.crashing.org>
To: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: "linuxppc-dev@ozlabs.org" <linuxppc-dev@ozlabs.org>,
	Jon Loeliger <jdl@jdl.com>
Subject: Re: [PATCH 2/8] Add uli1575 pci-bridge sector to MPC8641HPCN dts file.
Date: Sun, 3 Jun 2007 09:37:19 +0200	[thread overview]
Message-ID: <fed89a6610a7defad7a18bbe5da8b33a@kernel.crashing.org> (raw)
In-Reply-To: <1180826202.14025.12.camel@localhost.localdomain>

>> Feel free to ask for advice on IRC, on the list, or
>> elsewhere _before_ sending a patch.
>
> Bah, sending a patch is good too, don't be too hard there. I for one
> prefer the approach of sending a patch first.

I'm saying that if you're unsure about stuff, you can ask
about it -- sending a preliminary patch with the question
is one way, sure.

>>> Second, perhaps there is lingering legacy
>>> crap in some of the early DTS files that should be systematically
>>> cleaned up.
>>
>> Yes there is.  Maybe one day I'll get to it, I'm hoping
>> someone else will fix this though ;-)
>
> We probably need to improve the documentation in fact. Maybe with a 
> step
> by step HOWTO create your device-tree with examples for typical things
> like PCI etc.. and the rational of why for each of these, we chose to 
> do
> XXX instead of YYY...

A howto-like doc would help yes.  Rationale might help people
better understand how things work, too, sure.

>> The official (and unofficial) Open Firmware docs are quite
>> good.  Perhaps we need a big huge fat sign in the DTS docs
>> pointing to them.
>
> They aren't -that- good when you come out of the blue and don't know
> where to start.

Well most people don't even know those docs _exist_ it
seems.

> I think a tutorial based on building a tree for an
> example board step by step would be best.

Would be nice yes.


Segher

  reply	other threads:[~2007-06-03  7:37 UTC|newest]

Thread overview: 56+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-06-01 17:48 [PATCH 2/8] Add uli1575 pci-bridge sector to MPC8641HPCN dts file Jon Loeliger
2007-06-01 18:58 ` Segher Boessenkool
2007-06-01 21:45   ` Wade Farnsworth
2007-06-02  8:22     ` Segher Boessenkool
2007-06-02  8:53       ` Gabriel Paubert
2007-06-02  9:01         ` Segher Boessenkool
2007-06-02 19:53           ` Gabriel Paubert
2007-06-02 20:23             ` Segher Boessenkool
2007-06-03  0:01               ` Benjamin Herrenschmidt
2007-06-03  7:41                 ` Segher Boessenkool
2007-06-03  8:33                   ` Gabriel Paubert
2007-06-03  8:57                     ` Segher Boessenkool
2007-06-03  9:12                       ` Benjamin Herrenschmidt
2007-06-03 10:02                         ` Segher Boessenkool
2007-06-03 10:10                       ` Gabriel Paubert
2007-06-03 11:42                         ` Segher Boessenkool
2007-06-03 12:43                           ` Gabriel Paubert
2007-06-03 14:42                             ` Segher Boessenkool
2007-06-03 18:20                               ` Gabriel Paubert
2007-06-03 18:56                                 ` Segher Boessenkool
2007-06-03  9:07                     ` Benjamin Herrenschmidt
2007-06-03  9:59                       ` Segher Boessenkool
2007-06-03 14:50                     ` Jon Loeliger
2007-06-03 17:27                       ` Segher Boessenkool
2007-06-04 20:27                     ` Andy Fleming
2007-06-04 22:31                       ` Randy Vinson
2007-06-05 19:16                         ` Andy Fleming
2007-06-05 20:28                           ` Randy Vinson
2007-06-06  7:09                         ` Benjamin Herrenschmidt
2007-06-07 16:21                           ` Andy Fleming
2007-06-07 16:53                             ` Segher Boessenkool
2007-06-07 22:12                             ` Benjamin Herrenschmidt
2007-06-08  8:29                               ` Segher Boessenkool
2007-06-08  8:32                                 ` Benjamin Herrenschmidt
2007-06-05  6:05               ` Zang Roy-r61911
2007-06-02 23:52           ` Benjamin Herrenschmidt
2007-06-02 23:51         ` Benjamin Herrenschmidt
2007-06-03  8:59         ` Geert Uytterhoeven
2007-06-04 18:50       ` Jon Loeliger
2007-06-04 19:27         ` Segher Boessenkool
2007-06-01 23:28   ` Benjamin Herrenschmidt
2007-06-01 23:36   ` Jon Loeliger
2007-06-02  0:22     ` Benjamin Herrenschmidt
2007-06-02  8:28       ` Segher Boessenkool
2007-06-02 16:04         ` Jon Loeliger
2007-06-02 20:00           ` Segher Boessenkool
2007-06-02 23:16             ` Benjamin Herrenschmidt
2007-06-03  7:37               ` Segher Boessenkool [this message]
2007-06-04  0:16               ` Olof Johansson
2007-06-04  8:18                 ` Segher Boessenkool
2007-06-02  8:25     ` Segher Boessenkool
2007-06-02 23:50 ` Benjamin Herrenschmidt
2007-06-03  0:13   ` Gabriel Paubert
2007-06-03  7:42     ` Segher Boessenkool
2007-06-03  7:53       ` Gabriel Paubert
2007-06-04 18:49   ` Jon Loeliger

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=fed89a6610a7defad7a18bbe5da8b33a@kernel.crashing.org \
    --to=segher@kernel.crashing.org \
    --cc=benh@kernel.crashing.org \
    --cc=jdl@jdl.com \
    --cc=linuxppc-dev@ozlabs.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.