Re: [PATCH v2 0/7] Modernize vga_switcheroo by using device link for HDA

2018-03-13 Thread Lukas Wunner
On Sun, Mar 11, 2018 at 04:55:49PM +0100, Lukas Wunner wrote: > > On Sat, Mar 03, 2018 at 10:53:24AM +0100, Lukas Wunner wrote: > > > Modernize vga_switcheroo by using a device link to enforce a runtime PM > > > dependency from an HDA controller to the GPU it's integrated into, v2. > > > > > > htt

Re: [PATCH v2 0/7] Modernize vga_switcheroo by using device link for HDA

2018-03-13 Thread Bjorn Helgaas
On Sun, Mar 11, 2018 at 04:55:49PM +0100, Lukas Wunner wrote: > On Tue, Mar 06, 2018 at 11:29:40AM +0100, Daniel Vetter wrote: > > On Sat, Mar 03, 2018 at 10:53:24AM +0100, Lukas Wunner wrote: > > > Modernize vga_switcheroo by using a device link to enforce a runtime PM > > > dependency from an HDA

Re: [PATCH v2 0/7] Modernize vga_switcheroo by using device link for HDA

2018-03-13 Thread Daniel Vetter
On Mon, Mar 12, 2018 at 09:55:50PM +0100, Lukas Wunner wrote: > On Mon, Mar 12, 2018 at 05:54:47PM +0100, Daniel Vetter wrote: > > On Sun, Mar 11, 2018 at 04:55:49PM +0100, Lukas Wunner wrote: > > > > On Sat, Mar 03, 2018 at 10:53:24AM +0100, Lukas Wunner wrote: > > > > > Modernize vga_switcheroo b

Re: [PATCH v2 0/7] Modernize vga_switcheroo by using device link for HDA

2018-03-12 Thread Takashi Iwai
On Mon, 12 Mar 2018 21:55:50 +0100, Lukas Wunner wrote: > > On Mon, Mar 12, 2018 at 05:54:47PM +0100, Daniel Vetter wrote: > > On Sun, Mar 11, 2018 at 04:55:49PM +0100, Lukas Wunner wrote: > > > > On Sat, Mar 03, 2018 at 10:53:24AM +0100, Lukas Wunner wrote: > > > > > Modernize vga_switcheroo by u

Re: [PATCH v2 0/7] Modernize vga_switcheroo by using device link for HDA

2018-03-12 Thread Lukas Wunner
On Mon, Mar 12, 2018 at 05:54:47PM +0100, Daniel Vetter wrote: > On Sun, Mar 11, 2018 at 04:55:49PM +0100, Lukas Wunner wrote: > > > On Sat, Mar 03, 2018 at 10:53:24AM +0100, Lukas Wunner wrote: > > > > Modernize vga_switcheroo by using a device link to enforce a runtime PM > > > > dependency from

Re: [PATCH v2 0/7] Modernize vga_switcheroo by using device link for HDA

2018-03-12 Thread Daniel Vetter
On Sun, Mar 11, 2018 at 04:55:49PM +0100, Lukas Wunner wrote: > On Tue, Mar 06, 2018 at 11:29:40AM +0100, Daniel Vetter wrote: > > On Sat, Mar 03, 2018 at 10:53:24AM +0100, Lukas Wunner wrote: > > > Modernize vga_switcheroo by using a device link to enforce a runtime PM > > > dependency from an HDA

Re: [PATCH v2 0/7] Modernize vga_switcheroo by using device link for HDA

2018-03-11 Thread Lukas Wunner
On Tue, Mar 06, 2018 at 11:29:40AM +0100, Daniel Vetter wrote: > On Sat, Mar 03, 2018 at 10:53:24AM +0100, Lukas Wunner wrote: > > Modernize vga_switcheroo by using a device link to enforce a runtime PM > > dependency from an HDA controller to the GPU it's integrated into, v2. > > > > https://gith

Re: [PATCH v2 0/7] Modernize vga_switcheroo by using device link for HDA

2018-03-09 Thread Lukas Wunner
Hi Peter, thanks a million for the extensive testing and reviewing. I'll go through the issues you've found below, but it appears to me that they're all either in the "works as intended" category or are caused by something unrelated to this series: On Mon, Mar 05, 2018 at 09:58:31PM +0100, Pete

Re: [PATCH v2 0/7] Modernize vga_switcheroo by using device link for HDA

2018-03-06 Thread Daniel Vetter
On Sat, Mar 03, 2018 at 10:53:24AM +0100, Lukas Wunner wrote: > Modernize vga_switcheroo by using a device link to enforce a runtime PM > dependency from an HDA controller to the GPU it's integrated into, v2. > > Changes since v1: > > - Replace patch [1/7] to use pci_save_state() / pci_restore_st

Re: [PATCH v2 0/7] Modernize vga_switcheroo by using device link for HDA

2018-03-05 Thread Peter Wu
Hi Lukas, Sorry for the delay, I finally found some time to reviewd and test the patches and found some issues (some of them might already be present in v4.15 without your patches though, I did not try). Test environment: - Branch switcheroo_devlink_v2 (commit v4.15-20-gb33d50c5c6ad) - Laptop: Cl

[PATCH v2 0/7] Modernize vga_switcheroo by using device link for HDA

2018-03-03 Thread Lukas Wunner
Modernize vga_switcheroo by using a device link to enforce a runtime PM dependency from an HDA controller to the GPU it's integrated into, v2. Changes since v1: - Replace patch [1/7] to use pci_save_state() / pci_restore_state() for consistency between runtime PM code path of bound and unbound