]> git.karo-electronics.de Git - karo-tx-linux.git/log
karo-tx-linux.git
12 years agoStaging: silicom: Header-cleanup
DanielC [Tue, 11 Sep 2012 02:41:51 +0000 (19:41 -0700)]
Staging: silicom: Header-cleanup

Staging: silicom: Header cleanup of WD_STEP_COUNT_GET

Signed-off-by: Daniel Cotey <puff65537@bansheeslibrary.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging:wlan-ng: clean some more functions
Devendra Naga [Sun, 9 Sep 2012 13:11:02 +0000 (18:41 +0530)]
staging:wlan-ng: clean some more functions

the function p80211netdev_rx doesnt' need a return at the end
and also remove some new lines

the function p80211knetdev_set_mac_address doesn't need the result
variable assigned as it gets a return from p80211req_dorequest function

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging:wlan-ng: clean register_wlandev function
Devendra Naga [Sun, 9 Sep 2012 13:11:01 +0000 (18:41 +0530)]
staging:wlan-ng: clean register_wlandev function

we dont need i to just return the status of register_netdev

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging:wlan-ng: cleanup prism2sta_commsqual_defer and hfa384x_drvr_getconfig
Devendra Naga [Sun, 9 Sep 2012 13:11:00 +0000 (18:41 +0530)]
staging:wlan-ng: cleanup prism2sta_commsqual_defer and hfa384x_drvr_getconfig

the function prism2sta_commsqual_defer defines a goto done lable, which just
jumps to end of function, which we can achieve with out it

the hfa384x_drvr_getconfig doesn't need the result variable, we can
remove and just return the function

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging:wlan-ng: cleanup p80211skb_free and p80211skb_rxmeta_detach
Devendra Naga [Sun, 9 Sep 2012 13:10:59 +0000 (18:40 +0530)]
staging:wlan-ng: cleanup p80211skb_free and p80211skb_rxmeta_detach

these functions doesn't need return at the end of the function

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging:wlan-ng: clean coding style problems in p80211req_mibset_mibget
Devendra Naga [Sun, 9 Sep 2012 13:10:58 +0000 (18:40 +0530)]
staging:wlan-ng: clean coding style problems in p80211req_mibset_mibget

clean this function to follow kernel coding way

remove double tabs
remove spaces

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging:wlan-ng: p80211req_mibset_mibget should be void
Devendra Naga [Sun, 9 Sep 2012 13:10:57 +0000 (18:40 +0530)]
staging:wlan-ng: p80211req_mibset_mibget should be void

this function always returning 0, there are no callers that checks
the return of this function, make this function return void

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging:wlan-ng: clean p80211req_handlemsg
Devendra Naga [Sun, 9 Sep 2012 13:10:56 +0000 (18:40 +0530)]
staging:wlan-ng: clean p80211req_handlemsg

the switch case doesn't have a break statement of DIDmsg_dot11req_mibget,
and DIDmsg_dot11req_mibset cases, and also cleanup the code to follow
the kernel coding way

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging:wlan-ng: remove default case in the p80211req_handlemsg
Devendra Naga [Sun, 9 Sep 2012 13:10:55 +0000 (18:40 +0530)]
staging:wlan-ng: remove default case in the p80211req_handlemsg

default is redundant and remove it

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging:wlan-ng: cleanup hfa384x_ctlxout_callback
Devendra Naga [Sun, 9 Sep 2012 13:10:54 +0000 (18:40 +0530)]
staging:wlan-ng: cleanup hfa384x_ctlxout_callback

goto done is not required and actually having goto done does
jumps the program to end of the function and calls return.
instead call return directly

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging:wlan-ng: cleanup hfa384x_usbctlx_resptimerfn
Devendra Naga [Sun, 9 Sep 2012 13:10:53 +0000 (18:40 +0530)]
staging:wlan-ng: cleanup hfa384x_usbctlx_resptimerfn

goto done is not required and simple return is fine

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging:wlan-ng: fix sparse warnings
Devendra Naga [Sun, 9 Sep 2012 13:10:52 +0000 (18:40 +0530)]
staging:wlan-ng: fix sparse warnings

drivers/staging/wlan-ng/prism2fw.c:240:5: warning: symbol 'prism2_fwapply' was not declared. Should it be static?
drivers/staging/wlan-ng/prism2fw.c:379:5: warning: symbol 'crcimage' was not declared. Should it be static?
drivers/staging/wlan-ng/prism2fw.c:443:6: warning: symbol 'free_chunks' was not declared. Should it be static?
drivers/staging/wlan-ng/prism2fw.c:465:6: warning: symbol 'free_srecs' was not declared. Should it be static?
drivers/staging/wlan-ng/prism2fw.c:492:5: warning: symbol 'mkimage' was not declared. Should it be static?
drivers/staging/wlan-ng/prism2fw.c:585:5: warning: symbol 'mkpdrlist' was not declared. Should it be static?
drivers/staging/wlan-ng/prism2fw.c:659:5: warning: symbol 'plugimage' was not declared. Should it be static?
drivers/staging/wlan-ng/prism2fw.c:767:5: warning: symbol 'read_cardpda' was not declared. Should it be static?
drivers/staging/wlan-ng/prism2fw.c:857:5: warning: symbol 'read_fwfile' was not declared. Should it be static?
drivers/staging/wlan-ng/prism2fw.c:981:5: warning: symbol 'writeimage' was not declared. Should it be static?
drivers/staging/wlan-ng/prism2fw.c:1133:5: warning: symbol 'validate_identity' was not declared. Should it be static?

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging:wlan-ng: slove sparse warnings
Devendra Naga [Sun, 9 Sep 2012 13:10:51 +0000 (18:40 +0530)]
staging:wlan-ng: slove sparse warnings

drivers/staging/wlan-ng/prism2fw.c:127:18: warning: symbol 's3data' was not declared. Should it be static?
drivers/staging/wlan-ng/prism2fw.c:130:14: warning: symbol 'ns3plug' was not declared. Should it be static?
drivers/staging/wlan-ng/prism2fw.c:131:18: warning: symbol 's3plug' was not declared. Should it be static?
drivers/staging/wlan-ng/prism2fw.c:134:14: warning: symbol 'ns3crc' was not declared. Should it be static?
drivers/staging/wlan-ng/prism2fw.c:135:17: warning: symbol 's3crc' was not declared. Should it be static?
drivers/staging/wlan-ng/prism2fw.c:138:14: warning: symbol 'ns3info' was not declared. Should it be static?
drivers/staging/wlan-ng/prism2fw.c:139:18: warning: symbol 's3info' was not declared. Should it be static?
drivers/staging/wlan-ng/prism2fw.c:142:5: warning: symbol 'startaddr' was not declared. Should it be static?
drivers/staging/wlan-ng/prism2fw.c:145:14: warning: symbol 'nfchunks' was not declared. Should it be static?
drivers/staging/wlan-ng/prism2fw.c:146:17: warning: symbol 'fchunk' was not declared. Should it be static?
drivers/staging/wlan-ng/prism2fw.c:154:12: warning: symbol 'pda' was not declared. Should it be static?
drivers/staging/wlan-ng/prism2fw.c:155:21: warning: symbol 'nicid' was not declared. Should it be static?
drivers/staging/wlan-ng/prism2fw.c:156:20: warning: symbol 'rfid' was not declared. Should it be static?
drivers/staging/wlan-ng/prism2fw.c:157:20: warning: symbol 'macid' was not declared. Should it be static?
drivers/staging/wlan-ng/prism2fw.c:158:20: warning: symbol 'priid' was not declared. Should it be static?

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in ms.c
Toshiaki Yamane [Sun, 9 Sep 2012 13:01:35 +0000 (22:01 +0900)]
staging/rts_pstor: remove braces {} in ms.c

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks
-WARNING: braces {} are not necessary for any arm of this statement

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in rtsx_scsi.c
Toshiaki Yamane [Sat, 8 Sep 2012 12:53:24 +0000 (21:53 +0900)]
staging/rts_pstor: remove braces {} in rtsx_scsi.c

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks
-WARNING: braces {} are not necessary for any arm of this statement

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (release_sd_card)
Toshiaki Yamane [Fri, 7 Sep 2012 04:39:36 +0000 (13:39 +0900)]
staging/rts_pstor: remove braces {} in sd.c (release_sd_card)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_power_off_card3v3)
Toshiaki Yamane [Fri, 7 Sep 2012 04:39:22 +0000 (13:39 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_power_off_card3v3)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_hw_rst)
Toshiaki Yamane [Fri, 7 Sep 2012 04:39:09 +0000 (13:39 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_hw_rst)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_execute_write_data)
Toshiaki Yamane [Fri, 7 Sep 2012 04:38:56 +0000 (13:38 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_execute_write_data)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for any arm of this statement
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_execute_read_data)
Toshiaki Yamane [Fri, 7 Sep 2012 04:38:43 +0000 (13:38 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_execute_read_data)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for any arm of this statement
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_execute_no_data)
Toshiaki Yamane [Fri, 7 Sep 2012 04:38:28 +0000 (13:38 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_execute_no_data)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (get_rsp_type)
Toshiaki Yamane [Fri, 7 Sep 2012 04:38:15 +0000 (13:38 +0900)]
staging/rts_pstor: remove braces {} in sd.c (get_rsp_type)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_pass_thru_mode)
Toshiaki Yamane [Fri, 7 Sep 2012 04:38:02 +0000 (13:38 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_pass_thru_mode)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (ext_sd_get_rsp)
Toshiaki Yamane [Fri, 7 Sep 2012 04:37:47 +0000 (13:37 +0900)]
staging/rts_pstor: remove braces {} in sd.c (ext_sd_get_rsp)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (ext_sd_send_cmd_get_rsp)
Toshiaki Yamane [Fri, 7 Sep 2012 04:37:20 +0000 (13:37 +0900)]
staging/rts_pstor: remove braces {} in sd.c (ext_sd_send_cmd_get_rsp)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_rw)
Toshiaki Yamane [Fri, 7 Sep 2012 04:37:05 +0000 (13:37 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_rw)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for any arm of this statement
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_auto_tune_clock)
Toshiaki Yamane [Fri, 7 Sep 2012 04:36:52 +0000 (13:36 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_auto_tune_clock)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_stop_seq_mode)
Toshiaki Yamane [Fri, 7 Sep 2012 04:36:37 +0000 (13:36 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_stop_seq_mode)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (wait_data_buf_ready)
Toshiaki Yamane [Fri, 7 Sep 2012 04:36:24 +0000 (13:36 +0900)]
staging/rts_pstor: remove braces {} in sd.c (wait_data_buf_ready)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (reset_mmc_only)
Toshiaki Yamane [Fri, 7 Sep 2012 04:36:11 +0000 (13:36 +0900)]
staging/rts_pstor: remove braces {} in sd.c (reset_mmc_only)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (reset_sd_card)
Toshiaki Yamane [Fri, 7 Sep 2012 04:35:56 +0000 (13:35 +0900)]
staging/rts_pstor: remove braces {} in sd.c (reset_sd_card)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (reset_mmc)
Toshiaki Yamane [Fri, 7 Sep 2012 04:35:41 +0000 (13:35 +0900)]
staging/rts_pstor: remove braces {} in sd.c (reset_mmc)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for any arm of this statement
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (mmc_switch_timing_bus)
Toshiaki Yamane [Fri, 7 Sep 2012 04:35:28 +0000 (13:35 +0900)]
staging/rts_pstor: remove braces {} in sd.c (mmc_switch_timing_bus)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for any arm of this statement
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (mmc_test_switch_bus)
Toshiaki Yamane [Fri, 7 Sep 2012 04:35:14 +0000 (13:35 +0900)]
staging/rts_pstor: remove braces {} in sd.c (mmc_test_switch_bus)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for any arm of this statement
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (reset_sd)
Toshiaki Yamane [Fri, 7 Sep 2012 04:35:00 +0000 (13:35 +0900)]
staging/rts_pstor: remove braces {} in sd.c (reset_sd)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for any arm of this statement
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_check_wp_state)
Toshiaki Yamane [Fri, 7 Sep 2012 04:34:44 +0000 (13:34 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_check_wp_state)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_read_lba0)
Toshiaki Yamane [Fri, 7 Sep 2012 04:34:28 +0000 (13:34 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_read_lba0)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for any arm of this statement

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_init_power)
Toshiaki Yamane [Fri, 7 Sep 2012 04:34:14 +0000 (13:34 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_init_power)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_pull_ctl_enable)
Toshiaki Yamane [Fri, 7 Sep 2012 04:34:00 +0000 (13:34 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_pull_ctl_enable)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_prepare_reset)
Toshiaki Yamane [Fri, 7 Sep 2012 04:33:45 +0000 (13:33 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_prepare_reset)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for any arm of this statement
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_switch_clock)
Toshiaki Yamane [Fri, 7 Sep 2012 04:33:16 +0000 (13:33 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_switch_clock)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for any arm of this statement
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (mmc_ddr_tuning)
Toshiaki Yamane [Fri, 7 Sep 2012 04:32:56 +0000 (13:32 +0900)]
staging/rts_pstor: remove braces {} in sd.c (mmc_ddr_tuning)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_ddr_tuning)
Toshiaki Yamane [Fri, 7 Sep 2012 04:32:32 +0000 (13:32 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_ddr_tuning)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_sdr_tuning)
Toshiaki Yamane [Fri, 7 Sep 2012 04:31:49 +0000 (13:31 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_sdr_tuning)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_tuning_tx)
Toshiaki Yamane [Fri, 7 Sep 2012 04:31:19 +0000 (13:31 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_tuning_tx)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for any arm of this statement
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_ddr_pre_tuning_tx)
Toshiaki Yamane [Fri, 7 Sep 2012 04:31:04 +0000 (13:31 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_ddr_pre_tuning_tx)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_tuning_rx)
Toshiaki Yamane [Fri, 7 Sep 2012 04:30:45 +0000 (13:30 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_tuning_rx)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for any arm of this statement
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_search_final_phase)
Toshiaki Yamane [Fri, 7 Sep 2012 04:30:30 +0000 (13:30 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_search_final_phase)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for any arm of this statement
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_ddr_tuning_tx_cmd)
Toshiaki Yamane [Fri, 7 Sep 2012 04:30:16 +0000 (13:30 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_ddr_tuning_tx_cmd)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for any arm of this statement
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_sdr_tuning_tx_cmd)
Toshiaki Yamane [Fri, 7 Sep 2012 04:30:01 +0000 (13:30 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_sdr_tuning_tx_cmd)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (mmc_ddr_tunning_rx_cmd)
Toshiaki Yamane [Fri, 7 Sep 2012 04:29:46 +0000 (13:29 +0900)]
staging/rts_pstor: remove braces {} in sd.c (mmc_ddr_tunning_rx_cmd)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for any arm of this statement
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_ddr_tuning_rx_cmd)
Toshiaki Yamane [Fri, 7 Sep 2012 04:29:29 +0000 (13:29 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_ddr_tuning_rx_cmd)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_sdr_tuning_rx_cmd)
Toshiaki Yamane [Fri, 7 Sep 2012 04:29:15 +0000 (13:29 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_sdr_tuning_rx_cmd)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_switch_function)
Toshiaki Yamane [Fri, 7 Sep 2012 04:29:00 +0000 (13:29 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_switch_function)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for any arm of this statement
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_check_switch)
Toshiaki Yamane [Fri, 7 Sep 2012 04:28:44 +0000 (13:28 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_check_switch)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (downgrade_switch_mode)
Toshiaki Yamane [Fri, 7 Sep 2012 04:28:28 +0000 (13:28 +0900)]
staging/rts_pstor: remove braces {} in sd.c (downgrade_switch_mode)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_check_switch_mode)
Toshiaki Yamane [Fri, 7 Sep 2012 04:28:09 +0000 (13:28 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_check_switch_mode)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_check_spec)
Toshiaki Yamane [Fri, 7 Sep 2012 04:27:54 +0000 (13:27 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_check_spec)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_change_phase)
Toshiaki Yamane [Fri, 7 Sep 2012 04:27:38 +0000 (13:27 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_change_phase)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for any arm of this statement
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_voltage_switch)
Toshiaki Yamane [Fri, 7 Sep 2012 04:27:24 +0000 (13:27 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_voltage_switch)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_change_bank_voltage)
Toshiaki Yamane [Fri, 7 Sep 2012 04:27:08 +0000 (13:27 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_change_bank_voltage)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_wait_state_data_ready)
Toshiaki Yamane [Fri, 7 Sep 2012 04:26:35 +0000 (13:26 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_wait_state_data_ready)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_update_lock_status)
Toshiaki Yamane [Fri, 7 Sep 2012 04:26:16 +0000 (13:26 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_update_lock_status)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks
-WARNING: braces {} are not necessary for any arm of this statement

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_select_card)
Toshiaki Yamane [Fri, 7 Sep 2012 04:25:48 +0000 (13:25 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_select_card)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_set_init_para)
Toshiaki Yamane [Fri, 7 Sep 2012 04:25:08 +0000 (13:25 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_set_init_para)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_set_clock_divider)
Toshiaki Yamane [Fri, 7 Sep 2012 04:24:26 +0000 (13:24 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_set_clock_divider)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for any arm of this statement

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_choose_proper_clock)
Toshiaki Yamane [Fri, 7 Sep 2012 04:24:03 +0000 (13:24 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_choose_proper_clock)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for any arm of this statement

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_set_sample_push_timing)
Toshiaki Yamane [Fri, 7 Sep 2012 04:23:46 +0000 (13:23 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_set_sample_push_timing)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks
-WARNING: braces {} are not necessary for any arm of this statement

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_check_csd)
Toshiaki Yamane [Fri, 7 Sep 2012 04:23:29 +0000 (13:23 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_check_csd)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks
-WARNING: braces {} are not necessary for any arm of this statement

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_write_data)
Toshiaki Yamane [Fri, 7 Sep 2012 04:23:09 +0000 (13:23 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_write_data)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_read_data)
Toshiaki Yamane [Fri, 7 Sep 2012 04:22:43 +0000 (13:22 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_read_data)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging/rts_pstor: remove braces {} in sd.c (sd_send_cmd_get_rsp)
Toshiaki Yamane [Fri, 7 Sep 2012 04:16:27 +0000 (13:16 +0900)]
staging/rts_pstor: remove braces {} in sd.c (sd_send_cmd_get_rsp)

fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for single statement blocks
-WARNING: braces {} are not necessary for any arm of this statement

Signed-off-by: Toshiaki Yamane <yamanetoshi@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoStaging: panel: fix spinlock trylock failure on UP
Fengguang Wu [Sun, 9 Sep 2012 08:23:46 +0000 (16:23 +0800)]
Staging: panel: fix spinlock trylock failure on UP

Use spin_lock_irq() to quiet warning:

         [    8.232324] BUG: spinlock trylock failure on UP on CPU#0, reboot/85
         [    8.234138]  lock: c161c760, .magic: dead4ead, .owner: reboot/85, .owner_cpu: 0
         [    8.236132] Pid: 85, comm: reboot Not tainted 3.4.0-rc7-00656-g82163ed #5
         [    8.237965] Call Trace:
         [    8.238648]  [<c13dfd20>] ? printk+0x18/0x1a
         [    8.239827]  [<c122a5e0>] spin_dump+0x80/0xd0
         [    8.241016]  [<c122a652>] spin_bug+0x22/0x30
         [    8.242181]  [<c122a93b>] do_raw_spin_trylock+0x5b/0x70
         [    8.243611]  [<c13e8bae>] _raw_spin_trylock+0xe/0x60
         [    8.244975]  [<c1392230>] ? keypad_send_key.constprop.9+0xe0/0xe0
 ==>     [    8.246638]  [<c13922ea>] panel_scan_timer+0xba/0x570
         [    8.248019]  [<c1392230>] ? keypad_send_key.constprop.9+0xe0/0xe0
         [    8.249689]  [<c102f6f5>] run_timer_softirq+0x1e5/0x370
         [    8.251191]  [<c102f645>] ? run_timer_softirq+0x135/0x370
         [    8.252718]  [<c1392230>] ? keypad_send_key.constprop.9+0xe0/0xe0
         [    8.254462]  [<c102a592>] __do_softirq+0xc2/0x1c0
         [    8.255758]  [<c102a4d0>] ? local_bh_enable_ip+0x130/0x130
         [    8.257228]  <IRQ>  [<c102a855>] ? irq_exit+0x65/0x70
         [    8.258647]  [<c1013ff9>] ? smp_apic_timer_interrupt+0x49/0x80
         [    8.260226]  [<c13e96c1>] ? apic_timer_interrupt+0x31/0x38
         [    8.261737]  [<c12700e0>] ? drm_vm_open_locked+0x70/0xb0
         [    8.263166]  [<c122489a>] ? delay_tsc+0x1a/0x30
         [    8.264452]  [<c12248c9>] ? __delay+0x9/0x10
         [    8.265621]  [<c12248ec>] ? __const_udelay+0x1c/0x20
 ==>     [    8.266967]  [<c139136c>] ? lcd_clear_fast_p8+0x9c/0xe0
         [    8.268386]  [<c1391a66>] ? lcd_write+0x226/0x810
         [    8.269653]  [<c1367900>] ? md_set_readonly+0xc0/0xc0
         [    8.271013]  [<c122a9ed>] ? do_raw_spin_unlock+0x9d/0xe0
         [    8.272470]  [<c1392a98>] ? panel_lcd_print+0x38/0x40
         [    8.273837]  [<c1392ace>] ? panel_notify_sys+0x2e/0x60
         [    8.275224]  [<c1046634>] ? notifier_call_chain+0x84/0xb0
         [    8.276754]  [<c10469ce>] ? __blocking_notifier_call_chain+0x3e/0x60
         [    8.278576]  [<c1046a0a>] ? blocking_notifier_call_chain+0x1a/0x20
         [    8.280267]  [<c1036a14>] ? kernel_restart_prepare+0x14/0x40
         [    8.281901]  [<c1036a8e>] ? kernel_restart+0xe/0x50
         [    8.283216]  [<c1036ce9>] ? sys_reboot+0x149/0x1e0
         [    8.284532]  [<c10b3fb3>] ? handle_pte_fault+0x93/0xd70
         [    8.285956]  [<c1019e35>] ? do_page_fault+0x215/0x5e0
         [    8.287330]  [<c101a113>] ? do_page_fault+0x4f3/0x5e0
         [    8.288704]  [<c1045ac6>] ? up_read+0x16/0x30
         [    8.289890]  [<c101a113>] ? do_page_fault+0x4f3/0x5e0
         [    8.291252]  [<c10d4486>] ? iterate_supers+0x86/0xd0
         [    8.292615]  [<c122a9ed>] ? do_raw_spin_unlock+0x9d/0xe0
         [    8.294049]  [<c13e8dcd>] ? _raw_spin_unlock+0x1d/0x20
         [    8.295449]  [<c10d44ab>] ? iterate_supers+0xab/0xd0
         [    8.296795]  [<c10fb620>] ? __sync_filesystem+0xa0/0xa0
         [    8.298199]  [<c13e9b03>] ? sysenter_do_call+0x12/0x37
         [    8.306899] Restarting system.
         [    8.307747] machine restart

Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: fix ccg build when NET is not enabled
Randy Dunlap [Mon, 10 Sep 2012 16:03:41 +0000 (09:03 -0700)]
staging: fix ccg build when NET is not enabled

Fix build errors in ccg when CONFIG_NET is not enabled
by adding "depends on NET":

ERROR: "netif_carrier_on" [drivers/staging/ccg/g_ccg.ko] undefined!
ERROR: "netif_carrier_off" [drivers/staging/ccg/g_ccg.ko] undefined!
ERROR: "skb_realloc_headroom" [drivers/staging/ccg/g_ccg.ko] undefined!
ERROR: "skb_trim" [drivers/staging/ccg/g_ccg.ko] undefined!
ERROR: "netif_rx" [drivers/staging/ccg/g_ccg.ko] undefined!
ERROR: "ethtool_op_get_link" [drivers/staging/ccg/g_ccg.ko] undefined!
ERROR: "free_netdev" [drivers/staging/ccg/g_ccg.ko] undefined!
ERROR: "register_netdev" [drivers/staging/ccg/g_ccg.ko] undefined!
ERROR: "skb_push" [drivers/staging/ccg/g_ccg.ko] undefined!
ERROR: "skb_pull" [drivers/staging/ccg/g_ccg.ko] undefined!
ERROR: "dev_kfree_skb_any" [drivers/staging/ccg/g_ccg.ko] undefined!
ERROR: "skb_queue_tail" [drivers/staging/ccg/g_ccg.ko] undefined!
ERROR: "__alloc_skb" [drivers/staging/ccg/g_ccg.ko] undefined!
ERROR: "eth_type_trans" [drivers/staging/ccg/g_ccg.ko] undefined!
ERROR: "eth_validate_addr" [drivers/staging/ccg/g_ccg.ko] undefined!
ERROR: "skb_dequeue" [drivers/staging/ccg/g_ccg.ko] undefined!
ERROR: "unregister_netdev" [drivers/staging/ccg/g_ccg.ko] undefined!
ERROR: "__netif_schedule" [drivers/staging/ccg/g_ccg.ko] undefined!
ERROR: "skb_put" [drivers/staging/ccg/g_ccg.ko] undefined!
ERROR: "eth_mac_addr" [drivers/staging/ccg/g_ccg.ko] undefined!
ERROR: "dev_get_stats" [drivers/staging/ccg/g_ccg.ko] undefined!
ERROR: "alloc_etherdev_mqs" [drivers/staging/ccg/g_ccg.ko] undefined!

Signed-off-by: Randy Dunlap <rdunlap@xenotime.net>
Cc: Andrzej Pietrasiewicz <andrzej.p@samsung.com>
Cc: Mike Lockwood <lockwood@android.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoStaging: silicom: Force depend on module
DanielC [Tue, 11 Sep 2012 02:18:12 +0000 (19:18 -0700)]
Staging: silicom: Force depend on module

Staging: silicom: Force depend on module until monolithic build fixed

Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Signed-off-by: Daniel Cotey <puff65537@bansheeslibrary.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: ipack: make function tables const.
Stephen Hemminger [Mon, 10 Sep 2012 18:14:01 +0000 (11:14 -0700)]
staging: ipack: make function tables const.

Make some variables const:
  1. bus oerations table
  2. driver name
  3. tpci control register table

Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoStaging: silicom: C99 cleanup of libbp_sd.h
DanielC [Sat, 8 Sep 2012 04:32:27 +0000 (21:32 -0700)]
Staging: silicom: C99 cleanup of libbp_sd.h

Signed-off-by: Daniel Cotey <puff65537@bansheeslibrary.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoStaging: silicom: C99 cleanup of bypass.h
DanielC [Sat, 8 Sep 2012 04:31:17 +0000 (21:31 -0700)]
Staging: silicom: C99 cleanup of bypass.h

Signed-off-by: Daniel Cotey <puff65537@bansheeslibrary.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agoStaging: silicom: C99 cleanup of bp_ioctl.h
DanielC [Sat, 8 Sep 2012 04:29:14 +0000 (21:29 -0700)]
Staging: silicom: C99 cleanup of bp_ioctl.h

Signed-off-by: Daniel Cotey <puff65537@bansheeslibrary.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: add Silicom Bypass driver
DanielC [Fri, 7 Sep 2012 02:59:06 +0000 (19:59 -0700)]
staging: add Silicom Bypass driver

The Silicom Bypass Network Interface Cards (NICs) are network cards with
paired ports (2 or 4).  The pairs either act as a "wire" allowing the
network packets to pass or insert the device in between the two ports.
When paired with the on-board hardware watchdog or other failsafe, they
provide high availability for the network in the face of software
outages or maintenance.

Signed-off-by: Daniel Cotey <puff65537@bansheeslibrary.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: vt6655: vt6655_init_info function must be void type
Devendra Naga [Thu, 6 Sep 2012 18:38:06 +0000 (00:08 +0530)]
staging: vt6655: vt6655_init_info function must be void type

this is because it doesn't fail anywhere and returning a value
from it will be completely unnecesary.

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: vt6655: return ENOMEM rather than ENODEV when alloc_etherdev fail
Devendra Naga [Thu, 6 Sep 2012 18:38:05 +0000 (00:08 +0530)]
staging: vt6655: return ENOMEM rather than ENODEV when alloc_etherdev fail

when alloc_etherdev fails we should be returning ENOMEM, not ENODEV

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: vt6655: fix coding style problem at assigning netdev_ops
Devendra Naga [Thu, 6 Sep 2012 18:38:04 +0000 (00:08 +0530)]
staging: vt6655: fix coding style problem at assigning netdev_ops

we are using spaces at the beginning of the line, we should use
tabs instead

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: vt6655: fix coding style issues in device_get_options
Devendra Naga [Thu, 6 Sep 2012 18:38:03 +0000 (00:08 +0530)]
staging: vt6655: fix coding style issues in device_get_options

a) put the device_get_options functions' opening brace below
b) replace spaces with tabs

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Acked-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: vt6655: fix coding style warnings
Devendra Naga [Thu, 6 Sep 2012 18:38:02 +0000 (00:08 +0530)]
staging: vt6655: fix coding style warnings

a) replace spaces with tabs
b) put the opening brace of get_chip_name below it

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Acked-by: Marcos Paulo de Souza <marcos.souza.org@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: ramster: fix build warnings
Dan Magenheimer [Thu, 6 Sep 2012 17:01:14 +0000 (10:01 -0700)]
staging: ramster: fix build warnings

Fix build warnings resulting from in-progress work that was
not entirely ifdef'd out.

Signed-off-by: Dan Magenheimer <dan.magenheimer@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: tpci200: fix section mismatch warning
Stephen Hemminger [Fri, 7 Sep 2012 15:50:54 +0000 (08:50 -0700)]
staging: tpci200: fix section mismatch warning

PCI probe routines have to be named _probe to avoid section mismatch warning.
Found when doing 'make allmodconfig'

Signed-off-by: Stephen Hemminger <shemminger@vyatta.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: tidspbridge: move the dereference below the NULL test
Wei Yongjun [Fri, 7 Sep 2012 06:59:15 +0000 (14:59 +0800)]
staging: tidspbridge: move the dereference below the NULL test

The dereference should be moved below the NULL test.

spatch with a semantic match is used to found this.
(http://coccinelle.lip6.fr/)

Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: me4000: convert printk's to dev_printk's
H Hartley Sweeten [Sat, 8 Sep 2012 00:50:58 +0000 (17:50 -0700)]
staging: comedi: me4000: convert printk's to dev_printk's

A lot of the messages produced by this driver are just noise and
need to be removed. For now just convert them all to dev_printk's.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: me4000: remove me4000.h
H Hartley Sweeten [Sat, 8 Sep 2012 00:50:39 +0000 (17:50 -0700)]
staging: comedi: me4000: remove me4000.h

Move the remaining defines in this header to the c file. Nothing
in the header is needed by any other file.

While moving the defines, reorder them so that the bit defines
are associated with the register they go with. Also, convert the
bit defines to bit shifts to make them a bit clearer.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: me4000: move struct me4000_info definition
H Hartley Sweeten [Sat, 8 Sep 2012 00:50:19 +0000 (17:50 -0700)]
staging: comedi: me4000: move struct me4000_info definition

Move the struct me4000_info definition from the header to the c
file. This struct is not used by any other file.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: me4000: remove ME4000_CNT_* defines
H Hartley Sweeten [Sat, 8 Sep 2012 00:50:01 +0000 (17:50 -0700)]
staging: comedi: me4000: remove ME4000_CNT_* defines

Since this driver is using the 8253.h helpers, these defines are
no longer needed.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: me4000: cleanup me4000_cnt_insn_config()
H Hartley Sweeten [Sat, 8 Sep 2012 00:49:37 +0000 (17:49 -0700)]
staging: comedi: me4000: cleanup me4000_cnt_insn_config()

Absorb the cnt_reset and cnt_config helper functions. They are
now both just single line routines.

Remove the printk noise about "Invalid instruction length".

Fix the return value, the value should be the number of data values
used to perform the instruction. The GPCT_RESET instruction only
has one parameter.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: me4000: use the 8253 helper functions
H Hartley Sweeten [Sat, 8 Sep 2012 00:49:21 +0000 (17:49 -0700)]
staging: comedi: me4000: use the 8253 helper functions

The counter subdevice of this board is a standard 8254 compatible
counter/timer. Instead of open-coding the 8254 timer io, use the
helper functions provided by 8253.h.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: me4000: add namespace to reset_board()
H Hartley Sweeten [Sat, 8 Sep 2012 00:49:05 +0000 (17:49 -0700)]
staging: comedi: me4000: add namespace to reset_board()

Rename reset_board() to me4000_reset() so it has namespace associated
with this driver. Change it's return type to void, it always succeeds.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: me4000: fix the interrupt request/free
H Hartley Sweeten [Sat, 8 Sep 2012 00:48:48 +0000 (17:48 -0700)]
staging: comedi: me4000: fix the interrupt request/free

Only set the dev->irq when the request_irq is successful.
Use the dev->board_name for the name passed to request_irq.
Change the printk messages into dev_warn().
Make sure free_irq is called in the detach.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: me4000: combine the checks for valid io addresses
H Hartley Sweeten [Sat, 8 Sep 2012 00:48:26 +0000 (17:48 -0700)]
staging: comedi: me4000: combine the checks for valid io addresses

Combine the sanity checks for valid io addresses into one if().

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: me4000: remove program_regbase from private data
H Hartley Sweeten [Sat, 8 Sep 2012 00:48:09 +0000 (17:48 -0700)]
staging: comedi: me4000: remove program_regbase from private data

The program_regbase variable in the private data is only used
when the board is first attached. This variable holds the pci
base address used to read/write the xilinx part to upload the
firmware.

Use a local variable in the xilinx_download() function to hold
this address and remove program_regbase from the private data.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: me4000: use attach_pci callback
H Hartley Sweeten [Sat, 8 Sep 2012 00:47:41 +0000 (17:47 -0700)]
staging: comedi: me4000: use attach_pci callback

Convert this PCI driver to use the comedi PCI auto config attach
mechanism by adding an 'attach_pci' callback function. Since the
driver does not require any external configuration options, and
the legacy 'attach' callback is now optional, remove it.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
12 years agostaging: comedi: me4000: cleanup me4000_probe()
H Hartley Sweeten [Sat, 8 Sep 2012 00:47:20 +0000 (17:47 -0700)]
staging: comedi: me4000: cleanup me4000_probe()

Move the non pci probe related code out of the me4000_probe function
and back into the me4000_attach function in preparation of converting
this driver to the 'attach_pci' callback.

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