All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Vivi, Rodrigo" <rodrigo.vivi@intel.com>
To: "hdegoede@redhat.com" <hdegoede@redhat.com>,
	"joonas.lahtinen@linux.intel.com"
	<joonas.lahtinen@linux.intel.com>,
	"maarten.lankhorst@linux.intel.com"
	<maarten.lankhorst@linux.intel.com>,
	"mripard@kernel.org" <mripard@kernel.org>,
	"jani.nikula@linux.intel.com" <jani.nikula@linux.intel.com>,
	"tzimmermann@suse.de" <tzimmermann@suse.de>
Cc: "dri-devel@lists.freedesktop.org"
	<dri-devel@lists.freedesktop.org>,
	"daniel@ffwll.ch" <daniel@ffwll.ch>,
	"gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
	"intel-gfx@lists.freedesktop.org"
	<intel-gfx@lists.freedesktop.org>
Subject: Re: [GIT PULL] drm-misc + drm-intel: Add support for out-of-band hotplug notification
Date: Wed, 25 Aug 2021 16:03:43 +0000	[thread overview]
Message-ID: <b24d3733d0b0c79fa6fce4f4e696b0c3a338469a.camel@intel.com> (raw)
In-Reply-To: <15400f71-dfe6-3142-d191-596ef9af7e7a@redhat.com>

On Tue, 2021-08-24 at 18:48 +0200, Hans de Goede wrote:
> 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.)

Just to be clear and avoid confusion: This pull request does apply
cleanly on drm-misc-next nd drm-intel-next right now.

I'm just waiting for drm-misc-next maintainers to pull this to drm-
misc-next so I can pull it to drm-intel-next.

Maxime, is that your round now?
or Thomas?

Thanks,
Rodrigo.

> 
> 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(-)
> > > 
> > 
> 


WARNING: multiple messages have this Message-ID (diff)
From: "Vivi, Rodrigo" <rodrigo.vivi@intel.com>
To: "hdegoede@redhat.com" <hdegoede@redhat.com>,
	"joonas.lahtinen@linux.intel.com"
	<joonas.lahtinen@linux.intel.com>,
	"maarten.lankhorst@linux.intel.com"
	<maarten.lankhorst@linux.intel.com>,
	"mripard@kernel.org" <mripard@kernel.org>,
	"jani.nikula@linux.intel.com" <jani.nikula@linux.intel.com>,
	"tzimmermann@suse.de" <tzimmermann@suse.de>
Cc: "dri-devel@lists.freedesktop.org"
	<dri-devel@lists.freedesktop.org>,
	"daniel@ffwll.ch" <daniel@ffwll.ch>,
	"gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
	"intel-gfx@lists.freedesktop.org"
	<intel-gfx@lists.freedesktop.org>
Subject: Re: [Intel-gfx] [GIT PULL] drm-misc + drm-intel: Add support for out-of-band hotplug notification
Date: Wed, 25 Aug 2021 16:03:43 +0000	[thread overview]
Message-ID: <b24d3733d0b0c79fa6fce4f4e696b0c3a338469a.camel@intel.com> (raw)
In-Reply-To: <15400f71-dfe6-3142-d191-596ef9af7e7a@redhat.com>

On Tue, 2021-08-24 at 18:48 +0200, Hans de Goede wrote:
> 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.)

Just to be clear and avoid confusion: This pull request does apply
cleanly on drm-misc-next nd drm-intel-next right now.

I'm just waiting for drm-misc-next maintainers to pull this to drm-
misc-next so I can pull it to drm-intel-next.

Maxime, is that your round now?
or Thomas?

Thanks,
Rodrigo.

> 
> 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-25 16:04 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-08-20 17:50 [GIT PULL] drm-misc + drm-intel: Add support for out-of-band hotplug notification Hans de Goede
2021-08-20 17:50 ` [Intel-gfx] " Hans de Goede
2021-08-24  8:45 ` Jani Nikula
2021-08-24  8:45   ` [Intel-gfx] " Jani Nikula
2021-08-24 16:48   ` Hans de Goede
2021-08-24 16:48     ` [Intel-gfx] " Hans de Goede
2021-08-25 16:03     ` Vivi, Rodrigo [this message]
2021-08-25 16:03       ` Vivi, Rodrigo
2021-08-26  8:23       ` Maxime Ripard
2021-08-26  8:23         ` [Intel-gfx] " Maxime Ripard
2021-08-26 11:43         ` Vivi, Rodrigo
2021-08-26 11:43           ` [Intel-gfx] " Vivi, Rodrigo
2021-09-13 10:33           ` Hans de Goede
2021-09-13 10:33             ` [Intel-gfx] " 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=b24d3733d0b0c79fa6fce4f4e696b0c3a338469a.camel@intel.com \
    --to=rodrigo.vivi@intel.com \
    --cc=daniel@ffwll.ch \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=hdegoede@redhat.com \
    --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=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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.