[PATCH v2 1/2] drm: bridge: adv7511: fix reading edid segments

Frieder Schrempf frieder.schrempf at kontron.de
Thu Nov 30 16:03:01 UTC 2023


On 30.11.23 16:57, Frieder Schrempf wrote:
> Hi Emil,
> 
> On 27.10.23 14:22, Emil Abildgaard Svendsen wrote:
>> [Sie erhalten nicht häufig E-Mails von emas at bang-olufsen.dk. Weitere Informationen, warum dies wichtig ist, finden Sie unter https://aka.ms/LearnAboutSenderIdentification ]
>>
>> Currently reading EDID only works because usually only two EDID blocks
>> of 128 bytes is used. Where an EDID segment holds 256 bytes or two EDID
>> blocks. And the first EDID segment read works fine but E-EDID specifies
>> up to 128 segments.
>>
>> The logic is broken so change EDID segment index to multiple of 256
>> bytes and not 128 (block size).
>>
>> Fixes: 9c8af882bf12 ("drm: Add adv7511 encoder driver")
>>
>> Signed-off-by: Emil Svendsen <emas at bang-olufsen.dk>
>> ---
>> v2:
>>  - Split into two patches.
>>  - Add Fixes tag.
>>
>> v1: https://lore.kernel.org/all/20231026113029.575846-1-emas@bang-olufsen.dk/
> 
> Unfortunately this v2 series breaks my setup in the same way as I
> reported for v1 [1].

To be more specific: It's actually patch 2/2 that breaks it. Patch 1/2
alone seems to work fine in my case.


More information about the dri-devel mailing list