Commits · 170ce6d2c514dbf0dabd749e8dcb9943bb2c173f · Kirill Smelkov / linux

12 Jul, 2014 40 commits

staging: vt6656: baseband.c rename abyVT3184_AGC to vnt_vt3184_agc · 170ce6d2
Malcolm Priestley authored Jul 12, 2014
```
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
```
170ce6d2

staging: vt6656: baseband clean up VT3184 static variables · 4fd6e9f2

Malcolm Priestley authored Jul 12, 2014

Remove white space comments and arrange 8 bytes wide
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

4fd6e9f2

staging: vt6656: rf.c cleanup comments · ef13cfb4

Malcolm Priestley authored Jul 12, 2014

named comments are already referenced in revision history
in high-level comment.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

ef13cfb4

staging: vt6656: vnt_rf_write_embedded add reg length and IFREGCTL_REGW · f1c840d5

Malcolm Priestley authored Jul 12, 2014

The rf register always have  *_REG_LEN and IFREGCTL_REGW macros added.

*_REG_LEN is always 23(0x17) replace them with VNT_RF_REG_LEN.

Remove *_REG_LEN and IFREGCTL_REGW from tables and vnt_rf_set_txpower and
apply VNT_RF_REG_LEN and IFREGCTL_REGW in vnt_rf_write_embedded
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

f1c840d5

staging: vt6656: rf.c remove dead code RFaby11aChannelIndex · 0e2b6fe1

Malcolm Priestley authored Jul 12, 2014

Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

0e2b6fe1

staging: vt6656: rf.c clean up channel and power tables · 77bdc0a5

Malcolm Priestley authored Jul 12, 2014

Remove comments to channel number execpt where there is a
change to 5gHz channel 15

named comments are already referenced in revision history
in high-level comment.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

77bdc0a5

Staging: bcm: Transmit.c: Removed obvious comment · 2349c05a

Matthias Beyer authored Jul 12, 2014

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

2349c05a

Staging: bcm: Transmit.c: Line length / Whitespace cleanup · 358ed017

Matthias Beyer authored Jul 12, 2014

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

358ed017

Staging: bcm: Transmit.c: Replaced member accessing with variable · c971681a

Matthias Beyer authored Jul 12, 2014

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

c971681a

staging: vt6656: wcmd replace debug message with dev_dbg · f7f04857

Malcolm Priestley authored Jul 12, 2014

Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

f7f04857

staging: vt6656: vResetCommandTimer remove camel case · 4e3af0fa

Malcolm Priestley authored Jul 12, 2014

pDevice -> priv
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

4e3af0fa

staging: vt6656: bScheduleCommand remove camel case · f7e9ed41

Malcolm Priestley authored Jul 12, 2014

Camel case changes
pDevice -> priv
eCommand -> command
pbyItem0 -> item0
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

f7e9ed41

staging: vt6656: s_bCommandComplete clean up and remove camel case · d287093e

Malcolm Priestley authored Jul 12, 2014

pDevice -> priv
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

d287093e

staging: vt6656: s_bCommandComplete remove else from if · a675dc21

Malcolm Priestley authored Jul 12, 2014

if returns out so remove else
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

a675dc21

staging: vt6656: remove unused variables from struct tagCMD_ITEM · 18647c58

Malcolm Priestley authored Jul 12, 2014

Also remove local variables
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

18647c58

staging: vt6656: vRunCommand remove camel case. · cf07bb8e

Malcolm Priestley authored Jul 12, 2014

pDevice -> priv
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

cf07bb8e

staging: vt6656: wcmd remove unused commands · 43878a36

Malcolm Priestley authored Jul 12, 2014

WLAN_CMD_CHANGE_BBSENSITIVITY and WLAN_CMD_MAC_DISPOWERSAVING
are already implemented and don't need the command timer.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

43878a36

staging: vt6656: wcmd Remove command WLAN_CMD_RADIO · c06b1ad0

Malcolm Priestley authored Jul 12, 2014

Part of it has already been removed.  checking isr1 & ISR_GPIO3
serves no purpose.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

c06b1ad0

staging: vt6656: vCommandTimerWait remove camel case. · e30546d6

Malcolm Priestley authored Jul 12, 2014

camel case changes
pDevice -> priv
MSecond -> msecs
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

e30546d6

staging: lustre: fid: fid_request.c: checkpatch cleanup: align arguments to parenthesis · 913827ee
Greg Kroah-Hartman authored Jul 12, 2014
```
whitespace changes only - git diff -w shows no difference
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
```
913827ee

staging: lustre: remove cfs_time_before() · 699503bc

Greg Kroah-Hartman authored Jul 12, 2014

Just call time_before() instead.

Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

699503bc

staging: lustre: cfs_time_sub() must return unsigned long · 88c03eae

Greg Kroah-Hartman authored Jul 12, 2014

It looks like someone ment for it to return this value (there's a cast),
but the funtion says 'long' which causes problems when comparing values
from this function.

Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

88c03eae

staging: lustre: remove cfs_duration_t typedef · b2d201bd

Greg Kroah-Hartman authored Jul 12, 2014

Just use 'long' as that is what it was defined as.

Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

b2d201bd

staging: lustre: remove cfs_time_current_sec wrapper · 7264b8a5

Greg Kroah-Hartman authored Jul 12, 2014

Just call get_seconds() directly.

Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

7264b8a5

staging: lustre: remove cfs_time_beforeq wrapper · 82173c62

Greg Kroah-Hartman authored Jul 12, 2014

Use time_before_eq() call instead.

Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

82173c62

staging: lustre: remove cfs_time_t typedef · a649ad1d

Greg Kroah-Hartman authored Jul 12, 2014

Just use unsigned long everywhere, like the rest of the kernel does.

Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

a649ad1d

staging: lustre: remove unused functions from linux-time.h · 8718c63e

Greg Kroah-Hartman authored Jul 12, 2014

No one was using them, so delete them.

Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

8718c63e

staging: lustre: remove cfs_fs_time_t · 61b3508a

Greg Kroah-Hartman authored Jul 12, 2014

Use 'struct timespec' like the rest of the kernel does

Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

61b3508a

staging: lustre: remove linux-tcpip.h · fb4a1539

Greg Kroah-Hartman authored Jul 12, 2014

There were just a few macros being used, remove them and have the places
where they were being used to make the proper kernel api call instead.

Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

fb4a1539

staging: lustre: linux-tcpip.h: remove unused macros · e12517f3

Greg Kroah-Hartman authored Jul 11, 2014

Some macros are never used, so remove them.

Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

e12517f3

staging: lustre: fix makefile build breakage · 490e7dd4

Greg Kroah-Hartman authored Jul 11, 2014

Forgot to test with infiniband enabled.
Reported-by: kbuild test robot <fengguang.wu@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

490e7dd4

staging: lustre: remove socket_t typedef · e327dc88

Greg Kroah-Hartman authored Jul 11, 2014

Use 'struct socket' like the rest of the kernel does.

Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org

e327dc88

staging: lustre: remove ctl_table typedefs · 691bc0eb

Greg Kroah-Hartman authored Jul 11, 2014

No need for these, just use the "real" struct the kernel provides.

Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

691bc0eb

staging: lustre: remove linux-types.h · 9fc48d00

Greg Kroah-Hartman authored Jul 11, 2014

It's an empty file, so remove it.

Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

9fc48d00

staging: lustre: fix build breakage on microblaze · 86e5b63d

Greg Kroah-Hartman authored Jul 11, 2014

I missed an include path that microblaze found in it's build paths.

Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Reported-by: kbuild test robot <fengguang.wu@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

86e5b63d

staging: lustre: delete linux-lock.h · 6de9710b

Greg Kroah-Hartman authored Jul 11, 2014

It was a few empty defines, so just remove the thing.

Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

6de9710b

staging: lustre: linux-cpu.h · ebf7b778

Greg Kroah-Hartman authored Jul 11, 2014

Remove large commented out section for compatibility stuff that will
never be needed again.

Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

ebf7b778

staging: lustre: remove include/libcfs/linux/linux-bitops.h · d2bc118d

Greg Kroah-Hartman authored Jul 11, 2014

It was just includeing bitops.h so do that instead in the one place that
was calling it.

Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

d2bc118d

staging: lustre: make include/linux/libcfs/linux/linux-crypto.h private · 13636be4

Greg Kroah-Hartman authored Jul 11, 2014

There are only 2 functions that are actually called from it, so move it
to the local place where it is needed, no need for it to be "global" for
the rest of lustre to see it.

Also remove the unneeded EXPORT_SYMBOL() markings for these functions,
as no one else calls them.

Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

13636be4

staging: lustre: remove unneeded DECLARE_LL_PROC_PPOS_DECL macro · 30f38472

Greg Kroah-Hartman authored Jul 11, 2014

It did nothing, so remove it.

Cc: Andreas Dilger <andreas.dilger@intel.com>
Cc: Oleg Drokin <oleg.drokin@intel.com>
Cc: hpdd-discuss <hpdd-discuss@lists.01.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

30f38472