linux-toradex.git/drivers/watchdog/pcwd_usb.c, branch v3.2.64

USB: rename usb_buffer_alloc() and usb_buffer_free() users

2010-05-20T20:21:38+00:00

For more clearance what the functions actually do,

  usb_buffer_alloc() is renamed to usb_alloc_coherent()
  usb_buffer_free()  is renamed to usb_free_coherent()

They should only be used in code which really needs DMA coherency.

All call sites have been changed accordingly, except for staging
drivers.

Signed-off-by: Daniel Mack 
Cc: Alan Stern 
Cc: Pedro Ribeiro 
Signed-off-by: Greg Kroah-Hartman

[WATCHDOG] watchdog_info constify

2010-03-07T10:30:57+00:00

make the watchdog_info struct const where possible.

Signed-off-by: Wim Van Sebroeck

[WATCHDOG] More coding-style and trivial clean-up

2009-03-25T09:07:04+00:00

Some more cleaning-up of the watchdog drivers.

Signed-off-by: Wim Van Sebroeck

[PATCH 13/13] drivers/watchdog: use USB API functions rather than constants

2009-03-25T09:03:50+00:00

This set of patches introduces calls to the following set of functions:

usb_endpoint_dir_in(epd)
usb_endpoint_dir_out(epd)
usb_endpoint_is_bulk_in(epd)
usb_endpoint_is_bulk_out(epd)
usb_endpoint_is_int_in(epd)
usb_endpoint_is_int_out(epd)
usb_endpoint_num(epd)
usb_endpoint_type(epd)
usb_endpoint_xfer_bulk(epd)
usb_endpoint_xfer_control(epd)
usb_endpoint_xfer_int(epd)
usb_endpoint_xfer_isoc(epd)

In some cases, introducing one of these functions is not possible, and it
just replaces an explicit integer value by one of the following constants:

USB_ENDPOINT_XFER_BULK
USB_ENDPOINT_XFER_CONTROL
USB_ENDPOINT_XFER_INT
USB_ENDPOINT_XFER_ISOC

An extract of the semantic patch that makes these changes is as follows:
(http://www.emn.fr/x-info/coccinelle/)

// 
@r1@ struct usb_endpoint_descriptor *epd; @@

- ((epd->bmAttributes & \(USB_ENDPOINT_XFERTYPE_MASK\|3\)) ==
- \(USB_ENDPOINT_XFER_CONTROL\|0\))
+ usb_endpoint_xfer_control(epd)

@r5@ struct usb_endpoint_descriptor *epd; @@

- ((epd->bEndpointAddress & \(USB_ENDPOINT_DIR_MASK\|0x80\)) ==
-  \(USB_DIR_IN\|0x80\))
+ usb_endpoint_dir_in(epd)

@inc@
@@

#include 

@depends on !inc && (r1||r5)@
@@

+ #include 
  #include 
// 

Signed-off-by: Julia Lawall 
Signed-off-by: Wim Van Sebroeck

[PATCH] Switch all my contributions stuff to a single common address

2008-10-28T19:16:11+00:00

Signed-off-by: Alan Cox 
Signed-off-by: Wim Van Sebroeck

[WATCHDOG] more coding style clean-up's

2008-08-06T20:19:41+00:00

More coding style clean-up's.

Signed-off-by: Wim Van Sebroeck

[WATCHDOG] Coding style - Indentation - part 2

2008-08-06T13:21:09+00:00

This brings the watchdog drivers into line with coding style.
This patch takes cares of the indentation as described in chapter 1.
Main changes:
* Re-structure the ioctl switch call for all drivers as follows:
	switch (cmd) {
	case WDIOC_GETSUPPORT:
	case WDIOC_GETSTATUS:
	case WDIOC_GETBOOTSTATUS:
	case WDIOC_GETTEMP:
	case WDIOC_SETOPTIONS:
	case WDIOC_KEEPALIVE:
	case WDIOC_SETTIMEOUT:
	case WDIOC_GETTIMEOUT:
	case WDIOC_GETTIMELEFT:
	default:
	}

This to make the migration from the drivers to the uniform watchdog
device driver easier in the future.

Signed-off-by: Wim Van Sebroeck

[WATCHDOG] Coding style - Indentation - part 1

2008-08-06T13:21:05+00:00

This brings the watchdog drivers into line with coding style.
This patch takes cares of the indentation as described in chapter 1:
  The preferred way to ease multiple indentation levels in a switch
  statement is to align the "switch" and its subordinate "case"
  labels in the same column instead of "double-indenting" the "case"
  labels.

Signed-off-by: Wim Van Sebroeck

[WATCHDOG] pcwd: a couple of watchdogs escaped conversion

2008-08-06T13:03:11+00:00

Fix them up.  Once we know the long term plan the watchdogs can all get
shrunk massively anyway

Signed-off-by: Alan Cox 
Signed-off-by: Wim Van Sebroeck 
Signed-off-by: Andrew Morton

[WATCHDOG] Clean-up includes

2008-08-06T13:02:16+00:00

Use #include  instead of 
Use #include  instead of 
Clean-up includes.

Signed-off-by: Wim Van Sebroeck