Malcolm Priestley
2cd98cd5b8
staging: vt6656: rename vCommandTimerWait to vnt_cmd_timer_wait
...
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-13 12:22:34 -07:00
Malcolm Priestley
60cecd2857
staging: vt6656: Move and rename s_bCommandComplete to vnt_cmd_complete
...
Remove static declaration and moving for function visibility
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-13 12:22:34 -07:00
Malcolm Priestley
190c8190e1
staging: vt6656: dead code remove typedef enum tagCMD_STATUS
...
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-13 12:22:34 -07:00
Malcolm Priestley
33a60b87b6
staging: vt6656: replace camel case from command variables
...
camel case changes
uCmdDequeueIdx -> cmd_dequeue_idx
uCmdEnqueueIdx -> cmd_enqueue_idx
cbFreeCmdQueue -> free_cmd_queue
bCmdRunning -> cmd_running
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-13 12:22:34 -07:00
Malcolm Priestley
9a309289b2
staging: vt6656: remove unused variables bCmdClear and bNeedRadioOFF
...
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-13 12:22:34 -07:00
Malcolm Priestley
b595f9b89c
staging: vt6656: remove typedef struct tagCMD_ITEM
...
Since enum vnt_cmd is only member replace with array
enum vnt_cmd cmd_queue with size of CMD_Q_SIZE
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-13 12:22:34 -07:00
Malcolm Priestley
ad74e91d06
staging: vt6656: replace typedef enum tagCMD_STATE with enum vnt_cmd_state
...
Replacing variables.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-13 12:22:34 -07:00
Malcolm Priestley
80f0d0923f
staging: vt6656: replace typedef enum tagCMD_CODE with enum vnt_cmd
...
Replacing the variables.
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-13 12:22:34 -07:00
Jes Sorensen
f6a7125c82
staging: rtl8723au: Remove unncessary wrapper PHY_RFConfig8723A()
...
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-13 12:14:32 -07:00
Jes Sorensen
9464cac7c9
staging: rtl8723au: Remove obfuscating wrapper ODM_ConfigMACWithHeaderFile23a()
...
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-13 12:14:32 -07:00
Jes Sorensen
f467c2b214
staging: rtl8723au: Remove unused ODM_MacStatusQuery23a()
...
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-13 12:14:32 -07:00
Jes Sorensen
79756e687c
staging: rtl8723au: Remove ugly wrapper ODM_ConfigBBWithHeaderFile23a()
...
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-13 12:14:32 -07:00
Jes Sorensen
7bafabf2d9
staging: rtl8723au: Remove unused ODM_ReadAndConfig_PHY_REG_PG_8723A() and related code+data
...
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-13 12:14:32 -07:00
Jes Sorensen
863b61948e
staging: rtl8723au: Remove pointless wrappers for calling odm_ConfigRFReg_8723A()
...
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-13 12:14:31 -07:00
Jes Sorensen
d10ef6371e
staging: rtl8723au: Remove zero value calculation added to address
...
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-13 12:14:31 -07:00
Jes Sorensen
6b435adbcb
staging: rtl8723au: Eliminate wrapper ODM_ConfigRFWithHeaderFile23a()
...
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-13 12:14:31 -07:00
Jes Sorensen
a895042573
staging: rtl8723au: Call to ODM_ConfigRFWithHeaderFile23a path is identical for RF_PATH_[AB]
...
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-13 12:14:31 -07:00
Jes Sorensen
974fe27cec
staging: rtl8723au: Remove horrendous code obfuscation READ_AND_CONFIG*()
...
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-13 12:14:31 -07:00
Jeremiah Mahler
635c4efa17
staging: dgnc: DGNC_VERIFY_BOARD macro in do while block
...
Enclose the body of the multi-statement DGNC_VERIFY_BOARD macro inside a
do - while block as per Documentation/CodingStyle. Fixes 1 error found
by checkpatch.pl.
Cc: Lidza Louina <lidza.louina@gmail.com >
Signed-off-by: Jeremiah Mahler <jmmahler@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-13 12:08:03 -07:00
Greg Kroah-Hartman
1d3ade00c6
staging: lustre: remove proc function wrappers
...
ll_proc_dointvec and ll_proc_dolongvec didn't do anything except confuse
things due to a lack of a filep variable in the functions they were used
in. So remove them and just call the real proc functions. This enabled
the code to be unwound a bit and a few "empty" proc handlers were
removed and just passed directly to procfs instead.
Cc: Andreas Dilger <andreas.dilger@intel.com >
Cc: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-12 21:26:09 -07:00
Greg Kroah-Hartman
ceeb4aef56
staging: lustre: delete params_tree.h
...
It's not being used at all, so delete it.
Cc: Andreas Dilger <andreas.dilger@intel.com >
Cc: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-12 21:15:26 -07:00
Greg Kroah-Hartman
f267cdb464
staging: lustre: use CONFIG_PROC_FS
...
Don't rely on a "custom" LPROCFS define, it's not needed, just check for
the real thing. This will let us delete a whole .h file that is not
being used for anything other than one #define.
Cc: Andreas Dilger <andreas.dilger@intel.com >
Cc: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-12 21:13:25 -07:00
Greg Kroah-Hartman
06bf61b2a3
staging: lustre: don't use custom typedef for an inode
...
It's 'struct inode' so just use that.
Cc: Andreas Dilger <andreas.dilger@intel.com >
Cc: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-12 20:56:47 -07:00
Greg Kroah-Hartman
15191cac20
staging: lustre: remove ALLOC_ATOMIC_TRY
...
Come on, is it so hard to type "GFP_ATOMIC"? Look, it's less
characters!!!
Cc: Andreas Dilger <andreas.dilger@intel.com >
Cc: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-12 20:37:56 -07:00
Greg Kroah-Hartman
0918cf03c5
staging: lustre: remove kp30.h
...
It's empty, so remove it and move the #include lines into the one file
that used to include this header.
Cc: Andreas Dilger <andreas.dilger@intel.com >
Cc: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-12 20:32:03 -07:00
Greg Kroah-Hartman
55f5a824b9
staging: lustre: remove LPX64 define
...
Just use the proper modifier type...
Cc: Andreas Dilger <andreas.dilger@intel.com >
Cc: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-12 20:26:07 -07:00
Greg Kroah-Hartman
b0f5aad587
staging: lustre: remove LPU64 define
...
Just use the proper modifier type...
Cc: Andreas Dilger <andreas.dilger@intel.com >
Cc: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-12 20:25:57 -07:00
Greg Kroah-Hartman
f537dd2c3c
staging: lustre: remove LPD64 define
...
Just use the proper modifier type...
Cc: Andreas Dilger <andreas.dilger@intel.com >
Cc: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-12 18:41:09 -07:00
Greg Kroah-Hartman
f7941e4d2c
staging: lustre: remove LPX64i define
...
Just use the proper modifier type...
Cc: Andreas Dilger <andreas.dilger@intel.com >
Cc: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-12 18:31:44 -07:00
Greg Kroah-Hartman
eafc75596f
staging: lustre: remove LPLU, LPLD, and LPLX defines
...
They are variable modifiers, come on, no need for a special define just
for them...
Cc: Andreas Dilger <andreas.dilger@intel.com >
Cc: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-12 18:28:41 -07:00
Greg Kroah-Hartman
143f378cf5
staging: lustre: remove LPF64 #define
...
Just use the proper string for the variable type.
Also remove some variable types that are not used at all (LPO64 and
LPPID)
Cc: Andreas Dilger <andreas.dilger@intel.com >
Cc: Oleg Drokin <oleg.drokin@intel.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-12 18:23:18 -07:00
Greg Kroah-Hartman
fa8d5b4474
staging: lustre: remove linux-prim.h
...
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 >
2014-07-12 18:17:20 -07:00
Greg Kroah-Hartman
9b335e2b2f
staging: lustre: clean up some proc file declarations
...
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 >
2014-07-12 18:11:25 -07:00
Greg Kroah-Hartman
e432a77f5f
staging: lustre: remove hpdd-discuss list from TODO file
...
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 >
2014-07-12 18:01:57 -07:00
Greg Kroah-Hartman
3bdb7ce411
staging: lustre: remove LL_PROC_PROTO macro
...
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 >
2014-07-12 18:00:44 -07:00
Malcolm Priestley
0a16b63d54
staging: vt6656: baseband.c rename abyVT3184_VT3226D0 to vnt_vt3184_vt3226d0
...
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-12 13:29:38 -07:00
Malcolm Priestley
ab48ab222d
staging: vt6656: baseband.c remame abyVT3184_AL2230 to vnt_vt3184_al2230
...
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-12 13:29:38 -07:00
Malcolm Priestley
170ce6d2c5
staging: vt6656: baseband.c rename abyVT3184_AGC to vnt_vt3184_agc
...
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-12 13:29:38 -07:00
Malcolm Priestley
4fd6e9f286
staging: vt6656: baseband clean up VT3184 static variables
...
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 >
2014-07-12 13:29:38 -07:00
Malcolm Priestley
ef13cfb4dc
staging: vt6656: rf.c cleanup comments
...
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 >
2014-07-12 13:28:28 -07:00
Malcolm Priestley
f1c840d55b
staging: vt6656: vnt_rf_write_embedded add reg length and IFREGCTL_REGW
...
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 >
2014-07-12 13:28:27 -07:00
Malcolm Priestley
0e2b6fe1f4
staging: vt6656: rf.c remove dead code RFaby11aChannelIndex
...
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-12 13:28:27 -07:00
Malcolm Priestley
77bdc0a5c6
staging: vt6656: rf.c clean up channel and power tables
...
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 >
2014-07-12 13:28:27 -07:00
Matthias Beyer
2349c05a61
Staging: bcm: Transmit.c: Removed obvious comment
...
Signed-off-by: Matthias Beyer <mail@beyermatthias.de >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-12 13:21:45 -07:00
Matthias Beyer
358ed017c7
Staging: bcm: Transmit.c: Line length / Whitespace cleanup
...
Signed-off-by: Matthias Beyer <mail@beyermatthias.de >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-12 13:21:45 -07:00
Matthias Beyer
c971681aea
Staging: bcm: Transmit.c: Replaced member accessing with variable
...
Signed-off-by: Matthias Beyer <mail@beyermatthias.de >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-12 13:21:45 -07:00
Malcolm Priestley
f7f0485757
staging: vt6656: wcmd replace debug message with dev_dbg
...
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-12 13:20:14 -07:00
Malcolm Priestley
4e3af0fa67
staging: vt6656: vResetCommandTimer remove camel case
...
pDevice -> priv
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-12 13:20:14 -07:00
Malcolm Priestley
f7e9ed41c9
staging: vt6656: bScheduleCommand remove camel case
...
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 >
2014-07-12 13:20:14 -07:00
Malcolm Priestley
d287093e4c
staging: vt6656: s_bCommandComplete clean up and remove camel case
...
pDevice -> priv
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com >
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org >
2014-07-12 13:20:14 -07:00