<div style="font-family: arial, helvetica, sans-serif; font-size: 10pt">Hi Jani,<div><br></div><div>Sorry for the delay, our branches have diverged significantly, and it took me quite a while to merge your changes.</div><div>
On the bright sight, I've tested the patch on SandyBridge LDVS and IvyBridge eDP, and it works fine.</div><div><br></div><div>Regards,</div><div>Yuly.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Oct 26, 2012 at 9:44 AM, Paulo Zanoni <span dir="ltr"><<a href="mailto:przanoni@gmail.com" target="_blank">przanoni@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi<br>
<br>
2012/10/26 Jani Nikula <<a href="mailto:jani.nikula@intel.com">jani.nikula@intel.com</a>>:<br>
<div class="im">> [Dropped lkml, added intel-gfx]<br>
><br>
> Hi Yuly, here's a slightly modified version of your patch, rebased on<br>
> drm-intel-next-queued. I kept your authorship, but any new errors are<br>
> totally mine...<br>
><br>
> These are compile tested only; I'd appreciate if you could check it<br>
> still does what it says on the box!<br>
<br>
</div>I have nothing to add or remove. Tested on HSW eDP, used "xrandr" to<br>
alternate the property values. Works fine.<br>
<br>
Being consistent on the default value between LVDS and eDP is<br>
certainly a nice thing.<br>
<br>
For the 3 patches:<br>
Reviewed-by: Paulo Zanoni <<a href="mailto:paulo.r.zanoni@intel.com">paulo.r.zanoni@intel.com</a>><br>
Tested-by: Paulo Zanoni <<a href="mailto:paulo.r.zanoni@intel.com">paulo.r.zanoni@intel.com</a>><br>
<div class="im"><br>
><br>
> BR,<br>
> Jani.<br>
><br>
><br>
> Jani Nikula (1):<br>
>   drm/i915/lvds: move fitting mode from intel_lvds_connector to<br>
>     intel_panel<br>
><br>
> Yuly Novikov (2):<br>
>   drm/i915/dp: allow configuring eDP panel fitting scaling mode<br>
>   drm/i915/dp: change eDP default scaling mode to respect aspect ratio<br>
><br>
>  drivers/gpu/drm/i915/intel_dp.c   |   31 ++++++++++++++++++++++++++++++-<br>
>  drivers/gpu/drm/i915/intel_drv.h  |    1 +<br>
>  drivers/gpu/drm/i915/intel_lvds.c |   24 ++++++++++--------------<br>
>  3 files changed, 41 insertions(+), 15 deletions(-)<br>
><br>
> --<br>
> 1.7.9.5<br>
><br>
</div>> _______________________________________________<br>
> Intel-gfx mailing list<br>
> <a href="mailto:Intel-gfx@lists.freedesktop.org">Intel-gfx@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/intel-gfx" target="_blank">http://lists.freedesktop.org/mailman/listinfo/intel-gfx</a><br>
<span class="HOEnZb"><font color="#888888"><br>
<br>
<br>
--<br>
Paulo Zanoni<br>
</font></span></blockquote></div><br></div></div>