Messages in this thread | ![/](/images/icornerl.gif) | | Subject | Re: [PATCH 1/2] gpio: add pin biasing and drive mode to gpiolib | From | Ben Nizette <> | Date | Tue, 19 Apr 2011 08:16:12 +1000 |
| |
On 18/04/2011, at 9:59 PM, Mark Brown wrote:
> On Mon, Apr 18, 2011 at 06:50:12PM +1000, Ben Nizette wrote: > > Please fix your MUA to word wrap within paragraphs; I've reflowed your > text for legibility.
Sorry, I apologised in advance in my first email, I've made the mistake of travelling without a sane MUA :)
> >> Still my question stands, where is the driver ever better placed to >> make these calls than the board code? > > Even if it ends up being the board code using these APIs it seems > sensible to have a standard API that GPIO drivers can use to expose the > functionality. This will help with getting control into code Linux owns > (since people don't have to implement custom APIs) and will mean we can > do things like add control of this for device tree based boards.
Oh I'm all for platform-specific libraries abstracting this away from the board code if that helps, that's certainly the way that, eg, AVR32 does it. It just doesn't make sense to me to bounce from the board code in to 'generic' gpio code then back to platform-specific implementations when you could cut out the middle man.
--Ben.
| ![\](/images/icornerr.gif) |