On Fri, 08 Mar 2019, Robert Tarasov <[email protected]> wrote: > Fixed bug with reading of last EDID extra block in drm/udl driver. > Previouse approach read all the blocks except the last one. > > Signed-off-by: Robert Tarasov <[email protected]> > --- > drivers/gpu/drm/udl/udl_connector.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/udl/udl_connector.c > b/drivers/gpu/drm/udl/udl_connector.c > index c3dc1fd20cb4..68b221b9a01f 100644 > --- a/drivers/gpu/drm/udl/udl_connector.c > +++ b/drivers/gpu/drm/udl/udl_connector.c > @@ -70,7 +70,7 @@ static bool udl_get_edid(struct udl_device *udl, u8 > **result_buff, > memcpy(buff_ptr, block_buff, EDID_LENGTH); > kfree(block_buff); > buff_ptr += EDID_LENGTH; > - for (i = 1; i < extensions; ++i) { > + for (i = 1; i <= extensions; ++i) { > if (udl_get_edid_block(udl, i, buff_ptr)) { > buff_ptr += EDID_LENGTH; > } else {
Ugh. Why doesn't udl use drm_do_get_edid()? BR, Jani. -- Jani Nikula, Intel Open Source Graphics Center _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
