From: Lukas Wunner <lukas@wunner.de>
To: Daniel Vetter <daniel@ffwll.ch>,
Alex Deucher <alexander.deucher@amd.com>,
Sean Paul <seanpaul@chromium.org>,
Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
Gustavo Padovan <gustavo@padovan.org>,
Bjorn Helgaas <bhelgaas@google.com>
Cc: dri-devel@lists.freedesktop.org, nouveau@lists.freedesktop.org,
"Rafael J. Wysocki" <rafael.j.wysocki@intel.com>,
linux-pci@vger.kernel.org, Daniel Drake <dan@reactivated.net>,
Denis Lisov <dennis.lissov@gmail.com>,
Peter Wu <peter@lekensteyn.nl>,
Martin Lopatar <lopin@dataplex.cz>,
Maik Freudenberg <hhfeuer@gmx.de>, Takashi Iwai <tiwai@suse.com>,
Lyude Paul <lyude@redhat.com>,
Hans de Goede <hdegoede@redhat.com>,
alsa-devel@alsa-project.org, Mike Lothian <mike@fireburn.co.uk>,
Kai-Heng Feng <kai.heng.feng@canonical.com>
Subject: Re: [PATCH v2 0/7] Modernize vga_switcheroo by using device link for HDA
Date: Sun, 11 Mar 2018 16:55:49 +0100 [thread overview]
Message-ID: <20180311155549.GA11806@wunner.de> (raw)
In-Reply-To: <20180306102940.GG22212@phenom.ffwll.local>
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://github.com/l1k/linux/commits/switcheroo_devlink_v2
>
> This all looks really reasonable and like a good cleanup, but it's a bit
> too much detail so I'll punt review to someone else with more clue.
Patches [3/7] to [7/7] were reviewed by Peter Wu, the HDA bits in
patch [5/7] additionally by Takashi.
Patch [2/7] was acked by Bjorn. There was no ack for patch [1/7]
(authored by Rafael), but it adressed the objection Bjorn raised
against my original patch, so I'm assuming Bjorn is okay with it.
(Bjorn, please let me know if that isn't the case.)
The series has been tested on 5 systems, which raises the confidence:
2x AMD PowerXpress (Mike Lothian, Kai Heng Feng)
2x Nvidia Optimus (Denis Lisov, Peter Wu)
1x MacBook Pro
The issues found during Peter Wu's thorough testing appear to all
be unrelated to this series, as per my e-mail yesterday.
If there are no objections, I plan to push the series to
drm-misc-next by the middle of the coming week so that it
would still catch the last train to 4.17.
Thanks,
Lukas
next prev parent reply other threads:[~2018-03-11 15:55 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-03 9:53 [PATCH v2 0/7] Modernize vga_switcheroo by using device link for HDA Lukas Wunner
2018-03-03 9:53 ` [PATCH v2 1/7] PCI: Restore config space on runtime resume despite being unbound Lukas Wunner
2018-03-13 17:34 ` Bjorn Helgaas
2018-03-03 9:53 ` [PATCH v2 5/7] vga_switcheroo: Use device link for HDA controller Lukas Wunner
2018-03-05 20:58 ` [PATCH v2 0/7] Modernize vga_switcheroo by using device link for HDA Peter Wu
2018-03-10 6:09 ` Lukas Wunner
2018-03-06 10:29 ` Daniel Vetter
2018-03-11 15:55 ` Lukas Wunner [this message]
2018-03-12 16:54 ` Daniel Vetter
2018-03-13 17:36 ` Bjorn Helgaas
2018-03-14 6:07 ` Lukas Wunner
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180311155549.GA11806@wunner.de \
--to=lukas@wunner.de \
--cc=alexander.deucher@amd.com \
--cc=alsa-devel@alsa-project.org \
--cc=bhelgaas@google.com \
--cc=dan@reactivated.net \
--cc=daniel@ffwll.ch \
--cc=dennis.lissov@gmail.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=gustavo@padovan.org \
--cc=hdegoede@redhat.com \
--cc=hhfeuer@gmx.de \
--cc=kai.heng.feng@canonical.com \
--cc=linux-pci@vger.kernel.org \
--cc=lopin@dataplex.cz \
--cc=lyude@redhat.com \
--cc=maarten.lankhorst@linux.intel.com \
--cc=mike@fireburn.co.uk \
--cc=nouveau@lists.freedesktop.org \
--cc=peter@lekensteyn.nl \
--cc=rafael.j.wysocki@intel.com \
--cc=seanpaul@chromium.org \
--cc=tiwai@suse.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).