From patchwork Mon Jan 11 19:18:46 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Lee Jones X-Patchwork-Id: 12011451 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-11.6 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id C7F13C433DB for ; Mon, 11 Jan 2021 19:19:33 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 8241A22CAD for ; Mon, 11 Jan 2021 19:19:33 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 8241A22CAD Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linaro.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=dri-devel-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 7EE5A89DBF; Mon, 11 Jan 2021 19:19:32 +0000 (UTC) Received: from mail-wm1-x334.google.com (mail-wm1-x334.google.com [IPv6:2a00:1450:4864:20::334]) by gabe.freedesktop.org (Postfix) with ESMTPS id D51AD89DC1 for ; Mon, 11 Jan 2021 19:19:31 +0000 (UTC) Received: by mail-wm1-x334.google.com with SMTP id k10so242894wmi.3 for ; Mon, 11 Jan 2021 11:19:31 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=PECDtmeFRTTbINpo1MBoxeyYk3sK6YMNX5ISdYlRSfY=; b=bEP+YEmqCeDN3RyNe6EivxTw/yjX7MVFIrNcAawNmgxyS0evh90c8B23+z33XT1SNe CG6s6PcQvz60ZI6zOkh0SPduCuUXuwbzu4781tlmRDvvTUjgQwPyf+d3/fuFjHNe1gdq jrq8YrTUAJ/B2Hfog0n0OrhJ9qhJ4tHTew2TeqHVLFy4Zb0qiA9FM0FVvopLPi9PRL2e HU4P2AkKZXbx2ojbWO5LEmQP4S6ZT3jhdMXSBHU/tcvJuZH1K/L0gG5oE+WFDXxIHSpP PZQ4zZzlseVjhtcdgVDxmNrW1hmgeOvAx+ubJgF4isRsuJ1GOMGXrAsde/7Zl9qUquic xREg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=PECDtmeFRTTbINpo1MBoxeyYk3sK6YMNX5ISdYlRSfY=; b=bFjgFbajkIEAKwFNGkAdUI9CSA2PDddotxr65C9I68AB6NUICz+P/iUwnm2vnaG0gv H1glI/Bo2TsL5mYxxflBO+ZH7AeBRrklK0of+Z4vFeMWGOo++BMdB8l6Bm9OQwdk0nwR biC07cIDCmpF9epNSMrYqblP7xxDZW7czAgO84xcPcwtqqAue69QYC3QmzgaeoGH1ZxG Efui0kvzvJsSl4oLfAJ6pNlO4oo7/VaxinSf03D5E3WbEnOx+1FHufGHLcEWj+iisk9d mOBiH+bm86a2nxKQUwkpkFtmol/9/6p1RdmDmXFqWYOBJZXA9iKu3EXRVbxwoLS2efZU o64w== X-Gm-Message-State: AOAM533i39nFQxGi81G7MqtSuzaHozSvW0VUbHwUQQlkjVNUXS/xYnbU a94AX2HhncL0n0W5xXNLOVdG3A== X-Google-Smtp-Source: ABdhPJxjofujK1MYN201ahii9IuZL5GFpIteWPNIpWPJ1RVAHK4iaqUKUikzOi6JRd0GzRb/ggmMuw== X-Received: by 2002:a7b:c306:: with SMTP id k6mr292835wmj.52.1610392770402; Mon, 11 Jan 2021 11:19:30 -0800 (PST) Received: from dell.default ([91.110.221.229]) by smtp.gmail.com with ESMTPSA id n3sm778090wrw.61.2021.01.11.11.19.29 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 11 Jan 2021 11:19:29 -0800 (PST) From: Lee Jones To: lee.jones@linaro.org Subject: [PATCH 00/40] [Set 12] Rid W=1 warnings from GPU Date: Mon, 11 Jan 2021 19:18:46 +0000 Message-Id: <20210111191926.3688443-1-lee.jones@linaro.org> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Xiaojian Du , David Airlie , Mauro Rossi , linux-kernel@vger.kernel.org, amd-gfx@lists.freedesktop.org, Josip Pavic , Leo Li , Aurabindo Pillai , Tony Cheng , dri-devel@lists.freedesktop.org, Alex Deucher , Colin Ian King , Anthony Koo , =?utf-8?q?Christian_K=C3=B6nig?= Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This set is part of a larger effort attempting to clean-up W=1 kernel builds, which are currently overwhelmingly riddled with niggly little warnings. Lee Jones (40): drm/amd/display/dc/dce/dce_aux: Mark 'dce_aux_transfer_raw' as __maybe_unused drm/amd/display/dc/dce/dce_link_encoder: Remove unused variable 'value0' drm/amd/display/dc/gpio/hw_ddc: Remove unused variable 'reg2' drm/amd/display/dc/dce/dce_opp: Demote non-compliant kernel-doc headers drm/amd/display/dc/dce/dce_transform: Demote kernel-doc abuse drm/amd/display/dc/gpio/diagnostics/hw_translate_diag: Include our own header containing prototypes drm/amd/display/dc/irq/irq_service: Make local function static drm/amd/display/dc/gpio/diagnostics/hw_factory_diag: Fix struct declared inside parameter list error drm/amd/display/dc/gpio/diagnostics/hw_factory_diag: Include our own header containing prototypes drm/amd/display/dc/dce120/dce120_hw_sequencer: Encompass defines in same clause as their use drm/amd/display/dc/dce120/dce120_timing_generator: drm/amd/display/dc/dce120/Makefile: Ignore -Woverride-init warning drm/amd/display/dc/calcs/dce_calcs: Move some large variables from the stack to the heap drm/amd/display/dc/calcs/dce_calcs: Remove some large variables from the stack drm/amd/display/dc/dce120/dce120_resource: Staticify local functions drm/amd/display/dc/dce120/dce120_timing_generator: Demote non-kerneldoc headers drm/amd/display/dc/dce/dce_aux: Remove duplicate line causing 'field overwritten' issue drm/amd/display/dc/dce112/Makefile: Ignore -Woverride-init warning drm/amd/display/dc/dce/dce_opp: Remove duplicate entries causing 'field overwritten' issues drm/amd/display/dc/dce110/dce110_timing_generator: Remove unused variable 'value_crtc_vtotal' drm/amd/display/dc/dce110/dce110_compressor: Remove unused function 'dce110_get_required_compressed_surfacesize drm/amd/display/dc/dce110/dce110_hw_sequencer: Demote non-conformant kernel-doc header drm/amd/display/dc/dce110/dce110_mem_input_v: Make local functions static drm/amd/pm/swsmu/smu11/vangogh_ppt: Make local function 'vangogh_clk_dpm_is_enabled' static drm/amd/display/dc/dce120/dce120_timing_generator: Remove unused function 'dce120_timing_generator_get_position' drm/amd/display/dc/dce110/dce110_timing_generator: Demote kernel-doc abuses to standard function headers drm/amd/display/dc/dce110/dce110_compressor: Strip out unused function 'controller_id_to_index' drm/amd/display/dc/calcs/dce_calcs: Demote non-conformant kernel-doc function headers drm/amd/display/dc/dce112/dce112_resource: Make local functions and ones called by reference static drm/amd/display/dc/dce110/dce110_timing_generator_v: Demote kernel-doc abuse and line up comments drm/amd/display/dc/dce110/dce110_mem_input_v: Include our own header, containing prototypes drm/amd/display/dc/dce110/Makefile: Ignore -Woverride-init warning drm/amd/display/dc/dce110/dce110_resource: Make local functions invoked by reference static drm/amd/display/dc/dce110/dce110_transform_v: Demote kernel-doc abuse drm/amd/display/dc/dce60/dce60_timing_generator: Make 'dce60_configure_crc' invoked by reference static drm/amd/display/dc/dce100/dce100_resource: Make local functions and ones called by reference static drm/amd/display/dc/dce60/dce60_resource: Make local functions static drm/amd/display/dc/dce80/dce80_resource: Make local functions static drm/amd/display/dc/core/dc_surface: Demote kernel-doc abuse drm/amd/display/dc/core/dc_stream: Demote non-conformant kernel-doc headers .../gpu/drm/amd/display/dc/calcs/dce_calcs.c | 1141 +++++++++-------- .../gpu/drm/amd/display/dc/core/dc_stream.c | 5 +- .../gpu/drm/amd/display/dc/core/dc_surface.c | 2 +- drivers/gpu/drm/amd/display/dc/dce/dce_aux.c | 2 +- drivers/gpu/drm/amd/display/dc/dce/dce_aux.h | 1 - .../drm/amd/display/dc/dce/dce_link_encoder.c | 3 +- drivers/gpu/drm/amd/display/dc/dce/dce_opp.c | 16 +- drivers/gpu/drm/amd/display/dc/dce/dce_opp.h | 2 - .../drm/amd/display/dc/dce/dce_transform.c | 2 +- .../amd/display/dc/dce100/dce100_resource.c | 16 +- .../gpu/drm/amd/display/dc/dce110/Makefile | 2 + .../amd/display/dc/dce110/dce110_compressor.c | 55 - .../display/dc/dce110/dce110_hw_sequencer.c | 2 +- .../display/dc/dce110/dce110_mem_input_v.c | 17 +- .../amd/display/dc/dce110/dce110_resource.c | 16 +- .../dc/dce110/dce110_timing_generator.c | 76 +- .../dc/dce110/dce110_timing_generator_v.c | 19 +- .../display/dc/dce110/dce110_transform_v.c | 19 +- .../gpu/drm/amd/display/dc/dce112/Makefile | 2 + .../amd/display/dc/dce112/dce112_resource.c | 16 +- .../gpu/drm/amd/display/dc/dce120/Makefile | 2 + .../display/dc/dce120/dce120_hw_sequencer.c | 2 +- .../amd/display/dc/dce120/dce120_resource.c | 6 +- .../dc/dce120/dce120_timing_generator.c | 99 +- .../drm/amd/display/dc/dce60/dce60_resource.c | 16 +- .../display/dc/dce60/dce60_timing_generator.c | 4 +- .../drm/amd/display/dc/dce80/dce80_resource.c | 16 +- .../dc/gpio/diagnostics/hw_factory_diag.c | 1 + .../dc/gpio/diagnostics/hw_factory_diag.h | 2 + .../dc/gpio/diagnostics/hw_translate_diag.c | 1 + drivers/gpu/drm/amd/display/dc/gpio/hw_ddc.c | 7 +- .../gpu/drm/amd/display/dc/irq/irq_service.c | 2 +- .../gpu/drm/amd/pm/swsmu/smu11/vangogh_ppt.c | 4 +- 33 files changed, 746 insertions(+), 830 deletions(-) Cc: Alex Deucher Cc: amd-gfx@lists.freedesktop.org Cc: Anthony Koo Cc: Aric Cyr Cc: Aric Cyr Cc: Aurabindo Pillai Cc: "Christian König" Cc: Colin Ian King Cc: Daniel Vetter Cc: David Airlie Cc: dri-devel@lists.freedesktop.org Cc: Harry Wentland Cc: Josip Pavic Cc: Lee Jones Cc: Leo Li Cc: Mauro Rossi Cc: Tony Cheng Cc: Xiaojian Du