From patchwork Sun Dec 30 17:48:24 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sam Ravnborg X-Patchwork-Id: 10745145 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 6292313A4 for ; Sun, 30 Dec 2018 17:48:58 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 45129285A0 for ; Sun, 30 Dec 2018 17:48:58 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 38F21285A8; Sun, 30 Dec 2018 17:48:58 +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 775EE285A0 for ; Sun, 30 Dec 2018 17:48:57 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id B3C478984D; Sun, 30 Dec 2018 17:48:54 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from mail-lj1-x243.google.com (mail-lj1-x243.google.com [IPv6:2a00:1450:4864:20::243]) by gabe.freedesktop.org (Postfix) with ESMTPS id A56478984D for ; Sun, 30 Dec 2018 17:48:53 +0000 (UTC) Received: by mail-lj1-x243.google.com with SMTP id l15-v6so22320390lja.9 for ; Sun, 30 Dec 2018 09:48:53 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:from:to:cc:subject:date:message-id; bh=dFF8xyr2Vaw57wDmm/6wV3EIINyAuCDmINUT4XSQNjA=; b=tVt+ngobheLcpARf9JsBCnuEANc9Z04q5JgYDoWFp7tn5pUseRev1xlwAYaEOq0N5k XWyFpPkc/zoBBluGaA07/ZSeLVpBFVnQPzsChyv2qP/+rHL2+p2IWGDJMsbGgGNHl4s7 UZR7lVQD+9W4FL8n7R+glUCerug3xH5rHM2j3nf5XMC8Pzh8vdtOlAzT81Cf2mR9h32D oocG0MafR3RnzxljEYaS0J6R/qJwRGRxHMD1vVHCvTG1Psv5yAfOqS8SNQR906jhLYJS ey0JHe/IF4iEa9m5E4LNerBahnoATtjHidfMS0Kbt1B1fmQ1uEFFP8i9C5igNCsqQc9y F+aQ== X-Gm-Message-State: AJcUukd2LYBbbtTLhv1w5kf6N+wBOU3dGU0TGLhqRN9Dk4uYQyKJmGiu RyhPJXoDtUXGAAJX2xdZgKA= X-Google-Smtp-Source: ALg8bN73+yJZffcnZzjqiwwxyOz7qH4iziUcbJw4xOytqubUWD5QIAs71MhMki1F+/gxvchhDzdTsw== X-Received: by 2002:a2e:5c07:: with SMTP id q7-v6mr21256582ljb.119.1546192131690; Sun, 30 Dec 2018 09:48:51 -0800 (PST) Received: from jupiter.lan (18.158-248-194.customer.lyse.net. [158.248.194.18]) by smtp.gmail.com with ESMTPSA id q11sm8853170lfc.92.2018.12.30.09.48.50 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sun, 30 Dec 2018 09:48:51 -0800 (PST) From: Sam Ravnborg To: Daniel Vetter , David Airlie , Maarten Lankhorst , Maxime Ripard , Sean Paul , David Lechner , Laurent Pinchart , linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org Subject: [PATCH v2 0/14] drm: minimize drmP.h dependencies Date: Sun, 30 Dec 2018 18:48:24 +0100 Message-Id: <20181230174838.32330-1-sam@ravnborg.org> X-Mailer: git-send-email 2.12.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: Sam Ravnborg MIME-Version: 1.0 Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" X-Virus-Scanned: ClamAV using ClamSMTP The goal with this small series is to remove the last bits that prevents us from removing drmP.h driver-by-driver or file by file. - drmP.h is now stripped down to include files and forward declarations. - All header files in include/drm/ no longer include drmP.h. The series was made on top of drm-misc-next Build tested using arm and x86 all{yes,mod}config The patches are trivial but touches a lot of files, so a lot of people on cc: for the individual patches. I expect the full series to be applied to drm-misc-next There are still ~700 uses of drmP.h so a long way ahead of us to get rid of it. (The last mail said 900 but that was a typo) The kernel-doc parts that I authored in the following may require some extra info as I know too little about DRM to be able to write something really useful. Or at least review by someone with knowledge of DRM, but that goes for everything. v2: - DRM_SWITCH_POWER as enum (Daniel Vetter) - Prefer forward decalration over includes (Laurent Pinchart) - Updated drm_device to use kerneldoc style (Daniel Vetter) - Improved commit messages (David Lechner) - Split up patch when removing drmP.h from drm_gem_cma_helper.h - rebased on top of drm-misc-next - dropped patch already applied - added reviewed-by from Laurent Pinchart (Laurent Pinchart) - add drm_framebuffer.h patch - add kernel-doc comments to drm_util.h - moved EXPORT_SYMBOL_FOR_TESTS_ONLY to drm_util.h - added note to drmP.h not to add new stuff and not to use in new files Sam Sam Ravnborg (14): drm: drm_device.h: update comments to kerneldoc style drm: move DRM_SWITCH_POWER defines to drm_device.h drm: make drm_framebuffer.h self contained drm: make drm_file.h self contained drm: move drm_can_sleep() to drm_util.h drm: add kernel-doc comments to drm_util.h drm: move EXPORT_SYMBOL_FOR_TESTS_ONLY to drm_util.h drm: remove include of drmP.h from bridge/dw_hdmi.h drm: remove include of drmP.h from drm_encoder_slave.h drm: remove include of drmP.h from drm_modeset_helper.h drm/arc: do not reply on drmP.h from drm_gem_cma_helper.h drm/stm: do not reply on drmP.h from drm_gem_cma_helper.h drm/tinydrm: do not reply on drmP.h from drm_gem_cma_helper.h drm: remove drmP.h from drm_gem_cma_helper.h drivers/gpu/drm/amd/amdgpu/atom.c | 2 + drivers/gpu/drm/arc/arcpgu_crtc.c | 2 + drivers/gpu/drm/arc/arcpgu_drv.c | 6 + drivers/gpu/drm/arc/arcpgu_sim.c | 1 + drivers/gpu/drm/ast/ast_fb.c | 2 + drivers/gpu/drm/bridge/cdns-dsi.c | 2 + .../gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c | 4 + drivers/gpu/drm/cirrus/cirrus_fbdev.c | 1 + drivers/gpu/drm/drm_flip_work.c | 1 + drivers/gpu/drm/drm_framebuffer.c | 1 + drivers/gpu/drm/drm_modeset_helper.c | 2 + drivers/gpu/drm/mgag200/mgag200_fb.c | 1 + drivers/gpu/drm/msm/disp/mdp5/mdp5_smp.c | 1 + drivers/gpu/drm/omapdrm/omap_fbdev.c | 1 + drivers/gpu/drm/qxl/qxl_cmd.c | 2 + drivers/gpu/drm/radeon/atom.c | 2 + drivers/gpu/drm/radeon/radeon_legacy_encoders.c | 1 + drivers/gpu/drm/rcar-du/rcar_dw_hdmi.c | 1 + drivers/gpu/drm/rcar-du/rcar_lvds.c | 1 + drivers/gpu/drm/stm/drv.c | 6 +- drivers/gpu/drm/stm/ltdc.c | 9 +- drivers/gpu/drm/tinydrm/core/tinydrm-core.c | 3 + drivers/gpu/drm/tinydrm/core/tinydrm-pipe.c | 3 + drivers/gpu/drm/tinydrm/hx8357d.c | 1 + drivers/gpu/drm/tinydrm/ili9225.c | 2 + drivers/gpu/drm/tinydrm/ili9341.c | 1 + drivers/gpu/drm/tinydrm/mi0283qt.c | 1 + drivers/gpu/drm/tinydrm/mipi-dbi.c | 3 + drivers/gpu/drm/tinydrm/repaper.c | 1 + drivers/gpu/drm/tinydrm/st7586.c | 1 + drivers/gpu/drm/tinydrm/st7735r.c | 1 + drivers/gpu/drm/vc4/vc4_drv.h | 1 + include/drm/bridge/dw_hdmi.h | 6 +- include/drm/drmP.h | 24 +-- include/drm/drm_device.h | 228 +++++++++++++++------ include/drm/drm_encoder_slave.h | 1 - include/drm/drm_file.h | 1 + include/drm/drm_framebuffer.h | 5 +- include/drm/drm_gem_cma_helper.h | 5 +- include/drm/drm_modeset_helper.h | 6 +- include/drm/drm_util.h | 49 ++++- 41 files changed, 299 insertions(+), 93 deletions(-)