Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965414AbcCON5u (ORCPT ); Tue, 15 Mar 2016 09:57:50 -0400 Received: from mga11.intel.com ([192.55.52.93]:22476 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934119AbcCON5s (ORCPT ); Tue, 15 Mar 2016 09:57:48 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.24,339,1455004800"; d="scan'208";a="669940842" Date: Tue, 15 Mar 2016 15:57:43 +0200 From: Mika Westerberg To: Lukas Wunner 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: <20160315135743.GT1793@lahna.fi.intel.com> 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> <20160315133958.GA27793@wunner.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160315133958.GA27793@wunner.de> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1485 Lines: 31 On Tue, Mar 15, 2016 at 02:39:58PM +0100, Lukas Wunner wrote: > 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. If we disable the timer then we can't detect when a new device is connected to the port. I think in this case it might be better not to enable runtime PM for the port at all.