From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from bombadil.infradead.org ([198.137.202.9]:41917 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751085AbcAXSML (ORCPT ); Sun, 24 Jan 2016 13:12:11 -0500 Date: Sun, 24 Jan 2016 10:12:06 -0800 From: Darren Hart To: Josh Boyer Cc: Ike Panhc , platform-driver-x86@vger.kernel.org, "Linux-Kernel@Vger. Kernel. Org" , "stable@vger.kernel.org" Subject: Re: [PATCH] ideapad-laptop: Add Lenovo Yoga 700 to no_hw_rfkill dmi list Message-ID: <20160124181206.GJ1979@malice.jf.intel.com> References: <1453145573-15600-1-git-send-email-jwboyer@fedoraproject.org> <20160123072731.GI1979@malice.jf.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: stable-owner@vger.kernel.org List-ID: On Sat, Jan 23, 2016 at 07:57:23AM -0500, Josh Boyer wrote: > On Sat, Jan 23, 2016 at 2:27 AM, Darren Hart wrote: > > On Mon, Jan 18, 2016 at 02:32:53PM -0500, Josh Boyer wrote: > >> Like the Yoga 900 models the Lenovo Yoga 700 does not have a > >> hw rfkill switch, and trying to read the hw rfkill switch through the > >> ideapad module causes it to always reported blocking breaking wifi. > >> > >> This commit adds the Lenovo Yoga 700 to the no_hw_rfkill dmi list, fixing > >> the wifi breakage. > >> > >> BugLink: https://bugzilla.redhat.com/show_bug.cgi?id=1295272 > >> Tested-by: > >> Cc: stable@vger.kernel.org > >> Signed-off-by: Josh Boyer > >> --- > >> > >> This applies to the for-next branch of the platform-x86-drivers tree > >> > >> > >> drivers/platform/x86/ideapad-laptop.c | 7 +++++++ > >> 1 file changed, 7 insertions(+) > >> > >> diff --git a/drivers/platform/x86/ideapad-laptop.c b/drivers/platform/x86/ideapad-laptop.c > >> index d28db0e793df..51178626305d 100644 > >> --- a/drivers/platform/x86/ideapad-laptop.c > >> +++ b/drivers/platform/x86/ideapad-laptop.c > >> @@ -900,6 +900,13 @@ static const struct dmi_system_id no_hw_rfkill_list[] = { > >> }, > >> }, > >> { > >> + .ident = "Lenogo Yoga 700", > > > > Josh, > > > > Is this a typo? "Lenogo"? Please tell me it's a typo.... :-) > > Ugh, yes. A typo. One that apparently doesn't matter in > functionality because the tester used this exact patch and things > work, but embarrassing either way. > > Do you need me to send a v2, or is that something you can fixup? Normally I could have just fixed it up. I had some emergency travel on Friday until now, so couldn't get back to it. Thank you for resending. -- Darren Hart Intel Open Source Technology Center