On Thu, 2021-01-14 at 11:14 +0200, Pekka Paalanen wrote:
Hi,
please document somewhere that ends up in git history (commit message, code comments, description of the parameter would be the best but maybe there isn't enough space?) what Christian König explained in
https://lists.freedesktop.org/archives/dri-devel/2020-December/291254.html
that this is a stop-gap feature intended to be removed as soon as possible (when a better solution comes up, which could be years).
So far I have not seen a single mention of this intention in your patch submissions, and I think it is very important to make known.
Hi,
Thanks for the headsup, I shall add the relevant info in the next verison.
I also did not see an explanation of why this instead of manufacturing these video modes in userspace (an idea mentioned by Christian in the referenced email). I think that too should be part of a commit message.
This is an opt-in feature, which shall be superseded by a better solution. We also add a set of common modes for scaling similarly. Userspace can still add whatever mode they want. So I dont see a reason why this cant be in the kernel.
--
Regards, Aurabindo Pillai
Thanks, pq