From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-11.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, MENTIONS_GIT_HOSTING,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 615F4C4332B for ; Tue, 24 Mar 2020 11:47:14 +0000 (UTC) Received: from fraxinus.osuosl.org (smtp4.osuosl.org [140.211.166.137]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 38CDB20870 for ; Tue, 24 Mar 2020 11:47:14 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 38CDB20870 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=hadess.net Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=driverdev-devel-bounces@linuxdriverproject.org Received: from localhost (localhost [127.0.0.1]) by fraxinus.osuosl.org (Postfix) with ESMTP id 12E06861B2; Tue, 24 Mar 2020 11:47:14 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from fraxinus.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 6ghzODnDVI8o; Tue, 24 Mar 2020 11:47:11 +0000 (UTC) Received: from ash.osuosl.org (ash.osuosl.org [140.211.166.34]) by fraxinus.osuosl.org (Postfix) with ESMTP id 7FBA685FC7; Tue, 24 Mar 2020 11:47:11 +0000 (UTC) Received: from whitealder.osuosl.org (smtp1.osuosl.org [140.211.166.138]) by ash.osuosl.org (Postfix) with ESMTP id 4C6AA1BF477 for ; Tue, 24 Mar 2020 11:47:10 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by whitealder.osuosl.org (Postfix) with ESMTP id 3F08587988 for ; Tue, 24 Mar 2020 11:47:10 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from whitealder.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 2zsLC+MQXWFH for ; Tue, 24 Mar 2020 11:47:08 +0000 (UTC) X-Greylist: delayed 01:11:02 by SQLgrey-1.7.6 Received: from relay9-d.mail.gandi.net (relay9-d.mail.gandi.net [217.70.183.199]) by whitealder.osuosl.org (Postfix) with ESMTPS id 719FA87631 for ; Tue, 24 Mar 2020 11:47:08 +0000 (UTC) X-Originating-IP: 83.155.44.161 Received: from classic (mon69-7-83-155-44-161.fbx.proxad.net [83.155.44.161]) (Authenticated sender: hadess@hadess.net) by relay9-d.mail.gandi.net (Postfix) with ESMTPSA id 73EEEFF815; Tue, 24 Mar 2020 11:47:02 +0000 (UTC) Message-ID: <7aa74127978a73359ae95cd193bb3092d4536118.camel@hadess.net> Subject: Re: [PATCH resend 3] staging: rtl8188eu: Add rtw_led_enable module parameter From: Bastien Nocera To: Greg Kroah-Hartman Date: Tue, 24 Mar 2020 12:47:01 +0100 In-Reply-To: <20200324113840.GA2322042@kroah.com> References: <97d2ef68a6bcb7d1ece978eef6315e95732ca39d.camel@hadess.net> <20200324113840.GA2322042@kroah.com> User-Agent: Evolution 3.36.0 (3.36.0-1.fc32) MIME-Version: 1.0 X-BeenThere: driverdev-devel@linuxdriverproject.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux Driver Project Developer List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devel@driverdev.osuosl.org, linux-wireless@vger.kernel.org, Kalle Valo , Larry Finger Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: driverdev-devel-bounces@linuxdriverproject.org Sender: "devel" On Tue, 2020-03-24 at 12:38 +0100, Greg Kroah-Hartman wrote: > On Tue, Mar 24, 2020 at 11:36:00AM +0100, Bastien Nocera wrote: > > Make it possible to disable the LED, as it can be pretty annoying > > depending on where it's located. > > > > See also https://github.com/lwfinger/rtl8188eu/pull/304 for the > > out-of-tree version. > > > > Signed-off-by: Bastien Nocera > > --- > > drivers/staging/rtl8188eu/core/rtw_led.c | 6 ++++++ > > drivers/staging/rtl8188eu/include/drv_types.h | 2 ++ > > drivers/staging/rtl8188eu/os_dep/os_intfs.c | 5 +++++ > > 3 files changed, 13 insertions(+) > > > > diff --git a/drivers/staging/rtl8188eu/core/rtw_led.c > > b/drivers/staging/rtl8188eu/core/rtw_led.c > > index d1406cc99768..75a859accb7e 100644 > > --- a/drivers/staging/rtl8188eu/core/rtw_led.c > > +++ b/drivers/staging/rtl8188eu/core/rtw_led.c > > @@ -467,10 +467,16 @@ void blink_handler(struct LED_871x *pLed) > > > > void led_control_8188eu(struct adapter *padapter, enum > > LED_CTL_MODE LedAction) > > { > > + struct registry_priv *registry_par; > > + > > if (padapter->bSurpriseRemoved || padapter->bDriverStopped || > > !padapter->hw_init_completed) > > return; > > > > + registry_par = &padapter->registrypriv; > > + if (!registry_par->led_enable) > > + return; > > + > > if ((padapter->pwrctrlpriv.rf_pwrstate != rf_on && > > padapter->pwrctrlpriv.rfoff_reason > RF_CHANGE_BY_PS) && > > (LedAction == LED_CTL_TX || LedAction == LED_CTL_RX || > > diff --git a/drivers/staging/rtl8188eu/include/drv_types.h > > b/drivers/staging/rtl8188eu/include/drv_types.h > > index 35c0946bc65d..4ca828141d3f 100644 > > --- a/drivers/staging/rtl8188eu/include/drv_types.h > > +++ b/drivers/staging/rtl8188eu/include/drv_types.h > > @@ -67,6 +67,8 @@ struct registry_priv { > > u8 wmm_enable; > > u8 uapsd_enable; > > > > + u8 led_enable; > > + > > struct wlan_bssid_ex dev_network; > > > > u8 ht_enable; > > diff --git a/drivers/staging/rtl8188eu/os_dep/os_intfs.c > > b/drivers/staging/rtl8188eu/os_dep/os_intfs.c > > index 8907bf6bb7ff..ba55ae741215 100644 > > --- a/drivers/staging/rtl8188eu/os_dep/os_intfs.c > > +++ b/drivers/staging/rtl8188eu/os_dep/os_intfs.c > > @@ -47,6 +47,8 @@ static int rtw_acm_method;/* 0:By SW 1:By HW. */ > > static int rtw_wmm_enable = 1;/* default is set to enable the > > wmm. */ > > static int rtw_uapsd_enable; > > > > +static int rtw_led_enable = 1; > > + > > static int rtw_ht_enable = 1; > > /* 0 :disable, bit(0): enable 2.4g, bit(1): enable 5g */ > > static int rtw_cbw40_enable = 3; > > @@ -98,6 +100,7 @@ module_param(rtw_channel, int, 0644); > > module_param(rtw_wmm_enable, int, 0644); > > module_param(rtw_vrtl_carrier_sense, int, 0644); > > module_param(rtw_vcs_type, int, 0644); > > +module_param(rtw_led_enable, int, 0644); > > Ick, really? No, no nee module parameters, this is not the 1990's. > > This should be done on a per-device basis, using the correct apis. What API? _______________________________________________ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel