Commits · fa8d5b44740cd90551461e0ddbb34fbc6c914be5 · nexedi / linux

13 Jul, 2014 4 commits

staging: lustre: remove linux-prim.h · fa8d5b44

Greg Kroah-Hartman authored Jul 12, 2014

It's empty now, move the #include lines it was pulling in, into libcfs.h
and sort them to remove the duplicates.

Odds are they all aren't needed, but let's not break the build by
experimenting with removing them at this point in time.

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

fa8d5b44

staging: lustre: clean up some proc file declarations · 9b335e2b

Greg Kroah-Hartman authored Jul 12, 2014

This removes the DECLARE_PROC_HANDLER macro, and makes proc_call_handler
a static function (horrible name to be in a module).  This makes the
linux-prim.h file empty now, so it can be removed soon.

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

9b335e2b

staging: lustre: remove hpdd-discuss list from TODO file · e432a77f

Greg Kroah-Hartman authored Jul 12, 2014

It's a closed mailing list, what's the use of cc:ing patches there if
they never go through.  So remove it from the TODO file.

A real MAINTAINERS entry is probably a good thing for the Lustre code,
maybe someday someone will send a patch to do that...

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>

e432a77f

staging: lustre: remove LL_PROC_PROTO macro · 3bdb7ce4

Greg Kroah-Hartman authored Jul 12, 2014

Spell out what is happening with the proc file functions, it helps to
unwind the call chain mess here.

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>

3bdb7ce4

12 Jul, 2014 36 commits

staging: vt6656: baseband.c rename abyVT3184_VT3226D0 to vnt_vt3184_vt3226d0 · 0a16b63d
Malcolm Priestley authored Jul 12, 2014
```
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
```
0a16b63d
staging: vt6656: baseband.c remame abyVT3184_AL2230 to vnt_vt3184_al2230 · ab48ab22
Malcolm Priestley authored Jul 12, 2014
```
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
```
ab48ab22
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