Re: [PATCH 1/2] Staging: ks7010: mark symbols static where possible

2016-09-24 Thread Arnd Bergmann
On Saturday, September 24, 2016 9:34:41 AM CEST Baoyou Xie wrote:
> On 24 September 2016 at 01:10, Greg KH  wrote:
> 
> > On Fri, Sep 23, 2016 at 09:22:29PM +0800, Baoyou Xie wrote:
> > > We get 3 warnings when building kernel with W=1:
> > > drivers/staging/ks7010/ks7010_sdio.c:90:6: warning: no previous
> > prototype for 'ks_wlan_hw_sleep_doze_request' [-Wmissing-prototypes]
> > > drivers/staging/ks7010/ks7010_sdio.c:120:6: warning: no previous
> > prototype for 'ks_wlan_hw_sleep_wakeup_request' [-Wmissing-prototypes]
> > > drivers/staging/ks7010/ks7010_sdio.c:172:5: warning: no previous
> > prototype for '_ks_wlan_hw_power_save' [-Wmissing-prototypes]
> > >
> > > In fact, these functions are only used in the file in which they are
> > > declared and don't need a declaration, but can be made static.
> > > so this patch marks these functions with 'static'.
> > >
> > > Signed-off-by: Baoyou Xie 
> > > ---
> > >  drivers/staging/ks7010/ks7010_sdio.c | 6 +++---
> > >  1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > Does not apply cleanly, are you sure you are using the latest tree?
> >
> I worked on linux next-20160922 tag, could you tell me how to find the
> valid tree?

The file saw a couple of small changes in next-20160923, so I think
this was just unfortunate timing but you used the right tree.

staging can be moving very rapidly at times, so if you want to
be sure to get the right tree of the minute, you can
rebase on top of

git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git staging-testing

I usually just base on top of the linux-next of the day, which tends
to work well enough.

Arnd
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


[PATCH 2/2] rtlwifi: Add explicit values to hw_variables enum

2016-09-24 Thread Larry Finger
The entries in this enum may be referenced in debug output. Adding explicit
values simplifies the lookup.

Signed-off-by: Larry Finger 
---
 drivers/net/wireless/realtek/rtlwifi/wifi.h | 208 ++--
 1 file changed, 104 insertions(+), 104 deletions(-)

diff --git a/drivers/net/wireless/realtek/rtlwifi/wifi.h 
b/drivers/net/wireless/realtek/rtlwifi/wifi.h
index c5086c2..595f7d5 100644
--- a/drivers/net/wireless/realtek/rtlwifi/wifi.h
+++ b/drivers/net/wireless/realtek/rtlwifi/wifi.h
@@ -394,110 +394,110 @@ enum io_type {
 };
 
 enum hw_variables {
-   HW_VAR_ETHER_ADDR,
-   HW_VAR_MULTICAST_REG,
-   HW_VAR_BASIC_RATE,
-   HW_VAR_BSSID,
-   HW_VAR_MEDIA_STATUS,
-   HW_VAR_SECURITY_CONF,
-   HW_VAR_BEACON_INTERVAL,
-   HW_VAR_ATIM_WINDOW,
-   HW_VAR_LISTEN_INTERVAL,
-   HW_VAR_CS_COUNTER,
-   HW_VAR_DEFAULTKEY0,
-   HW_VAR_DEFAULTKEY1,
-   HW_VAR_DEFAULTKEY2,
-   HW_VAR_DEFAULTKEY3,
-   HW_VAR_SIFS,
-   HW_VAR_R2T_SIFS,
-   HW_VAR_DIFS,
-   HW_VAR_EIFS,
-   HW_VAR_SLOT_TIME,
-   HW_VAR_ACK_PREAMBLE,
-   HW_VAR_CW_CONFIG,
-   HW_VAR_CW_VALUES,
-   HW_VAR_RATE_FALLBACK_CONTROL,
-   HW_VAR_CONTENTION_WINDOW,
-   HW_VAR_RETRY_COUNT,
-   HW_VAR_TR_SWITCH,
-   HW_VAR_COMMAND,
-   HW_VAR_WPA_CONFIG,
-   HW_VAR_AMPDU_MIN_SPACE,
-   HW_VAR_SHORTGI_DENSITY,
-   HW_VAR_AMPDU_FACTOR,
-   HW_VAR_MCS_RATE_AVAILABLE,
-   HW_VAR_AC_PARAM,
-   HW_VAR_ACM_CTRL,
-   HW_VAR_DIS_Req_Qsize,
-   HW_VAR_CCX_CHNL_LOAD,
-   HW_VAR_CCX_NOISE_HISTOGRAM,
-   HW_VAR_CCX_CLM_NHM,
-   HW_VAR_TxOPLimit,
-   HW_VAR_TURBO_MODE,
-   HW_VAR_RF_STATE,
-   HW_VAR_RF_OFF_BY_HW,
-   HW_VAR_BUS_SPEED,
-   HW_VAR_SET_DEV_POWER,
-
-   HW_VAR_RCR,
-   HW_VAR_RATR_0,
-   HW_VAR_RRSR,
-   HW_VAR_CPU_RST,
-   HW_VAR_CHECK_BSSID,
-   HW_VAR_LBK_MODE,
-   HW_VAR_AES_11N_FIX,
-   HW_VAR_USB_RX_AGGR,
-   HW_VAR_USER_CONTROL_TURBO_MODE,
-   HW_VAR_RETRY_LIMIT,
-   HW_VAR_INIT_TX_RATE,
-   HW_VAR_TX_RATE_REG,
-   HW_VAR_EFUSE_USAGE,
-   HW_VAR_EFUSE_BYTES,
-   HW_VAR_AUTOLOAD_STATUS,
-   HW_VAR_RF_2R_DISABLE,
-   HW_VAR_SET_RPWM,
-   HW_VAR_H2C_FW_PWRMODE,
-   HW_VAR_H2C_FW_JOINBSSRPT,
-   HW_VAR_H2C_FW_MEDIASTATUSRPT,
-   HW_VAR_H2C_FW_P2P_PS_OFFLOAD,
-   HW_VAR_FW_PSMODE_STATUS,
-   HW_VAR_INIT_RTS_RATE,
-   HW_VAR_RESUME_CLK_ON,
-   HW_VAR_FW_LPS_ACTION,
-   HW_VAR_1X1_RECV_COMBINE,
-   HW_VAR_STOP_SEND_BEACON,
-   HW_VAR_TSF_TIMER,
-   HW_VAR_IO_CMD,
-
-   HW_VAR_RF_RECOVERY,
-   HW_VAR_H2C_FW_UPDATE_GTK,
-   HW_VAR_WF_MASK,
-   HW_VAR_WF_CRC,
-   HW_VAR_WF_IS_MAC_ADDR,
-   HW_VAR_H2C_FW_OFFLOAD,
-   HW_VAR_RESET_WFCRC,
-
-   HW_VAR_HANDLE_FW_C2H,
-   HW_VAR_DL_FW_RSVD_PAGE,
-   HW_VAR_AID,
-   HW_VAR_HW_SEQ_ENABLE,
-   HW_VAR_CORRECT_TSF,
-   HW_VAR_BCN_VALID,
-   HW_VAR_FWLPS_RF_ON,
-   HW_VAR_DUAL_TSF_RST,
-   HW_VAR_SWITCH_EPHY_WoWLAN,
-   HW_VAR_INT_MIGRATION,
-   HW_VAR_INT_AC,
-   HW_VAR_RF_TIMING,
-
-   HAL_DEF_WOWLAN,
-   HW_VAR_MRC,
-   HW_VAR_KEEP_ALIVE,
-   HW_VAR_NAV_UPPER,
-
-   HW_VAR_MGT_FILTER,
-   HW_VAR_CTRL_FILTER,
-   HW_VAR_DATA_FILTER,
+   HW_VAR_ETHER_ADDR = 0x0,
+   HW_VAR_MULTICAST_REG = 0x1,
+   HW_VAR_BASIC_RATE = 0x2,
+   HW_VAR_BSSID = 0x3,
+   HW_VAR_MEDIA_STATUS= 0x4,
+   HW_VAR_SECURITY_CONF= 0x5,
+   HW_VAR_BEACON_INTERVAL = 0x6,
+   HW_VAR_ATIM_WINDOW = 0x7,
+   HW_VAR_LISTEN_INTERVAL = 0x8,
+   HW_VAR_CS_COUNTER = 0x9,
+   HW_VAR_DEFAULTKEY0 = 0xa,
+   HW_VAR_DEFAULTKEY1 = 0xb,
+   HW_VAR_DEFAULTKEY2 = 0xc,
+   HW_VAR_DEFAULTKEY3 = 0xd,
+   HW_VAR_SIFS = 0xe,
+   HW_VAR_R2T_SIFS = 0xf,
+   HW_VAR_DIFS = 0x10,
+   HW_VAR_EIFS = 0x11,
+   HW_VAR_SLOT_TIME = 0x12,
+   HW_VAR_ACK_PREAMBLE = 0x13,
+   HW_VAR_CW_CONFIG = 0x14,
+   HW_VAR_CW_VALUES = 0x15,
+   HW_VAR_RATE_FALLBACK_CONTROL= 0x16,
+   HW_VAR_CONTENTION_WINDOW = 0x17,
+   HW_VAR_RETRY_COUNT = 0x18,
+   HW_VAR_TR_SWITCH = 0x19,
+   HW_VAR_COMMAND = 0x1a,
+   HW_VAR_WPA_CONFIG = 0x1b,
+   HW_VAR_AMPDU_MIN_SPACE = 0x1c,
+   HW_VAR_SHORTGI_DENSITY = 0x1d,
+   HW_VAR_AMPDU_FACTOR = 0x1e,
+   HW_VAR_MCS_RATE_AVAILABLE = 0x1f,
+   HW_VAR_AC_PARAM = 0x20,
+   HW_VAR_ACM_CTRL = 0x21,
+   HW_VAR_DIS_Req_Qsize = 0x22,
+   HW_VAR_CCX_CHNL_LOAD = 0x23,
+   HW_VAR_CCX_NOISE_HISTOGRAM = 0x24,
+   HW_VAR_CCX_CLM_NHM = 0x25,
+   HW_VAR_TxOPLimit = 0x26,
+   HW_VAR_TURBO_MODE = 0x27,
+   HW_VAR_RF_STATE = 0x28,
+   HW_VAR_RF_OFF_BY_HW = 0x29,
+   HW_VAR_BUS_SPEED = 0x2a,
+   HW_VAR_SET_DEV_POWER = 0x2b,
+
+   HW_VAR_RCR = 0x2c,
+   HW_VAR_RATR_0 = 0x2d,
+   HW_VAR_RRSR = 0x2e,
+   HW_VA

[PATCH 1/2] rtlwifi: Add HAL_DEF_WOWLAN case to *_get_hw() routines

2016-09-24 Thread Larry Finger
Only rtl8821ae implements WOWLAN; however, the other drivers may receive
a call requesting information about this mode. The other drivers need to
ignore the request rather than logging that the default branch of the
switch statement has been reached.

Reported by: Jean Delvare 
Signed-off-by: Larry Finger 
---

This patch depends on "realtek: Add switch variable to 'switch case not
processed' messages" submitted by Joe Perches.

Larry
---

 drivers/net/wireless/realtek/rtlwifi/rtl8188ee/hw.c | 2 ++
 drivers/net/wireless/realtek/rtlwifi/rtl8192ce/hw.c | 2 ++
 drivers/net/wireless/realtek/rtlwifi/rtl8192de/hw.c | 2 ++
 drivers/net/wireless/realtek/rtlwifi/rtl8192ee/hw.c | 2 ++
 drivers/net/wireless/realtek/rtlwifi/rtl8192se/hw.c | 2 ++
 drivers/net/wireless/realtek/rtlwifi/rtl8723ae/hw.c | 2 ++
 drivers/net/wireless/realtek/rtlwifi/rtl8723be/hw.c | 2 ++
 7 files changed, 14 insertions(+)

diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/hw.c 
b/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/hw.c
index 3285117..37d6efc 100644
--- a/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/hw.c
+++ b/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/hw.c
@@ -355,6 +355,8 @@ void rtl88ee_get_hw_reg(struct ieee80211_hw *hw, u8 
variable, u8 *val)
 
*((u64 *)(val)) = tsf;
break; }
+   case HAL_DEF_WOWLAN:
+   break;
default:
RT_TRACE(rtlpriv, COMP_ERR, DBG_EMERG,
 "switch case %#x not processed\n", variable);
diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8192ce/hw.c 
b/drivers/net/wireless/realtek/rtlwifi/rtl8192ce/hw.c
index 6d308f9..a47be73 100644
--- a/drivers/net/wireless/realtek/rtlwifi/rtl8192ce/hw.c
+++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192ce/hw.c
@@ -141,6 +141,8 @@ void rtl92ce_get_hw_reg(struct ieee80211_hw *hw, u8 
variable, u8 *val)
 
break;
}
+   case HAL_DEF_WOWLAN:
+   break;
default:
RT_TRACE(rtlpriv, COMP_ERR, DBG_EMERG,
 "switch case %#x not processed\n", variable);
diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8192de/hw.c 
b/drivers/net/wireless/realtek/rtlwifi/rtl8192de/hw.c
index 5369011..d91f8bb 100644
--- a/drivers/net/wireless/realtek/rtlwifi/rtl8192de/hw.c
+++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192de/hw.c
@@ -164,6 +164,8 @@ void rtl92de_get_hw_reg(struct ieee80211_hw *hw, u8 
variable, u8 *val)
case HW_VAR_INT_AC:
*((bool *)(val)) = rtlpriv->dm.disable_tx_int;
break;
+   case HAL_DEF_WOWLAN:
+   break;
default:
RT_TRACE(rtlpriv, COMP_ERR, DBG_EMERG,
 "switch case %#x not processed\n", variable);
diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8192ee/hw.c 
b/drivers/net/wireless/realtek/rtlwifi/rtl8192ee/hw.c
index 47bb6d8..ebf663e 100644
--- a/drivers/net/wireless/realtek/rtlwifi/rtl8192ee/hw.c
+++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192ee/hw.c
@@ -338,6 +338,8 @@ void rtl92ee_get_hw_reg(struct ieee80211_hw *hw, u8 
variable, u8 *val)
*((u64 *)(val)) = tsf;
}
break;
+   case HAL_DEF_WOWLAN:
+   break;
default:
RT_TRACE(rtlpriv, COMP_ERR, DBG_DMESG,
 "switch case %#x not processed\n", variable);
diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8192se/hw.c 
b/drivers/net/wireless/realtek/rtlwifi/rtl8192se/hw.c
index 5bad9c9..52e4430 100644
--- a/drivers/net/wireless/realtek/rtlwifi/rtl8192se/hw.c
+++ b/drivers/net/wireless/realtek/rtlwifi/rtl8192se/hw.c
@@ -77,6 +77,8 @@ void rtl92se_get_hw_reg(struct ieee80211_hw *hw, u8 variable, 
u8 *val)
*((bool *)(val)) = rtlpriv->dm.current_mrc_switch;
break;
}
+   case HAL_DEF_WOWLAN:
+   break;
default: {
RT_TRACE(rtlpriv, COMP_ERR, DBG_EMERG,
 "switch case %#x not processed\n", variable);
diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8723ae/hw.c 
b/drivers/net/wireless/realtek/rtlwifi/rtl8723ae/hw.c
index 2bf603b..f8be0bd 100644
--- a/drivers/net/wireless/realtek/rtlwifi/rtl8723ae/hw.c
+++ b/drivers/net/wireless/realtek/rtlwifi/rtl8723ae/hw.c
@@ -141,6 +141,8 @@ void rtl8723e_get_hw_reg(struct ieee80211_hw *hw, u8 
variable, u8 *val)
 
break;
}
+   case HAL_DEF_WOWLAN:
+   break;
default:
RT_TRACE(rtlpriv, COMP_ERR, DBG_LOUD,
 "switch case %#x not processed\n", variable);
diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8723be/hw.c 
b/drivers/net/wireless/realtek/rtlwifi/rtl8723be/hw.c
index 999c1ac..aba60c3 100644
--- a/drivers/net/wireless/realtek/rtlwifi/rtl8723be/hw.c
+++ b/drivers/net/wireless/realtek/rtlwifi/rtl8723be/hw.c
@@ -348,6 +348,8 @@ void rtl8723be_ge

[PATCHv2] staging: ion: Align cases with switch

2016-09-24 Thread Christopher Pezley
The preferred indentation for cases and switches has the cases at
the same level as the switch.

Signed-off-by: Christopher H. Pezley 
---
 drivers/staging/android/ion/ion_of.c | 18 +-
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/drivers/staging/android/ion/ion_of.c 
b/drivers/staging/android/ion/ion_of.c
index de0899a..3c050b0 100644
--- a/drivers/staging/android/ion/ion_of.c
+++ b/drivers/staging/android/ion/ion_of.c
@@ -58,15 +58,15 @@ int ion_setup_heap_common(struct platform_device *parent,
int ret = 0;
 
switch (heap->type) {
-   case ION_HEAP_TYPE_CARVEOUT:
-   case ION_HEAP_TYPE_CHUNK:
-   if (heap->base && heap->size)
-   return 0;
-
-   ret = of_reserved_mem_device_init(heap->priv);
-   break;
-   default:
-   break;
+   case ION_HEAP_TYPE_CARVEOUT:
+   case ION_HEAP_TYPE_CHUNK:
+   if (heap->base && heap->size)
+   return 0;
+
+   ret = of_reserved_mem_device_init(heap->priv);
+   break;
+   default:
+   break;
}
 
return ret;
-- 
2.7.4

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


[PATCH 0/2] rtlwifi: Add missing values to case statements and simplify debugging

2016-09-24 Thread Larry Finger
Routines *_get_hw_reg() for some of the drivers were missing entries for
HAL_DEF_WOWLAN. No action should be taken for these drivers.

Patch 1 depends on "realtek: Add switch variable to 'switch case not
processed' messages" submitted by Joe Perches. That patch will log
missing cases with the hex value. To aid in debugging, the values for
entries in the hardware variables enum are explicitly defined in hex. 

Larry Finger (2):
  rtlwifi: Add HAL_DEF_WOWLAN case to *_get_hw() routines
  rtlwifi: Add explicit values to hw_variables enum

 .../net/wireless/realtek/rtlwifi/rtl8188ee/hw.c|   2 +
 .../net/wireless/realtek/rtlwifi/rtl8192ce/hw.c|   2 +
 .../net/wireless/realtek/rtlwifi/rtl8192de/hw.c|   2 +
 .../net/wireless/realtek/rtlwifi/rtl8192ee/hw.c|   2 +
 .../net/wireless/realtek/rtlwifi/rtl8192se/hw.c|   2 +
 .../net/wireless/realtek/rtlwifi/rtl8723ae/hw.c|   2 +
 .../net/wireless/realtek/rtlwifi/rtl8723be/hw.c|   2 +
 drivers/net/wireless/realtek/rtlwifi/wifi.h| 208 ++---
 8 files changed, 118 insertions(+), 104 deletions(-)

-- 
2.6.6

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


[PATCH] greybus: audio: fix uninitialized variable errors found by cppcheck

2016-09-24 Thread Vaibhav Agarwal
Currently, if info is null, the dev_err message is dereferencing an
uninitialized module pointer.  Instead, it should use codec->dev pointer
in dev_err call and better align with other err msg in this function.

Also, ret variable might be used uninitialized in a specific case.
Avoid using it this way.

Found using static analysis with cppcheck:
Checking drivers/staging/greybus/audio_topology.c...
[drivers/staging/greybus/audio_topology.c:175]: (error) Uninitialized
variable: module
[drivers/staging/greybus/audio_topology.c:495]: (error) Uninitialized
variable: ret

Reported-by: Colin Ian King 
Signed-off-by: Vaibhav Agarwal 
---
 drivers/staging/greybus/audio_topology.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/greybus/audio_topology.c 
b/drivers/staging/greybus/audio_topology.c
index f9f33817a092..b6251691a33d 100644
--- a/drivers/staging/greybus/audio_topology.c
+++ b/drivers/staging/greybus/audio_topology.c
@@ -172,7 +172,7 @@ static int gbcodec_mixer_ctl_info(struct snd_kcontrol 
*kcontrol,
info = (struct gb_audio_ctl_elem_info *)data->info;
 
if (!info) {
-   dev_err(module->dev, "NULL info for %s\n", uinfo->id.name);
+   dev_err(codec->dev, "NULL info for %s\n", uinfo->id.name);
return -EINVAL;
}
 
@@ -489,10 +489,11 @@ static int gbcodec_mixer_dapm_ctl_put(struct snd_kcontrol 
*kcontrol,
dev_err_ratelimited(codec->dev,
"%d:Error in %s for %s\n", ret,
__func__, kcontrol->id.name);
+   return ret;
}
}
 
-   return ret;
+   return 0;
 }
 
 #define SOC_DAPM_MIXER_GB(xname, kcount, data) \
-- 
2.7.4

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


Re: [PATCH 2/2] rtlwifi: Add explicit values to hw_variables enum

2016-09-24 Thread Larry Finger

On 09/24/2016 12:43 PM, Joe Perches wrote:

On Sat, 2016-09-24 at 11:57 -0500, Larry Finger wrote:

The entries in this enum may be referenced in debug output. Adding explicit
values simplifies the lookup.


A negative aspect to this style is forced renumbering
if a HW_VAR entry is inserted.

Is that possible?


Any new one would be added at the end.

Larry


___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


Re: [PATCH 2/2] rtlwifi: Add explicit values to hw_variables enum

2016-09-24 Thread Joe Perches
On Sat, 2016-09-24 at 11:57 -0500, Larry Finger wrote:
> The entries in this enum may be referenced in debug output. Adding explicit
> values simplifies the lookup.

A negative aspect to this style is forced renumbering
if a HW_VAR entry is inserted.

Is that possible?

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


Re: [PATCH 1/2] rtlwifi: Add HAL_DEF_WOWLAN case to *_get_hw() routines

2016-09-24 Thread Joe Perches
On Sat, 2016-09-24 at 11:57 -0500, Larry Finger wrote:
> Only rtl8821ae implements WOWLAN; however, the other drivers may receive
> a call requesting information about this mode. The other drivers need to
> ignore the request rather than logging that the default branch of the
> switch statement has been reached.

How about later rationalizing the DBG_EMERG vs DBG_LOUD differences too?

> diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/hw.c 
> b/drivers/net/wireless/realtek/rtlwifi/rtl8188ee/hw.c
[]
> @@ -355,6 +355,8 @@ void rtl88ee_get_hw_reg(struct ieee80211_hw *hw, u8 
> variable, u8 *val)
[]
>   default:
>   RT_TRACE(rtlpriv, COMP_ERR, DBG_EMERG,
>    "switch case %#x not processed\n", variable);
[]
> diff --git a/drivers/net/wireless/realtek/rtlwifi/rtl8723ae/hw.c 
> b/drivers/net/wireless/realtek/rtlwifi/rtl8723ae/hw.c
[]
> @@ -141,6 +141,8 @@ void rtl8723e_get_hw_reg(struct ieee80211_hw *hw, u8 
> variable, u8 *val)
[]
>   default:
>   RT_TRACE(rtlpriv, COMP_ERR, DBG_LOUD,
>    "switch case %#x not processed\n", variable);

etc...
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


Re: [PATCH] staging: sm750fb: rename getChipType to get_chip_type

2016-09-24 Thread Moshe Green
On Fri, Sep 23, 2016 at 02:13:52PM +0200, Greg KH wrote:
> On Thu, Sep 22, 2016 at 09:15:45PM +0300, Moshe Green wrote:
> > Rename CamelCased function getChipType to get_chip_type.
> > This issue was found by checkpatch.pl
> 
> As this is a global function, can you rename it to something like
> "sm750_get_chip_type()"?  Having a driver-specific function called
> get_chip_type() is not good.
> 
> thanks,
> 
> greg k-h

Will do.
Thanks,

Moshe Green
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel