Hi Joonyoung, 2015-01-30 Joonyoung Shim <jy0922.shim@xxxxxxxxxxx>: > Hi, > > On 01/23/2015 09:43 PM, Gustavo Padovan wrote: > > From: Daniel Kurtz <djkurtz@xxxxxxxxxxxx> > > > > The 'mode' is the modeline information which specifies the ideal mode > > requested by the mode set initiator (usually userspace). > > The 'adjusted_mode' is the actual hardware mode after all the crtcs > > and encoders have had a chance to "fix it up". > > > > The adjusted_mode starts as a duplicate of the mode in > > drm_crtc_helper_set_mode(), and gets modified as required. There is no > > reason to touch the original requested mode. > > > > Agree, but is there any side effect after this commit? Should we save > adjusted_mode to other variable and use it? I haven't seen any. Tested on peach pi and snow. Do we have any reason to save it now? I don't we have a user for it now. Gustavo _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel