Commits · 9b335e2b2f2e92b1134035818c9ae9316bbddca3 · nexedi / linux

13 Jul, 2014 3 commits

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 37 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

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