lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 22 Apr 2014 19:31:50 +0530 From: navin patidar <navin.patidar@...il.com> To: Dan Carpenter <dan.carpenter@...cle.com> Cc: Greg KH <gregkh@...uxfoundation.org>, Larry.Finger@...inger.net, devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH 03/10] staging: rtl8188eu: Remove _ReadLEDSetting() function. Hi dan it's just a cleanup patch and i also tested all patches of this patch series on the hardware (TP-LINK TL-WN723N). my mistake i should have added "Tested-by" tag for all patches. Tested-by: navin patidar <navin.patidar@...il.com> regards, --navin-patidar On Tue, Apr 22, 2014 at 3:41 PM, Dan Carpenter <dan.carpenter@...cle.com> wrote: > On Sun, Apr 20, 2014 at 02:17:11PM +0530, navin patidar wrote: >> _ReadLEDSetting() doesn't read led settings this function actually >> initialize member variables of struct led_priv, we should do that >> inside rtl8188eu_InitSwLeds(). >> > > This seems like a bugfix or is it just a cleanup? It's not clear to me > from the patch description. Have you tested this patch? > > regards, > dan carpenter > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists