All of lore.kernel.org
 help / color / mirror / Atom feed
From: Matt Sealey <matt@genesi-usa.com>
To: Segher Boessenkool <segher@kernel.crashing.org>
Cc: David Woodhouse <dwmw2@infradead.org>,
	bbrv@genesi-usa.com, Paul Mackerras <paulus@samba.org>,
	Linux PPC DEV <linuxppc-dev@ozlabs.org>
Subject: Re: [PATCH] Probe Efika platform before CHRP.
Date: Sun, 07 Jan 2007 22:10:57 +0000	[thread overview]
Message-ID: <45A16FF1.5000401@genesi-usa.com> (raw)
In-Reply-To: <f0c5df8b1d2bcc1abcd04894e0210cc1@kernel.crashing.org>



Why is it so funny? I'd love to be enlightened rather than just plain
insulted about it.

I don't think any of that can be taken into account by simply giving things
*names* and then having a 3-week discussion and committee hearing about how
someone gave it a DIFFERENT name. It is even more strange to me, that nobody
HAS a standard name for the devices on the MPC5200B, yet we are arguing
about what would look nicer in the Linux source code?

Small example:

mpc5200b-fec, mpc5200-fec, mpc52xx-fec doesn't tell you anything, and
then with the 5500 or 5120 or something, you then have worked out you
named your original devices wrong, and now need to make it compatible
with those? Do you switch features on and off based on a string comparison?
What if a certain chip revision has a bug you need to work around (there
are plenty in the original MPC5200!), is that meant to be encoded in the
"compatible" property too, somehow? Or would you check the SVR too? If
you would check that, why not use this as the basis of the support for
that driver? For a PCI device, you are given basically a 32-bit UID for
each device, which is attached to a unique domain, host, bus, device
and function number. You manage well here without giving it names.

An Intel processor might return a string for CPUID - mine says it's a
"Intel(R) Pentium(R) M 1.70GHz". That string is absolutely useless in
determining it's compatibility. It's just a name. It may as well say
"Fight Famine In Rwanda" for all the good it does.

I think there are better ways and better places to encode certain
properties of the system as a whole (as the system is a lot more than
just that single chip) than having a bunch of strings in a property
which claim that it is compatible with something else, strictly defined
naming conventions and so on across chip ranges. Although we are talking
here mostly about two boards with the same chip basically - Lite5200
and Efika, there may be more boards with similar hardware supported,
extra hardware supported, new chips which look very much like the 5200
but have slightly different or bugfixed operation (and here is my point)
which I do not think you can encode in names and compatibile names.

-- 
Matt Sealey <matt@genesi-usa.com>
Genesi, Manager, Developer Relations

Segher Boessenkool wrote:
>> (especially a C-based one, OpenBIOS
>> is 90% Forth which is a terrible lock-in)
> 
>> Isn't the e300 PVR and e300 SVR, or any other device identifier on the
>> chip, a much better differentiator for drivers, than a named 
>> compatibility
>> flag?
> 
> Only 7 days into the new year, and already we get some great
> entries for best-joke-of-2007!  Too bad they're factually
> incorrect.
> 
> 
> Segher
> 

  reply	other threads:[~2007-01-07 22:11 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-12-19  6:23 Bug fixes for 2.6.20 Paul Mackerras
2006-12-19  9:54 ` [PATCH] Probe Efika platform before CHRP David Woodhouse
2006-12-19 11:39   ` Benjamin Herrenschmidt
2006-12-19 11:55     ` Sylvain Munaut
2006-12-19 14:29       ` David Woodhouse
2006-12-19 15:46         ` Grant Likely
2006-12-19 19:58         ` Benjamin Herrenschmidt
2006-12-19 22:33           ` Sylvain Munaut
2007-01-02 21:29           ` David Woodhouse
2007-01-02 21:37             ` Benjamin Herrenschmidt
2007-01-02 22:04               ` David Woodhouse
2007-01-02 23:04                 ` Grant Likely
     [not found]                   ` <45A01416.6080401@genesi-usa.com>
2007-01-06 22:23                     ` Grant Likely
2007-01-06 23:13                       ` David Woodhouse
2007-01-06 23:37                         ` Grant Likely
2007-01-07  2:55                       ` Sylvain Munaut
2007-01-07  9:11                         ` Raquel Velasco and Bill Buck
2007-03-31 13:15                           ` David Woodhouse
2007-03-31 13:18                             ` Raquel Velasco and Bill Buck
2007-03-31 13:21                               ` David Woodhouse
2007-05-09 11:42                               ` Olaf Hering
2007-05-09 11:45                                 ` Raquel Velasco and Bill Buck
2007-01-07 20:09                       ` Matt Sealey
2007-01-07 20:24                         ` Segher Boessenkool
2007-01-07 22:10                           ` Matt Sealey [this message]
2007-01-07 22:35                             ` Benjamin Herrenschmidt
2007-01-07 22:35                             ` Sylvain Munaut
2007-01-07 23:04                               ` Segher Boessenkool
2007-01-07 23:07                               ` Matt Sealey
2007-01-07 23:38                                 ` Sylvain Munaut
2007-01-08  0:37                                 ` Benjamin Herrenschmidt
2007-01-07 22:32                           ` Matt Sealey
2007-01-07 22:39                           ` Matt Sealey
2007-01-07 23:27                             ` Segher Boessenkool
2007-01-07 21:16                         ` Benjamin Herrenschmidt
2007-01-08  2:17                         ` David Woodhouse
2007-01-08  9:18                           ` Matt Sealey
2007-01-08 12:52                             ` Sylvain Munaut
2007-01-08 13:16                               ` Sylvain Munaut
2007-01-08 17:25                                 ` Matt Sealey
2007-01-08 15:44                               ` Sylvain Munaut
2007-01-08 17:25                                 ` Matt Sealey
2007-01-08 17:24                               ` Matt Sealey
2007-01-08 17:59                                 ` Sylvain Munaut
2006-12-19 19:02 ` [PATCH] powerpc export rtas_set_slot_reset() Linas Vepstas
2006-12-19 22:08   ` Brian King
2006-12-19 19:06 ` [PATCH] powerpc fixup error message Linas Vepstas
2006-12-19 20:00 ` [PATCH] powerpc initialize pci device channel state Linas Vepstas

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=45A16FF1.5000401@genesi-usa.com \
    --to=matt@genesi-usa.com \
    --cc=bbrv@genesi-usa.com \
    --cc=dwmw2@infradead.org \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=paulus@samba.org \
    --cc=segher@kernel.crashing.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.