[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20161108001029.GB10184@xfiles>
Date: Mon, 7 Nov 2016 22:10:31 -0200
From: Marcos Paulo de Souza <marcos.souza.org@...il.com>
To: Darren Hart <dvhart@...radead.org>
Cc: corentin.chary@...il.com, acpi4asus-user@...ts.sourceforge.net,
platform-driver-x86@...r.kernel.org, linux-kernel@...r.kernel.org,
gregkh@...uxfoundation.org
Subject: Re: [PATCH v2] platform/x86/asus-nb-wmi.c: Add X45U quirk
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@...il.com>
> > > > Cc: <stable@...r.kernel.org>
> > > >
> > > > ---
> > > > 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@...r.kernel.org> # 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?
I had some problem to send the patch through git send-email and mutt,
because the stable address and the version always got merged. So Laura
Abbott helped me with the tip of --suppress-cc=bodycc, and then I added
stable address manually.
Is there a new procedure to send a patch to stable, without being this
tricky? Is there is, I would be happy to send a patch to fix our
documentation, since even people from kernel newbies are suffering by
the same problem[1]
Thanks in advance.
[1]
https://lists.kernelnewbies.org/pipermail/kernelnewbies/2016-April/016194.html
>
> --
> Darren Hart
> Intel Open Source Technology Center
Powered by blists - more mailing lists