lkml.org 
[lkml]   [2008]   [Jan]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [RFC, PATCH, take 2] watchdog on generic gpio
Date
Le lundi 14 janvier 2008, Marc Pignat a écrit :
> Hi Florian!
> I understand your wish, but...
> You told me that your plaform doesn't implement the generic gpio interface
> (yet?), so this driver can't work for you.

You understood me wrong, I told you that not all platforms, actually only
AVR32 and ARM seem to make use of David Brownell's gpiolib, but the MIPS
board I am working with supports the "old" generic GPIO API which required
you to define your own wrappers for gpio_set/get_value and such. For both
implementations the config symbol is GENERIC_GPIO, which can somehow be
confusing.
>
> This is not the only function to be changed, start, stop and keepalive (and
> probably suspend/resume) should be changed, and when the 'Uniform Watchdog
> Device Driver' will be merged, writing your driver dealing with more gpio
> will be almost trivial.

I use only one gpio bit as well, only the keepalive function should be
changed. And even if we need the other ones to be changed, let's assume we
can pass the necessary callback to the platform driver, and checking wether
they are defined and call them or not is just trivial. This will not bloat
the driver.

>
> Please make sure to look a the right place, the git tree for the unified
> driver you have given is 17 month old... here is the latest:
> http://git.kernel.org/?p=linux/kernel/git/wim/linux-2.6-watchdog-mm.git;a=s
>ummary
>
> I'm trying to get this work merged because I use it, it works for me and
> I'm almost sure I haven't got the only embedded system using a max823
> connected to a gpio.

Yes, I understand, but having a max823-centric driver does not make it as
general as it could be.

>
> Let me know if you want some help implementing your driver!

It already exists and his merged as mtx1_wdt, I just want to get rid of it if
any generic gpio wathchdog driver can fit my needs and others.

As a general comments, Wim did not answer yet, I would like to hear from him.
--
Cordialement, Florian Fainelli
------------------------------
--
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/

\
 
 \ /
  Last update: 2008-01-14 15:05    [W:0.069 / U:0.132 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site