Re: [PATCH v3 1/2] drm/panel: simple: Add Waveshare 13.3" panel support

[Marek Vasut <marek.vasut@mailbox.org>]

On 1/20/26 10:33 AM, Geert Uytterhoeven wrote:

Hello Geert,

>>> That looks like the plain standard 60Hz Full HD mode.
>>> Is there really no other copy of that structure available in the kernel?
>>>
>>> auo_t215hvn01_mode is almost the same, except for .clock and .flags.
>>> drivers/video/fbdev/core/modedb.c has the same mode, but in a different
>>> structure.
>>
>> The panel-simple.c is full of similar-ish panel timings . The timings
>> above are adapted from the waveshare DTO for this panel. What else would
>> you suggest I do/use for this device ?
> 
> I am just wondering if there is a better way for panels that use
> standard timings.

I sent a support request to waveshare to get hw details, but I am not 
holding my breath.

I don't think this panel should use standard timings in the first place, 
it should use timings which match the panel, but those are unknown, 
because the content of the metal case is unknown. For now, the timings 
come from the waveshare DTO, which only by coincidence matches the 
standard timings, but that may have to be updated in the future, at 
which point the use of standard timings would have to be removed anyway.

I don't think we should press for standard timings for this particular 
panel. For other panels, the timings come from their EDID/DPCD/... so 
that is already a solved problem.