From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org 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 smtp.lore.kernel.org (Postfix) with ESMTPS id C0FE9C433F5 for ; Tue, 5 Apr 2022 17:34:25 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id EA75D10ED9B; Tue, 5 Apr 2022 17:34:24 +0000 (UTC) Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by gabe.freedesktop.org (Postfix) with ESMTPS id 6E00C10ED94 for ; Tue, 5 Apr 2022 17:34:23 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1649180063; x=1680716063; h=from:to:subject:date:message-id:mime-version: content-transfer-encoding; bh=8SsqxWSX/Tv11FBHGeA6ir0gv2IbNe/kMwlS7xwgPq0=; b=WcqEyuz4ApKzIWb7u4J6RxxGe1vns9tKycvAcWRfBH+If36qVxLYgLy5 fSZMHO0AouyfVsIdiOG+zntreroBwIPpk2z/x/MmL96NHmQfVimVM0y8x MZonuoLUMyxMyfWbyNxd5y9xrqqzW2/5YATmGf75LL6JZZxhyP5dJKOhS y13/26/+GTJCVEosuyyi6EM92HxFJIu2UG2opYNWFxR3vmlt1rv2EXr8B wu0vbeedcRAGgw5n60qSczVjEVzVa37QtfjaGnfthYouddl+AORHw5oDk 1ciqn8JEC5tmNyvma18tolvKXlo1/7q5EaB6VnLIL+foeYCINcOPJ6gek Q==; X-IronPort-AV: E=McAfee;i="6200,9189,10308"; a="259648155" X-IronPort-AV: E=Sophos;i="5.90,236,1643702400"; d="scan'208";a="259648155" Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga104.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 05 Apr 2022 10:34:13 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.90,236,1643702400"; d="scan'208";a="651984273" Received: from stinkpipe.fi.intel.com (HELO stinkbox) ([10.237.72.51]) by fmsmga002.fm.intel.com with SMTP; 05 Apr 2022 10:34:11 -0700 Received: by stinkbox (sSMTP sendmail emulation); Tue, 05 Apr 2022 20:34:10 +0300 From: Ville Syrjala To: intel-gfx@lists.freedesktop.org Date: Tue, 5 Apr 2022 20:33:48 +0300 Message-Id: <20220405173410.11436-1-ville.syrjala@linux.intel.com> X-Mailer: git-send-email 2.35.1 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Subject: [Intel-gfx] [PATCH v2 00/22] drm/i915/bios: Rework BDB block handling and PNPID->panel_type matching 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: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" From: Ville Syrjälä Several changes to our BDB block handling: 1) The current way of trusting the version checks to avoid out of bounds accesses to the BDB blocks is fragile. We might just get the version check wrong, or the VBT may be corrupted/malicious. So instead of doing blind accesses into the original data let's make a copy of each block with a gauranteed minimum size. 2) The LFP data table pointer stuff is a horrible mess currently. Let's make that sensible by verifying the pointers ahead of time, which allows us to trust them 100% when we acually parse the actual data block. 3) There's more stuff at the tail end of the LFP data block we need to parse. The variable size of the fp_timing table makes that a bit awkward, but with the pointer validation in place it's not too horrible. 4) Modern VBTs (seen it on TGL/ADL-P/CML so far) no longer include the LFP data table pointers block (41) in the VBT. In order to keep the rest of the code working as is we must therefore generate the pointers block from scratch. New stuff in v2: - Make all the BDB block copies up front - Split the VBT parsing into two parts and add the EDID PNPID->panel_type matching because many modern machines need it Ville Syrjälä (22): drm/i915/bios: Use the cached BDB version drm/i915/bios: Make copies of VBT data blocks drm/i915/bios: Use the copy of the LFP data table always drm/i915/bios: Validate LFP data table pointers drm/i915/bios: Trust the LFP data pointers drm/i915/bios: Validate the panel_name table drm/i915/bios: Reorder panel DTD parsing drm/i915/bios: Generate LFP data table pointers if the VBT lacks them drm/i915/bios: Get access to the tail end of the LFP data block drm/i915/bios: Assume panel_type==0 if the VBT has bogus data drm/i915/bios: Split parse_driver_features() into two parts drm/i915/bios: Split VBT parsing to global vs. panel specific parts drm/i915/pps: Split PPS init+sanitize in two drm/i915/pps: Reinit PPS delays after VBT has been fully parsed drm/i915/bios: Do panel specific VBT parsing later drm/i915/bios: Extract get_panel_type() drm/i915/bios: Refactor panel_type code drm/i915/bios: Determine panel type via PNPID match drm/i915/bios: Parse the seamless DRRS min refresh rate drm/i915: Respect VBT seamless DRRS min refresh rate drm/edid: Extract drm_edid_decode_mfg_id() drm/i915/bios: Dump PNPID and panel name drivers/gpu/drm/i915/display/icl_dsi.c | 2 + drivers/gpu/drm/i915/display/intel_bios.c | 883 ++++++++++++++---- drivers/gpu/drm/i915/display/intel_bios.h | 3 + drivers/gpu/drm/i915/display/intel_dp.c | 4 + drivers/gpu/drm/i915/display/intel_lvds.c | 2 + drivers/gpu/drm/i915/display/intel_panel.c | 10 +- drivers/gpu/drm/i915/display/intel_pps.c | 35 +- drivers/gpu/drm/i915/display/intel_pps.h | 1 + drivers/gpu/drm/i915/display/intel_sdvo.c | 3 + drivers/gpu/drm/i915/display/intel_vbt_defs.h | 22 + drivers/gpu/drm/i915/display/vlv_dsi.c | 2 + drivers/gpu/drm/i915/i915_drv.h | 2 + include/drm/drm_edid.h | 21 +- 13 files changed, 810 insertions(+), 180 deletions(-) -- 2.35.1