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 [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
