Am Mittwoch, den 30.05.2012, 13:52 +0200 schrieb Daniel Vetter: Typo in ?change? in the commit message. > We should only frob adjusted_mode. This is in preparation of > a massive patch by Laurent Pinchart to make the mode argument > const. > > After the prevous prep patch to use adjusted_mode->clock instead of previous > mode->clock the only thing left is to clean up things a bit. I've > opted to pass in an adjust_mode param to dp_adjust_dithering because > that way we can be sure to avoid duplicating this logic - which was > the cause behind a dp link bw calculation bug in the past. > > Also mark the mode argument of pch_panel_fitting const. > > v2: Split up the mode->clock => adjusted_mode->clock change, > as suggested by Chris Wilson. > > Reported-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com> > Signed-off-by: Daniel Vetter <daniel.vetter at ffwll.ch> > --- > drivers/gpu/drm/i915/intel_dp.c | 12 ++++++------ > drivers/gpu/drm/i915/intel_drv.h | 2 +- > drivers/gpu/drm/i915/intel_panel.c | 2 +- > 3 files changed, 8 insertions(+), 8 deletions(-) [?] Thanks, Paul -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 198 bytes Desc: This is a digitally signed message part URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20120530/cce1bdbc/attachment.pgp>