Re: [PATCH v2 0/3] drm: Store USB device in struct drm_device

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi

Am 03.11.20 um 12:09 schrieb Hans de Goede:
> Hi,
> 
> On 11/3/20 11:36 AM, Thomas Zimmermann wrote:
>> The drivers gm12u320 and udl operate on USB devices. They leave the PCI
>> device in struct drm_device empty and store the USB device in their own
>> driver structure. It's expected that DRM core and helpers only touch the
>> PCI-device field for actual PCI devices.
>>
>> Fix this special case by upcasting struct drm_device.dev to the USB
>> device. The drivers' udev variables are being removed.
>>
>> v2:
>> 	* upcast USB device from struct drm_device.dev (Daniel)
>>
>> Thomas Zimmermann (3):
>>   drm: Add USB helpers
>>   drm/tiny/gm12u320: Retrieve USB device from struct drm_device.dev
>>   drm/udl: Retrieve USB device from struct drm_device.dev
> 
> Thanks.
> 
> The entire series looks good to me:
> 
> Reviewed-by: Hans de Goede <hdegoede@xxxxxxxxxx>

Thanks!

> 
> Note you may want to wait with pushing this to drm-misc until the
> first patch gets at least one other review.

Following Daniel's request, I'll drop the first patch and put the helper
into the drivers.

> 
> Or at least give others some time to possibly react :)

Sure, I'll merge it later this week.

Best regards
Thomas

> 
> Regards,
> 
> Hans
> 
> 
> 
> 
>>
>>  Documentation/gpu/drm-internals.rst |  5 +++
>>  drivers/gpu/drm/tiny/gm12u320.c     | 52 +++++++++++++----------------
>>  drivers/gpu/drm/udl/udl_connector.c |  9 ++---
>>  drivers/gpu/drm/udl/udl_drv.c       |  3 --
>>  drivers/gpu/drm/udl/udl_drv.h       |  1 -
>>  drivers/gpu/drm/udl/udl_main.c      | 25 ++++++++------
>>  include/drm/drm_usb_helper.h        | 25 ++++++++++++++
>>  7 files changed, 73 insertions(+), 47 deletions(-)
>>  create mode 100644 include/drm/drm_usb_helper.h
>>
>> --
>> 2.29.0
>>
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel@xxxxxxxxxxxxxxxxxxxxx
> 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

Attachment: OpenPGP_0x680DC11D530B7A23.asc
Description: application/pgp-keys

Attachment: OpenPGP_signature
Description: OpenPGP digital signature

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel

[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux