Re: [PATCH v6 0/8] drm: zynqmp_dp: IRQ cleanups and debugfs support
From: Tomi Valkeinen
Date: Wed Oct 02 2024 - 10:53:10 EST
Hi,
On 01/10/2024 21:31, Sean Anderson wrote:
On 8/9/24 15:35, Sean Anderson wrote:
This series cleans up the zyqnmp_dp IRQ and locking situation. Once
that's done, it adds debugfs support. The intent is to enable compliance
testing or to help debug signal-integrity issues.
I think the patches 1-7 look fine, and I think I can pick those already
to drm-misc if you're ok with that.
I'm a bit unsure about patch 8, probably mainly because I don't have
experience with the compliance testing.
How have you tested this? With some DP analyzer/tester, I presume?
I think none of this (patch 8) is needed by almost anybody. Even among
zynqmp_dp developers I assume it's very rare to have the hardware for
this. I wonder if it would make sense to have the debugfs and related
code behind a compile option (which would be nice as the code wouldn't
even compiled in), or maybe a module parameter (which would be nice as
then "anyone" can easily enable it for compliance testing). What do you
think?
I also somehow recall that there was some discussion earlier about
how/if other drivers support compliance testing. But I can't find the
discussion. Do you remember if there was such discussion, and what was
the conclusion? With a quick look, everything in the debugfs looks
generic, not xilinx specific.
Tomi