Re: kern/39211: ubsa device does no longer work since January 2008

[Matthias Drochner <M.Drochner%fz-juelich.de@localhost>]

The following reply was made to PR kern/39211; it has been noted by GNATS.

From: Matthias Drochner <M.Drochner%fz-juelich.de@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: kern-bug-people%NetBSD.org@localhost, gnats-admin%NetBSD.org@localhost,
        netbsd-bugs%NetBSD.org@localhost
Subject: Re: kern/39211: ubsa device does no longer work since January 2008 
Date: Thu, 31 Jul 2008 17:31:50 +0200

 This is a multipart MIME message.
 
 --==_Exmh_66776297067510
 Content-Type: text/plain; charset=us-ascii
 
 
 You are right, the code was messed up when "uhmodem"
 was added. Generally, if a USB driver attaches to an
 interface, it is not allowed to set the configuration
 later because this potentially invalidates all interfaces.
 So ubsa should attach as whole device as it did before.
 I'm not so sure about the config index. Your error message
 suggests that your device supports only one configuration
 and thus can only be set to index 0. But the code before
 the uhmodem addition seems to use index 1...
 (I've actually never seen a USB device with more than
 a single configuration.)
 
 Could you try the appended patch, as a starting point?
 
 best regards
 Matthias
 
 
 
 
 
 -------------------------------------------------------------------
 -------------------------------------------------------------------
 Forschungszentrum Juelich GmbH
 52425 Juelich
 
 Sitz der Gesellschaft: Juelich
 Eingetragen im Handelsregister des Amtsgerichts Dueren Nr. HR B 3498
 Vorsitzende des Aufsichtsrats: MinDir'in Baerbel Brumme-Bothe
 Geschaeftsfuehrung: Prof. Dr. Achim Bachem (Vorsitzender),
 Dr. Ulrich Krafft (stellv. Vorsitzender), Prof. Dr. Harald Bolt,
 Dr. Sebastian M. Schmidt
 -------------------------------------------------------------------
 -------------------------------------------------------------------
 
 --==_Exmh_66776297067510
 Content-Type: text/plain ; name="ubsa.txt"; charset=us-ascii
 Content-Description: ubsa.txt
 Content-Disposition: attachment; filename="ubsa.txt"
 
 #
 # old_revision [bc9b40deb2273f24a32fed91af7d896bc85babaf]
 #
 # patch "sys/dev/usb/files.usb"
 #  from [86c1e7701940f91829fdefa1c88ca932230936e1]
 #    to [d7134e8fae7fb3a268ac3701073a28e0b040fc29]
 # 
 # patch "sys/dev/usb/ubsavar.h"
 #  from [bb368d41d9ff889f7bd6f6c48f9ce4c9ab04c7fb]
 #    to [400d4a6f9924ee5c43d4fb7914d12835151909d2]
 #
 ============================================================
 --- sys/dev/usb/files.usb      86c1e7701940f91829fdefa1c88ca932230936e1
 +++ sys/dev/usb/files.usb      d7134e8fae7fb3a268ac3701073a28e0b040fc29
 @@ -237,8 +237,7 @@ device     ubsa: ucombus, ubsa_common
  define        ubsa_common
  file  dev/usb/ubsa_common.c           ubsa_common
  device        ubsa: ucombus, ubsa_common
 -#attach       ubsa at usbdevif
 -attach        ubsa at usbifif
 +attach        ubsa at usbdevif
  file  dev/usb/ubsa.c                  ubsa
  
  # Huawei E220 3G/HSDPA modem (ubsa)
 ============================================================
 --- sys/dev/usb/ubsavar.h      bb368d41d9ff889f7bd6f6c48f9ce4c9ab04c7fb
 +++ sys/dev/usb/ubsavar.h      400d4a6f9924ee5c43d4fb7914d12835151909d2
 @@ -55,7 +55,7 @@
  
  #define       UBSA_MODVER             1       /* module version */
  
 -#define       UBSA_DEFAULT_CONFIG_INDEX       1
 +#define       UBSA_DEFAULT_CONFIG_INDEX       0
  #define       UBSA_IFACE_INDEX_OFFSET 0
  
  #define       UBSA_INTR_INTERVAL      100     /* ms */
 
 --==_Exmh_66776297067510--