<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Mar 13, 2019 at 2:26 AM Jani Nikula <<a href="mailto:jani.nikula@linux.intel.com">jani.nikula@linux.intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">You'll also get support for debugfs and firmware loader EDID override<br>
mechanisms for free.<br></blockquote><div><br></div><div>Yep :) </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Signed-off-by missing!<br></blockquote><div><br></div><div>Fixed.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> read_buff = kmalloc(2, GFP_KERNEL);<br>
<br>
A follow-up cleanup might be to switch to using "u8 read_buff[2];"<br>
instead of kmallocing it.<br>
<br>
I don't claim to understand how the usb stuff works, but otherwise the<br>
patch looks good to me. Nice refactoring!<br><br></blockquote><div><br></div><span style="font-size:13px">"The buffer passed to usb_control_msg may end up in scatter-gather list, and </span><span style="font-size:13px">may thus not be on the stack. Having it on the stack usually works on x86,   </span><span style="font-size:13px">but </span><span style="font-size:13px">not on other archs.</span><span style="font-size:13px"> "</span></div></div>