On Fri, 03 Jan 2020, Lee Shawn C <[email protected]> wrote:
> Run intel_dp_compliance would failed at video pattern related
> test case sometimes. DP test applet read incorrect test type
> from kernel to cause this symptom. Add a "\n" (newline) in
> seq_printf() then DP test applet will get proper parameters.
>
> Cc: Manasi Navare <[email protected]>
> Cc: Jani Nikula <[email protected]>
> Cc: Daniel Vetter <[email protected]>
> Cc: Ville Syrjala <[email protected]>
> Cc: Cooper Chiou <[email protected]>
> Signed-off-by: Lee Shawn C <[email protected]>
Fixes: eb3394faeb97 ("drm/i915: Add debugfs test control files for Displayport
compliance testing")
Reviewed-by: Jani Nikula <[email protected]>
> ---
> drivers/gpu/drm/i915/i915_debugfs.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c
> b/drivers/gpu/drm/i915/i915_debugfs.c
> index 0ac98e39eb75..74180158a909 100644
> --- a/drivers/gpu/drm/i915/i915_debugfs.c
> +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> @@ -3167,7 +3167,7 @@ static int i915_displayport_test_data_show(struct
> seq_file *m, void *data)
> intel_dp = enc_to_intel_dp(&encoder->base);
> if (intel_dp->compliance.test_type ==
> DP_TEST_LINK_EDID_READ)
> - seq_printf(m, "%lx",
> + seq_printf(m, "%lx\n",
> intel_dp->compliance.test_data.edid);
> else if (intel_dp->compliance.test_type ==
> DP_TEST_LINK_VIDEO_PATTERN) {
> @@ -3209,7 +3209,7 @@ static int i915_displayport_test_type_show(struct
> seq_file *m, void *data)
>
> if (encoder && connector->status == connector_status_connected)
> {
> intel_dp = enc_to_intel_dp(&encoder->base);
> - seq_printf(m, "%02lx", intel_dp->compliance.test_type);
> + seq_printf(m, "%02lx\n",
> intel_dp->compliance.test_type);
> } else
> seq_puts(m, "0");
> }
--
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
