Hi Jani,

Sorry for the delay, our branches have diverged significantly, and it took
me quite a while to merge your changes.
On the bright sight, I've tested the patch on SandyBridge LDVS and
IvyBridge eDP, and it works fine.

Regards,
Yuly.


On Fri, Oct 26, 2012 at 9:44 AM, Paulo Zanoni <przan...@gmail.com> wrote:

> Hi
>
> 2012/10/26 Jani Nikula <jani.nik...@intel.com>:
> > [Dropped lkml, added intel-gfx]
> >
> > Hi Yuly, here's a slightly modified version of your patch, rebased on
> > drm-intel-next-queued. I kept your authorship, but any new errors are
> > totally mine...
> >
> > These are compile tested only; I'd appreciate if you could check it
> > still does what it says on the box!
>
> I have nothing to add or remove. Tested on HSW eDP, used "xrandr" to
> alternate the property values. Works fine.
>
> Being consistent on the default value between LVDS and eDP is
> certainly a nice thing.
>
> For the 3 patches:
> Reviewed-by: Paulo Zanoni <paulo.r.zan...@intel.com>
> Tested-by: Paulo Zanoni <paulo.r.zan...@intel.com>
>
> >
> > BR,
> > Jani.
> >
> >
> > Jani Nikula (1):
> >   drm/i915/lvds: move fitting mode from intel_lvds_connector to
> >     intel_panel
> >
> > Yuly Novikov (2):
> >   drm/i915/dp: allow configuring eDP panel fitting scaling mode
> >   drm/i915/dp: change eDP default scaling mode to respect aspect ratio
> >
> >  drivers/gpu/drm/i915/intel_dp.c   |   31 ++++++++++++++++++++++++++++++-
> >  drivers/gpu/drm/i915/intel_drv.h  |    1 +
> >  drivers/gpu/drm/i915/intel_lvds.c |   24 ++++++++++--------------
> >  3 files changed, 41 insertions(+), 15 deletions(-)
> >
> > --
> > 1.7.9.5
> >
> > _______________________________________________
> > Intel-gfx mailing list
> > Intel-gfx@lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
>
>
> --
> Paulo Zanoni
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to