On Tue, Apr 26, 2016 at 01:08:35PM -0400, Robert Foss wrote:
>
>
> On 04/26/2016 01:00 PM, Ville Syrjälä wrote:
> >On Tue, Apr 26, 2016 at 12:55:54PM -0400, Robert Foss wrote:
> >>
> >>
> >>On 04/26/2016 10:11 AM, Daniel Vetter wrote:
> >>>On Fri, Apr 22, 2016 at 05:25:00PM -0400, robert.foss@xxxxxxxxxxxxx wrote:
> >>>>From: Robert Foss <robert.foss@xxxxxxxxxxxxx>
> >>>>
> >>>>Add support for DRM_MODE_CONNECTOR_LVDS and DRM_MODE_CONNECTOR_DSI
> >>>>connectors.
> >>>>
> >>>>Signed-off-by: Robert Foss <robert.foss@xxxxxxxxxxxxx>
> >>>>---
> >>>> tests/kms_panel_fitting.c | 12 +++++++++---
> >>>> 1 file changed, 9 insertions(+), 3 deletions(-)
> >>>>
> >>>>diff --git a/tests/kms_panel_fitting.c b/tests/kms_panel_fitting.c
> >>>>index 60a22b0..933776c 100644
> >>>>--- a/tests/kms_panel_fitting.c
> >>>>+++ b/tests/kms_panel_fitting.c
> >>>>@@ -140,9 +140,15 @@ static void test_panel_fitting(data_t *d)
> >>>> for_each_connected_output(display, output) {
> >>>> drmModeModeInfo *mode, native_mode;
> >>>>
> >>>>- if (output->config.connector->connector_type !=
> >>>>- DRM_MODE_CONNECTOR_eDP)
> >>>>- continue;
> >>>>+ switch (output->config.connector->connector_type)
> >>>>+ {
> >>>>+ case DRM_MODE_CONNECTOR_LVDS:
> >>>>+ case DRM_MODE_CONNECTOR_eDP:
> >>>>+ case DRM_MODE_CONNECTOR_DSI:
> >>>>+ break;
> >>>>+ default:
> >>>>+ continue;
> >>>>+ }
> >>>
> >>>I think ville commented that we should instead check for "scaling mode",
> >>>and we discussed this on irc. Am I accidentally looking at an old patch
> >>>series?
> >>>-Daniel
> >>>
> >>No, this isn't an old patch, I misunderstood where our conversation ended.
> >>
> >>"scaling mode" is an intel specific extension as far as I understand it,
> >>which is somewhat the opposite of generalizing i-g-t tests.
> >>
> >>Is there an architecture independent way forward?
> >
> >A quick poke at cscope tells me "scaling mode" is used by amdgpu, gma500,
> >i915, nouveau, and radeon. Seems fairly independent to me.
> >
> If that is the case, and it seems to be, "scaling mode" is definitely
> worth pursuing.
Perhaps also send in a kernel patch to update the docbook table? It's
awful, but the best thing we have right now ...
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
