Received: by 2002:a05:6358:11c7:b0:104:8066:f915 with SMTP id i7csp1167719rwl; Fri, 7 Apr 2023 10:58:32 -0700 (PDT) X-Google-Smtp-Source: AKy350afynoojJPRYklN62WePkuioBqELjMLrGfxQmttIReXBfdGmKGL/6VM4YlGyeCGguVFLaNv X-Received: by 2002:a17:90a:44:b0:23f:9448:89c2 with SMTP id 4-20020a17090a004400b0023f944889c2mr3368731pjb.7.1680890312262; Fri, 07 Apr 2023 10:58:32 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1680890312; cv=none; d=google.com; s=arc-20160816; b=XfIryFSx6rS/USd1m7vRKT1qK0LSov+hUW57rEw4ZUFWQMAFi5jx65JjoZwmgiPLdZ QiMiQoIy8WJIwO+5OpD9jmFxJZo73HkLxVnjggdw0R6g9C4Re42Pm6zgTY97p/fVpU8W G4zRFYdx/UiYM9rO6A+Vl/rMokZRkSVYrd49Sx0P1exlSIKOIGNl256bM4vxFmMIKMfh NJNUmQUoH4fci0QJDyvlfsB3XC8ND7nwNVTWwgNbtLxnhXMUakZe5tDx5g+0Su4/bZrH qwGVOV7pitC5ENJB6cTnWWv+ssQW2scgoq1SQbDq7icPwhgRVNCFfC41OnMx2f5HMRw0 cAkA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:cc:to:subject :message-id:date:from:in-reply-to:references:mime-version :dkim-signature; bh=KYBbdfYMz/vjcXkNGYT26LJaYJDJYJgOQo589rbtS2s=; b=kw0/ZZhRTVkTRF5rMNM1bEIGA3QK9uwMcTKo4tTIq24BWkZKYqtXL4HqV+UHLYCuUy 6c3m6uUNeb5dxyci7CgQUesvpP+TAsr9aobiJxeYRkvW7zOXwcgZNC7YVFuBhR7Qx3XX efWVrEbF8ziWN7XJYpoFN/OUhSdrLr2w83xMTcfTDuNES9Bl+KP9JiAEFKd70AJBOCsp Rp/Zy/+xHlvIlnANcS1sA34KYxRLT0YhuF6SQtKjg4Y/DyOXJJzLUTzo5YuZo0JOHmVg mpKTHvfEB8vWvCGPlhb9WIgnLMfBdQn5Gr12rokjeirWl5g1QPeNcK5f95hX0So12l3b Enug== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@google.com header.s=20210112 header.b=DqKYdsHf; 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=REJECT sp=REJECT dis=NONE) header.from=google.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id u5-20020a170902e80500b001a11cf69a4dsi151884plg.226.2023.04.07.10.58.19; Fri, 07 Apr 2023 10:58:32 -0700 (PDT) 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=@google.com header.s=20210112 header.b=DqKYdsHf; 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=REJECT sp=REJECT dis=NONE) header.from=google.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229589AbjDGRxG (ORCPT + 99 others); Fri, 7 Apr 2023 13:53:06 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60652 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229656AbjDGRxC (ORCPT ); Fri, 7 Apr 2023 13:53:02 -0400 Received: from mail-pf1-x430.google.com (mail-pf1-x430.google.com [IPv6:2607:f8b0:4864:20::430]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0BD725597 for ; Fri, 7 Apr 2023 10:53:01 -0700 (PDT) Received: by mail-pf1-x430.google.com with SMTP id y2so26947949pfw.9 for ; Fri, 07 Apr 2023 10:53:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; t=1680889980; x=1683481980; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=KYBbdfYMz/vjcXkNGYT26LJaYJDJYJgOQo589rbtS2s=; b=DqKYdsHfDx8zdLJVjdTiTFTABvHWdnCrcEmv1w5HVb7vZJgYb9t4uDuX/Cr9FthsY+ sNlQf5412FR4vITwMaeTaoKN8vXOBxK5I09sKXDtFLOCDcucYYJqntHqjO3K/idXejnx mpjwZtwsKYITG3OUusv3MCa9zgiWgd4qcE93WnOc5b19hJYSRsJv9ND+hyeP42g88eNa 4M5aWWIG7t2lc67vCrkzFbzHfaMpEH2orbYitLkGcLjlPGiRbePJKxB3XQzuoavzhYGq dWt2ylU++nmF7XJzcfQQHXSf1nOsa4/z1PObYwy+i3A5Vf3jwiWGU3HMphpiMtUKlzUK V5Qg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1680889980; x=1683481980; h=content-transfer-encoding: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=KYBbdfYMz/vjcXkNGYT26LJaYJDJYJgOQo589rbtS2s=; b=OcgXBvVe7Kd/gv6bT1mpUFMYgwXoZKEeH2BadAqnPGUX6lttzh2D74ayeUEG/ZPaCO kWHMzj667ZiAWm6me+HmbsmiFv8Rr2qleuLlKla6aMMD5ghd+TqHCpNHZviDDOjYSQYQ 3SYayPoNaD/t7pDzvBNKQ7MxJFLxZVR2yPC5ULpiWt83XvrKQYGNjJdJagGb2cPPJdyK ttGsD0DWpHRlR/xMeDC4OmNPA8by95HkryVMtQatCqY1IDTAS2bR5gCB284JGVUNCunE pOPKqIwR/PhAXf4RKEWHBmxtV9iV5O0k+XC3HFyvSLlqk7MwQvgBhDMAAqNuFGlJTwA4 N8/A== X-Gm-Message-State: AAQBX9cZAGnZLp2hdzM14n3/qn86KHF9XU21r88SIDY4mCLjt+7PoAPo k3O7rfzrwjgBUfNiDCC663e7cyeRh1Dbjm6tVD01/g== X-Received: by 2002:a65:5b43:0:b0:50f:5f89:2a9d with SMTP id y3-20020a655b43000000b0050f5f892a9dmr671939pgr.1.1680889980187; Fri, 07 Apr 2023 10:53:00 -0700 (PDT) MIME-Version: 1.0 References: <20230325134503.1335510-1-trix@redhat.com> In-Reply-To: <20230325134503.1335510-1-trix@redhat.com> From: Nick Desaulniers Date: Fri, 7 Apr 2023 10:52:49 -0700 Message-ID: Subject: Re: [PATCH] drm/amd/display: remove unused matching_stream_ptrs variable To: Jimmy.Kizito@amd.com Cc: harry.wentland@amd.com, 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, nathan@kernel.org, Jun.Lei@amd.com, wenjing.liu@amd.com, Cruise.Hung@amd.com, amd-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, llvm@lists.linux.dev, Tom Rix Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Spam-Status: No, score=-15.7 required=5.0 tests=DKIMWL_WL_MED,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,ENV_AND_HDR_SPF_MATCH, RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS,T_PDS_OTHER_BAD_TLD, USER_IN_DEF_DKIM_WL,USER_IN_DEF_SPF_WL autolearn=unavailable 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 Jimmy, can you review? The change LGTM; but I'm not sure if there was something else intended here= . On Sat, Mar 25, 2023 at 6:45=E2=80=AFAM Tom Rix wrote: > > clang with W=3D1 reports > drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_link_enc_cfg.c:625:6: er= ror: > variable 'matching_stream_ptrs' set but not used [-Werror,-Wunused-but-= set-variable] > int matching_stream_ptrs =3D 0; > ^ > This variable is not used so remove it. > > Signed-off-by: Tom Rix > --- > drivers/gpu/drm/amd/display/dc/core/dc_link_enc_cfg.c | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_link_enc_cfg.c b/driv= ers/gpu/drm/amd/display/dc/core/dc_link_enc_cfg.c > index 41198c729d90..30c0644d4418 100644 > --- a/drivers/gpu/drm/amd/display/dc/core/dc_link_enc_cfg.c > +++ b/drivers/gpu/drm/amd/display/dc/core/dc_link_enc_cfg.c > @@ -622,7 +622,6 @@ bool link_enc_cfg_validate(struct dc *dc, struct dc_s= tate *state) > int i, j; > uint8_t valid_count =3D 0; > uint8_t dig_stream_count =3D 0; > - int matching_stream_ptrs =3D 0; > int eng_ids_per_ep_id[MAX_PIPES] =3D {0}; > int ep_ids_per_eng_id[MAX_PIPES] =3D {0}; > int valid_bitmap =3D 0; > @@ -645,9 +644,7 @@ bool link_enc_cfg_validate(struct dc *dc, struct dc_s= tate *state) > struct link_enc_assignment assignment =3D state->res_ctx.= link_enc_cfg_ctx.link_enc_assignments[i]; > > if (assignment.valid) { > - if (assignment.stream =3D=3D state->streams[i]) > - matching_stream_ptrs++; > - else > + if (assignment.stream !=3D state->streams[i]) > valid_stream_ptrs =3D false; > } > } > -- > 2.27.0 > --=20 Thanks, ~Nick Desaulniers