linux-kernel - Re: [PATCH v4 3/3] dell-laptop: Use dell-rbtn instead i8042 filter when possible

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 27 May 2015 19:52:13 -0700
From:	Darren Hart <dvhart@...radead.org>
To:	Pali Rohár <pali.rohar@...il.com>
Cc:	platform-driver-x86@...r.kernel.org, linux-kernel@...r.kernel.org,
	Gabriele Mazzotta <gabriele.mzt@...il.com>,
	Alex Hung <alex.hung@...onical.com>,
	Matthew Garrett <mjg59@...f.ucam.org>,
	Dmitry Torokhov <dmitry.torokhov@...il.com>,
	Valdis.Kletnieks@...edu, Darren Hart <dvhart@...ux.intel.com>
Subject: Re: [PATCH v4 3/3] dell-laptop: Use dell-rbtn instead i8042 filter
 when possible

On Wed, May 27, 2015 at 11:28:25PM +0200, Pali Rohár wrote:
> Until now module dell-laptop registered rfkill device which used i8042
> filter function for receiving HW switch rfkill events (handling special
> keycode).
> 
> But for some dell laptops there is native ACPI driver dell-rbtn which can
> receive rfkill events (without i8042 hooks).
> 
> So this patch will combine best from both sides. It will use native ACPI
> driver dell-rbtn for receiving events and dell-laptop SMBIOS interface for
> enabling or disabling radio devices. If ACPI driver or device will not be
> available fallback to i8042 filter function will be used.
> 
> Patch also changes module_init() to late_initcall() to ensure that init
> function will be called after initializing dell-rbtn.c driver.
> 
> Signed-off-by: Pali Rohár <pali.rohar@...il.com>
> Tested-by: Gabriele Mazzotta <gabriele.mzt@...il.com>
> Signed-off-by: Darren Hart <dvhart@...ux.intel.com>


Several basic checkpatch.pl errors in this one. Please always run checkpatch.pl
before submitting patches, these sorts of issues should be caught before hitting
the mailing list.

Please provide some details on the scenarios you have tested to verify you have
addressed the build and runtime issues reported.

Thanks,

-- 
Darren Hart
Intel Open Source Technology Center
--
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/