linux-kernel - Re: [PATCH] input: driver for USB VoIP phones with CM109 chipset #2

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 04 Aug 2008 22:34:27 +0200
From:	"Alfred E. Heggestad" <aeh@...org>
To:	Oliver Neukum <oliver@...kum.org>
CC:	Dmitry Torokhov <dmitry.torokhov@...il.com>,
	linux-input@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] input: driver for USB VoIP phones with CM109 chipset
 #2

Oliver Neukum wrote:
> Am Montag 04 August 2008 00:04:58 schrieb Alfred E. Heggestad:
>> Oliver Neukum wrote:
>>> Am Samstag 26 Juli 2008 17:55:35 schrieb Alfred E. Heggestad:
>>>> Jul 26 17:32:33 io kernel: hald-addon-keyb[23629]: segfault at 6f6f6c5f ip b7e9855b sp bf9d89dc error 4 in libc-2.6.1.so[b7e28000+144000]
>>>> Jul 26 17:32:43 io kernel: cm109: cm109_urb_ctl_callback: usb_submit_urb (urb_irq) failed -22
>>>> Jul 26 17:33:21 io last message repeated 6 times
>>>> Jul 26 17:34:28 io last message repeated 17 times
>>> This should not happen. Please provide more context. If possible
>>> recompile your kernel with CONFIG_USB_DEBUG.
>>>
>> Hi Oliver
>>
>> sorry for the delay, I was out travelling with no access to CM109 hardware..
>>
>> I recompiled the latest kernel from input-branch with CONFIG_USB_DEBUG enabled,
>> the version reported is 2.6.27-rc1. Below you can see the syslog and the
>> context of events:
> 
> Looks like urb_irq is not properly initialised. Is there really nothing in the syslog?
> Please print out urb->interval
> 

urb->interval == 32

I added some more debugging in the driver, and after the module is loaded,
I press "TAB" in a shell to force Bell/Buzzer.


the syslog reports:


;;;
;;; Insert device:
;;;

Aug  4 22:20:43 io kernel: cm109: Keymap for Komunikate KIP1000 phone loaded
Aug  4 22:20:43 io kernel: cm109 1-2:1.3: usb_probe_interface
Aug  4 22:20:43 io kernel: cm109 1-2:1.3: usb_probe_interface - got id
Aug  4 22:20:43 io kernel: cm109: urb_irq init: interval=32
Aug  4 22:20:43 io kernel: input: CM109 USB driver as /class/input/input13
Aug  4 22:20:43 io kernel: cm109: cm109_urb_ctl_callback: buzzer_pending=0, ctl_urb_pending=0
Aug  4 22:20:43 io kernel: uhci_hcd 0000:00:1f.2: reserve dev 2 ep83-INT, period 32, phase 16, 14 us
Aug  4 22:20:43 io kernel: usbcore: registered new interface driver cm109
Aug  4 22:20:43 io kernel: cm109: CM109 phone driver: 20080726 (C) Alfred E. Heggestad

;;;
;;; Bell/Alarm -> buzzer
;;;

Aug  4 22:20:50 io kernel: cm109: cm109_toggle_buzzer_async: ctl_urb_pending=0 buzzer_pending=0
Aug  4 22:20:50 io kernel: cm109: cm109_submit_buzz_toggle: buzzer_state=1
Aug  4 22:20:50 io kernel: cm109: cm109_urb_ctl_callback: buzzer_pending=0, ctl_urb_pending=1
Aug  4 22:20:50 io kernel: cm109: cm109_urb_ctl_callback: usb_submit_urb (urb_irq) failed -22     <----- error -EINVAL
Aug  4 22:20:50 io kernel: cm109: cm109_toggle_buzzer_async: ctl_urb_pending=0 buzzer_pending=0
Aug  4 22:20:50 io kernel: cm109: cm109_submit_buzz_toggle: buzzer_state=0
Aug  4 22:20:50 io kernel: cm109: cm109_toggle_buzzer_async: ctl_urb_pending=1 buzzer_pending=0
Aug  4 22:20:50 io kernel: cm109: cm109_urb_ctl_callback: buzzer_pending=1, ctl_urb_pending=1
Aug  4 22:20:50 io kernel: cm109: cm109_submit_buzz_toggle: buzzer_state=0
Aug  4 22:20:50 io kernel: cm109: cm109_urb_ctl_callback: buzzer_pending=0, ctl_urb_pending=1
Aug  4 22:20:50 io kernel: cm109: cm109_urb_ctl_callback: usb_submit_urb (urb_irq) failed -22     <---- error -EINVAL

;;;
;;; unplug device:
;;;

Aug  4 22:20:58 io kernel: usbcore: deregistering interface driver cm109
Aug  4 22:20:58 io kernel: uhci_hcd 0000:00:1f.2: shutdown urb cec1a760 ep3in-intr
Aug  4 22:20:58 io kernel: cm109: urb_irq_cb: urb_irq->interval=32
Aug  4 22:20:58 io kernel: uhci_hcd 0000:00:1f.2: release dev 2 ep83-INT, period 32, phase 16, 14 us
Aug  4 22:20:58 io kernel: cm109: cm109_toggle_buzzer_sync: usb_control_msg() failed 4




/alfred
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/