Received: by 2002:a05:6358:16cc:b0:ea:6187:17c9 with SMTP id r12csp2529903rwl; Fri, 6 Jan 2023 07:39:24 -0800 (PST) X-Google-Smtp-Source: AMrXdXtNPuvdjjhVWdMWiAS3RaiYTdR4fX4TDyily/so7/hINam+COM9TEbbK1drXKgb8limLVPB X-Received: by 2002:a17:907:ca14:b0:84d:b49:8afd with SMTP id uk20-20020a170907ca1400b0084d0b498afdmr5967545ejc.75.1673019564390; Fri, 06 Jan 2023 07:39:24 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1673019564; cv=none; d=google.com; s=arc-20160816; b=wzRmdvyvnuZL3rTugYF+9u74o4Sww++7oFr3XxpBAHHaE3mMMwmQb9u5UN/fjrzZZ6 l3RjIlHBq7oAQRFCgpJETquFOB4H8xvYHc3wyZxo6n/dpuvExsmPoDbU2yX27aVjmk11 Ua/EyVbqL9XujRDH2DGuln5GGDvz2TkZzGK6EhUVYljqdCBnvOUs7VXa9WpFVFI6DY/Z wnhyLtPfIKk5WkgLY5P4skguaygkRBSjOATmJ5qJZqu5+4+OVSrjKNzdKLhZJtZcdCml XAjYTtYL5G26MkOcbrqx3LyShhFNzR7nfYy+vsk79Xn6UYVPJGnKqiVQ7oeEgmwM6TVw rETw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:in-reply-to :references:mime-version:dkim-signature; bh=ab3tdx57EQZ6xsAca1KbY5FPLARshJ6+T7eu3+BdUuU=; b=rVBAaviJEjOcM3eqCD/E5pKbFtdl+FY3j5dxNDEeAGq4e0AdnSHCAj4T8OoXPxdHYF KpZcDPMazAFGFbzSaVYyKgwbXoLnThoDv4HNETRspLGdUbjDhhC1Gyj79E3vl5Llju+K i9hwbL1LZ6A+R3Bac7LaVDcVhzW4MOJz0lMkQzIdbnpLvuyG/U8X2mb+L+J41lH/ueBY 4NAPp1q0d3tVuZ+RiYAppF4zJpSKdzjkNm8OaoYiQ42R4Grnjsx2fwlGV2FohL/HPv85 sMBmUb6Do+6+deTi45B1NxY9/ikx1rRza0yhAxLxiv0GY7mK/85MHXZzy5nIa2HGYDtf WsAQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=R+nbt7Dk; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id sg41-20020a170907a42900b007dbea9fb8dasi1618204ejc.641.2023.01.06.07.39.11; Fri, 06 Jan 2023 07:39:24 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=R+nbt7Dk; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233380AbjAFP2Z (ORCPT + 54 others); Fri, 6 Jan 2023 10:28:25 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42506 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235017AbjAFP2V (ORCPT ); Fri, 6 Jan 2023 10:28:21 -0500 Received: from mail-oi1-x236.google.com (mail-oi1-x236.google.com [IPv6:2607:f8b0:4864:20::236]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C32F21A820 for ; Fri, 6 Jan 2023 07:28:20 -0800 (PST) Received: by mail-oi1-x236.google.com with SMTP id s187so1292777oie.10 for ; Fri, 06 Jan 2023 07:28:20 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=ab3tdx57EQZ6xsAca1KbY5FPLARshJ6+T7eu3+BdUuU=; b=R+nbt7DkHgGcPeOelWU5aKN10vnIzExJ73cYyrBcZxDumkoPwVyZ/LBRIBi1dFqYEO HmMMo5aKtfNiebY/OmMpDt6rcED3/51Yw4d3ylCuH3xxhyjpkGMMD09mfgx7o99FTFIG 91Etl5cKDGMMG8ny63WurR6tOfI6w3Ilc/Tlj7dEILx0ZkBH8Gdm2wQVR3+OlAKBZ3CM 16UZK3RKtAHlAy00yvJvNl4puQNR3KJg+4jrdYoSmuWXcnk31joQLUlc20h5D8G+aCix ENOz8Cr3k1JuVzer9pBDUWCtmquWIk5ZgYlBuLQs3+uOlo2Hjhj3V9+Wal6NTeA/a2qJ YcLw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=ab3tdx57EQZ6xsAca1KbY5FPLARshJ6+T7eu3+BdUuU=; b=ScZN4ls72DujlotsCyvUlEGtw42hd/l/socUEHjwbKzB4XUlrIVLB37z9daj6hT79/ uULPaldOyuG7+R+O5KvbaP+xM0CjYsZ498MjIKAB5DeLqlrspTWbwUHmnixx6ctfRIvS fBjpQsh2m/hFLVTbwb6Tbe6u+pnxH3xiwH6MQgVtfKfDxQQToisLEIbUbR8hT2/5HPHW 5ClwhaUHhJaNTreDmH347TpXqbvbx/t35raNbUFHXlpLqVBX4kwMbWuy7oFih/r3AeDr 2TfL8hCeILDkt5E0TlAPtkZblLV4kdVY7yzC3wjglaisDHpRyNMw1IyaG1TzZc3fwGw3 bFow== X-Gm-Message-State: AFqh2kqbJiqFiOd2Rr5aOIw9yWcpBID9QLDd6qHDUdamgorHkPtWWXa7 mvd6bwbuXtY7CC8CXO7euZo8p+pNXybhJcDNX58= X-Received: by 2002:a05:6808:2994:b0:35b:f5f7:3ed0 with SMTP id ex20-20020a056808299400b0035bf5f73ed0mr3738323oib.46.1673018900122; Fri, 06 Jan 2023 07:28:20 -0800 (PST) MIME-Version: 1.0 References: <20221216102318.197994-1-yiyang13@huawei.com> <1800775a-5788-b60b-87cb-9b09c7e7c177@amd.com> In-Reply-To: <1800775a-5788-b60b-87cb-9b09c7e7c177@amd.com> From: Alex Deucher Date: Fri, 6 Jan 2023 10:28:07 -0500 Message-ID: Subject: Re: [PATCH -next] drm/amd/display: Remove redundant assignment to variable dc To: Harry Wentland Cc: Yi Yang , sunpeng.li@amd.com, Rodrigo.Siqueira@amd.com, alexander.deucher@amd.com, christian.koenig@amd.com, Xinhui.Pan@amd.com, airlied@gmail.com, daniel@ffwll.ch, dri-devel@lists.freedesktop.org, amd-gfx@lists.freedesktop.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset="UTF-8" X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM, RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Applied. Thanks! Alex On Thu, Jan 5, 2023 at 2:20 PM Harry Wentland wrote: > > On 12/16/22 05:23, Yi Yang wrote: > > Smatch report warning as follows: > > > > Line 53679: drivers/gpu/drm/amd/display/dc/core/dc_stream.c:402 > > dc_stream_set_cursor_position() warn: variable dereferenced before > > check 'stream' > > > > The value of 'dc' has been assigned after check whether 'stream' is > > NULL. Fix it by remove redundant assignment. > > > > Signed-off-by: Yi Yang > > If this didn't blow up until now we might not even need > the NULL check below, but either way this is correct and > > Reviewed-by: Harry Wentland > > Harry > > > --- > > drivers/gpu/drm/amd/display/dc/core/dc_stream.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_stream.c b/drivers/gpu/drm/amd/display/dc/core/dc_stream.c > > index 20e534f73513..78d31bb875d1 100644 > > --- a/drivers/gpu/drm/amd/display/dc/core/dc_stream.c > > +++ b/drivers/gpu/drm/amd/display/dc/core/dc_stream.c > > @@ -408,7 +408,7 @@ bool dc_stream_set_cursor_position( > > struct dc_stream_state *stream, > > const struct dc_cursor_position *position) > > { > > - struct dc *dc = stream->ctx->dc; > > + struct dc *dc; > > bool reset_idle_optimizations = false; > > > > if (NULL == stream) { >