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

2007-09-06 Thread Olof Johansson
On Thu, Sep 06, 2007 at 04:30:23PM -0500, Scott Wood wrote: > On Thu, Sep 06, 2007 at 10:57:28PM +0200, Segher Boessenkool wrote: > > >>>BTW, IEEE1275 seems to disagree: > > >> > > >>No it doesn't. > > > > > >"...in conventional usage the string begins with the name of the > > >device's > > >manuf

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

2007-09-06 Thread Scott Wood
On Thu, Sep 06, 2007 at 10:57:28PM +0200, Segher Boessenkool wrote: > >>>BTW, IEEE1275 seems to disagree: > >> > >>No it doesn't. > > > >"...in conventional usage the string begins with the name of the > >device's > >manufacturer". > > You cut that sentence short here, it continues: "as with the

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

2007-09-06 Thread Segher Boessenkool
>>> BTW, IEEE1275 seems to disagree: >> >> No it doesn't. > > "...in conventional usage the string begins with the name of the > device's > manufacturer". You cut that sentence short here, it continues: "as with the name property." > Even if you want to quibble about the manner in which the > ma

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

2007-09-06 Thread Scott Wood
On Thu, Sep 06, 2007 at 09:36:07PM +0200, Segher Boessenkool wrote: > >BTW, IEEE1275 seems to disagree: > > No it doesn't. "...in conventional usage the string begins with the name of the device's manufacturer". Even if you want to quibble about the manner in which the manufacturer is specified,

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

2007-09-06 Thread Segher Boessenkool
> BTW, IEEE1275 seems to disagree: No it doesn't. > A manufacturer-dependent string that generally specifies the model > name > and number (including revision level) for this device. The format of > the text string is arbitrary, although in conventional usage the > string > begins with

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

2007-09-06 Thread Segher Boessenkool
>>> + 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

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

2007-09-06 Thread Scott Wood
On Thu, Sep 06, 2007 at 02:12:55PM -0500, Scott Wood wrote: > On Thu, Sep 06, 2007 at 08:48:12PM +0200, Segher Boessenkool wrote: > > So it should be "Adder MPC875", then :-) > > Any particular reason to leave out potentially useful information in a > field that is for human consumption? > > Othe

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

2007-09-06 Thread Scott Wood
On Thu, Sep 06, 2007 at 08:48:12PM +0200, Segher Boessenkool wrote: > >+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

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

2007-09-06 Thread Segher Boessenkool
> + 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

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

2007-09-06 Thread Scott Wood
On Thu, Sep 06, 2007 at 07:50:32PM +0200, Segher Boessenkool wrote: > >>>+ 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 un

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

2007-09-06 Thread Segher Boessenkool
>>> + 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. Segher _

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

2007-09-06 Thread Scott Wood
On Thu, Sep 06, 2007 at 04:08:43PM +0200, Segher Boessenkool wrote: > >+/ { > >+model = "Analogue & Micro Adder MPC875"; > > This should probably be just "MPC875". There's more than one board with an MPC875 on it. -Scott ___ Linuxppc-dev mailing li

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

2007-09-06 Thread Segher Boessenkool
> +/ { > + model = "Analogue & Micro Adder MPC875"; This should probably be just "MPC875". Segher ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev