[igt-dev] [PATCH i-g-t 3/3] kms/test: Pass correct pipe value to print pipe name
Jani Nikula
jani.nikula at linux.intel.com
Tue Mar 10 08:20:33 UTC 2020
On Tue, 10 Mar 2020, "Khajapasha, Mohammed" <mohammed.khajapasha at intel.com> wrote:
> Are you saying, we should get rid of pipe enum and make it array index
> based? Is my understanding correct? At present, every IGT subtest
> iterates over pipe list using "enum pipe" and this is getting passed
> down.
I am saying do not conflate enum pipe and crtc index.
If you call something a pipe, it should be the pipe, not crtc index. And
vice versa.
It gets really confusing really quick if you call something a pipe, and
then go on to convert it into a pipe, as if it wasn't a pipe already.
In i915 we've removed all assumptions that pipe == crtc index. So should
igt.
BR,
Jani.
--
Jani Nikula, Intel Open Source Graphics Center
More information about the igt-dev
mailing list