[PATCH] drm/udl: Refactor edid retreiving in UDL driver

Robert Tarasov tutankhamen at chromium.org
Wed Mar 13 18:36:56 UTC 2019


On Wed, Mar 13, 2019 at 2:26 AM Jani Nikula <jani.nikula at linux.intel.com>
wrote:

> You'll also get support for debugfs and firmware loader EDID override
> mechanisms for free.
>

Yep :)


> Signed-off-by missing!
>

Fixed.

 read_buff = kmalloc(2, GFP_KERNEL);
>
> A follow-up cleanup might be to switch to using "u8 read_buff[2];"
> instead of kmallocing it.
>
> I don't claim to understand how the usb stuff works, but otherwise the
> patch looks good to me. Nice refactoring!
>
>
"The buffer passed to usb_control_msg may end up in scatter-gather list,
and may thus not be on the stack. Having it on the stack usually works on
x86,   but not on other archs. "
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20190313/1da9ae0a/attachment-0001.html>


More information about the dri-devel mailing list