Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752207AbbG1QSt (ORCPT ); Tue, 28 Jul 2015 12:18:49 -0400 Received: from mail.fireflyinternet.com ([87.106.93.118]:52996 "EHLO fireflyinternet.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751599AbbG1QSs (ORCPT ); Tue, 28 Jul 2015 12:18:48 -0400 X-Default-Received-SPF: pass (skip=forwardok (res=PASS)) x-ip-name=78.156.65.138; Date: Tue, 28 Jul 2015 17:18:31 +0100 From: Chris Wilson To: Benjamin Tissoires Cc: Daniel Vetter , intel-gfx@lists.freedesktop.org, Todd Broch , linux-kernel@vger.kernel.org Subject: Re: [Intel-gfx] [PATCH 2/3] drm/i915: hide errors when probing for a reverse display port Message-ID: <20150728161831.GF16528@nuc-i3427.alporthouse.com> Mail-Followup-To: Chris Wilson , Benjamin Tissoires , Daniel Vetter , intel-gfx@lists.freedesktop.org, Todd Broch , linux-kernel@vger.kernel.org References: <1438099409-25456-1-git-send-email-benjamin.tissoires@redhat.com> <1438099409-25456-3-git-send-email-benjamin.tissoires@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1438099409-25456-3-git-send-email-benjamin.tissoires@redhat.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 947 Lines: 23 On Tue, Jul 28, 2015 at 12:03:28PM -0400, Benjamin Tissoires wrote: > We check the polarity of the attached dp, so it is normal to fail. > Do not send errors to the users. if (probe) DRM_DEBUG else DRM_ERROR is fairly offensive. It strikes me that you could make each of these functions report the failure to the caller (have the caller even do so error handling!) and as part of that have the caller report an error and so demote all of these to DRM_DEBUG_KMS(). Who knows, actually doing some error handling may make monitor training more reliable! Or we may even get carried away and report the failure all the way back to userspace. -Chris -- Chris Wilson, Intel Open Source Technology Centre -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/