Greg KH wrote :
> 
> Hi,
> 
> Here's a patch against 2.4.10-ac8 that adds a driver to enable a USB
> IrDA device as a simple serial device, like a serial IrDA device can be
> used.  This is helpful for users that do not have a IrDA support on
> the other side of their Ir connection (like some embedded systems.)
> 
> The driver only binds to one specific USB IrDA device right now as it's
> all I have to test it with.
> 
> thanks,
> 
> greg k-h

        I just have a few comments...
        1) There is more USB-IrDA devices than those you list in the
driver, and most IrDA-USB devices don't follow the spec in some way or
another. Please check .../drivers/net/irda/irda-usb.c for the gory
details...
        2) The idea is great, but I think the overall architecture
could be improved. The way you have done it leads to code duplication
(irda-usb.c + ir-usb.c) and work only for a small subset of the IrDA
devices (i.e. don't for for other FIR devices such as NSC, SMC, ALI,
SigmaTel-USB...). The API of IrDA driver is simple and well defined,
so it would be IMHO much more efficient to do a simple TTY wrapper
that would work over *any* IrDA driver. I think that LIRC is moving
this way...
        3) The Linux-IrDA stack already allow you to talk to embedded
devices with no IrDA support. This is called the Ultra protocol (very
simple, no handshake, no management frames). I can give you some
examples if you want...

        Have fun...

        Jean

_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to