From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751618AbdJXN7d (ORCPT ); Tue, 24 Oct 2017 09:59:33 -0400 Received: from mout.web.de ([212.227.17.12]:56910 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750897AbdJXN7a (ORCPT ); Tue, 24 Oct 2017 09:59:30 -0400 Subject: [PATCH 1/2] drm/i915/dp: Delete an unnecessary goto statement in intel_dp_sink_crc() From: SF Markus Elfring To: dri-devel@lists.freedesktop.org, intel-gfx@lists.freedesktop.org, David Airlie , Jani Nikula , Joonas Lahtinen , Rodrigo Vivi Cc: LKML , kernel-janitors@vger.kernel.org References: <668a4b56-091d-d4e7-2204-daaec6675e11@users.sourceforge.net> Message-ID: Date: Tue, 24 Oct 2017 15:59:17 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: <668a4b56-091d-d4e7-2204-daaec6675e11@users.sourceforge.net> Content-Type: text/plain; charset=utf-8 Content-Language: en-GB Content-Transfer-Encoding: 8bit X-Provags-ID: V03:K0:/fQJ+TaGB1flD2HL99eNIjO1/asL6RYgoTI5JiFY9314HcwE+qU UhFMs4hLHX4tOM+TQTrHpVvk5rjo5jAVCP198KsW/4QFn1ghWgd0l4MWzpN3kG1vbLgTs0u Z2vuSszpwXbDt6eubzU6bQkkaKHIHBcLlPQxuQGofPjIXQ5FlHDfpCh60HVn2DuImF4bdDE q04qTcZh5mkuKAfYO5Tjg== X-UI-Out-Filterresults: notjunk:1;V01:K0:zLw6wF5s+fA=:9Dfq73gaoCHR6QnjdfDXfy f8OyG8TtFeoczI2mHN+G8U5cIXACMF/MoPyMJPF73soXBgdvacjxO9sGq7bwbaDnkg18VZagf G/6qCCUo9WMYKlbuiGj9n7mpcqt+b43Y2Z3m4veP9qvm9N633IC3aTNDBwGzqr81+QxbB983g /klaPtE6V9rZVV//b8kVUwQOglaFmm2TxxphFMj231q5PfO9UkN+/mbSFQq8jofoxfOadDu3N 7lyJy2lPoxuax0UQqL3mwbTTuE9iOBgfycTyauLVlp54kyrrXej3to7KAhnpmyXhTRZOO4/tI ju4pqcAu9VJxraH9etFgDCptdmNFbTPFSRS7oNFtMNk5Qo+IXmRllCRastxIQkANia8cGdduQ g4xPMXJvEXk90guXk/Geb/o14N3I70UnflNcos2NhTCimdkyFnn95ekQkp94/J/Sg+1rBsQrP MgrkI+ndLVAf/tkUJ3hAyd4RRLMJPlkRr+Mk2Vky90N6pGOkdJz6GxpNX8+9+i4BiFAvgVuv/ F7PV45oNYN43mnZShcT9h1xcW2j52+XqUrVQSrjddIaSKZGJK/yg1Ax7OaFA5pmeflJCzgMpQ sbGb+/Ps/wrt8cD53z201Qbwa/4TTuDGHsrD2UnrKdfs8722FN7Dh54f+bLh/IiDwuNCYfaEv Hw2mXwWc/OOh/Q2TgYfqlraQ5RLV1EUcOy/AXzbj7vpdVzbopS4u3dJ+A7fUlscLzLDX+lRpv /0r1Ak+vAOZZmqp4s9dhPyL2rQZt3I4jxUH01fVAGMMLsWe0wiwgEvCewdPQyGf86Cfp76YH/ VTQotLbfixVxR3UzIKN8UPswiyrvbTPWMJoedFO3hSmidOTZcg= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Markus Elfring Date: Tue, 24 Oct 2017 15:15:20 +0200 A jump was specified for a location which was directly behind. Thus remove such an unnecessary goto statement. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/gpu/drm/i915/intel_dp.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c index ca48bce23a6f..342f1a1fa085 100644 --- a/drivers/gpu/drm/i915/intel_dp.c +++ b/drivers/gpu/drm/i915/intel_dp.c @@ -3994,10 +3994,8 @@ int intel_dp_sink_crc(struct intel_dp *intel_dp, u8 *crc) goto stop; } - if (drm_dp_dpcd_read(&intel_dp->aux, DP_TEST_CRC_R_CR, crc, 6) < 0) { + if (drm_dp_dpcd_read(&intel_dp->aux, DP_TEST_CRC_R_CR, crc, 6) < 0) ret = -EIO; - goto stop; - } stop: intel_dp_sink_crc_stop(intel_dp); -- 2.14.3