From: Segher Boessenkool <segher@kernel.crashing.org>
To: Scott Wood <scottwood@freescale.com>
Cc: linuxppc-dev@ozlabs.org
Subject: Re: [PATCH 8/9] 8xx: Adder 875 support
Date: Thu, 6 Sep 2007 21:32:50 +0200 [thread overview]
Message-ID: <5df2b075e2e4821ecd03ea02b69fd2f3@kernel.crashing.org> (raw)
In-Reply-To: <20070906191255.GA22663@ld0162-tx32.am.freescale.net>
>>>>>>> + model = "Analogue & Micro Adder MPC875";
>>>>>>
>>>>>> This should probably be just "MPC875".
>>>>>
>>>>> There's more than one board with an MPC875 on it.
>>>>
>>>> "model" is the model name the vendor uses. It isn't supposed to
>>>> be unique globally, nor does it say what CPU is on the board.
>>>
>>> The name the vendor uses is "Adder MPC875". The name of the vendor
>>> is
>>> "Analogue & Micro", which seems useful to include, even if it's not
>>> mandatory that it be globally unique.
>>
>> So it should be "Adder MPC875", then :-)
>
> Any particular reason to leave out potentially useful information in a
> field that is for human consumption?
Because it isn't for human consumption (only), and has well-defined
semantics (although slightly fuzzy).
> Other than Because Segher Says So(tm). :-)
"Because the OF standard says so" :-)
Segher
next prev parent reply other threads:[~2007-09-06 19:32 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-09-05 19:27 [PATCH 1/9] 8xx: Fix CONFIG_PIN_TLB Scott Wood
2007-09-05 19:27 ` [PATCH 2/9] 8xx: Infrastructure code cleanup Scott Wood
2007-09-05 19:27 ` [PATCH 3/9] 8xx: Add pin and clock setting functions Scott Wood
2007-09-05 19:27 ` [PATCH 4/9] 8xx: Work around CPU15 erratum Scott Wood
2007-09-05 19:28 ` [PATCH 5/9] 8xx: Don't call non-existent Soft_emulate_8xx from SoftwareEmulation Scott Wood
2007-09-05 19:28 ` [PATCH 6/9] 8xx: Set initial memory limit Scott Wood
2007-09-05 19:28 ` [PATCH 7/9] 8xx: mpc885ads cleanup Scott Wood
2007-09-05 19:28 ` [PATCH 8/9] 8xx: Adder 875 support Scott Wood
2007-09-06 14:08 ` Segher Boessenkool
2007-09-06 14:16 ` Scott Wood
2007-09-06 17:50 ` Segher Boessenkool
2007-09-06 18:09 ` Scott Wood
2007-09-06 18:48 ` Segher Boessenkool
2007-09-06 19:12 ` Scott Wood
2007-09-06 19:20 ` Scott Wood
2007-09-06 19:36 ` Segher Boessenkool
2007-09-06 19:56 ` Scott Wood
2007-09-06 20:57 ` Segher Boessenkool
2007-09-06 21:30 ` Scott Wood
2007-09-06 23:45 ` Olof Johansson
2007-09-06 19:32 ` Segher Boessenkool [this message]
2007-09-05 19:28 ` [PATCH 9/9] 8xx: Embedded Planet EP88xC support Scott Wood
2007-09-05 20:36 ` [PATCH 1/9] 8xx: Fix CONFIG_PIN_TLB Dan Malek
2007-09-05 20:53 ` Scott Wood
2007-09-05 20:59 ` Scott Wood
2007-09-05 22:27 ` Dan Malek
2007-09-06 3:11 ` Scott Wood
2007-09-05 22:08 ` Dan Malek
2007-09-05 22:23 ` Scott Wood
2007-09-05 22:42 ` Dan Malek
2007-09-06 3:01 ` Scott Wood
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=5df2b075e2e4821ecd03ea02b69fd2f3@kernel.crashing.org \
--to=segher@kernel.crashing.org \
--cc=linuxppc-dev@ozlabs.org \
--cc=scottwood@freescale.com \
/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.