Subject: Re: [PATCH 8/9] 8xx: Adder 875 support

From: Segher Boessenkool <[email-masked]>
Date: Thu, 6 Sep 2007 21:32:50 +0200

>>>>>>> +	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