On Wed, Jul 04, 2012 at 08:19:24PM -0300, Eugeni Dodonov wrote: > On 07/04/2012 02:41 PM, Paulo Zanoni wrote: > > 2012/6/28 Eugeni Dodonov <eugeni.dodonov at intel.com>: > >> This pollutes dmesg output even if we do not have FBC for the device, so > >> move the DRM_DEBUG_KMS statement lower. > >> > >> v2: just kill the message as suggested by Daniel. > >> > >> Signed-off-by: Eugeni Dodonov <eugeni.dodonov at intel.com> > > > > The message first says we "move the statement lower", but then later > > it says we kill the message :) > > With that fixed: > > > > Reviewed-by: Paulo Zanoni <paulo.r.zanoni at intel.com> > > > > I'm happy with both v1 and v2. > > I'll let Daniel decide the bike shed color on this one :). We can always > say that 'move it down' was a slang term for 'killing' later :). I've already merged it, but forgot to send out the ack mail ;-) -Daniel -- Daniel Vetter Mail: daniel at ffwll.ch Mobile: +41 (0)79 365 57 48
