Devicetree
 help / color / mirror / Atom feed
From: David Gibson <david@gibson.dropbear.id.au>
To: Matt Sealey <matt@genesi-usa.com>
Cc: Mitch Bradley <wmb@firmworks.com>,
	linuxppc-dev list <linuxppc-dev@ozlabs.org>,
	devicetree-discuss list <devicetree-discuss@ozlabs.org>
Subject: Re: GPIO - marking individual pins (not) available in device tree
Date: Mon, 27 Oct 2008 10:53:03 +1100	[thread overview]
Message-ID: <20081026235303.GE22339@yookeroo.seuss> (raw)
In-Reply-To: <4904DD76.7070706@genesi-usa.com>

On Sun, Oct 26, 2008 at 04:13:26PM -0500, Matt Sealey wrote:
>
>
> Mitch Bradley wrote:
>>
>> I don't use device_type much, if at all, anymore.  Generic name +  
>> compatible just works better than device_type + specific name.  When
> > I write code that has to find a node that is suitable for a given purpose,
> > I look for the existence of suitable methods and perhaps other properties.
> > I was just too hard to keep the list of device_type values properly
> > synchronized with all the possible things that you might want to infer
> > from that set of names.
>
> The simple problem comes when you define a device_type for everything,
> I do agree it's best not to add any *MORE* that aren't in the IEEE1275
> or CHRP etc. bindings, but for those that still exist and are well
> defined (serial port probably the best, but network devices too) I
> think we should keep using them where possible and where relevant.

device_type in 1275 defines the runtime method interface.  It's *not*
for declaring the general class of the device, although it often
matches that in practice.  Drivers which attempt to use it this way
are buggy.

So, in the  case of a real OF implementation,  yes, you should include
device_type values as specified by 1275.  Assuming of course that your
implementation  really  does  implement  the OF  method  binding  that
matches the stated device_type.  However, flattened trees clearly
can't provide the method interface, and so shouldn't declare the
device_type.

In practice, we do suggest including device_type in certain, limited,
circumstances precisely because there are a whole bunch of buggy
drivers out there which match (at least partly) on device_type.  We
don't want to break these gratuitously, but neither do we want to
encourage any further spread of using device_type incorrectly for
driver matching.  Hence the current policy.

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

  reply	other threads:[~2008-10-26 23:53 UTC|newest]

Thread overview: 52+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-23 21:32 GPIO - marking individual pins (not) available in device tree Matt Sealey
2008-10-23 22:22 ` Mitch Bradley
2008-10-23 23:05   ` Matt Sealey
2008-10-24  0:52     ` Mitch Bradley
     [not found]       ` <49011C42.2020101-D5eQfiDGL7eakBO8gow8eQ@public.gmane.org>
2008-10-24  3:29         ` David Gibson
     [not found]           ` <20081024032944.GE4267-787xzQ0H9iRg7VrjXcPTGA@public.gmane.org>
2008-10-24  4:17             ` Mitch Bradley
     [not found]               ` <49014C69.8020408-D5eQfiDGL7eakBO8gow8eQ@public.gmane.org>
2008-10-24  4:45                 ` David Gibson
2008-10-24 22:14                   ` Matt Sealey
2008-10-26 23:47                     ` David Gibson
2008-10-27 15:40                       ` Matt Sealey
2008-10-27 18:34                         ` Anton Vorontsov
2008-10-27 18:56                           ` Matt Sealey
     [not found]                             ` <49060EC3.1070704-sEEEE4iEDtaXzmuOJsdVMQ@public.gmane.org>
2008-10-27 20:10                               ` Anton Vorontsov
     [not found]                                 ` <20081027201023.GA18642-wnGakbxT3iijyJ0x5qLZdcN33GVbZNy3@public.gmane.org>
2008-10-27 21:56                                   ` Matt Sealey
     [not found]                                     ` <49063929.3060900-sEEEE4iEDtaXzmuOJsdVMQ@public.gmane.org>
2008-10-27 23:12                                       ` Anton Vorontsov
2008-10-27 23:40                                         ` Anton Vorontsov
2008-10-28  0:47                                           ` Matt Sealey
     [not found]                                         ` <20081027231221.GA22008-wnGakbxT3iijyJ0x5qLZdcN33GVbZNy3@public.gmane.org>
2008-10-28  1:11                                           ` Matt Sealey
2008-10-28  2:37                                             ` Anton Vorontsov
     [not found]                                               ` <20081028023722.GA30057-wnGakbxT3iijyJ0x5qLZdcN33GVbZNy3@public.gmane.org>
2008-10-28 16:53                                                 ` Matt Sealey
2008-10-28 17:39                                                   ` Grant Likely
     [not found]                                                     ` <fa686aa40810281039v4f22a464u3eb1d421e1226fc8-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2008-10-28 19:46                                                       ` Matt Sealey
     [not found]                         ` <4905E0DC.104-sEEEE4iEDtaXzmuOJsdVMQ@public.gmane.org>
2008-10-28  0:15                           ` David Gibson
2008-10-28  0:51                             ` Matt Sealey
2008-10-28  1:50                               ` David Gibson
     [not found]                               ` <4906620B.7080308-sEEEE4iEDtaXzmuOJsdVMQ@public.gmane.org>
2008-10-28  5:20                                 ` Grant Likely
2008-10-24 22:03       ` Matt Sealey
2008-10-24 22:20         ` Stephen Neuendorffer
2008-10-26 21:39           ` Matt Sealey
2008-10-24 23:44         ` Mitch Bradley
     [not found]           ` <49025DE0.6070403-D5eQfiDGL7eakBO8gow8eQ@public.gmane.org>
2008-10-26 21:13             ` Matt Sealey
2008-10-26 23:53               ` David Gibson [this message]
     [not found]                 ` <20081026235303.GE22339-787xzQ0H9iRg7VrjXcPTGA@public.gmane.org>
2008-10-27 16:12                   ` Matt Sealey
     [not found]                     ` <4905E857.6040206-sEEEE4iEDtaXzmuOJsdVMQ@public.gmane.org>
2008-10-27 16:35                       ` Scott Wood
2008-10-27 17:05                         ` Matt Sealey
2008-10-27 17:25                           ` Scott Wood
2008-10-27 17:49                             ` Matt Sealey
2008-10-27 17:54                               ` Scott Wood
     [not found]                                 ` <49060047.2010809-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
2008-10-28  0:38                                   ` David Gibson
2008-10-28  0:34                       ` David Gibson
2008-10-24  4:58     ` David Gibson
     [not found]   ` <4900F90B.80703-D5eQfiDGL7eakBO8gow8eQ@public.gmane.org>
2008-10-24  3:27     ` David Gibson
2008-10-24 16:41 ` Anton Vorontsov
2008-10-24 17:01   ` Anton Vorontsov
     [not found]     ` <20081024170128.GA20577-wnGakbxT3iijyJ0x5qLZdcN33GVbZNy3@public.gmane.org>
2008-10-24 22:17       ` Matt Sealey
2008-10-24 22:37         ` Anton Vorontsov
  -- strict thread matches above, loose matches on Subject: below --
2008-10-28 13:31 Konstantinos Margaritis
2008-10-28 14:11 ` Anton Vorontsov
2008-10-28 14:15 ` Grant Likely
     [not found]   ` <fa686aa40810280715m3275a302jd43ab517d2d455fc-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2008-10-28 17:06     ` Matt Sealey
2008-10-28 17:32       ` Grant Likely
2008-10-28 23:37 ` David Gibson

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=20081026235303.GE22339@yookeroo.seuss \
    --to=david@gibson.dropbear.id.au \
    --cc=devicetree-discuss@ozlabs.org \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=matt@genesi-usa.com \
    --cc=wmb@firmworks.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox