DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws;
  s=s1024; d=pacbell.net;
  h=Received:X-Yahoo-SMTP:X-YMail-OSG:X-Yahoo-Newman-Property:From:To:Subject:Date:User-Agent:Cc:References:In-Reply-To:MIME-Version:Content-Type:Content-Transfer-Encoding:Content-Disposition:Message-Id;
  b=xhJKW7mgIzTcNUykz1goXvY278CE3rcNYOI/SFI84x4oERWCKXxGVTXzF8ffWpoo8D5NKHxPKwo5nsipcG00wvq4KJmICnbWZHAjmMnxgPCS4f2M6MZ6PZk3lM2PPjavRQHw+GAdTM5Qqf3V8Twyy3bgTrWNZo0voIxFJ5HufZI=  ;
From: David Brownell <david-b@pacbell.net>
To: Baruch Siach <baruch@tkos.co.il>
Subject: Re: [PATCH v4] gpio: driver for PrimeCell PL061 GPIO controller
Date: Mon, 15 Jun 2009 16:33:02 -0700
User-Agent: KMail/1.9.10
Cc: Andrew Morton <akpm@linux-foundation.org>, linux-kernel@vger.kernel.org,
       linux-arm-kernel@lists.arm.linux.org.uk, linux@arm.linux.org.uk
References: <1244399935-23128-1-git-send-email-baruch@tkos.co.il> <20090610004447.78b84cd5.akpm@linux-foundation.org> <20090610075633.GD10382@tarshish>
In-Reply-To: <20090610075633.GD10382@tarshish>
MIME-Version: 1.0
Content-Type: text/plain;
  charset="us-ascii"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
Message-Id: <200906151633.03217.david-b@pacbell.net>
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1258
Lines: 32

On Wednesday 10 June 2009, Baruch Siach wrote:
> > Should the gpio core provide a primitive to check that a gpio has been
> > properly requested rathe rthan open-coding it here?
> 
> Probably. 

I happen to strongly dislike primitives that are fundamentally
MT-unsafe, except in non-threaded environments ... so I would
disagree about that.  :)

If you want to see if a GPIO is available, allocate it.
The only answer you can really depend on is "yes, it's
mine now".  You can't rely on other threads not allocating,
or not de-allocating, the GPIO later ... unless they can't,
because it's not theirs.

But that's beside the point, since GPIO drivers shouldn't
be in the business of requesting the GPIOs they manage.
Such layer violations are another sign that you may not be
thinking very clearly about things.

In this case, whoever allocated that GPIO for IRQ purposes
should have done the gpio_request() before handing out
the gpio_to_irq() results to however many drivers ended
up needing to use it.

- Dave
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/