Re: [PATCH][RFC] Remove support for the orphaned, obsolete Digi EPCAdriver.

From: Robert P. J. Day
Date: Mon Jul 02 2007 - 10:23:02 EST


On Mon, 2 Jul 2007, Alan Cox wrote:

> > > > This driver requires a number of user-space tools. They can be acquired from
> > > > http://www.digi.com, but only works with 2.4 kernels."
> > >
> > > So fix them or merge the newer digi drivers first.
> >
> > ok, but what then does it mean to read that this driver:
> >
> > 1) *requires* a number of user-space tools, and
> > 2) these tools don't work with the 2.6 kernel?
>
> Last time I poked at the tools they were working under 2.6 with some
> trivial compile fixes. Ideally this driver would be trimmed to ISA
> only and the newer dgrs drivers merged but latter half appears to be
> something digi have no interest in

ah, that wasn't obvious from the text. in any event, there's still
some cosmetic cleanup that can be done if that driver's going to stick
around.

first, from Documentation/ioctl-number.txt:

...
Documentation/ioctl-number.txt:'d' F0-FF linux/digi1.h
Documentation/ioctl-number.txt:'e' all linux/digi1.h conflict!
...

there *is* no header file linux/digi1.h -- there is, however, a
drivers/char/digi1.h.

also, there are references to a README.epca file in various places:

Documentation/kernel-parameters.txt: See drivers/char/README.epca and
Documentation/digiepca.txt:drivers/char/README.epca for more details. Note,
drivers/char/epca.c:/* See README.epca for change history --DAT*/

but there is no README.epca file. just a couple observations.

rday
--
========================================================================
Robert P. J. Day
Linux Consulting, Training and Annoying Kernel Pedantry
Waterloo, Ontario, CANADA

http://fsdev.net/wiki/index.php?title=Main_Page
========================================================================
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/