summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2014-07-13staging: vt6656: replace camel case from command variablesMalcolm Priestley
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-13staging: vt6656: remove unused variables bCmdClear and bNeedRadioOFFMalcolm Priestley
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-13staging: vt6656: remove typedef struct tagCMD_ITEMMalcolm Priestley
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-13staging: vt6656: replace typedef enum tagCMD_STATE with enum vnt_cmd_stateMalcolm Priestley
Replacing variables. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-13staging: vt6656: replace typedef enum tagCMD_CODE with enum vnt_cmdMalcolm Priestley
Replacing the variables. Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-13staging: rtl8723au: Remove unncessary wrapper PHY_RFConfig8723A()Jes Sorensen
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-13staging: rtl8723au: Remove obfuscating wrapper ODM_ConfigMACWithHeaderFile23a()Jes Sorensen
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-13staging: rtl8723au: Remove unused ODM_MacStatusQuery23a()Jes Sorensen
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-13staging: rtl8723au: Remove ugly wrapper ODM_ConfigBBWithHeaderFile23a()Jes Sorensen
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-13staging: rtl8723au: Remove unused ODM_ReadAndConfig_PHY_REG_PG_8723A() and ↵Jes Sorensen
related code+data Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-13staging: rtl8723au: Remove pointless wrappers for calling ↵Jes Sorensen
odm_ConfigRFReg_8723A() Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-13staging: rtl8723au: Remove zero value calculation added to addressJes Sorensen
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-13staging: rtl8723au: Eliminate wrapper ODM_ConfigRFWithHeaderFile23a()Jes Sorensen
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-13staging: rtl8723au: Call to ODM_ConfigRFWithHeaderFile23a path is identical ↵Jes Sorensen
for RF_PATH_[AB] Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-13staging: rtl8723au: Remove horrendous code obfuscation READ_AND_CONFIG*()Jes Sorensen
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-13staging: dgnc: DGNC_VERIFY_BOARD macro in do while blockJeremiah Mahler
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-13staging: lustre: remove proc function wrappersGreg Kroah-Hartman
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-13staging: lustre: delete params_tree.hGreg Kroah-Hartman
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-13staging: lustre: use CONFIG_PROC_FSGreg Kroah-Hartman
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-13staging: lustre: don't use custom typedef for an inodeGreg Kroah-Hartman
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-13staging: lustre: remove ALLOC_ATOMIC_TRYGreg Kroah-Hartman
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-13staging: lustre: remove kp30.hGreg Kroah-Hartman
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-13staging: lustre: remove LPX64 defineGreg Kroah-Hartman
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-13staging: lustre: remove LPU64 defineGreg Kroah-Hartman
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-13staging: lustre: remove LPD64 defineGreg Kroah-Hartman
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-13staging: lustre: remove LPX64i defineGreg Kroah-Hartman
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-13staging: lustre: remove LPLU, LPLD, and LPLX definesGreg Kroah-Hartman
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-13staging: lustre: remove LPF64 #defineGreg Kroah-Hartman
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-13staging: lustre: remove linux-prim.hGreg Kroah-Hartman
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-13staging: lustre: clean up some proc file declarationsGreg Kroah-Hartman
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-13staging: lustre: remove hpdd-discuss list from TODO fileGreg Kroah-Hartman
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-13staging: lustre: remove LL_PROC_PROTO macroGreg Kroah-Hartman
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-12staging: vt6656: baseband.c rename abyVT3184_VT3226D0 to vnt_vt3184_vt3226d0Malcolm Priestley
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-12staging: vt6656: baseband.c remame abyVT3184_AL2230 to vnt_vt3184_al2230Malcolm Priestley
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-12staging: vt6656: baseband.c rename abyVT3184_AGC to vnt_vt3184_agcMalcolm Priestley
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-12staging: vt6656: baseband clean up VT3184 static variablesMalcolm Priestley
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-12staging: vt6656: rf.c cleanup commentsMalcolm Priestley
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-12staging: vt6656: vnt_rf_write_embedded add reg length and IFREGCTL_REGWMalcolm Priestley
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-12staging: vt6656: rf.c remove dead code RFaby11aChannelIndexMalcolm Priestley
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-12staging: vt6656: rf.c clean up channel and power tablesMalcolm Priestley
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-12Staging: bcm: Transmit.c: Removed obvious commentMatthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-12Staging: bcm: Transmit.c: Line length / Whitespace cleanupMatthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-12Staging: bcm: Transmit.c: Replaced member accessing with variableMatthias Beyer
Signed-off-by: Matthias Beyer <mail@beyermatthias.de> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-12staging: vt6656: wcmd replace debug message with dev_dbgMalcolm Priestley
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-12staging: vt6656: vResetCommandTimer remove camel caseMalcolm Priestley
pDevice -> priv Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-12staging: vt6656: bScheduleCommand remove camel caseMalcolm Priestley
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-12staging: vt6656: s_bCommandComplete clean up and remove camel caseMalcolm Priestley
pDevice -> priv Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-12staging: vt6656: s_bCommandComplete remove else from ifMalcolm Priestley
if returns out so remove else Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-12staging: vt6656: remove unused variables from struct tagCMD_ITEMMalcolm Priestley
Also remove local variables Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
2014-07-12staging: vt6656: vRunCommand remove camel case.Malcolm Priestley
pDevice -> priv Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>