From patchwork Fri Jan 24 13:25:21 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Jani Nikula X-Patchwork-Id: 11350227 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 9010B138C for ; Fri, 24 Jan 2020 13:25:03 +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 782862075D for ; Fri, 24 Jan 2020 13:25:03 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 782862075D Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=intel.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=intel-gfx-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 9808972A68; Fri, 24 Jan 2020 13:25:01 +0000 (UTC) X-Original-To: intel-gfx@lists.freedesktop.org Delivered-To: intel-gfx@lists.freedesktop.org Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by gabe.freedesktop.org (Postfix) with ESMTPS id B439272A68 for ; Fri, 24 Jan 2020 13:25:00 +0000 (UTC) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga008.jf.intel.com ([10.7.209.65]) by orsmga101.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 24 Jan 2020 05:25:00 -0800 X-IronPort-AV: E=Sophos;i="5.70,357,1574150400"; d="scan'208";a="221022653" Received: from omarkovx-mobl.ger.corp.intel.com (HELO localhost) ([10.249.37.60]) by orsmga008-auth.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 24 Jan 2020 05:24:57 -0800 From: Jani Nikula To: intel-gfx@lists.freedesktop.org Date: Fri, 24 Jan 2020 15:25:21 +0200 Message-Id: X-Mailer: git-send-email 2.20.1 MIME-Version: 1.0 Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo Subject: [Intel-gfx] [RFC 00/33] drm/i915/display: mass conversion to intel_de_*() register accessors X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Jani Nikula , Lucas De Marchi Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" Hey all, So I sent [1] to convert some forcewake register accessors... but what if we just ripped off the bandage once and for all? It's going to hurt, a lot, but we'd get it done. This completely rids us of the "dev_priv" dependency in display/. All the patches here are per-file and independent of each other. We could also pick and apply the ones that are least likely to conflict. Opinions? BR, Jani. PS. I didn't bother looking at the checkpatch warnings this may generate at this point. I just used the --linux-spacing option for spatch, and closed my eyes. I completely scripted the generation of the series, apart from just a couple of build fixes. Cc: Chris Wilson Cc: Joonas Lahtinen Cc: Lucas De Marchi Cc: Rodrigo Vivi Cc: Ville Syrjälä [1] https://patchwork.freedesktop.org/series/72476/ Jani Nikula (33): drm/i915/icl_dsi: use intel_de_*() functions for register access drm/i915/audio: use intel_de_*() functions for register access drm/i915/cdclk: use intel_de_*() functions for register access drm/i915/color: use intel_de_*() functions for register access drm/i915/combo_phy: use intel_de_*() functions for register access drm/i915/crt: use intel_de_*() functions for register access drm/i915/ddi: use intel_de_*() functions for register access drm/i915/display: use intel_de_*() functions for register access drm/i915/display_power: use intel_de_*() functions for register access drm/i915/dp: use intel_de_*() functions for register access drm/i915/dpio_phy: use intel_de_*() functions for register access drm/i915/dpll_mgr: use intel_de_*() functions for register access drm/i915/dp_mst: use intel_de_*() functions for register access drm/i915/dsb: use intel_de_*() functions for register access drm/i915/dvo: use intel_de_*() functions for register access drm/i915/fbc: use intel_de_*() functions for register access drm/i915/fifo_underrun: use intel_de_*() functions for register access drm/i915/gmbus: use intel_de_*() functions for register access drm/i915/hdcp: use intel_de_*() functions for register access drm/i915/hdmi: use intel_de_*() functions for register access drm/i915/lpe_audio: use intel_de_*() functions for register access drm/i915/lvds: use intel_de_*() functions for register access drm/i915/overlay: use intel_de_*() functions for register access drm/i915/panel: use intel_de_*() functions for register access drm/i915/pipe_crc: use intel_de_*() functions for register access drm/i915/psr: use intel_de_*() functions for register access drm/i915/sdvo: use intel_de_*() functions for register access drm/i915/sprite: use intel_de_*() functions for register access drm/i915/tv: use intel_de_*() functions for register access drm/i915/vdsc: use intel_de_*() functions for register access drm/i915/vga: use intel_de_*() functions for register access drm/i915/vlv_dsi: use intel_de_*() functions for register access drm/i915/vlv_dsi_pll: use intel_de_*() functions for register access drivers/gpu/drm/i915/display/icl_dsi.c | 271 ++-- drivers/gpu/drm/i915/display/intel_audio.c | 112 +- drivers/gpu/drm/i915/display/intel_cdclk.c | 133 +- drivers/gpu/drm/i915/display/intel_color.c | 204 +-- .../gpu/drm/i915/display/intel_combo_phy.c | 66 +- drivers/gpu/drm/i915/display/intel_crt.c | 51 +- drivers/gpu/drm/i915/display/intel_ddi.c | 474 +++---- drivers/gpu/drm/i915/display/intel_display.c | 1171 +++++++++-------- .../drm/i915/display/intel_display_power.c | 294 +++-- drivers/gpu/drm/i915/display/intel_dp.c | 234 ++-- drivers/gpu/drm/i915/display/intel_dp_mst.c | 11 +- drivers/gpu/drm/i915/display/intel_dpio_phy.c | 77 +- drivers/gpu/drm/i915/display/intel_dpll_mgr.c | 388 +++--- drivers/gpu/drm/i915/display/intel_dsb.c | 24 +- drivers/gpu/drm/i915/display/intel_dvo.c | 34 +- drivers/gpu/drm/i915/display/intel_fbc.c | 106 +- .../drm/i915/display/intel_fifo_underrun.c | 37 +- drivers/gpu/drm/i915/display/intel_gmbus.c | 74 +- drivers/gpu/drm/i915/display/intel_hdcp.c | 142 +- drivers/gpu/drm/i915/display/intel_hdmi.c | 258 ++-- .../gpu/drm/i915/display/intel_lpe_audio.c | 14 +- drivers/gpu/drm/i915/display/intel_lvds.c | 57 +- drivers/gpu/drm/i915/display/intel_overlay.c | 45 +- drivers/gpu/drm/i915/display/intel_panel.c | 255 ++-- drivers/gpu/drm/i915/display/intel_pipe_crc.c | 20 +- drivers/gpu/drm/i915/display/intel_psr.c | 70 +- drivers/gpu/drm/i915/display/intel_sdvo.c | 30 +- drivers/gpu/drm/i915/display/intel_sprite.c | 320 +++-- drivers/gpu/drm/i915/display/intel_tv.c | 138 +- drivers/gpu/drm/i915/display/intel_vdsc.c | 413 +++--- drivers/gpu/drm/i915/display/intel_vga.c | 7 +- drivers/gpu/drm/i915/display/vlv_dsi.c | 345 ++--- drivers/gpu/drm/i915/display/vlv_dsi_pll.c | 49 +- 33 files changed, 3158 insertions(+), 2766 deletions(-) Acked-by: Chris Wilson Acked-by: Rodrigo Vivi Acked-by: Joonas Lahtinen