From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH] drm/i915/sdvo: Fix up debug output to not split lines Date: Wed, 27 Nov 2013 22:53:05 +0100 Message-ID: <20131127215305.GC27344@phenom.ffwll.local> References: <1385550619-31901-1-git-send-email-daniel.vetter@ffwll.ch> <1385564581-13015-1-git-send-email-daniel.vetter@ffwll.ch> <878uw9ooyb.fsf@gaia.fi.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-ea0-f174.google.com (mail-ea0-f174.google.com [209.85.215.174]) by gabe.freedesktop.org (Postfix) with ESMTP id DB220FBAF7 for ; Wed, 27 Nov 2013 13:52:22 -0800 (PST) Received: by mail-ea0-f174.google.com with SMTP id b10so5099833eae.5 for ; Wed, 27 Nov 2013 13:52:22 -0800 (PST) Content-Disposition: inline In-Reply-To: <878uw9ooyb.fsf@gaia.fi.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces@lists.freedesktop.org Errors-To: intel-gfx-bounces@lists.freedesktop.org To: Mika Kuoppala Cc: Daniel Vetter , Intel Graphics Development List-Id: intel-gfx@lists.freedesktop.org On Wed, Nov 27, 2013 at 05:15:56PM +0200, Mika Kuoppala wrote: > Daniel Vetter writes: > > > It leads to a big mess when stuff interleaves. Especially with the new > > patch I've submitted for the drm core to no longer artificially split > > up debug messages. > > > > v2: The size parameter to snprintf includes the terminating 0, but the > > return value does not. Adjust the logic accordingly. Spotted by Mika. > > > > Cc: Mika Kuoppala > > Signed-off-by: Daniel Vetter > > Reviewed-by: Mika Kuoppala Queued for -next, thanks for the review. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch