On Saturday 26 November 2011 04:41:08 Frank Wille wrote:
> The driver is called ppbusgpio.c, or just "gpio", because the ppbus under
> NetBSD attaches 17 GPIO pins to control all parallel port lines via
> /dev/gpio0.

we've already got a "gpio" driver that sits on top of the Linux GPIO sysfs 
interface.  we can't have another driver using the same name.  could you take 
a look at src/tap/cable/gpio.c and see if you could add NetBSD support to that 
instead ?  i think you'd just have to update the accessor funcs ...

> You find a diff file on r2012 attached. It is complete, also in respect to
> auto configuration (a new lowlevel driver "gpio" can be selected). I also
> fixed some bugs with missing "(unsigned char)" casts, which cause error
> messages since gcc 4.5.

i'd like to address this separately.  i've been building under Linux with 
gcc-4.5 and gcc-4.6 and haven't seen any warnings.

POSIX says that isspace() and friends take an int, so i don't see why we need 
to cast a char to unsigned char to avoid warnings there.
        http://pubs.opengroup.org/onlinepubs/9699919799/functions/isspace.html
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.

------------------------------------------------------------------------------
All the data continuously generated in your IT infrastructure 
contains a definitive record of customers, application performance, 
security threats, fraudulent activity, and more. Splunk takes this 
data and makes sense of it. IT sense. And common sense.
http://p.sf.net/sfu/splunk-novd2d
_______________________________________________
UrJTAG-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/urjtag-development

Reply via email to