From: Thomas Zimmermann <tzimmermann@suse.de> To: Daniel Vetter <daniel@ffwll.ch> Cc: hudson@trmm.net, markus@raatikainen.cc, Sam Ravnborg <sam@ravnborg.org>, USB list <linux-usb@vger.kernel.org>, dri-devel <dri-devel@lists.freedesktop.org>, Tyler Hardin <th020394@gmail.com>, Lubomir Rintel <lkundrak@v3.sk>, pontus.fuchs@gmail.com, peter@stuge.se Subject: Re: [PATCH v4 1/3] drm/uapi: Add USB connector type Date: Thu, 21 Jan 2021 11:01:11 +0100 [thread overview] Message-ID: <1ea4e6e4-0806-dba1-a424-47f178dc882f@suse.de> (raw) In-Reply-To: <CAKMK7uHiQ3i-Rz_y_3joR2Zi3fA=1qp8MdGZ9w9PUcGoWT3urw@mail.gmail.com> [-- Attachment #1.1: Type: text/plain, Size: 2870 bytes --] Hi Am 21.01.21 um 09:27 schrieb Daniel Vetter: > On Thu, Jan 21, 2021 at 8:45 AM Thomas Zimmermann <tzimmermann@suse.de> wrote: >> >> Hi Noralf, >> >> glad to hear from you! Welcome back! >> >> Am 20.01.21 um 18:42 schrieb Daniel Vetter: >>> On Wed, Jan 20, 2021 at 6:10 PM Noralf Trønnes <noralf@tronnes.org> wrote: >>>> >>>> Add a connector type for USB connected display panels. >>>> >>>> Signed-off-by: Noralf Trønnes <noralf@tronnes.org> >>>> --- >>>> include/uapi/drm/drm_mode.h | 1 + >>>> 1 file changed, 1 insertion(+) >>>> >>>> diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h >>>> index fed66a03c7ae..33024cc5d26e 100644 >>>> --- a/include/uapi/drm/drm_mode.h >>>> +++ b/include/uapi/drm/drm_mode.h >>>> @@ -367,6 +367,7 @@ enum drm_mode_subconnector { >>>> #define DRM_MODE_CONNECTOR_DPI 17 >>>> #define DRM_MODE_CONNECTOR_WRITEBACK 18 >>>> #define DRM_MODE_CONNECTOR_SPI 19 >>>> +#define DRM_MODE_CONNECTOR_USB 20 >> >> I would not call it USB. I could imagine that at some point a generic >> USB protocol could serve simple displays (i.e. in the sense of USB HID >> or data or imaging). (Maybe Thunderbold already counts.) Anyway, USB >> should be reserved for this case. > > We end up calling those DisplayPort, since that's what's being > transported over thunderbolt or usb-C. So the usb connector would be > called usb-C. I think the reason we don't do fancy connector names is > that adding them is a bit a pain. Plus drm/i915 specifically has some > very quirky connector enumerating that doesn't match much with reality > unfortunately anyway :-/ In the case of the other USB drivers, IIRC we use the connector type that is at the output (i.e., HDMI in the case of udl). I think we should do the same here. Or use 'Unknown'. Best regards Thomas > -Daniel > >> >> Best regards >> Thomas >> >>> >>> Beware, new connector types have in the past resulted in userspace >>> burning&crashing. Maybe it's become better ... >>> >>> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch> >>>> >>>> /** >>>> * struct drm_mode_get_connector - Get connector metadata. >>>> -- >>>> 2.23.0 >>>> >>>> _______________________________________________ >>>> dri-devel mailing list >>>> dri-devel@lists.freedesktop.org >>>> https://lists.freedesktop.org/mailman/listinfo/dri-devel >>> >>> >>> >> >> -- >> Thomas Zimmermann >> Graphics Driver Developer >> SUSE Software Solutions Germany GmbH >> Maxfeldstr. 5, 90409 Nürnberg, Germany >> (HRB 36809, AG Nürnberg) >> Geschäftsführer: Felix Imendörffer >> > > -- Thomas Zimmermann Graphics Driver Developer SUSE Software Solutions Germany GmbH Maxfeldstr. 5, 90409 Nürnberg, Germany (HRB 36809, AG Nürnberg) Geschäftsführer: Felix Imendörffer [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 840 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Thomas Zimmermann <tzimmermann@suse.de> To: Daniel Vetter <daniel@ffwll.ch> Cc: hudson@trmm.net, markus@raatikainen.cc, peter@stuge.se, USB list <linux-usb@vger.kernel.org>, dri-devel <dri-devel@lists.freedesktop.org>, Tyler Hardin <th020394@gmail.com>, Lubomir Rintel <lkundrak@v3.sk>, pontus.fuchs@gmail.com, Sam Ravnborg <sam@ravnborg.org> Subject: Re: [PATCH v4 1/3] drm/uapi: Add USB connector type Date: Thu, 21 Jan 2021 11:01:11 +0100 [thread overview] Message-ID: <1ea4e6e4-0806-dba1-a424-47f178dc882f@suse.de> (raw) In-Reply-To: <CAKMK7uHiQ3i-Rz_y_3joR2Zi3fA=1qp8MdGZ9w9PUcGoWT3urw@mail.gmail.com> [-- Attachment #1.1.1: Type: text/plain, Size: 2870 bytes --] Hi Am 21.01.21 um 09:27 schrieb Daniel Vetter: > On Thu, Jan 21, 2021 at 8:45 AM Thomas Zimmermann <tzimmermann@suse.de> wrote: >> >> Hi Noralf, >> >> glad to hear from you! Welcome back! >> >> Am 20.01.21 um 18:42 schrieb Daniel Vetter: >>> On Wed, Jan 20, 2021 at 6:10 PM Noralf Trønnes <noralf@tronnes.org> wrote: >>>> >>>> Add a connector type for USB connected display panels. >>>> >>>> Signed-off-by: Noralf Trønnes <noralf@tronnes.org> >>>> --- >>>> include/uapi/drm/drm_mode.h | 1 + >>>> 1 file changed, 1 insertion(+) >>>> >>>> diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h >>>> index fed66a03c7ae..33024cc5d26e 100644 >>>> --- a/include/uapi/drm/drm_mode.h >>>> +++ b/include/uapi/drm/drm_mode.h >>>> @@ -367,6 +367,7 @@ enum drm_mode_subconnector { >>>> #define DRM_MODE_CONNECTOR_DPI 17 >>>> #define DRM_MODE_CONNECTOR_WRITEBACK 18 >>>> #define DRM_MODE_CONNECTOR_SPI 19 >>>> +#define DRM_MODE_CONNECTOR_USB 20 >> >> I would not call it USB. I could imagine that at some point a generic >> USB protocol could serve simple displays (i.e. in the sense of USB HID >> or data or imaging). (Maybe Thunderbold already counts.) Anyway, USB >> should be reserved for this case. > > We end up calling those DisplayPort, since that's what's being > transported over thunderbolt or usb-C. So the usb connector would be > called usb-C. I think the reason we don't do fancy connector names is > that adding them is a bit a pain. Plus drm/i915 specifically has some > very quirky connector enumerating that doesn't match much with reality > unfortunately anyway :-/ In the case of the other USB drivers, IIRC we use the connector type that is at the output (i.e., HDMI in the case of udl). I think we should do the same here. Or use 'Unknown'. Best regards Thomas > -Daniel > >> >> Best regards >> Thomas >> >>> >>> Beware, new connector types have in the past resulted in userspace >>> burning&crashing. Maybe it's become better ... >>> >>> Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch> >>>> >>>> /** >>>> * struct drm_mode_get_connector - Get connector metadata. >>>> -- >>>> 2.23.0 >>>> >>>> _______________________________________________ >>>> dri-devel mailing list >>>> dri-devel@lists.freedesktop.org >>>> https://lists.freedesktop.org/mailman/listinfo/dri-devel >>> >>> >>> >> >> -- >> Thomas Zimmermann >> Graphics Driver Developer >> SUSE Software Solutions Germany GmbH >> Maxfeldstr. 5, 90409 Nürnberg, Germany >> (HRB 36809, AG Nürnberg) >> Geschäftsführer: Felix Imendörffer >> > > -- Thomas Zimmermann Graphics Driver Developer SUSE Software Solutions Germany GmbH Maxfeldstr. 5, 90409 Nürnberg, Germany (HRB 36809, AG Nürnberg) Geschäftsführer: Felix Imendörffer [-- Attachment #1.2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 840 bytes --] [-- Attachment #2: Type: text/plain, Size: 160 bytes --] _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2021-01-21 10:02 UTC|newest] Thread overview: 53+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-01-20 17:00 [PATCH v4 0/3] Generic USB Display driver Noralf Trønnes 2021-01-20 17:00 ` Noralf Trønnes 2021-01-20 17:00 ` [PATCH v4 1/3] drm/uapi: Add USB connector type Noralf Trønnes 2021-01-20 17:00 ` Noralf Trønnes 2021-01-20 17:42 ` Daniel Vetter 2021-01-20 17:42 ` Daniel Vetter 2021-01-21 7:45 ` Thomas Zimmermann 2021-01-21 7:45 ` Thomas Zimmermann 2021-01-21 8:27 ` Daniel Vetter 2021-01-21 8:27 ` Daniel Vetter 2021-01-21 8:40 ` Simon Ser 2021-01-21 8:40 ` Simon Ser 2021-01-21 10:01 ` Thomas Zimmermann [this message] 2021-01-21 10:01 ` Thomas Zimmermann 2021-01-21 18:07 ` Noralf Trønnes 2021-01-21 18:07 ` Noralf Trønnes 2021-01-22 7:54 ` Thomas Zimmermann 2021-01-22 7:54 ` Thomas Zimmermann 2021-01-22 7:59 ` Thomas Zimmermann 2021-01-22 7:59 ` Thomas Zimmermann 2021-01-22 11:44 ` Noralf Trønnes 2021-01-22 11:44 ` Noralf Trønnes 2021-01-22 12:47 ` Thomas Zimmermann 2021-01-22 12:47 ` Thomas Zimmermann 2021-01-22 14:35 ` Noralf Trønnes 2021-01-22 14:35 ` Noralf Trønnes 2021-01-22 14:55 ` Thomas Zimmermann 2021-01-22 14:55 ` Thomas Zimmermann 2021-01-22 18:39 ` Noralf Trønnes 2021-01-22 18:39 ` Noralf Trønnes 2021-01-20 17:00 ` [PATCH v4 2/3] drm/probe-helper: Check epoch counter in output_poll_execute() Noralf Trønnes 2021-01-20 17:00 ` Noralf Trønnes 2021-01-20 17:41 ` Daniel Vetter 2021-01-20 17:41 ` Daniel Vetter 2021-01-20 17:00 ` [PATCH v4 3/3] drm: Add Generic USB Display driver Noralf Trønnes 2021-01-20 17:00 ` Noralf Trønnes 2021-01-20 18:02 ` Daniel Vetter 2021-01-24 16:17 ` Noralf Trønnes 2021-01-24 16:17 ` Noralf Trønnes 2021-01-21 9:59 ` [PATCH v4 0/3] " Thomas Zimmermann 2021-01-21 9:59 ` Thomas Zimmermann 2021-01-21 10:20 ` Simon Ser 2021-01-21 10:20 ` Simon Ser 2021-01-23 11:46 ` Noralf Trønnes 2021-01-23 11:46 ` Noralf Trønnes 2021-01-23 18:47 ` Lubomir Rintel 2021-01-23 18:47 ` Lubomir Rintel 2021-01-24 18:38 ` Lubomir Rintel 2021-01-24 18:38 ` Lubomir Rintel 2021-01-24 20:51 ` Noralf Trønnes 2021-01-24 20:51 ` Noralf Trønnes 2021-02-21 11:14 ` Noralf Trønnes 2021-02-21 11:14 ` Noralf Trønnes
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=1ea4e6e4-0806-dba1-a424-47f178dc882f@suse.de \ --to=tzimmermann@suse.de \ --cc=daniel@ffwll.ch \ --cc=dri-devel@lists.freedesktop.org \ --cc=hudson@trmm.net \ --cc=linux-usb@vger.kernel.org \ --cc=lkundrak@v3.sk \ --cc=markus@raatikainen.cc \ --cc=peter@stuge.se \ --cc=pontus.fuchs@gmail.com \ --cc=sam@ravnborg.org \ --cc=th020394@gmail.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: linkBe 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.