linux-toradex.git/drivers/usb/atm, branch v3.4.2

module_param: make bool parameters really bool (drivers & misc)

2012-01-12T23:02:20+00:00

module_param(bool) used to counter-intuitively take an int.  In
fddd5201 (mid-2009) we allowed bool or int/unsigned int using a messy
trick.

It's time to remove the int/unsigned int option.  For this version
it'll simply give a warning, but it'll break next kernel version.

Acked-by: Mauro Carvalho Chehab 
Signed-off-by: Rusty Russell

USB: convert drivers/usb/* to use module_usb_driver()

2011-11-18T17:34:02+00:00

This converts the drivers in drivers/usb/* to use the
module_usb_driver() macro which makes the code smaller and a bit
simpler.

Added bonus is that it removes some unneeded kernel log messages about
drivers loading and/or unloading.

Cc: Simon Arlott 
Cc: Duncan Sands 
Cc: Matthieu CASTET 
Cc: Stanislaw Gruszka 
Cc: Pete Zaitcev 
Cc: Oliver Neukum 
Cc: Juergen Stuber 
Cc: Cesar Miquel 
Cc: Matthew Dharm 
Cc: Matthew Wilcox 
Cc: Sarah Sharp 
Cc: Kuninori Morimoto 
Cc: Felipe Balbi 
Cc: Lucas De Marchi 
Cc: Michael Hund 
Cc: Zack Parsons 
Cc: Melchior FRANZ 
Cc: Tomoki Sekiyama 
Cc: Dan Carpenter 
Signed-off-by: Greg Kroah-Hartman

drivers: usb: atm: ueagle-atm: Add missing const qualifier

2011-07-08T21:51:30+00:00

Added missing const qualifier as flagged by checkpatch.pl

Signed-off-by: Chris Forbes 
Signed-off-by: Greg Kroah-Hartman

drivers: usb: atm: ueagle-atm: use __packed

2011-07-08T21:51:29+00:00

Replaced __attribute__ ((packed)) with __packed;

Signed-off-by: Chris Forbes 
Signed-off-by: Greg Kroah-Hartman

drivers/usb/atm: use printk_ratelimited() instead of printk_ratelimit()

2011-07-01T21:43:25+00:00

Since printk_ratelimit() shouldn't be used anymore (see comment in
include/linux/printk.h), replace it with printk_ratelimited()

Signed-off-by: Manuel Zerpies 
Signed-off-by: Greg Kroah-Hartman

Fix common misspellings

2011-03-31T14:26:23+00:00

Fixes generated by 'codespell' and manually reviewed.

Signed-off-by: Lucas De Marchi

USB: ueagle-atm: use system_wq instead of dedicated workqueues

2011-01-23T03:38:26+00:00

With cmwq, there's no reason to use separate workqueues.  Drop
uea_softc->work_q and use system_wq instead.  The used work item is
sync flushed on driver detach.

Signed-off-by: Tejun Heo 
Cc: Stanislaw Gruszka 
Signed-off-by: Greg Kroah-Hartman

Merge branch 'for-2.6.38' of git://git.kernel.org/pub/scm/linux/kernel/git/tj/wq

2011-01-08T00:58:04+00:00

* 'for-2.6.38' of git://git.kernel.org/pub/scm/linux/kernel/git/tj/wq: (33 commits)
  usb: don't use flush_scheduled_work()
  speedtch: don't abuse struct delayed_work
  media/video: don't use flush_scheduled_work()
  media/video: explicitly flush request_module work
  ioc4: use static work_struct for ioc4_load_modules()
  init: don't call flush_scheduled_work() from do_initcalls()
  s390: don't use flush_scheduled_work()
  rtc: don't use flush_scheduled_work()
  mmc: update workqueue usages
  mfd: update workqueue usages
  dvb: don't use flush_scheduled_work()
  leds-wm8350: don't use flush_scheduled_work()
  mISDN: don't use flush_scheduled_work()
  macintosh/ams: don't use flush_scheduled_work()
  vmwgfx: don't use flush_scheduled_work()
  tpm: don't use flush_scheduled_work()
  sonypi: don't use flush_scheduled_work()
  hvsi: don't use flush_scheduled_work()
  xen: don't use flush_scheduled_work()
  gdrom: don't use flush_scheduled_work()
  ...

Fixed up trivial conflict in drivers/media/video/bt8xx/bttv-input.c
as per Tejun.

ueagle-atm: fix PHY signal initialization race

2010-12-28T21:41:41+00:00

A race exists when initializing ueagle-atm devices where the generic atm
device may not yet be created before the driver attempts to initialize
it's PHY signal state, which checks whether the atm device has been
created or not.  This often causes the sysfs 'carrier' attribute to be
'1' even though no signal has actually been found.

uea_probe
   usbatm_usb_probe
      driver->bind (uea_bind)
         uea_boot
            kthread_run(uea_kthread)     uea_kthread
      usbatm_atm_init                       uea_start_reset
         atm_dev_register                      UPDATE_ATM_SIGNAL

UPDATE_ATM_SIGNAL checks whether the ATM device has been created and if
not, will not update the PHY signal state.  Because of the race that
does not always happen in time, and the PHY signal state remains
ATM_PHY_SIG_FOUND even though no signal exists.

To fix the race, just create the kthread during initialization, and only
after initialization is complete, start the thread that reboots the
device and initializes PHY state.

[ 3030.490931] uea_probe: calling usbatm_usb_probe
[ 3030.490946] ueagle-atm 8-2:1.0: usbatm_usb_probe: trying driver ueagle-atm with vendor=1110, product=9031, ifnum  0
[ 3030.493691] uea_bind: setting usbatm
[ 3030.496932] usb 8-2: [ueagle-atm] using iso mode
[ 3030.497283] ueagle-atm 8-2:1.0: usbatm_usb_probe: using 3021 byte buffer for rx channel 0xffff880125953508
   
[ 3030.497292] usb 8-2: [ueagle-atm] (re)booting started
   
[ 3030.497298] uea_start_reset: atm dev (null)
   
[ 3030.497306] ueagle-atm 8-2:1.0: usbatm_usb_probe: using 3392 byte buffer for tx channel 0xffff8801259535b8
[ 3030.497374] usbatm_usb_probe: about to init
[ 3030.497379] usbatm_usb_probe: calling usbatm_atm_init
   
[ 3030.497384] usbatm_atm_init: creating atm device!

Signed-off-by: Dan Williams 
Signed-off-by: David S. Miller

usb: don't use flush_scheduled_work()

2010-12-24T15:14:20+00:00

flush_scheduled_work() is being deprecated.  Directly flush or cancel
work items instead.

* u_ether, isp1301_omap, speedtch conversions are straight-forward.

* ochi-hcd should only flush when quirk_nec() is true as otherwise the
  work wouldn't have been initialized.

* In oti6858, cancel_delayed_work() + flush_scheduled_work() ->
  cancel_delayed_work_sync().

Signed-off-by: Tejun Heo 
Acked-by: Greg Kroah-Hartman 
Cc: David Brownell 
Cc: Duncan Sands 
Cc: linux-usb@vger.kernel.org