Please make usbserial a module again (CONFIG_USB_SERIAL=m) in 2.6.31

Adding a bit more info and tagging regression-potential:

ogasawara@emiko:~/ubuntu-jaunty/debian/config$ grep -rn "CONFIG_USB_SERIAL=" *
amd64/config:3318:CONFIG_USB_SERIAL=m
armel/config.imx51:1549:CONFIG_USB_SERIAL=m
armel/config.ixp4xx:1200:CONFIG_USB_SERIAL=m
i386/config:3557:CONFIG_USB_SERIAL=m
lpia/config:3259:CONFIG_USB_SERIAL=m

ogasawara@emiko:~/ubuntu-karmic/debian/config$ grep -rn "CONFIG_USB_SERIAL=" *
amd64/config.common.amd64:602:CONFIG_USB_SERIAL=y
config.common.ports:3343:CONFIG_USB_SERIAL=m
i386/config.common.i386:608:CONFIG_USB_SERIAL=y
lpia/config.common.lpia:614:CONFIG_USB_SERIAL=y

Hi Noah,

I've built a test kernel which enables usbserial (CONFIG_USB_SERIAL) as a module:

http://people.canonical.com/~ogasawara/lp397189/

Could you please test and let me know your results? Thanks

Hi,

I can confirm your test kernel works as expected on the Soekris Net5501 I'm using for testing.
The USB 3G modem (Huawei E620) can now be reconfigured properly (with the help of usb_modeswitch) and works for connecting to the Internet.

# dmesg |egrep -i 'ttyUSB|option|usb.*serial'
[ 0.000000] please try 'cgroup_disable=memory' option if you don't want memory cgroups
[ 10.484764] usbcore: registered new interface driver usbserial
[ 10.520719] USB Serial support registered for generic
[ 10.616771] usbcore: registered new interface driver usbserial_generic
[ 10.621882] usbserial: USB Serial Driver core
[ 10.653338] USB Serial support registered for GSM modem (1-port)
[ 10.658593] option 1-1:1.0: GSM modem (1-port) converter detected
[ 10.663766] usb 1-1: GSM modem (1-port) converter now attached to ttyUSB0
[ 10.666825] option 1-1:1.1: GSM modem (1-port) converter detected
[ 10.693464] usb 1-1: GSM modem (1-port) converter now attached to ttyUSB1
[ 10.756655] option 1-1:1.2: GSM modem (1-port) converter detected
[ 10.793938] usb 1-1: GSM modem (1-port) converter now attached to ttyUSB2
[ 10.886104] usbcore: registered new interface driver option
[ 10.889408] option: v0.7.2:USB Driver for GSM modems

Thanks for testing Noah. I've submitted this patch to the Ubuntu kernel team:

https://lists.ubuntu.com/archives/kernel-team/2009-July/006640.html

Patched has been committed to the Karmic tree.

ogasawara@emiko:~/ubuntu-karmic$ git log -p 352c7cd1b6b33b0ca8a05ff25e8e635b0aa7508a
commit 352c7cd1b6b33b0ca8a05ff25e8e635b0aa7508a
Author: Tim Gardner <email address hidden>
Date: Fri Jul 24 10:23:13 2009 -0600

    UBUNTU: [Config] Enabled CONFIG_SERIAL=m

    BugLink: http://bugs.launchpad.net/bugs/397189

    Slightly refactored from the original patch since it was produced before
    the common configs were developed.

    Signed-off-by: Leann Ogasawara <email address hidden>
    Signed-off-by: Tim Gardner <email address hidden>

This bug was fixed in the package linux - 2.6.31-4.22

---------------
linux (2.6.31-4.22) karmic; urgency=low

  [ Amit Kucheria ]

  * ARM: IMX51: Add SoC and board support for Freescale mx51 platform
  * ARM: IMX51: Move board-mx51* header files to the correct location
  * ARM: IMX51: Clean-up the craziness of including mxc_uart.h _everywhere_
  * ARM: IMX51: Make it compile.
  * ARM: IMX51: Unconditionally disable CONFIG_GPIOLIB
  * ARM: IMX51: Make ARCH_MXC auto-enable ARCH_MXC_CANONICAL
  * ARM: IMX51: Enable ZONE_DMA for ARCH_MXC
  * ARM: IMX51: Compile-in the IMX51 cpufreq driver by default
  * ARM: IMX51: Fix isl29003 HWMON driver for i2c changes
  * ARM: USB: musb: Refer to musb_otg_timer_func under correct #ifdef
  * ARM: staging: udlfb: Add vmalloc.h include
  * UBUNTU [Config]: Bring imx51 config upto date with other flavours

  [ Brad Figg ]

  * ARM: IMX51: Hack to add defines for DMA_MODE_READ/WRITE/MASK
  * ARM: IMX51: codec structure was moved to the card structure
  * ARM: IMX51: snd_soc_machine structure replaced with snd_soc_card.
  * ARM: IMX51: Get to bus_id by calling dev_name.
  * ARM: IMX51: Get to bus_id by calling dev_name.
  * ARM: IMX51: Added external declaration for mxc_map_io.
  * ARM: IMX51: Removed the mxc_gpio_port structure.
  * ARM: IMX51: mxc_timer_init prototype
  * ARM: IMX51: Fix up merge error in Kconfig
  * ARM: IMX51: Changed from snd_card_new to snd_card_create

  [ Dinh Nguyen ]

  * ARM: IMX51: Fix plat-mxc/timer.c to handle imx51
  * ARM: IMX51: Minimal changes for USB to work on 2.6.31
  * ARM: IMX51: Babbage 2.5 needs a different system revision
  * ARM: IMX51: IPU irq handler deadlock fix

  [ Tim Gardner ]

  * [Config] Enabled CONFIG_CAN=m
    - LP: #327243
  * [Config] Enabled CONFIG_SERIAL=m
    - LP: #397189

 -- Tim Gardner <email address hidden> Fri, 24 Jul 2009 06:19:10 -0600