Hi,
On 11/3/20 12:30 PM, Thomas Zimmermann wrote:
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@redhat.com
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.
Ok, that works for me.
Or at least give others some time to possibly react :)
Sure, I'll merge it later this week.
My remark was mostly to give Daniel time to react...
BTW I missed Daniel's reaction again. Now I have figured out why though for some reason RH's email system is marking Daniels mails as spam, so they were in my spam folder ?
Regards,
Hans