ADV7511/13 implementing atomic ops and bus formats

Jesse Van Gavere jesse.vangavere at scioteq.com
Thu Jan 2 07:18:09 UTC 2025


Hello Dmitry,

> The atomic_check() function verifies if the passed set of changes (in the form of drm_*_state) is valid from the driver's point of view. If you have nothing to check, it's fine to skip the function.

I'll look over the other examples a bit then but I think the reason for implementing it was I thought TIDSS seemed to expect it.

> I'd say, it's fine to ommit the output formats handling. From what I can see only dw-hdmi does sensible job of calculating output bus formats for HDMI Bus. The it66121 simply returns MEDIA_BUS_FMT_RGB888_1X24.

Yeah the output formats are also not well defined in the ADV75xx docs so I'll best leave that be for now.

> No, each bus (DSI, DPI, etc) have it's own data formats. This means 
> ADV7511 (D[35:0]), ADV7513 (D[23:0]) and ADV7533/35 (4x D-PHY lanes + clock lane) have different input formats.

...

> Just uapi patch with explanation and justification.

Got it, I'll make inventory of what's all out there first then and send it.

> --
> With best wishes
> Dmitry

Thank you very much for all the information, should really help in writing the first patches to implement this!

Best regards,
Jesse


More information about the dri-devel mailing list