From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lukas Wunner Subject: Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10 machines. Date: Tue, 15 Mar 2016 14:39:58 +0100 Message-ID: <20160315133958.GA27793@wunner.de> References: <1457504045-12738-1-git-send-email-airlied@gmail.com> <2736217.utxPzJExd5@vostro.rjw.lan> <20160311105815.GS1796@lahna.fi.intel.com> <2759763.dR5D0KopkA@vostro.rjw.lan> <20160314094335.GE1796@lahna.fi.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <20160314094335.GE1796@lahna.fi.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Mika Westerberg Cc: "Rafael J. Wysocki" , Linux PCI , "linux-pm@vger.kernel.org" , "Rafael J. Wysocki" , Linux Kernel Mailing List , dri-devel , ACPI Devel Maling List List-Id: linux-acpi@vger.kernel.org SGkgTWlrYSwKCk9uIE1vbiwgTWFyIDE0LCAyMDE2IGF0IDExOjQzOjM1QU0gKzAyMDAsIE1pa2Eg V2VzdGVyYmVyZyB3cm90ZToKPiBPbiBNb24sIE1hciAxNCwgMjAxNiBhdCAxMjoxOToxNFBNICsx MDAwLCBEYXZlIEFpcmxpZSB3cm90ZToKPiA+IE9uIDExIE1hcmNoIDIwMTYgYXQgMjM6NDUsIFJh ZmFlbCBKLiBXeXNvY2tpIDxyandAcmp3eXNvY2tpLm5ldD4gd3JvdGU6Cj4gPiA+IE9uIEZyaWRh eSwgTWFyY2ggMTEsIDIwMTYgMTI6NTg6MTUgUE0gTWlrYSBXZXN0ZXJiZXJnIHdyb3RlOgo+ID4g Pj4gT24gVGh1LCBNYXIgMTAsIDIwMTYgYXQgMDk6NTc6MDlQTSArMDEwMCwgUmFmYWVsIEouIFd5 c29ja2kgd3JvdGU6Cj4gPiA+PiA+ID4gSXQgZG9lc24ndCBzZWVtIHRvIGRvIGFueSBydW50aW1l IFBNLAo+ID4gPj4gPiA+IEkgZG8gd29uZGVyIGlmIHBjaWVwb3J0IHNob3VsZCBiZSBkb2luZyBp dCdzIG93biBydW50aW1lIFBNIGhhbmRsaW5nLAo+ID4gPj4gPiA+IGJ1dCB0aGF0IGlzIGEKPiA+ ID4+ID4gPiBsYXJnZXIgdGFzayB0aGFuIEknbSB0aGlua2luZyB0byB0YWNrbGUgaGVyZS4KPiA+ ID4+ID4KPiA+ID4+ID4gUENJZSBwb3J0cyBkb24ndCBkbyBQTSAtIHlldC4gIE1pa2EgaGFzIHBv c3RlZCBhIHNlcmllcyBvZiBwYXRjaGVzIHRvIGltcGxlbWVudAo+ID4gPj4gPiB0aGF0LCBob3dl dmVyLCB0aGF0IGFyZSB3YWl0aW5nIGZvciBjb21tZW50cyBub3c6Cj4gPiA+PiA+Cj4gPiA+PiA+ IGh0dHBzOi8vcGF0Y2h3b3JrLmtlcm5lbC5vcmcvcGF0Y2gvODQ1MzMxMS8KPiA+ID4+ID4gaHR0 cHM6Ly9wYXRjaHdvcmsua2VybmVsLm9yZy9wYXRjaC84NDUzMzgxLwo+ID4gPj4gPiBodHRwczov L3BhdGNod29yay5rZXJuZWwub3JnL3BhdGNoLzg0NTMzOTEvCj4gPiA+PiA+IGh0dHBzOi8vcGF0 Y2h3b3JrLmtlcm5lbC5vcmcvcGF0Y2gvODQ1MzQxMS8KPiA+ID4+ID4gaHR0cHM6Ly9wYXRjaHdv cmsua2VybmVsLm9yZy9wYXRjaC84NDUzMzcxLwo+ID4gPj4gPiBodHRwczovL3BhdGNod29yay5r ZXJuZWwub3JnL3BhdGNoLzg0NTMzNTEvCgpJZiBhIHBjaWVocCBwb3J0IGlzIHJ1bnRpbWUgc3Vz cGVuZGVkIGFuZCBwY2llaHBfcG9sbF9tb2RlIGlzIGVuYWJsZWQsCnRoZSBwb2xsIHRpbWVyIG5l ZWRzIHRvIGJlIGRpc2FibGVkIGFuZCBsYXRlciByZWVuYWJsZWQgb24gcnVudGltZSByZXN1bWUu CgpEb24ndCB3ZSBuZWVkIHRvIGFkZCBydW50aW1lIHBtIGNhbGxiYWNrcyB0byBzdHJ1Y3QgcGNp ZV9wb3J0X3NlcnZpY2VfZHJpdmVyCmZvciB0aGF0LCBhbmQgY2FsbCB0aGVtIGZyb20gcGNpZV9w b3J0X3J1bnRpbWVfKigpPwoKQmVzdCByZWdhcmRzLAoKTHVrYXMKX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVsIG1haWxpbmcgbGlzdApkcmkt ZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3Jn L21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg== From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965269AbcCONjF (ORCPT ); Tue, 15 Mar 2016 09:39:05 -0400 Received: from mailout2.hostsharing.net ([83.223.90.233]:56907 "EHLO mailout2.hostsharing.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750817AbcCONiz (ORCPT ); Tue, 15 Mar 2016 09:38:55 -0400 Date: Tue, 15 Mar 2016 14:39:58 +0100 From: Lukas Wunner To: Mika Westerberg Cc: Dave Airlie , "linux-pm@vger.kernel.org" , Linux PCI , "Rafael J. Wysocki" , "Rafael J. Wysocki" , Linux Kernel Mailing List , dri-devel , ACPI Devel Maling List Subject: Re: [PATCH 1/2] vga_switcheroo: add power support for windows 10 machines. Message-ID: <20160315133958.GA27793@wunner.de> References: <1457504045-12738-1-git-send-email-airlied@gmail.com> <2736217.utxPzJExd5@vostro.rjw.lan> <20160311105815.GS1796@lahna.fi.intel.com> <2759763.dR5D0KopkA@vostro.rjw.lan> <20160314094335.GE1796@lahna.fi.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160314094335.GE1796@lahna.fi.intel.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Mika, On Mon, Mar 14, 2016 at 11:43:35AM +0200, Mika Westerberg wrote: > On Mon, Mar 14, 2016 at 12:19:14PM +1000, Dave Airlie wrote: > > On 11 March 2016 at 23:45, Rafael J. Wysocki wrote: > > > On Friday, March 11, 2016 12:58:15 PM Mika Westerberg wrote: > > >> On Thu, Mar 10, 2016 at 09:57:09PM +0100, Rafael J. Wysocki wrote: > > >> > > It doesn't seem to do any runtime PM, > > >> > > I do wonder if pcieport should be doing it's own runtime PM handling, > > >> > > but that is a > > >> > > larger task than I'm thinking to tackle here. > > >> > > > >> > PCIe ports don't do PM - yet. Mika has posted a series of patches to implement > > >> > that, however, that are waiting for comments now: > > >> > > > >> > https://patchwork.kernel.org/patch/8453311/ > > >> > https://patchwork.kernel.org/patch/8453381/ > > >> > https://patchwork.kernel.org/patch/8453391/ > > >> > https://patchwork.kernel.org/patch/8453411/ > > >> > https://patchwork.kernel.org/patch/8453371/ > > >> > https://patchwork.kernel.org/patch/8453351/ If a pciehp port is runtime suspended and pciehp_poll_mode is enabled, the poll timer needs to be disabled and later reenabled on runtime resume. Don't we need to add runtime pm callbacks to struct pcie_port_service_driver for that, and call them from pcie_port_runtime_*()? Best regards, Lukas