* [PATCH][v4.2.y-ckt]ACPI / LPSS: allow to use specific PM domain during ->probe()
@ 2016-05-06 14:05 Joseph Salisbury
2016-05-07 4:47 ` Greg KH
2016-05-08 17:11 ` Greg KH
0 siblings, 2 replies; 6+ messages in thread
From: Joseph Salisbury @ 2016-05-06 14:05 UTC (permalink / raw)
To: Kamal Mostafa; +Cc: stable@vger.kernel.org, Andy Shevchenko, Rafael J. Wysocki
Hello,
Please consider including mainline commit
de16d55206c5ce82ad44590b01718fbe2323853c in the next v4.2.y-ckt
release. It was
included in the mainline tree as of v4.5-rc1. It has been tested and
confirmed
to resolve http://bugs.launchpad.net/bugs/1549354 .
commit de16d55206c5ce82ad44590b01718fbe2323853c
Author: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Date: Fri Dec 4 23:49:19 2015 +0200
ACPI / LPSS: allow to use specific PM domain during ->probe()
Sincerely,
Joseph Salisbury
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH][v4.2.y-ckt]ACPI / LPSS: allow to use specific PM domain during ->probe()
2016-05-06 14:05 [PATCH][v4.2.y-ckt]ACPI / LPSS: allow to use specific PM domain during ->probe() Joseph Salisbury
@ 2016-05-07 4:47 ` Greg KH
2016-05-08 17:11 ` Greg KH
1 sibling, 0 replies; 6+ messages in thread
From: Greg KH @ 2016-05-07 4:47 UTC (permalink / raw)
To: Joseph Salisbury
Cc: Kamal Mostafa, stable@vger.kernel.org, Andy Shevchenko,
Rafael J. Wysocki
On Fri, May 06, 2016 at 10:05:52AM -0400, Joseph Salisbury wrote:
> Hello,
>
> Please consider including mainline commit
> de16d55206c5ce82ad44590b01718fbe2323853c in the next v4.2.y-ckt
> release. It was
> included in the mainline tree as of v4.5-rc1. It has been tested and
> confirmed
> to resolve http://bugs.launchpad.net/bugs/1549354 .
>
> commit de16d55206c5ce82ad44590b01718fbe2323853c
> Author: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> Date: Fri Dec 4 23:49:19 2015 +0200
>
> ACPI / LPSS: allow to use specific PM domain during ->probe()
>
Why shouldn't it also go into 4.4-stable?
thanks,
greg k-h
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH][v4.2.y-ckt]ACPI / LPSS: allow to use specific PM domain during ->probe()
2016-05-06 14:05 [PATCH][v4.2.y-ckt]ACPI / LPSS: allow to use specific PM domain during ->probe() Joseph Salisbury
2016-05-07 4:47 ` Greg KH
@ 2016-05-08 17:11 ` Greg KH
2016-05-09 0:04 ` Joseph Salisbury
1 sibling, 1 reply; 6+ messages in thread
From: Greg KH @ 2016-05-08 17:11 UTC (permalink / raw)
To: Joseph Salisbury
Cc: Kamal Mostafa, stable@vger.kernel.org, Andy Shevchenko,
Rafael J. Wysocki
On Fri, May 06, 2016 at 10:05:52AM -0400, Joseph Salisbury wrote:
> Hello,
>
> Please consider including mainline commit
> de16d55206c5ce82ad44590b01718fbe2323853c in the next v4.2.y-ckt
> release. It was
> included in the mainline tree as of v4.5-rc1. It has been tested and
> confirmed
> to resolve http://bugs.launchpad.net/bugs/1549354 .
>
> commit de16d55206c5ce82ad44590b01718fbe2323853c
> Author: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> Date: Fri Dec 4 23:49:19 2015 +0200
>
> ACPI / LPSS: allow to use specific PM domain during ->probe()
>
It doesn't apply to 4.4-stable, why would it apply to 4.2?
confused,
greg k-h
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH][v4.2.y-ckt]ACPI / LPSS: allow to use specific PM domain during ->probe()
2016-05-08 17:11 ` Greg KH
@ 2016-05-09 0:04 ` Joseph Salisbury
2016-05-09 11:21 ` Andy Shevchenko
0 siblings, 1 reply; 6+ messages in thread
From: Joseph Salisbury @ 2016-05-09 0:04 UTC (permalink / raw)
To: Greg KH
Cc: Kamal Mostafa, stable@vger.kernel.org, Andy Shevchenko,
Rafael J. Wysocki
On 05/08/2016 01:11 PM, Greg KH wrote:
> On Fri, May 06, 2016 at 10:05:52AM -0400, Joseph Salisbury wrote:
>> Hello,
>>
>> Please consider including mainline commit
>> de16d55206c5ce82ad44590b01718fbe2323853c in the next v4.2.y-ckt
>> release. It was
>> included in the mainline tree as of v4.5-rc1. It has been tested and
>> confirmed
>> to resolve http://bugs.launchpad.net/bugs/1549354 .
>>
>> commit de16d55206c5ce82ad44590b01718fbe2323853c
>> Author: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
>> Date: Fri Dec 4 23:49:19 2015 +0200
>>
>> ACPI / LPSS: allow to use specific PM domain during ->probe()
>>
> It doesn't apply to 4.4-stable, why would it apply to 4.2?
>
> confused,
>
> greg k-h
Hi Greg,
Commit de16d55 is an amendment to commit 01ac170, per the commit
message. Commit 01ac170 landed in mainline as of 3.19-rc1 and commit
de16d55 landed in mainline as of v4.5-rc1. However, neither commits were
cc'd to stable.
This request for 4.2.y was to prevent a revert of a revert due to a bug[0]. In 4.2.y-ckt(And in mainline), commit 01ac170 was reverted by commit 05c7343(Commit b5f88dd in mainline). This introduced bug 1549354[0] in Ubuntu. This bug reports that the Elantech touchpad stopped working.
Commit 01ac170 was not reverted in v4.4.y, so it shouldn't have introduced the same bug. The commit message for b5f88dd says: "Fixes: 3df2da968744", but it was not cc'd to stable, so I'm not sure if you plan on including it in 4.4.y. If you do, commit 01ac170 will be needed to prevent the bug.
I also should have mentioned commit de16d55 will cause a build failure because it requires commit 14b6257 as a prerequisite. So if you include commit b5f88dd1d
in 4.4.y, you will need commits de16d55 and 14b6257a5 to prevent the indicated bug.
Thanks,
Joe
[0] http://pad.lv/1549354
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH][v4.2.y-ckt]ACPI / LPSS: allow to use specific PM domain during ->probe()
2016-05-09 0:04 ` Joseph Salisbury
@ 2016-05-09 11:21 ` Andy Shevchenko
2016-05-09 11:42 ` Greg KH
0 siblings, 1 reply; 6+ messages in thread
From: Andy Shevchenko @ 2016-05-09 11:21 UTC (permalink / raw)
To: Joseph Salisbury, Greg KH
Cc: Kamal Mostafa, stable@vger.kernel.org, Rafael J. Wysocki
On Sun, 2016-05-08 at 20:04 -0400, Joseph Salisbury wrote:
> On 05/08/2016 01:11 PM, Greg KH wrote:
> >
> > On Fri, May 06, 2016 at 10:05:52AM -0400, Joseph Salisbury wrote:
> > >
> > > Hello,
> > >
> > > Please consider including mainline commit
> > > de16d55206c5ce82ad44590b01718fbe2323853c in the next v4.2.y-ckt
> > > release. It was
> > > included in the mainline tree as of v4.5-rc1. It has been tested
> > > and
> > > confirmed
> > > to resolve http://bugs.launchpad.net/bugs/1549354 .
> > >
> > > commit de16d55206c5ce82ad44590b01718fbe2323853c
> > > Author: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> > > Date: Fri Dec 4 23:49:19 2015 +0200
> > >
> > > ACPI / LPSS: allow to use specific PM domain during ->probe()
> > >
> > It doesn't apply to 4.4-stable, why would it apply to 4.2?
> >
> > confused,
> >
> > greg k-h
> Hi Greg,
>
> Commit de16d55 is an amendment to commit 01ac170, per the commit
> message. Commit 01ac170 landed in mainline as of 3.19-rc1 and commit
> de16d55 landed in mainline as of v4.5-rc1. However, neither commits
> were
> cc'd to stable.
>
> This request for 4.2.y was to prevent a revert of a revert due to a
> bug[0]. In 4.2.y-ckt(And in mainline), commit 01ac170 was reverted by
> commit 05c7343(Commit b5f88dd in mainline). This introduced bug
> 1549354[0] in Ubuntu. This bug reports that the Elantech touchpad
> stopped working.
>
> Commit 01ac170 was not reverted in v4.4.y, so it shouldn't have
> introduced the same bug. The commit message for b5f88dd says: "Fixes:
> 3df2da968744", but it was not cc'd to stable, so I'm not sure if you
> plan on including it in 4.4.y. If you do, commit 01ac170 will be
> needed to prevent the bug.
>
> I also should have mentioned commit de16d55 will cause a build failure
> because it requires commit 14b6257 as a prerequisite. So if you
> include commit b5f88dd1d
> in 4.4.y, you will need commits de16d55 and 14b6257a5 to prevent the
> indicated bug.
It might be not the best idea to cherry-pick only those two since
culprit commit in your case is a part of a bigger fix, i.e.
commit eebb3e8d8aaf ("ACPI / LPSS: override power state for LPSS DMA
device"). So, it would be better to have that fix to land in the kernel
if you care.
And just a side note, please use the agreed form of commit reference
(for a mainline commits), i.e
commit <12+ hash> ("<title>")
> [0] http://pad.lv/1549354
--
Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Intel Finland Oy
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH][v4.2.y-ckt]ACPI / LPSS: allow to use specific PM domain during ->probe()
2016-05-09 11:21 ` Andy Shevchenko
@ 2016-05-09 11:42 ` Greg KH
0 siblings, 0 replies; 6+ messages in thread
From: Greg KH @ 2016-05-09 11:42 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Joseph Salisbury, Kamal Mostafa, stable@vger.kernel.org,
Rafael J. Wysocki
On Mon, May 09, 2016 at 02:21:15PM +0300, Andy Shevchenko wrote:
> On Sun, 2016-05-08 at 20:04 -0400, Joseph Salisbury wrote:
> > On 05/08/2016 01:11 PM, Greg KH wrote:
> > >
> > > On Fri, May 06, 2016 at 10:05:52AM -0400, Joseph Salisbury wrote:
> > > >
> > > > Hello,
> > > > ����
> > > > Please consider including mainline commit
> > > > de16d55206c5ce82ad44590b01718fbe2323853c in the next v4.2.y-ckt
> > > > release.��It was
> > > > included in the mainline tree as of v4.5-rc1.��It has been tested
> > > > and
> > > > confirmed
> > > > to resolve http://bugs.launchpad.net/bugs/1549354 .
> > > > ����
> > > > commit de16d55206c5ce82ad44590b01718fbe2323853c
> > > > Author: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> > > > Date:���Fri Dec 4 23:49:19 2015 +0200
> > > >
> > > > ����ACPI / LPSS: allow to use specific PM domain during ->probe()
> > > >
> > > It doesn't apply to 4.4-stable, why would it apply to 4.2?
> > >
> > > confused,
> > >
> > > greg k-h
> > Hi Greg,
> >
> > Commit de16d55 is an amendment to commit 01ac170, per the commit
> > message. Commit 01ac170 landed in mainline as of 3.19-rc1 and commit
> > de16d55 landed in mainline as of v4.5-rc1. However, neither commits
> > were
> > cc'd to stable.
> >
> > This request for 4.2.y was to prevent a revert of a revert due to a
> > bug[0].��In 4.2.y-ckt(And in mainline), commit 01ac170 was reverted by
> > commit 05c7343(Commit b5f88dd in mainline).��This introduced bug
> > 1549354[0] in Ubuntu.��This bug reports that the Elantech touchpad
> > stopped working.��
> >
> > Commit 01ac170 was not reverted in v4.4.y, so it shouldn't have
> > introduced the same bug.��The commit message for b5f88dd says: "Fixes:
> > 3df2da968744",��but it was not cc'd to stable, so I'm not sure if you
> > plan on including it in 4.4.y.��If you do, commit 01ac170 will be
> > needed to prevent the bug.
> >
> > I also should have mentioned commit de16d55 will cause a build failure
> > because it requires commit 14b6257 as a prerequisite.��So if you
> > include commit b5f88dd1d
> > in 4.4.y, you will need commits de16d55 and 14b6257a5 to prevent the
> > indicated bug.
>
> It might be not the best idea to cherry-pick only those two since
> culprit commit in your case is a part of a bigger fix, i.e.�
> commit eebb3e8d8aaf ("ACPI / LPSS: override power state for LPSS DMA
> device"). So, it would be better to have that fix to land in the kernel
> if you care.
This is crazy. I think I've taken that last patch, so I'm going to
leave 4.4-stable alone for now unless someone tells me I've messed
something up...
thanks,
greg k-h
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-05-09 11:42 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-05-06 14:05 [PATCH][v4.2.y-ckt]ACPI / LPSS: allow to use specific PM domain during ->probe() Joseph Salisbury
2016-05-07 4:47 ` Greg KH
2016-05-08 17:11 ` Greg KH
2016-05-09 0:04 ` Joseph Salisbury
2016-05-09 11:21 ` Andy Shevchenko
2016-05-09 11:42 ` Greg KH
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).