Hi all,

if I interpret the following line
--
#define IS_ISDN(sc) \
        (le16_to_cpu(sc->usb_dev->descriptor.bcdDevice) & 0x80)
--
correctly, the annex a/b (pots/isdn) autodetection works be checking for the 
0x80 bit in the USB revision number. This fails at least for the "Devolo 
MicroLink ADSL Fun USB ADSL-Modem", which is sold as being German T-DSL 
compatible (T-DSL is *always* annex b, irrespective of the telephone line 
being pots/isdn) and has
--
Vendor=1039 ProdID=2110 Rev=40.02
--
and does work with isdn if the correct firmware is loaded into it.

The wiki suggests copying around the files in /lib/firmware/ueagle-atm which 
does work but which is IMHO no clean solution. Neither for regular users 
(should we really tell them they have to mess around with the firmware 
images?) nor for distributors (they will probably refuse to introduce 
configuration dialogs which will copy around firmware image files). Besides 
that, copying around the firmware files is neither logical nor consistent: 
The file names include 'p' for pots and 'i' for isdn, copying the files 
around results in bogus file names.

Thus I'd suggest to introduce a new module parameter called "pots_isdn" with 
the following possible values:
0 = auto (default)
1 = pots
2 = isdn

Modifying the IS_ISDN macro accordingly should be easy.

Comments, please?

Cheers, Johann

_______________________________________________
Ueagleatm-dev mailing list
Ueagleatm-dev@gna.org
https://mail.gna.org/listinfo/ueagleatm-dev

Reply via email to