Re: [PATCH] Add intel_panel_fitter tool

[Chris Wilson <chris@chris-wilson.co.uk>]

On Fri, 03 Feb 2012 10:39:18 -0800, Ben Widawsky <ben@bwidawsk.net> wrote:
> On 02/03/12 08:44, Paulo Zanoni wrote:
> > 2012/2/3 Chris Wilson<chris@chris-wilson.co.uk>:
> >> I don't this should an installable tool for use by end-users. Direct
> >> register access from userspace to registers that the driver owns? I'm
> >> wary of installing even root-only tools that can screw us over in
> >> unpredictable ways.
> >> -Chris
> >
> > My idea is to write a patch that adds kernel peroperties to solve
> > this, but while people still don't have kernel 3.4, the tool would be
> > useful.
> >
> > If adding this tool to i-g-t is not a good idea, that's fine too: I
> > still plan to write the kernel patches, so people wanting overscan
> > compensation will have to use the kernel patch instead of the tool.
> 
> I didn't read that email from Chris as don't add it. I read it as,
> don't have it installed if they do make install or something. So I
> think he should clarify himself. If this benefits end users now - I'm
> all for it.

As Ben said, I meant that I view this a debug/development tool and not
the final end-user solution. I think it is useful to distribute such
tools in i-g-t so that we can have testers, but I don't want it packaged
by distributions and used as the preferred/correct mode of operation.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre