Not strictly related to this patch.

On Thu, 26 Nov 2015, [email protected] wrote:
> The .get_config() hooks should not reference anything in crtc->config,
> everything should be based on the passed in pipe_config instead.

This is the kind of authoritative comment that I would appreciate seeing
in the kernel-doc for the hooks in struct intel_encoder. We have
practically nothing of the sort. We still seem to rely more on oral
tradition than actual documentation. (Not that there's anything wrong
with oral tradition per se, it's just that we don't regularly sit around
campfires reciting encoder hook rules or make folk songs about them.)

BR,
Jani.


-- 
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to