From patchwork Mon Nov 5 23:30:52 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jordan Crouse X-Patchwork-Id: 10669425 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 7BD94175A for ; Mon, 5 Nov 2018 23:31:16 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 719782A00C for ; Mon, 5 Nov 2018 23:31:16 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 6E63D29C98; Mon, 5 Nov 2018 23:31:16 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.2 required=2.0 tests=BAYES_00,MAILING_LIST_MULTI, RCVD_IN_DNSWL_MED autolearn=ham version=3.3.1 Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 144D82A00C for ; Mon, 5 Nov 2018 23:31:16 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 2058A6E238; Mon, 5 Nov 2018 23:31:13 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from smtp.codeaurora.org (smtp.codeaurora.org [198.145.29.96]) by gabe.freedesktop.org (Postfix) with ESMTPS id E903C6E1E6; Mon, 5 Nov 2018 23:31:11 +0000 (UTC) Received: by smtp.codeaurora.org (Postfix, from userid 1000) id A81A661653; Mon, 5 Nov 2018 23:31:08 +0000 (UTC) Received: from jcrouse-lnx.qualcomm.com (i-global254.qualcomm.com [199.106.103.254]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: jcrouse@smtp.codeaurora.org) by smtp.codeaurora.org (Postfix) with ESMTPSA id B68466163D; Mon, 5 Nov 2018 23:31:06 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org B68466163D From: Jordan Crouse To: freedreno@lists.freedesktop.org Subject: [PATCH v2 00/11] DPU cleanups Date: Mon, 5 Nov 2018 16:30:52 -0700 Message-Id: <20181105233103.7657-1-jcrouse@codeaurora.org> X-Mailer: git-send-email 2.18.0 X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org, bzwang@chromium.org, seanpaul@chromium.org, abhinavk@codeaurora.org, sam@ravnborg.org MIME-Version: 1.0 Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" X-Virus-Scanned: ClamAV using ClamSMTP I've been working on various methods to automate code cleanup strategies and I'm using dpu as my guinea pig. I started out by trying to identify unused or lightly static inline functions and then that morphed to very small functions in general and that then identified a few general areas of cleanup such as debugfs init/destroy and the dpu_hw_blk_init users. Changes in v2: Addressed review comments (thanks!) and added even more debugfs cleanup Jordan Crouse (11): drm/msm/dpu: Remove dpu_dbg drm/msm/dpu: Use DEFINE_SHOW_ATTRIBUTE drm/msm/dpu: Remove dpu_crtc_get_mixer_height drm/msm/dpu: Remove dpu_crtc_is_enabled() drm/msm/dpu: Remove unused functions drm/msm/dpu: Cleanup callers of dpu_hw_blk_init drm/msm: Make irq_postinstall optional drm/msm/dpu: Remove dpu_irq and unused functions drm/msm/dpu: Cleanup the debugfs functions drm/msm/dpu: Further cleanups for static inline functions drm/msm/dpu: Clean up dpu_media_info.h static inline functions drivers/gpu/drm/msm/Makefile | 4 +- drivers/gpu/drm/msm/disp/dpu1/dpu_core_irq.c | 59 +- drivers/gpu/drm/msm/disp/dpu1/dpu_core_irq.h | 16 +- drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c | 122 +- drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.h | 7 +- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 76 +- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.h | 32 - drivers/gpu/drm/msm/disp/dpu1/dpu_dbg.c | 2393 ----------------- drivers/gpu/drm/msm/disp/dpu1/dpu_dbg.h | 103 - drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 55 +- .../gpu/drm/msm/disp/dpu1/dpu_encoder_phys.h | 2 +- .../drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c | 12 +- .../drm/msm/disp/dpu1/dpu_encoder_phys_vid.c | 1 - drivers/gpu/drm/msm/disp/dpu1/dpu_hw_blk.c | 10 +- drivers/gpu/drm/msm/disp/dpu1/dpu_hw_blk.h | 2 +- .../gpu/drm/msm/disp/dpu1/dpu_hw_catalog.h | 9 - drivers/gpu/drm/msm/disp/dpu1/dpu_hw_ctl.c | 18 +- drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.c | 18 +- drivers/gpu/drm/msm/disp/dpu1/dpu_hw_intf.h | 10 - drivers/gpu/drm/msm/disp/dpu1/dpu_hw_lm.c | 24 +- drivers/gpu/drm/msm/disp/dpu1/dpu_hw_lm.h | 5 - .../gpu/drm/msm/disp/dpu1/dpu_hw_pingpong.c | 18 +- .../gpu/drm/msm/disp/dpu1/dpu_hw_pingpong.h | 10 - drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.c | 21 +- drivers/gpu/drm/msm/disp/dpu1/dpu_hw_sspp.h | 10 - drivers/gpu/drm/msm/disp/dpu1/dpu_hw_top.c | 20 +- drivers/gpu/drm/msm/disp/dpu1/dpu_hw_top.h | 10 - drivers/gpu/drm/msm/disp/dpu1/dpu_hw_vbif.c | 1 - drivers/gpu/drm/msm/disp/dpu1/dpu_irq.c | 66 - drivers/gpu/drm/msm/disp/dpu1/dpu_irq.h | 59 - drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 188 +- drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h | 8 - drivers/gpu/drm/msm/disp/dpu1/dpu_mdss.c | 8 +- drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c | 108 +- drivers/gpu/drm/msm/disp/dpu1/dpu_vbif.c | 24 +- drivers/gpu/drm/msm/disp/dpu1/dpu_vbif.h | 15 +- .../gpu/drm/msm/disp/dpu1/msm_media_info.h | 164 +- drivers/gpu/drm/msm/msm_drv.c | 6 +- 38 files changed, 235 insertions(+), 3479 deletions(-) delete mode 100644 drivers/gpu/drm/msm/disp/dpu1/dpu_dbg.c delete mode 100644 drivers/gpu/drm/msm/disp/dpu1/dpu_dbg.h delete mode 100644 drivers/gpu/drm/msm/disp/dpu1/dpu_irq.c delete mode 100644 drivers/gpu/drm/msm/disp/dpu1/dpu_irq.h