Re: [PATCH v2] x86/platform/intel-mid: Retrofit pci_platform_pm_ops ->get_state hook

[Andy Shevchenko <andriy.shevchenko@linux.intel.com>]

On Wed, 2016-10-19 at 20:04 +0200, Lukas Wunner wrote:
> On Wed, Oct 19, 2016 at 06:48:01PM +0300, Andy Shevchenko wrote:
> > On Wed, 2016-10-19 at 10:10 -0500, Bjorn Helgaas wrote:
> > > On Sun, Oct 09, 2016 at 02:46:29PM +0200, Lukas Wunner wrote:


> However so far only 4.9-rc1 is broken and we still have like 7 weeks
> to get the fix in.

One week less. :-)

> > > Fixes: 5823d0893ec2 ("x86/platform/intel-mid: Add Power Management
> > > Unit driver")
> > > Acked-by: Bjorn Helgaas <bhelgaas@google.com>
> > > 
> > > Sounds like this should be marked for stable, since v4.8 contains
> > > 5823d0893ec2 and is apparently broken?
> > 
> > At that point there was no "feature" commit in the tree. Perhaps the
> > Fixes tag should point to the "feature" commit instead. Am I right,
> > Lukas?
> 
> Right, the Fixes tag should be:
> 
> Fixes: cc7cc02bada8 ("PCI: Query platform firmware for device power
> state")
> 
> And this shouldn't get backported to 4.8, the issue is only present
> in 4.9-rc1 so far.
> 
> @Ingo: Do you want me to repost with Bjorn's ack and the correct
> Fixes: tag?

I didn't noticed the patch in last urgent pull request, so, I assume you
need to send v3.

-- 
Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Intel Finland Oy