GitHub/moto-9609/android_kernel_motorola_exynos9610.git
11 years agostaging: lustre: remove almost all crypto layer wrappers
Greg Kroah-Hartman [Sat, 3 Aug 2013 06:29:11 +0000 (14:29 +0800)]
staging: lustre: remove almost all crypto layer wrappers

Almost all of these are just a straight function name rename, so fix
them all up to call the crypto layer properly, no need for a #define to
hide things.

Cc: Peng Tao <tao.peng@emc.com>
Cc: Andreas Dilger <andreas.dilger@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: lustre: remove cfs_register_sysctl_table macro
Greg Kroah-Hartman [Sat, 3 Aug 2013 06:00:50 +0000 (14:00 +0800)]
staging: lustre: remove cfs_register_sysctl_table macro

It was a wrapper around register_sysctl_table, so just remove it.

Cc: Peng Tao <tao.peng@emc.com>
Cc: Andreas Dilger <andreas.dilger@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: dgnc: Remove version.h header inclusion in dgnc_trace.c
Sachin Kamat [Fri, 2 Aug 2013 09:10:18 +0000 (14:40 +0530)]
staging: dgnc: Remove version.h header inclusion in dgnc_trace.c

version.h header inclusion is not necessary as detected by
versioncheck.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: dgnc: Remove version.h header inclusion in dgnc_sysfs.c
Sachin Kamat [Fri, 2 Aug 2013 09:10:17 +0000 (14:40 +0530)]
staging: dgnc: Remove version.h header inclusion in dgnc_sysfs.c

version.h header inclusion is not necessary as detected by
versioncheck.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: dgnc: Remove version.h header inclusion in dgnc_neo.c
Sachin Kamat [Fri, 2 Aug 2013 09:10:16 +0000 (14:40 +0530)]
staging: dgnc: Remove version.h header inclusion in dgnc_neo.c

version.h header inclusion is not necessary as detected by
versioncheck.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: dgnc: Remove version.h header inclusion in dgnc_mgmt.c
Sachin Kamat [Fri, 2 Aug 2013 09:10:15 +0000 (14:40 +0530)]
staging: dgnc: Remove version.h header inclusion in dgnc_mgmt.c

version.h header inclusion is not necessary as detected by
versioncheck.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: dgnc: Include version.h header in dgnc_kcompat.h
Sachin Kamat [Fri, 2 Aug 2013 09:10:14 +0000 (14:40 +0530)]
staging: dgnc: Include version.h header in dgnc_kcompat.h

version.h header inclusion is necessary as detected by
versioncheck.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: dgnc: Remove version.h header inclusion in dgnc_driver.h
Sachin Kamat [Fri, 2 Aug 2013 09:10:13 +0000 (14:40 +0530)]
staging: dgnc: Remove version.h header inclusion in dgnc_driver.h

version.h header inclusion is not necessary as detected by
versioncheck.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: dgnc: Remove version.h header inclusion in dgnc_cls.c
Sachin Kamat [Fri, 2 Aug 2013 09:10:12 +0000 (14:40 +0530)]
staging: dgnc: Remove version.h header inclusion in dgnc_cls.c

version.h header inclusion is not necessary as detected by
versioncheck.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agoStaging: winbond: wb35reg: fixed some line over 80 characters
Iker Pedrosa [Fri, 2 Aug 2013 11:37:56 +0000 (13:37 +0200)]
Staging: winbond: wb35reg: fixed some line over 80 characters

Fixed some coding style issues

Signed-off-by: Iker Pedrosa <ikerpedrosam@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: dgnc: Remove duplicate header file inclusion
Sachin Kamat [Fri, 2 Aug 2013 09:15:26 +0000 (14:45 +0530)]
staging: dgnc: Remove duplicate header file inclusion

sched.h was included twice.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: lustre: remove RETURN macro
Greg Kroah-Hartman [Sat, 3 Aug 2013 02:35:28 +0000 (10:35 +0800)]
staging: lustre: remove RETURN macro

We have a kernel-wide function tracing system, so use that instead of
rolling a custom one just for one filesystem.

Cc: Peng Tao <tao.peng@emc.com>
Cc: Andreas Dilger <andreas.dilger@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: lustre: remove EXIT macro
Greg Kroah-Hartman [Fri, 2 Aug 2013 22:01:58 +0000 (06:01 +0800)]
staging: lustre: remove EXIT macro

We have a kernel-wide function tracing system, so use that instead of
rolling a custom one just for one filesystem.

Cc: Peng Tao <tao.peng@emc.com>
Cc: Andreas Dilger <andreas.dilger@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: lustre: remove RETURN_EXIT macro
Greg Kroah-Hartman [Fri, 2 Aug 2013 10:20:34 +0000 (18:20 +0800)]
staging: lustre: remove RETURN_EXIT macro

We have a kernel-wide function tracing system, so use that instead of
rolling a custom one just for one filesystem.

Cc: Peng Tao <tao.peng@emc.com>
Cc: Andreas Dilger <andreas.dilger@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: lustre: remove ENTRY macro
Greg Kroah-Hartman [Fri, 2 Aug 2013 10:14:51 +0000 (18:14 +0800)]
staging: lustre: remove ENTRY macro

We have a kernel-wide function tracing system, so use that instead of
rolling a custom one just for one filesystem.

Cc: Peng Tao <tao.peng@emc.com>
Cc: Andreas Dilger <andreas.dilger@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: lustre: remove ENTRY/EXIT_NESTING stuff
Greg Kroah-Hartman [Fri, 2 Aug 2013 08:24:50 +0000 (16:24 +0800)]
staging: lustre: remove ENTRY/EXIT_NESTING stuff

These macros were empty, so remove them in anticipation of removing the
entire ENTRY/EXIT macros.

Cc: Peng Tao <tao.peng@emc.com>
Cc: Andreas Dilger <andreas.dilger@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: lustre: remove ll_invalidate_bdev()
Greg Kroah-Hartman [Fri, 2 Aug 2013 08:17:04 +0000 (16:17 +0800)]
staging: lustre: remove ll_invalidate_bdev()

It was a wrapper around invalidate_bdev(), totally useless.

Cc: Peng Tao <tao.peng@emc.com>
Cc: Andreas Dilger <andreas.dilger@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: lustre: remove ll_vfs_symlink()
Greg Kroah-Hartman [Fri, 2 Aug 2013 08:11:35 +0000 (16:11 +0800)]
staging: lustre: remove ll_vfs_symlink()

No one was ever even using it.

Cc: Peng Tao <tao.peng@emc.com>
Cc: Andreas Dilger <andreas.dilger@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: lustre: remove ll_generic_permission()
Greg Kroah-Hartman [Fri, 2 Aug 2013 08:08:01 +0000 (16:08 +0800)]
staging: lustre: remove ll_generic_permission()

It was a wrapper around generic_permission(), totally useless.

Cc: Peng Tao <tao.peng@emc.com>
Cc: Andreas Dilger <andreas.dilger@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: lustre: remove ll_permission()
Greg Kroah-Hartman [Fri, 2 Aug 2013 08:01:26 +0000 (16:01 +0800)]
staging: lustre: remove ll_permission()

It was a wrapper around inode_permission(), totally useless.

Cc: Peng Tao <tao.peng@emc.com>
Cc: Andreas Dilger <andreas.dilger@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: lustre: remove ll_generic_file_llseek_size()
Greg Kroah-Hartman [Fri, 2 Aug 2013 07:56:22 +0000 (15:56 +0800)]
staging: lustre: remove ll_generic_file_llseek_size()

It was a wrapper around generic_file_llseek_size(), totally useless.

Cc: Peng Tao <tao.peng@emc.com>
Cc: Andreas Dilger <andreas.dilger@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: lustre: remove ll_dentry_open()
Greg Kroah-Hartman [Fri, 2 Aug 2013 07:51:34 +0000 (15:51 +0800)]
staging: lustre: remove ll_dentry_open()

It was a wrapper around dentry_open(), totally useless.

Cc: Peng Tao <tao.peng@emc.com>
Cc: Andreas Dilger <andreas.dilger@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: lustre: remove ll_blkdev_put()
Greg Kroah-Hartman [Fri, 2 Aug 2013 07:46:32 +0000 (15:46 +0800)]
staging: lustre: remove ll_blkdev_put()

It was a wrapper around blkdev_put(), totally useless.

Cc: Peng Tao <tao.peng@emc.com>
Cc: Andreas Dilger <andreas.dilger@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging/lustre: lloop depends on BLOCK
Xiong Zhou [Fri, 2 Aug 2013 05:50:48 +0000 (13:50 +0800)]
staging/lustre: lloop depends on BLOCK

Add a config option for llite/lloop in lustre driver, making it depends
on BLOCK to fix this better:
drivers/staging/lustre/lustre/fid/../include/linux/lustre_compat25.h:117:2:
error: implicit declaration of function â€˜unregister_blkdev'

Also, remove the wrapper ll_unregister_blkdev which depends on BLOCK in
the header and just call unregister_blkdev in lloop.c based on Peng Tao's
comment. Drop the redundant dependency on STAGING for LUSTRE_FS, remove
some unnecessary jdb header files which depends on BLOCK btw.

Signed-off-by: Xiong Zhou <jencce.kernel@gmail.com>
Reviewed-by: Peng Tao <tao.peng@emc.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: ozwpan: Set farewell report length.
Rupesh Gujare [Thu, 1 Aug 2013 17:45:02 +0000 (18:45 +0100)]
staging: ozwpan: Set farewell report length.

Fixes a bug where we were not setting length field causing wrong
report size to be copied.

Signed-off-by: Rupesh Gujare <rupesh.gujare@atmel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: ozwpan: Increase farewell report size.
Rupesh Gujare [Thu, 1 Aug 2013 17:45:01 +0000 (18:45 +0100)]
staging: ozwpan: Increase farewell report size.

Farewell report size can be bigger than one byte, increase array
size to accomodate maximum 32 bytes of farewell report.

Signed-off-by: Rupesh Gujare <rupesh.gujare@atmel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: ozwpan: Drop oldest ISOC frame instead of dropping latest.
Rupesh Gujare [Thu, 1 Aug 2013 17:40:03 +0000 (18:40 +0100)]
staging: ozwpan: Drop oldest ISOC frame instead of dropping latest.

In case of ISOC transfer, if TX queue is full then we start
dropping latest frame, instead we should drop oldest frame &
add latest frame to TX queue.

Signed-off-by: Rupesh Gujare <rupesh.gujare@atmel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: ozwpan: Use kernel strncmp function.
Rupesh Gujare [Thu, 1 Aug 2013 17:40:02 +0000 (18:40 +0100)]
staging: ozwpan: Use kernel strncmp function.

Use kernel's strncmp() function instead of defining same within driver.

Signed-off-by: Rupesh Gujare <rupesh.gujare@atmel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: ozwpan: Mark string as const
Rupesh Gujare [Thu, 1 Aug 2013 17:40:01 +0000 (18:40 +0100)]
staging: ozwpan: Mark string as const

Make sure that we mark const string so that it does not get modified.

Signed-off-by: Rupesh Gujare <rupesh.gujare@atmel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: ozwpan: Use kernel list function for managing interface list.
Rupesh Gujare [Thu, 1 Aug 2013 17:40:00 +0000 (18:40 +0100)]
staging: ozwpan: Use kernel list function for managing interface list.

Managing interface list, is easier if we use kernel list_* API
than managing it on our own.

Signed-off-by: Rupesh Gujare <rupesh.gujare@atmel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: imx-drm: add missing MFD_SYSCON dependency for LDB
Jingoo Han [Thu, 1 Aug 2013 00:32:29 +0000 (09:32 +0900)]
staging: imx-drm: add missing MFD_SYSCON dependency for LDB

Added missing MFD_SYSCON dependency for LVDS display bridge
in order to fix the following link error.

drivers/staging/imx-drm/imx-ldb.c:484: undefined reference to `syscon_regmap_lookup_by_phandle'

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Acked-by: Philipp Zabel <p.zabel@pengutronix.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: dgnc: add dgnc digi driver
Lidza Louina [Thu, 1 Aug 2013 21:00:20 +0000 (17:00 -0400)]
staging: dgnc: add dgnc digi driver

This patch adds the DGNC driver. This is a TTY Serial Port
Driver for the Digi International Neo and Classic PCI based product
line by Digi International <http://www.digi.com>.

This driver isn't hooked up to the build system because it doesn't
build, it merely adds the driver written by Digi to the kernel tree so
that it can be cleaned up and fixed up properly over time.

Cc: Mark Hounschell <markh@compro.net>
Signed-off-by: Lidza Louina <lidza.louina@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: octeon-usb: eliminate remaining typedef unions
Aaro Koskinen [Tue, 30 Jul 2013 20:43:12 +0000 (23:43 +0300)]
staging: octeon-usb: eliminate remaining typedef unions

Remaining typedef unions can be trivially eliminated with the pattern:

foo_t -> union foo

Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: octeon-usb: cvmx_usb_control_header_t -> union cvmx_usb_control_header
Aaro Koskinen [Tue, 30 Jul 2013 20:43:11 +0000 (23:43 +0300)]
staging: octeon-usb: cvmx_usb_control_header_t -> union cvmx_usb_control_header

Replace cvmx_usb_control_header_t with union cvmx_usb_control_header.

Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: octeon-usb: cvmx_usb_internal_state_t -> struct cvmx_usb_internal_state
Aaro Koskinen [Tue, 30 Jul 2013 20:43:10 +0000 (23:43 +0300)]
staging: octeon-usb: cvmx_usb_internal_state_t -> struct cvmx_usb_internal_state

Replace cvmx_usb_internal_state_t with struct cvmx_usb_internal_state.

Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: octeon-usb: cvmx_usb_tx_fifo_t -> struct cvmx_usb_tx_fifo
Aaro Koskinen [Tue, 30 Jul 2013 20:43:09 +0000 (23:43 +0300)]
staging: octeon-usb: cvmx_usb_tx_fifo_t -> struct cvmx_usb_tx_fifo

Replace cvmx_usb_tx_fifo_t with struct cvmx_usb_tx_fifo_t.

Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: octeon-usb: cvmx_usb_pipe_list_t -> struct cvmx_usb_pipe_list
Aaro Koskinen [Tue, 30 Jul 2013 20:43:08 +0000 (23:43 +0300)]
staging: octeon-usb: cvmx_usb_pipe_list_t -> struct cvmx_usb_pipe_list

Replace cvmx_usb_pipe_list_t with struct cvmx_usb_pipe_list.

Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: octeon-usb: cvmx_usb_pipe_t -> struct cvmx_usb_pipe
Aaro Koskinen [Tue, 30 Jul 2013 20:43:07 +0000 (23:43 +0300)]
staging: octeon-usb: cvmx_usb_pipe_t -> struct cvmx_usb_pipe

Replace cvmx_usb_pipe_t with struct cvmx_usb_pipe.

Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: octeon-usb: cvmx_usb_transaction_t -> struct cvmx_usb_transaction
Aaro Koskinen [Tue, 30 Jul 2013 20:43:06 +0000 (23:43 +0300)]
staging: octeon-usb: cvmx_usb_transaction_t -> struct cvmx_usb_transaction

Replace cvmx_usb_transaction_t with struct cvmx_usb_transaction.

Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: octeon-usb: cvmx_usb_state_t -> struct cvmx_usb_state
Aaro Koskinen [Tue, 30 Jul 2013 20:43:05 +0000 (23:43 +0300)]
staging: octeon-usb: cvmx_usb_state_t -> struct cvmx_usb_state

Replace cvmx_usb_state_t with struct cvmx_usb_state.

Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: octeon-usb: cvmx_usb_iso_packet_t -> struct cvmx_usb_iso_packet
Aaro Koskinen [Tue, 30 Jul 2013 20:43:04 +0000 (23:43 +0300)]
staging: octeon-usb: cvmx_usb_iso_packet_t -> struct cvmx_usb_iso_packet

Replace cvmx_usb_iso_packet_t with struct cvmx_usb_iso_packet.

Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: octeon-usb: cvmx_usb_port_status_t -> struct cvmx_usb_port_status
Aaro Koskinen [Tue, 30 Jul 2013 20:43:03 +0000 (23:43 +0300)]
staging: octeon-usb: cvmx_usb_port_status_t -> struct cvmx_usb_port_status

Replace cvmx_usb_port_status_t with struct cvmx_usb_port_status.

Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: octeon-usb: cvmx_usb_stage_t -> enum cvmx_usb_stage
Aaro Koskinen [Tue, 30 Jul 2013 20:43:02 +0000 (23:43 +0300)]
staging: octeon-usb: cvmx_usb_stage_t -> enum cvmx_usb_stage

Replace cvmx_usb_stage_t with enum cvmx_usb_stage.

Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: octeon-usb: cvmx_usb_transaction_flags_t -> enum cvmx_usb_transaction_flags
Aaro Koskinen [Tue, 30 Jul 2013 20:43:01 +0000 (23:43 +0300)]
staging: octeon-usb: cvmx_usb_transaction_flags_t -> enum cvmx_usb_transaction_flags

Replace cvmx_usb_transaction_flags_t with enum cvmx_usb_transaction_flags.

Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: octeon-usb: cvmx_usb_isochronous_flags_t -> enum cvmx_usb_isochronous_flags
Aaro Koskinen [Tue, 30 Jul 2013 20:43:00 +0000 (23:43 +0300)]
staging: octeon-usb: cvmx_usb_isochronous_flags_t -> enum cvmx_usb_isochronous_flags

Replace cvmx_usb_isochronous_flags_t with enum cvmx_usb_isochronous_flags.

Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: octeon-usb: cvmx_usb_pipe_flags_t -> enum cvmx_usb_pipe_flags
Aaro Koskinen [Tue, 30 Jul 2013 20:42:59 +0000 (23:42 +0300)]
staging: octeon-usb: cvmx_usb_pipe_flags_t -> enum cvmx_usb_pipe_flags

Replace cvmx_usb_pipe_flags_t with enum cvmx_usb_pipe_flags.

Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: octeon-usb: cvmx_usb_initialize_flags_t -> enum cvmx_usb_initialize_flags
Aaro Koskinen [Tue, 30 Jul 2013 20:42:58 +0000 (23:42 +0300)]
staging: octeon-usb: cvmx_usb_initialize_flags_t -> enum cvmx_usb_initialize_flags

Replace cvmx_usb_initialize_flags_t with enum cvmx_usb_initialize_flags.

Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: octeon-usb: cvmx_usb_callback_t -> enum cvmx_usb_callback
Aaro Koskinen [Tue, 30 Jul 2013 20:42:57 +0000 (23:42 +0300)]
staging: octeon-usb: cvmx_usb_callback_t -> enum cvmx_usb_callback

Replace cvmx_usb_callback_t with enum cvmx_usb_callback.

Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: octeon-usb: cvmx_usb_complete_t -> enum cvmx_usb_complete
Aaro Koskinen [Tue, 30 Jul 2013 20:42:56 +0000 (23:42 +0300)]
staging: octeon-usb: cvmx_usb_complete_t -> enum cvmx_usb_complete

Replace cvmx_usb_complete_t with enum cvmx_usb_complete.

Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: octeon-usb: cvmx_usb_direction_t -> enum cvmx_usb_direction
Aaro Koskinen [Tue, 30 Jul 2013 20:42:55 +0000 (23:42 +0300)]
staging: octeon-usb: cvmx_usb_direction_t -> enum cvmx_usb_direction

Replace cvmx_usb_direction_t with enum cvmx_usb_direction.

Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: octeon-usb: cvmx_usb_transfer_t -> enum cvmx_usb_transfer
Aaro Koskinen [Tue, 30 Jul 2013 20:42:54 +0000 (23:42 +0300)]
staging: octeon-usb: cvmx_usb_transfer_t -> enum cvmx_usb_transfer

Replace cvmx_usb_transfer_t with enum cvmx_usb_transfer.

Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: octeon-usb: cvmx_usb_speed_t -> enum cvmx_usb_speed
Aaro Koskinen [Tue, 30 Jul 2013 20:42:53 +0000 (23:42 +0300)]
staging: octeon-usb: cvmx_usb_speed_t -> enum cvmx_usb_speed

Replace cvmx_usb_speed_t with enum cvmx_usb_speed.

Signed-off-by: Aaro Koskinen <aaro.koskinen@iki.fi>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: ozwpan: High resolution timers
Rupesh Gujare [Tue, 30 Jul 2013 12:31:50 +0000 (13:31 +0100)]
staging: ozwpan: High resolution timers

Current implementation assumes HZ = 1000 for calculating
all internal timer intervals, which creates problem on
platforms where HZ != 1000.

As well we need resolution of less than 10 mSec for heartbeat
calculation, this creates problem on some platforms where HZ is
configured as HZ = 100, or around, which restricts us to timer interval
of 10 mSec. This is particularly found on embedded devices.

This patch moves on to use high resolution timers to calculate
all timer intervals as it allows us to have very small resolution
of timer interval, removing dependency on HZ.

Signed-off-by: Rupesh Gujare <rupesh.gujare@atmel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: dt9812: Resolve different base types warnings.
Shaun Laing [Tue, 30 Jul 2013 17:29:43 +0000 (11:29 -0600)]
staging: comedi: dt9812: Resolve different base types warnings.

Resolves warnings from the "sparse" checker of the form "warning: incorrect
type in assignment (different base types)".

Signed-off-by: Shaun Laing <shaun@xresource.ca>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Reviewed-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: allow ISA and PC/104 drivers on non-ISA systems
H Hartley Sweeten [Tue, 30 Jul 2013 16:39:49 +0000 (09:39 -0700)]
staging: comedi: allow ISA and PC/104 drivers on non-ISA systems

Embedded systems with a PC/104 bus might have a configuration that
does not have ISA enabled. This creates a problem in Comedi where
the PC/104 drivers cannot be enabled.

Remove the 'depends on ISA' for the "Comedi ISA and PC/104 drivers"
menu to allow the user to select these drivers on non-ISA systems.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: usbduxsigma: absorb usbduxsigma_attach_common() into caller
H Hartley Sweeten [Tue, 30 Jul 2013 00:48:50 +0000 (17:48 -0700)]
staging: comedi: usbduxsigma: absorb usbduxsigma_attach_common() into caller

This function is only called by usbduxsigma_auto_attach(). Absorb it
and refactor the code a bit to match the usbdux driver.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: usbdux: tidy up usbdux_free_usb_buffers()
H Hartley Sweeten [Tue, 30 Jul 2013 00:48:25 +0000 (17:48 -0700)]
staging: comedi: usbdux: tidy up usbdux_free_usb_buffers()

For aesthetics, pass the comedi_device pointer to this function
instead of the private data pointer.

The usb_kill_urb() calls are not needed. All the urbs were unlinked
before this function was called.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: usbdux: tidy up usbdux_detach()
H Hartley Sweeten [Tue, 30 Jul 2013 00:48:04 +0000 (17:48 -0700)]
staging: comedi: usbdux: tidy up usbdux_detach()

Make sure the usb intfdata is always cleared when the device is
detached.

Refactor the code to remove an indent level.

Always unlink the urbs when the device is detached.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: usbduxsigma: always clear the usb intfdata in (*detach)
H Hartley Sweeten [Tue, 30 Jul 2013 00:47:44 +0000 (17:47 -0700)]
staging: comedi: usbduxsigma: always clear the usb intfdata in (*detach)

Make sure the usb intfdata is always cleared when the device is
detached.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: usbduxsigma: move urb unlink into usbduxsigma_detach()
H Hartley Sweeten [Tue, 30 Jul 2013 00:47:26 +0000 (17:47 -0700)]
staging: comedi: usbduxsigma: move urb unlink into usbduxsigma_detach()

For aesthetics, move the final force unlink of the urbs from
usbduxsigma_free_usb_buffers() to usbduxsigma_detach().

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: usbduxsigma: init 'pwm_buf_sz' in usbduxsigma_auto_attach()
H Hartley Sweeten [Tue, 30 Jul 2013 00:47:06 +0000 (17:47 -0700)]
staging: comedi: usbduxsigma: init 'pwm_buf_sz' in usbduxsigma_auto_attach()

Initialize this variable with then_ai_urbs and n_ao_urbs in the
attach. Then usbduxsigma_alloc_usb_buffers() can just check the
variable to determine if the urb needs to be allocated.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: usbduxsigma: 'dac_commands' does not need to be kzalloc()'d
H Hartley Sweeten [Tue, 30 Jul 2013 00:46:46 +0000 (17:46 -0700)]
staging: comedi: usbduxsigma: 'dac_commands' does not need to be kzalloc()'d

The 'dac_commands' buffer is used to pass the ao channel list from
usbduxsigma_ao_cmd() to the urb callback, usbduxsigma_ao_urb_complete().
This buffer does not need to be allocated.

Change it into a simple array of the correct size. Rename the variable
to 'ao_chanlist' to clarify what it actually is.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: usbdux: tidy up usbdux_alloc_usb_buffers()
H Hartley Sweeten [Tue, 30 Jul 2013 00:46:27 +0000 (17:46 -0700)]
staging: comedi: usbdux: tidy up usbdux_alloc_usb_buffers()

Tidy up the error path by doing all the basic allocations then checking
for failures.

We have the comedi device now so fill in the urb->context.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: usbduxsigma: return void from usbduxsigma_pwm_pattern()
H Hartley Sweeten [Tue, 30 Jul 2013 00:46:07 +0000 (17:46 -0700)]
staging: comedi: usbduxsigma: return void from usbduxsigma_pwm_pattern()

This function never fails and currently returns '1' so that the
usbduxsigma_pwm_write() function returns to proper value (insn->n).

It's also called by usbduxsigma_pwm_config() which also returns the
'1' value. The proper return code for this function should be '0' to
indicate success.

Change the return type of the function to void and just have the
callers return the proper values.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: usbdux: return void from usbdux_pwm_pattern()
H Hartley Sweeten [Tue, 30 Jul 2013 00:45:47 +0000 (17:45 -0700)]
staging: comedi: usbdux: return void from usbdux_pwm_pattern()

This function never fails and currently returns '1' so that the
usbdux_pwm_write() function returns to proper value (insn->n).

It's also called by usbdux_pwm_config() which also returns the '1'
value. The proper return code for this function should be '0' to
indicate success.

Change the return type of the function to void and just have the
callers return the proper values.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: usbdux: tidy up usbdux_pwm_config()
H Hartley Sweeten [Tue, 30 Jul 2013 00:45:28 +0000 (17:45 -0700)]
staging: comedi: usbdux: tidy up usbdux_pwm_config()

Rename the local variable used for the private data pointer to 'devpriv'.

Tidy up the function a bit.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: usbdux: remove usbdux_pwm_read()
H Hartley Sweeten [Tue, 30 Jul 2013 00:45:09 +0000 (17:45 -0700)]
staging: comedi: usbdux: remove usbdux_pwm_read()

The pwm subdevice is not readable so this callback function is not
needed. Remove it.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: usbdux: tidy up usbdux_pwm_write()
H Hartley Sweeten [Tue, 30 Jul 2013 00:44:50 +0000 (17:44 -0700)]
staging: comedi: usbdux: tidy up usbdux_pwm_write()

Remove the sanity check of the private data. This function can only be
called if the private data was successfully allocated in the attach.

Tidy up the function a bit.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: usbdux: tidy up usbdux_pwm_pattern()
H Hartley Sweeten [Tue, 30 Jul 2013 00:44:32 +0000 (17:44 -0700)]
staging: comedi: usbdux: tidy up usbdux_pwm_pattern()

Rename the local variable used for the private data pointer to 'devpriv'.

Remove the sanity check of the private data. This function can only be
called if the private data was successfully allocated in the attach.

Tidy up the function to make it more concise.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: usbdux: tidy up usbdux_pwm_period()
H Hartley Sweeten [Tue, 30 Jul 2013 00:44:14 +0000 (17:44 -0700)]
staging: comedi: usbdux: tidy up usbdux_pwm_period()

Rename the local variable used for the private data pointer to 'devpriv'.

Fix the fx2delay calculation so it does not use floating point values.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: usbduxsigma: remove the usb endpoint defines
H Hartley Sweeten [Tue, 30 Jul 2013 00:43:55 +0000 (17:43 -0700)]
staging: comedi: usbduxsigma: remove the usb endpoint defines

The endpoint defines are each only used in one place and don't help
clarify the code. Remove the defines and just open code the values.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: usbduxsigma: generalize the ai/ao usb_kill_urb()
H Hartley Sweeten [Tue, 30 Jul 2013 00:43:36 +0000 (17:43 -0700)]
staging: comedi: usbduxsigma: generalize the ai/ao usb_kill_urb()

Generalize a helper function to replace for() loops in usbduxsigma_{ai,ao}_stop()
that call usb_kill_urb() to unlink all the urbs.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: usbdux: generalize the usb_submit_urb functions
H Hartley Sweeten [Tue, 30 Jul 2013 00:43:16 +0000 (17:43 -0700)]
staging: comedi: usbdux: generalize the usb_submit_urb functions

Generalize a helper function to replace usbduxsub_submit_{in,out}urbs().

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: vt6656: device.h unused variables
Malcolm Priestley [Tue, 30 Jul 2013 20:08:54 +0000 (21:08 +0100)]
staging: vt6656: device.h unused variables

wACKDuration
wRTSTransmitLen
byRTSServiceField
byRTSSignalField

Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: vt6656: dead code wCTSDuration.
Malcolm Priestley [Tue, 30 Jul 2013 20:03:13 +0000 (21:03 +0100)]
staging: vt6656: dead code wCTSDuration.

Always set to zero, so remove.

Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: xillybus: Fixed sparse errors
Eli Billauer [Wed, 31 Jul 2013 08:22:43 +0000 (11:22 +0300)]
staging: xillybus: Fixed sparse errors

Changes:

* xillybus.h: __iomem added to struct xilly_endpoint -> registers to suppress
  "different address spaces" errors.

* xillybus_core.c: __user added as required for the same reason.

* The two member names of struct xilly_endpoint_hardware of the form
  sync_single_for_{cpu,device} were changed to something that won't look
  like the well-known functions.

* All *.c files: Variables and functions made static as required.

Reported-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Eli Billauer <eli.billauer@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: xillybus: Removed dependency on OF_DEVICE in Kconfig
Eli Billauer [Wed, 31 Jul 2013 08:22:42 +0000 (11:22 +0300)]
staging: xillybus: Removed dependency on OF_DEVICE in Kconfig

This variable was removed from the kernel a while ago.

Signed-off-by: Eli Billauer <eli.billauer@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: das16: tidy up multi-line comments
H Hartley Sweeten [Mon, 29 Jul 2013 21:08:31 +0000 (14:08 -0700)]
staging: comedi: das16: tidy up multi-line comments

Tidy up the multi-line comments to follow the CodingStyle.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: das16: remove some unnecessary parentheses
H Hartley Sweeten [Mon, 29 Jul 2013 21:08:13 +0000 (14:08 -0700)]
staging: comedi: das16: remove some unnecessary parentheses

These parentheses are not necessary.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: das16: tidy up user ai/ao range initialization
H Hartley Sweeten [Mon, 29 Jul 2013 21:07:53 +0000 (14:07 -0700)]
staging: comedi: das16: tidy up user ai/ao range initialization

The allocation of the user range tables could fail. Make sure to check
for it.

Change the kmalloc()'s to kzalloc()'s to make sure the allocated range
tables are initialized to a known state.

Change the local variables so they can be used for both the ai and ao
range initialization and use shorter names to keep the lines < 80 chars.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: das16: remove an unnecessary check in the attach
H Hartley Sweeten [Mon, 29 Jul 2013 21:07:30 +0000 (14:07 -0700)]
staging: comedi: das16: remove an unnecessary check in the attach

If the user does not specify the analog output range the private data
'user_ao_range_table' will not be allocated. The comedi core will detect
the NULL s->range_table and automatically set it to &range_unknown.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: das16: fix das16_detach()
H Hartley Sweeten [Mon, 29 Jul 2013 21:07:10 +0000 (14:07 -0700)]
staging: comedi: das16: fix das16_detach()

The function das16_reset() needs a valid dev->iobase. Since the iobase
is requested after the devpriv has been allocated in the attach, move
the das16_reset() call into the if (devpriv) and add a if (dev->iobase).

Also, move the release of the extra iobase into the if (devpriv) to
prevent an invalid memory access.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: das16: change MODULE_DESCRIPTION
H Hartley Sweeten [Mon, 29 Jul 2013 21:06:52 +0000 (14:06 -0700)]
staging: comedi: das16: change MODULE_DESCRIPTION

Change the MODULE_DESCRIPTION to something useful instead of the
generic "Comedi low-level driver".

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: das16: tidy up private data definition
H Hartley Sweeten [Mon, 29 Jul 2013 21:06:34 +0000 (14:06 -0700)]
staging: comedi: das16: tidy up private data definition

Remove the comments and add some whitespace to help readability.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: das16: cleanup comedi_error() messages
H Hartley Sweeten [Mon, 29 Jul 2013 21:06:16 +0000 (14:06 -0700)]
staging: comedi: das16: cleanup comedi_error() messages

Some of these messages are missing the terminating '\n' and most of them
have the quoted string split across lines.

Change all of them to dev_err() messages and fix the issues.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: das16: fix a > 80 char line issue
H Hartley Sweeten [Mon, 29 Jul 2013 21:05:59 +0000 (14:05 -0700)]
staging: comedi: das16: fix a > 80 char line issue

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: das16: absorb das1600_mode_detect() into caller
H Hartley Sweeten [Mon, 29 Jul 2013 21:05:37 +0000 (14:05 -0700)]
staging: comedi: das16: absorb das1600_mode_detect() into caller

This function is only called during the board attach to determine the
pacer clock speed for the boards capable of bursting.

Absorb it into the caller and remove the attach noise.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: das16: cleanup printk message in board attach
H Hartley Sweeten [Mon, 29 Jul 2013 21:05:20 +0000 (14:05 -0700)]
staging: comedi: das16: cleanup printk message in board attach

Convert this printk message into a dev_err().

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: das16: fail attach if board id is incorrect
H Hartley Sweeten [Mon, 29 Jul 2013 21:05:00 +0000 (14:05 -0700)]
staging: comedi: das16: fail attach if board id is incorrect

This driver probes the board id during the attach to ensure that the
user is attempting to attach to the correct board type. Currently an
error message is output if the ids are not consistent but the attach
continues anyway.

Make the attach fail if the id bits do not match. Also, cleanup the
printk messages and use a dev_err() only if the probe fails.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: das16: tidy up the register map defines
H Hartley Sweeten [Mon, 29 Jul 2013 21:04:41 +0000 (14:04 -0700)]
staging: comedi: das16: tidy up the register map defines

Add the suffix _REG to all the register defines to better indicate
what they are.

Add a prefix to all the register bit defines to avoid namespace clashes
and to better indicate what they are.

Tidy up the whitesapce.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: das16: tidy up 8255 digital i/o subdevice
H Hartley Sweeten [Mon, 29 Jul 2013 21:04:13 +0000 (14:04 -0700)]
staging: comedi: das16: tidy up 8255 digital i/o subdevice

The 8255 digital i/o subdevice is only available on some of the boards
supported by this driver. Currently the 'i8255_offset' member of the
boardinfo is used to indicate that it is available.

To clarify this support, add a 'has_8255' bit-field flag to the boardinfo.
We can then use this flag to conditionally allocate the space for the
subdevice only when it is needed.

Also, make sure to check that subdev_8255_init() is successful and propagate
any errors.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: das16: remove 'i8254_offset' from boardinfo
H Hartley Sweeten [Mon, 29 Jul 2013 21:03:46 +0000 (14:03 -0700)]
staging: comedi: das16: remove 'i8254_offset' from boardinfo

All the boards supported by this driver have an 8254 timer at offset
0x0c. Remove this unused information from the boardinfo.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: das16: move boardinfo declaration
H Hartley Sweeten [Mon, 29 Jul 2013 21:03:28 +0000 (14:03 -0700)]
staging: comedi: das16: move boardinfo declaration

Now that the boardinfo does not require any forward declarations,
move it near the definition of the struct.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: das16: remove unnecessary test in das16_interrupt()
H Hartley Sweeten [Mon, 29 Jul 2013 21:03:11 +0000 (14:03 -0700)]
staging: comedi: das16: remove unnecessary test in das16_interrupt()

The timer is only started by the ai subdevice in das16_cmd_exec()
and this function can only be called if the device is attached.

Remove the unnecessary sanity check.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: das16: move some functions to improve readability
H Hartley Sweeten [Mon, 29 Jul 2013 21:02:48 +0000 (14:02 -0700)]
staging: comedi: das16: move some functions to improve readability

For aesthetics, move some of the functions to make the code easier
to follow.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: das16: tidy up das16_ai_insn_read()
H Hartley Sweeten [Mon, 29 Jul 2013 21:02:22 +0000 (14:02 -0700)]
staging: comedi: das16: tidy up das16_ai_insn_read()

Factor out the loop that waits for the ai conversion to complete.

Tidy up the function a bit.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: das16: introduce das16_ai_{en, dis}able() helpers
H Hartley Sweeten [Mon, 29 Jul 2013 20:58:36 +0000 (13:58 -0700)]
staging: comedi: das16: introduce das16_ai_{en, dis}able() helpers

Introduce a couple helper functions to enable and disable the
analog input interrupt/dma conversions.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: das16: remove the register map comments
H Hartley Sweeten [Mon, 29 Jul 2013 20:58:17 +0000 (13:58 -0700)]
staging: comedi: das16: remove the register map comments

This is just extra cruft. Remove it.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: das16: remove unused static const variable
H Hartley Sweeten [Mon, 29 Jul 2013 20:57:58 +0000 (13:57 -0700)]
staging: comedi: das16: remove unused static const variable

This variable is not used in the driver. Remove it.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
11 years agostaging: comedi: das16: remove unused DAS16_SIZE
H Hartley Sweeten [Mon, 29 Jul 2013 20:57:38 +0000 (13:57 -0700)]
staging: comedi: das16: remove unused DAS16_SIZE

This define is not used in the driver. Remove it.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>