Re: [PATCH v2] platform/x86/asus-nb-wmi.c: Add X45U quirk
From: Marcos Paulo de Souza
Date: Mon Nov 07 2016 - 15:14:17 EST
Hi Darren,
On Mon, Nov 07, 2016 at 10:35:05AM -0800, Darren Hart wrote:
> On Sun, Nov 06, 2016 at 12:05:12AM -0200, Marcos Paulo de Souza wrote:
> > Hi Darren,
> >
> > On Sat, Nov 05, 2016 at 11:24:04AM -0700, Darren Hart wrote:
> > > On Mon, Oct 31, 2016 at 11:56:10PM -0200, Marcos Paulo de Souza wrote:
> > > > Without this patch, the Asus X45U wireless card can't be turned
> > > > on (hard-blocked), but after a suspend/resume it just starts working.
> > > >
> > > > Following this bug report[1], there are other cases like this one, but
> > > > this Asus is the only model that I can test.
> > > >
> > > > [1] https://ubuntuforums.org/showthread.php?t=2181558
> > > >
> > > > Signed-off-by: Marcos Paulo de Souza <marcos.souza.org@xxxxxxxxx>
> > > > Cc: <stable@xxxxxxxxxxxxxxx>
> > > >
> > > > ---
> > > > This patch applies smoothly in 4.8.6 and 4.4.29. I hope now I followed
> > > > the instructions correctly.
> > >
> > > Hi Marcos,
> > >
> > > You'll find exact steps in the stable_kernel_rules.txt document regarding how to
> > > annotate the commit message with Cc lines indicating to which versions this
> > > patch should be applied.
> > >
> > > If, for example, you have verified that this patch is relevant only to 4.4 and
> > > forward, you would include:
> > >
> > > Cc: <stable@xxxxxxxxxxxxxxx> # 4.4.x-
> > >
> > > But this should mean that the patch is explicitly broken or otherwise
> > > inappropriate for 4.3 and earlier.
> >
> > Yes, I have misread the documentation. I could apply the patch as is just in
> > the versions 4.8.6 and 4.4.29, but maybe I could rework te patch earlier versions.
> >
> > But, aside from stable tree, do you agree with this change, since it fixes the
> > wireless problem in this laptop?
>
> Yes, no objection to the patch. It happened to land right about the time we were
> having the "be better about how you manage your stable commits" discussion at
> kernel summit, so it got some extra focus :-)
>
> Care to resend with the Cc stable tag corrected?
Absolutely not. I just sent a new version with the version along with
the stable email.
Thanks for your patience in this subject :)
>
> --
> Darren Hart
> Intel Open Source Technology Center