public inbox for intel-gfx@lists.freedesktop.org
 help / color / mirror / Atom feed
From: Hans de Goede <hdegoede@redhat.com>
To: Jani Nikula <jani.nikula@linux.intel.com>,
	Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
	Maxime Ripard <mripard@kernel.org>,
	Thomas Zimmermann <tzimmermann@suse.de>,
	Joonas Lahtinen <joonas.lahtinen@linux.intel.com>,
	Rodrigo Vivi <rodrigo.vivi@intel.com>
Cc: "dri-devel@lists.freedesktop.org"
	<dri-devel@lists.freedesktop.org>,
	intel-gfx <intel-gfx@lists.freedesktop.org>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Daniel Vetter <daniel@ffwll.ch>
Subject: Re: [Intel-gfx] [GIT PULL] drm-misc + drm-intel: Add support for out-of-band hotplug notification
Date: Tue, 24 Aug 2021 18:48:59 +0200	[thread overview]
Message-ID: <15400f71-dfe6-3142-d191-596ef9af7e7a@redhat.com> (raw)
In-Reply-To: <871r6jgrin.fsf@intel.com>

Hi,

On 8/24/21 10:45 AM, Jani Nikula wrote:
> On Fri, 20 Aug 2021, Hans de Goede <hdegoede@redhat.com> wrote:
>> Hello drm-misc and drm-intel maintainers,
>>
>> My "Add support for out-of-band hotplug notification" patchset:
>> https://patchwork.freedesktop.org/series/93763/
>>
>> Is ready for merging now, as discussed on IRC I based this series
>> on top drm-tip and when trying to apply the i915 parts on top
>> of drm-misc this fails due to conflict.
>>
>> So as Jani suggested here is a pull-req for a topic-branch with the
>> entire set, minus the troublesome i915 bits. Once this has been merged
>> into both drm-misc-next and drm-intel-next I can push the 2 i915
>> patch do drm-intel-next on top of the merge.
>>
>> Note there are also 2 drivers/usb/typec patches in here these
>> have Greg KH's Reviewed-by for merging through the drm tree,
>> Since this USB code does not change all that much. I also checked
>> and the drm-misc-next-2021-08-12 base of this tree contains the
>> same last commit to the modified file as usb-next.
>>
>> Daniel Vetter mentioned on IRC that it might be better for you to simply
>> pick-up the series directly from patchwork, that is fine too in that
>> case don't forget to add:
>>
>> Reviewed-by: Lyude Paul <lyude@redhat.com>
>>
>> To the entire series (given in a reply to the cover-letter)
>>
>> And:
>>
>> Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
>>
>> To the usb/typec patches (patch 7/8), this was given in reply
>> to a previous posting of the series and I forgot to add this
>> in the resend.
> 
> Since this is mostly touching drm core, I think it should be merged to
> drm-misc-next first, and drm-intel-next after. Please let us know.

I agree this should go to drm-misc-next first.

(I was planning on pushing this to drm-misc-next myself,
but then ended up going with the topic branch because of the
conflict in the i915 bits.)

Regards,

Hans



>> The following changes since commit c7782443a88926a4f938f0193041616328cf2db2:
>>
>>   drm/bridge: ti-sn65dsi86: Avoid creating multiple connectors (2021-08-12 09:56:09 -0700)
>>
>> are available in the Git repository at:
>>
>>   git://git.kernel.org/pub/scm/linux/kernel/git/hansg/linux.git drm-misc-intel-oob-hotplug-v1
>>
>> for you to fetch changes up to 7f811394878535ed9a6849717de8c2959ae38899:
>>
>>   usb: typec: altmodes/displayport: Notify drm subsys of hotplug events (2021-08-20 12:35:59 +0200)
>>
>> ----------------------------------------------------------------
>> Topic branch for drm-misc / drm-intel for OOB hotplug support for Type-C connectors
>>
>> ----------------------------------------------------------------
>> Hans de Goede (6):
>>       drm/connector: Give connector sysfs devices there own device_type
>>       drm/connector: Add a fwnode pointer to drm_connector and register with ACPI (v2)
>>       drm/connector: Add drm_connector_find_by_fwnode() function (v3)
>>       drm/connector: Add support for out-of-band hotplug notification (v3)
>>       usb: typec: altmodes/displayport: Make dp_altmode_notify() more generic
>>       usb: typec: altmodes/displayport: Notify drm subsys of hotplug events
>>
>>  drivers/gpu/drm/drm_connector.c          | 79 +++++++++++++++++++++++++++++
>>  drivers/gpu/drm/drm_crtc_internal.h      |  2 +
>>  drivers/gpu/drm/drm_sysfs.c              | 87 +++++++++++++++++++++++++++-----
>>  drivers/usb/typec/altmodes/Kconfig       |  1 +
>>  drivers/usb/typec/altmodes/displayport.c | 58 +++++++++++++--------
>>  include/drm/drm_connector.h              | 25 +++++++++
>>  6 files changed, 217 insertions(+), 35 deletions(-)
>>
> 


  reply	other threads:[~2021-08-24 16:49 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-08-20 17:50 [Intel-gfx] [GIT PULL] drm-misc + drm-intel: Add support for out-of-band hotplug notification Hans de Goede
2021-08-24  8:45 ` Jani Nikula
2021-08-24 16:48   ` Hans de Goede [this message]
2021-08-25 16:03     ` Vivi, Rodrigo
2021-08-26  8:23       ` Maxime Ripard
2021-08-26 11:43         ` Vivi, Rodrigo
2021-09-13 10:33           ` Hans de Goede

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=15400f71-dfe6-3142-d191-596ef9af7e7a@redhat.com \
    --to=hdegoede@redhat.com \
    --cc=daniel@ffwll.ch \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=jani.nikula@linux.intel.com \
    --cc=joonas.lahtinen@linux.intel.com \
    --cc=maarten.lankhorst@linux.intel.com \
    --cc=mripard@kernel.org \
    --cc=rodrigo.vivi@intel.com \
    --cc=tzimmermann@suse.de \
    /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