From patchwork Sun May 26 17:35:28 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sam Ravnborg X-Patchwork-Id: 10961831 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 EAE4F6C5 for ; Sun, 26 May 2019 17:35:51 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id D87AD28A7E for ; Sun, 26 May 2019 17:35:51 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id CCB7528A95; Sun, 26 May 2019 17:35:51 +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=unavailable 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 6FA3C28A7E for ; Sun, 26 May 2019 17:35:51 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 9A40B89247; Sun, 26 May 2019 17:35:46 +0000 (UTC) X-Original-To: intel-gfx@lists.freedesktop.org Delivered-To: intel-gfx@lists.freedesktop.org Received: from mail-lf1-x144.google.com (mail-lf1-x144.google.com [IPv6:2a00:1450:4864:20::144]) by gabe.freedesktop.org (Postfix) with ESMTPS id C4BBF89247; Sun, 26 May 2019 17:35:44 +0000 (UTC) Received: by mail-lf1-x144.google.com with SMTP id u27so10420462lfg.10; Sun, 26 May 2019 10:35:44 -0700 (PDT) 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 :mime-version:content-transfer-encoding; bh=Kidib+d4bYJPnpha2/4NzS+7WnpA00ibIeDKlPx2bIY=; b=pV35EgCSH3M4kHH/IGbxXcjjAGotpZoz2VIe3oT9liw55dHaDXl0b8Bz5LQm4DOl2Z 9J+GRw1SqPMTmOg+6x1ZJeByP6urz2vN8o256pgFwbeUqLcVN+YMR6td+VAGCrl60VWy nCRAEPG+KW4GTSfPvfAMmcsH4NqpvIqb7Uk7avr2TEsZ0SzTKH3WV+VfzqDaU3A5js1k ZSmeij0tCXUYuh0GGnerLwvwvG3FZ0rYqPt6PGFuYlXBdRiGHkl/2d3qtoXU0x/SQbN0 v3zIopDJ+jJzxUGb+P2VDwQmAmQDIbWjQPV8UDzSodO5UEnldCoe7zK4XHKED9zpbEYg PqMA== X-Gm-Message-State: APjAAAVClDRBeqxBbx7LJX/k9OkzuAkwlQGg2y/W2lN+IhJMmp8DTsXI CgtOw2N0VSbaA6nmTQgkOeTiOKVr X-Google-Smtp-Source: APXvYqzTfotnnd84M1t+oMWbqEcP5uobZ723fOew59GuJHMa/p5cfTODQSVypVcM8YiNO9CIEbk6Fg== X-Received: by 2002:a19:a887:: with SMTP id r129mr22883059lfe.16.1558892142679; Sun, 26 May 2019 10:35:42 -0700 (PDT) Received: from saturn.lan (18.158-248-194.customer.lyse.net. [158.248.194.18]) by smtp.gmail.com with ESMTPSA id n9sm1774489ljj.10.2019.05.26.10.35.41 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sun, 26 May 2019 10:35:41 -0700 (PDT) From: Sam Ravnborg To: dri-devel@lists.freedesktop.org Date: Sun, 26 May 2019 19:35:28 +0200 Message-Id: <20190526173535.32701-1-sam@ravnborg.org> X-Mailer: git-send-email 2.20.1 MIME-Version: 1.0 X-Mailman-Original-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=Kidib+d4bYJPnpha2/4NzS+7WnpA00ibIeDKlPx2bIY=; b=t6S32PxUsxkl6x7cJrY2D0MhK/glcNWNeSGWwmY8gWZAUM+dWww45YEiX1IMFVDDYa bF0eOjm7KPMLVOUnmYY0GerNfT+NgCtwjJqJe5x7tdV6SeTuXsDm6BnG67cv/nOo3Ilq VApYQ9Hynq7eTQp8EV+RGCkO0/lTTX1STsV6wUw7fNdUxU5OcXCL2EDL7GcBmhYOKkgG Z2J5p8j73wA2uYO0mYuoBXaRWuGo+AVXO+x10KURjw+x8O7epLfT0pnhMpySqTxOHW54 3blkWR6dhPuXO+Hr+7wk9JE9wq5mehy1isKZDgj3O/MihED1L4Efs3gTRZoMFF3OYSDT g9gw== Subject: [Intel-gfx] [PATCH v2 0/7] drm: make headers self-contained and drop drmP.h X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Maxime Ripard , intel-gfx@lists.freedesktop.org, David Airlie Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" X-Virus-Scanned: ClamAV using ClamSMTP While removing use of drmP.h from files in drm/* I noticed that I had to add the same include files due to dependencies in the header files. It is better to let the header files be self-contained and let the users pull in only the additional headers files required. So I went ahead and made the relevant header files self-contained. (I did not check if this made any includes redundant in some files, I do not have tooling in place to do so). Daniel suggested to add support for testing that they stay self contained. Jani Nikula has sent a patch to kbuild to make this part of the kbuild machinery. I have used it locally and as soon as it lands in kbuild I will start using it for drm. We could have duplicated the infrastructure now but that seemed too much code chrunch. This patchset include the actual removal of drmP.h as one big patch. This is build tested on alpha (always interesting), arm, arm64, x86 etc. For all files touched the following was done: - include files divided up in blocks in following order: linux/* video/* drm/* "" - within each block the include files are sorted alphabetically v2: - use same ordering af blocks - move includes down below license text - added patch with actual drmP.h removal - reworded some subjects to make them more descriptive - fixed a few spelling erros in changelogs (but a few may remain) Sam Sam Ravnborg (7): drm: make drm/drm_auth.h self contained drm: make drm/drm_legacy.h self-contained drm: make drm_crtc_internal.h self-contained drm: make drm_internal.h self-contained drm: make drm_legacy.h self-contained drm: make drm_trace.h self-contained drm: drop use of drmP.h in drm/* drivers/gpu/drm/ati_pcigart.c | 5 ++++- drivers/gpu/drm/drm_agpsupport.c | 11 +++++++++-- drivers/gpu/drm/drm_atomic.c | 9 +++++++-- drivers/gpu/drm/drm_atomic_helper.c | 11 +++++++---- drivers/gpu/drm/drm_atomic_state_helper.c | 7 ++++--- drivers/gpu/drm/drm_auth.c | 10 ++++++++-- drivers/gpu/drm/drm_blend.c | 9 ++++++--- drivers/gpu/drm/drm_bufs.c | 21 ++++++++++++++++----- drivers/gpu/drm/drm_client.c | 2 +- drivers/gpu/drm/drm_color_mgmt.c | 8 ++++++-- drivers/gpu/drm/drm_context.c | 8 +++++++- drivers/gpu/drm/drm_crtc_helper.c | 14 ++++++++------ drivers/gpu/drm/drm_crtc_internal.h | 24 ++++++++++++++++++++---- drivers/gpu/drm/drm_debugfs.c | 13 ++++++++----- drivers/gpu/drm/drm_debugfs_crc.c | 9 ++++++++- drivers/gpu/drm/drm_dma.c | 6 +++++- drivers/gpu/drm/drm_drv.c | 9 ++++++--- drivers/gpu/drm/drm_dumb_buffers.c | 4 +++- drivers/gpu/drm/drm_encoder.c | 4 +++- drivers/gpu/drm/drm_fb_helper.c | 19 ++++++++++++------- drivers/gpu/drm/drm_file.c | 11 +++++++---- drivers/gpu/drm/drm_flip_work.c | 6 ++++-- drivers/gpu/drm/drm_fourcc.c | 2 +- drivers/gpu/drm/drm_framebuffer.c | 13 +++++++++---- drivers/gpu/drm/drm_gem.c | 8 ++++++-- drivers/gpu/drm/drm_gem_cma_helper.c | 11 ++++++----- drivers/gpu/drm/drm_gem_framebuffer_helper.c | 1 - drivers/gpu/drm/drm_hashtab.c | 10 +++++++--- drivers/gpu/drm/drm_internal.h | 10 +++++++++- drivers/gpu/drm/drm_ioc32.c | 9 ++++++--- drivers/gpu/drm/drm_ioctl.c | 22 ++++++++++++++-------- drivers/gpu/drm/drm_irq.c | 13 +++++++++---- drivers/gpu/drm/drm_kms_helper_common.c | 3 ++- drivers/gpu/drm/drm_lease.c | 15 ++++++++++----- drivers/gpu/drm/drm_legacy.h | 4 ++++ drivers/gpu/drm/drm_legacy_misc.c | 7 ++++++- drivers/gpu/drm/drm_lock.c | 8 ++++++-- drivers/gpu/drm/drm_memory.c | 9 +++++++-- drivers/gpu/drm/drm_mm.c | 9 +++++---- drivers/gpu/drm/drm_mode_config.c | 6 +++++- drivers/gpu/drm/drm_mode_object.c | 9 +++++++-- drivers/gpu/drm/drm_modes.c | 7 +++++-- drivers/gpu/drm/drm_modeset_lock.c | 2 +- drivers/gpu/drm/drm_of.c | 5 +++-- drivers/gpu/drm/drm_pci.c | 11 ++++++++--- drivers/gpu/drm/drm_plane_helper.c | 9 +++++---- drivers/gpu/drm/drm_prime.c | 7 +++++-- drivers/gpu/drm/drm_print.c | 7 ++++++- drivers/gpu/drm/drm_probe_helper.c | 7 ++++--- drivers/gpu/drm/drm_property.c | 7 ++++++- drivers/gpu/drm/drm_rect.c | 4 +++- drivers/gpu/drm/drm_scatter.c | 9 +++++++-- drivers/gpu/drm/drm_scdc_helper.c | 2 +- drivers/gpu/drm/drm_simple_kms_helper.c | 5 +++-- drivers/gpu/drm/drm_syncobj.c | 13 +++++++++---- drivers/gpu/drm/drm_sysfs.c | 15 +++++++++++---- drivers/gpu/drm/drm_trace.h | 2 ++ drivers/gpu/drm/drm_trace_points.c | 3 ++- drivers/gpu/drm/drm_vblank.c | 12 +++++++++--- drivers/gpu/drm/drm_vm.c | 14 +++++++++++++- drivers/gpu/drm/drm_vma_manager.c | 6 +++--- drivers/gpu/drm/drm_vram_mm_helper.c | 4 +++- drivers/gpu/drm/drm_writeback.c | 6 ++++-- include/drm/drm_auth.h | 11 +++++++++-- include/drm/drm_legacy.h | 12 ++++++------ 65 files changed, 406 insertions(+), 163 deletions(-) Acked-by: Daniel Vetter