Anton Vorontsov wrote:
On Thu, Jul 03, 2008 at 01:20:18PM -0500, Kumar Gala wrote:
Guys,

        /* deprecated; */
        device_type = "i2c";

How about "deprecated but kept for compatibility with true Open Firmware
implementations"?

Seriously, you can't have a binding for "OF" and then cut out that part of the
standard at a whim. It should be there (at least for those parts which are
governed by a client interface API, like display, serial etc. but cutting it
off takes away all it's meaning, plus Linux implementations STILL keep searching
that property along with "compatible", and probably will for all time that
real IEEE1275 bindings are in use), and if it's not, then what used to go into
there needs to be in compatible.

As a nit, I don't think making it into an example makes the documentation any 
better
to read at all, in fact I found it harder to read. You may as well throw away 
all
the docs altogether and produce a well-commented example set if that is the 
route
this is going down. I think having everything as a plaintext file, while nice 
and
accessible for everyone, might have something to do with that though.

--
Matt Sealey <[EMAIL PROTECTED]>
Genesi, Manager, Developer Relations
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev

Reply via email to