aboutsummaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2014-11-26staging: fwserial: remove multiple blank linesLe Tan1-3/+0
This patch fixes the multiple blank lines issue complained by checkpatch.pl by removing useless blank lines. Signed-off-by: Le Tan <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: panel: Use better names for two defined valuesMariusz Gorski1-8/+8
Give DEFAULT_KEYPAD and DEFAULT_LCD defines better names, so that their meaning is emphasized. Signed-off-by: Mariusz Gorski <[email protected]> Acked-By: Willy Tarreau <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: panel: Reorder module parameter declarationsMariusz Gorski1-35/+38
Change the order of the module parameter declarations so that it matches the Kconfig order. Signed-off-by: Mariusz Gorski <[email protected]> Acked-By: Willy Tarreau <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: panel: Reorder DEFAULT_* values redefinesMariusz Gorski1-13/+13
Change the order of the DEFAULT_* values redefines so that it matches the Kconfig order. Signed-off-by: Mariusz Gorski <[email protected]> Acked-By: Willy Tarreau <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: panel: Reorder initial DEFAULT_* definesMariusz Gorski1-4/+4
Change the order of the initial DEFAULT_* defines so that it matches the Kconfig order. Signed-off-by: Mariusz Gorski <[email protected]> Acked-By: Willy Tarreau <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: octeon-usb: eliminate cvmx_dprintf() usageAaro Koskinen1-3/+9
Eliminate cvmx_dprintf() usage and use dev_err() instead. Signed-off-by: Aaro Koskinen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: octeon-usb: move helper functionsAaro Koskinen1-10/+10
Move helper functions to make them visible to all functions. Signed-off-by: Aaro Koskinen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: octeon-ethernet: drop multiple NAPI instancesAaro Koskinen3-106/+10
Octeon Ethernet driver is currently trying to wake up multiple CPU cores for parallel NAPI processing when needed, to cope with the fact that all network interfaces are handled through the same receive queue processed by a single core/interrupt (see http://marc.info/?l=linux-kernel&m=137209784914565&w=2). This partially overlaps with generic RPS functionality. Commit a6a39a7fcff5 ("staging: octeon-ethernet: disable load balance for receiving packet when CONFIG_RPS is enabled") already disabled this when RPS is enabled. Let's delete this code altogether as it has issues like packet reordering and potential problems with CPU offlining/onlining. Tested on EdgeRouter Lite. Signed-off-by: Aaro Koskinen <[email protected]> Acked-By: David Daney <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: android: ion: Deletion of unnecessary checks before two function callsMarkus Elfring3-12/+6
The functions ion_heap_destroy() and vfree() perform also input parameter validation. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: vt6655: rf.c clean up function commentsMalcolm Priestley1-28/+31
Signed-off-by: Malcolm Priestley <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: vt6655: rf.c Removed unneeded commentsMalcolm Priestley1-134/+0
Type and function that are not present comments Signed-off-by: Malcolm Priestley <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: vt6655: rf.c: Remove author commentsMalcolm Priestley1-18/+3
Moving names to file header comment Signed-off-by: Malcolm Priestley <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: vt6655: rf.c camel calse vnt_private rename pDevice to privMalcolm Priestley1-36/+36
Signed-off-by: Malcolm Priestley <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: vt6655: IFRFbWriteEmbedded replace __iomem with vnt_privateMalcolm Priestley2-42/+45
Fixing callers to vnt_private Signed-off-by: Malcolm Priestley <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: vt6655: rf remove __iomem *dwIoBase from functionsMalcolm Priestley3-15/+24
replacing with vnt_private and removing dereferencing from callers Signed-off-by: Malcolm Priestley <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: vt6655: card.c move __iomem into functions.Malcolm Priestley4-15/+20
Removing dereferencing from callers Signed-off-by: Malcolm Priestley <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: vt6655: card Remove function vUpdateIFSMalcolm Priestley3-67/+0
vUpdateIFS does the same operations of CARDbSetPhyParameter there is not need to call this function. Signed-off-by: Malcolm Priestley <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: vt6655: always keep abyCurrentNetAddr aligned to 2Malcolm Priestley1-1/+1
Because the driver now uses ether_addr_copy the alignment of this variable must be maintained at 2. Signed-off-by: Malcolm Priestley <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: vt6655: change typedef enum _VIA_PKT_TYPE to macroMalcolm Priestley2-14/+11
Replacing for u8 type and removing cast for byBBType. Signed-off-by: Malcolm Priestley <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: vt6655: replace typedef enum _VIA_BB_TYPE with macrosMalcolm Priestley1-7/+4
Replacing type with u8 Signed-off-by: Malcolm Priestley <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: vt6655: CARDbSetPhyParameter remove unused parametersMalcolm Priestley3-9/+4
wCapInfo byERPField, *pvSupportRateIEs *pvExtSupportRateIEs Signed-off-by: Malcolm Priestley <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: vt6655: dead code remove typedef enum _CARD_PHY_TYPEMalcolm Priestley2-8/+0
Removing unused eCurrentPHYType Signed-off-by: Malcolm Priestley <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: vt6655: s_vCalculateOFDMRParameter use bb_type for CARD_PHY_TYPEMalcolm Priestley1-10/+10
using u8 for bb_type and replacing PHY_TYPE_11A with BB_TYPE_11A Signed-off-by: Malcolm Priestley <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: vt6655: CARDvSetRSPINF us bb_type for CARD_PHY_TYPEMalcolm Priestley2-11/+11
Using u8 for bb_type Signed-off-by: Malcolm Priestley <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: vt6655: CARDbSetPhyParameter change CARD_PHY_TYPE to bb_typeMalcolm Priestley2-5/+5
using u8 for bb_type Signed-off-by: Malcolm Priestley <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: vt6655: dead code remove CARDbAddBasicRateMalcolm Priestley2-13/+0
we nonlonger set rates in the driver Signed-off-by: Malcolm Priestley <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: vt6655: CARDvUpdateBasicTopRate use basic_rates to find top rateMalcolm Priestley1-2/+2
basic_rates should now be used to find it. Signed-off-by: Malcolm Priestley <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: lustre: ldlm: Remove space before braces for defined() checkAndreas Ruprecht2-2/+2
checkpatch complains about two places where a space is prohibited before the braces for in "#if defined()" check. This patch removes these warnings. Signed-off-by: Andreas Ruprecht <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: lustre: ldlm: Remove unnecessary braces at ifsAndreas Ruprecht2-7/+4
checkpatch complains about three places where braces are not necessary. This patch removes the braces. Signed-off-by: Andreas Ruprecht <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: lustre: ldlm: Remove unnecessary line continuationsAndreas Ruprecht1-2/+2
checkpatch finds two unnecessary line continuations in ldlm_lockd.c which are removed by this patch. Signed-off-by: Andreas Ruprecht <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: lustre: ldlm: Fix warning about unneeded return statementAndreas Ruprecht1-1/+0
checkpatch complains about an unneeded return statement in the function search_granted_lock(), which is removed by this patch. Signed-off-by: Andreas Ruprecht <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: lustre: ldlm: Fix initialization of static variablesAndreas Ruprecht2-2/+2
checkpatch complains about static variables being initialized to 0 which is unnecessary. This patch removes the initialization. Signed-off-by: Andreas Ruprecht <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: lustre: ldlm: Fix indentation errors for switch-caseAndreas Ruprecht2-25/+25
checkpatch complains about two indentation errors where the case statements are on a higher indentation level than the corresponding switch statement. This patch removes these errors. Signed-off-by: Andreas Ruprecht <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: lustre: ldlm: Fix warning about missing spacesAndreas Ruprecht3-7/+8
checkpatch.pl complains about a number of places where spaces were missing. This patch fixes all of these errors. For better readability, the warning of hitting the 80 character per line limit in line 1349 introduced by this change is ignored deliberately. Signed-off-by: Andreas Ruprecht <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: lustre: ldlm: Fix overlong linesAndreas Ruprecht8-30/+49
checkpatch complains about a number of lines which are over the 80 character line limit. This patch changes most of them to comply, the few remaining lines with warnings are not changed for readability reasons. Signed-off-by: Andreas Ruprecht <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: lustre: ldlm: Add missing newlines after declarationsAndreas Ruprecht9-1/+29
checkpatch.pl complains about a number of missing newlines after declarations. This patch gets rid of these warnings. Signed-off-by: Andreas Ruprecht <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging:rtl8723au: fix sparse warning: incorrect type in assignmentKinka Huang1-2/+2
Signed-off-by: Kinka Huang <[email protected]> Acked-by: Jes Sorensen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging:rtl8723au: Fix sparse warning cast to restricted __le16Tobenna P. Igwe1-1/+1
This patch fixes the following sparse warning: drivers/staging/rtl8723au/hal/rtl8723a_bt-coexist.c:343:36: warning: cast to restricted __le16 by using the le16_to_cpus function. Signed-off-by: Tobenna P. Igwe <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: rtl8723au: fix sparse warningAleh Suprunovich1-78/+0
drivers/staging/rtl8723au/hal/rtl8723a_hal_init.c:747:1: warning: symbol 'rtl8723a_EfusePgPacketRead' was not declared. Should it be static? Function 'rtl8723a_EfusePgPacketRead' seems to be unused in current staging code. Before, it was available as 'static s32 Hal_EfusePgPacketRead', but code that was using it removed, in the same commit as rename and signature change to 'bool rtl8723a_EfusePgPacketRead' has taken place. Signed-off-by: Aleh Suprunovich <[email protected]> Signed-off-by: Jes Sorensen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging:rtl8723au: core: Fix checkpatch warningSanjeev Sharma1-21/+21
This is a patch to the rtw_cmd.c file that fixes following Warning by introducing temporary structure. WARNING: line over 80 characters Signed-off-by: Sanjeev Sharma <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging:rtl8723au: core: Added missing space reported by checkpatch.Sanjeev Sharma1-2/+2
This is a patch to the rtw_cmd.c file that fixes following Error. ERROR: spaces required around that '>' (ctx:WxV) Signed-off-by: Sanjeev Sharma <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: rtl8723au: Remove sw led handlingJes Sorensen14-2252/+0
The rtl8723au relies on hw led support, so no point carrying a large unused sw led infrastructure around. Signed-off-by: Jes Sorensen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: rtl8723au: Remove no-op function Hal_InitChannelPlan23a()Jes Sorensen3-10/+0
Signed-off-by: Jes Sorensen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: rtl8723au: Remove unused rtl8723a_update_txdesc() and child functionsJes Sorensen2-249/+0
Signed-off-by: Jes Sorensen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: rtl8723au: pxmitframe->frame_tag is never set to a value > 8Jes Sorensen2-6/+7
No point of masking out high bits since we never set the value to anything exceeding bits 0-3. Signed-off-by: Jes Sorensen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: rtl8723au: rtw_dump_xframe(): Use proper ETH_P_* typesJes Sorensen1-4/+4
Signed-off-by: Jes Sorensen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: rtl8723au: portctrl(): Remove a pile of unnecessary clutterJes Sorensen1-36/+4
Signed-off-by: Jes Sorensen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: rtl8723au: rtw_put_snap23a(): Make it staticJes Sorensen2-21/+20
Signed-off-by: Jes Sorensen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging: rtl8723au: rtw_put_snap23a(): Use put_unaligned to set protocolJes Sorensen1-1/+1
Reported-by: Arend van Spriel <[email protected]> Signed-off-by: Jes Sorensen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>
2014-11-26staging rtl8723au: rtw_put_smap23a(): Use common SNAP header definesJes Sorensen1-19/+7
Signed-off-by: Jes Sorensen <[email protected]> Signed-off-by: Greg Kroah-Hartman <[email protected]>