<div dir="ltr">I've already submitted the new one.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Mar 12, 2019 at 1:02 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">On Mon, 11 Mar 2019, Robert Tarasov <<a href="mailto:tutankhamen@chromium.org" target="_blank">tutankhamen@chromium.org</a>> wrote:<br>
> Yes, you're right. Will prepare another patch which will use<br>
> drm_do_get_edid()<br>
<br>
Oh, you might want to have this patch first with cc: stable and all<br>
that, and do the cleanup afterwards.<br>
<br>
BR,<br>
Jani.<br>
<br>
<br>
><br>
> On Mon, Mar 11, 2019 at 4:33 AM Jani Nikula <<a href="mailto:jani.nikula@linux.intel.com" target="_blank">jani.nikula@linux.intel.com</a>><br>
> wrote:<br>
><br>
>> On Fri, 08 Mar 2019, Robert Tarasov <<a href="mailto:tutankhamen@chromium.org" target="_blank">tutankhamen@chromium.org</a>> wrote:<br>
>> > Fixed bug with reading of last EDID extra block in drm/udl driver.<br>
>> > Previouse approach read all the blocks except the last one.<br>
>> ><br>
>> > Signed-off-by: Robert Tarasov <<a href="mailto:tutankhamen@chromium.org" target="_blank">tutankhamen@chromium.org</a>><br>
>> > ---<br>
>> >  drivers/gpu/drm/udl/udl_connector.c | 2 +-<br>
>> >  1 file changed, 1 insertion(+), 1 deletion(-)<br>
>> ><br>
>> > diff --git a/drivers/gpu/drm/udl/udl_connector.c<br>
>> b/drivers/gpu/drm/udl/udl_connector.c<br>
>> > index c3dc1fd20cb4..68b221b9a01f 100644<br>
>> > --- a/drivers/gpu/drm/udl/udl_connector.c<br>
>> > +++ b/drivers/gpu/drm/udl/udl_connector.c<br>
>> > @@ -70,7 +70,7 @@ static bool udl_get_edid(struct udl_device *udl, u8<br>
>> **result_buff,<br>
>> >                       memcpy(buff_ptr, block_buff, EDID_LENGTH);<br>
>> >                       kfree(block_buff);<br>
>> >                       buff_ptr += EDID_LENGTH;<br>
>> > -                     for (i = 1; i < extensions; ++i) {<br>
>> > +                     for (i = 1; i <= extensions; ++i) {<br>
>> >                               if (udl_get_edid_block(udl, i, buff_ptr)) {<br>
>> >                                       buff_ptr += EDID_LENGTH;<br>
>> >                               } else {<br>
>><br>
>> Ugh. Why doesn't udl use drm_do_get_edid()?<br>
>><br>
>> BR,<br>
>> Jani.<br>
>><br>
>><br>
>> --<br>
>> Jani Nikula, Intel Open Source Graphics Center<br>
>><br>
<br>
-- <br>
Jani Nikula, Intel Open Source Graphics Center<br>
</blockquote></div>