From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by mx.groups.io with SMTP id smtpd.web08.1280.1662685631700254532 for ; Thu, 08 Sep 2022 18:07:12 -0700 Authentication-Results: mx.groups.io; dkim=fail reason="unable to parse pub key" header.i=@intel.com header.s=intel header.b=PtCQmhom; spf=pass (domain: intel.com, ip: 192.55.52.88, mailfrom: nathaniel.l.desimone@intel.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1662685631; x=1694221631; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=3R8CI1PeSzptejQZFyk6ItXKzSXpcF5vh9PGN3hX+wc=; b=PtCQmhomNC9/MJlfcRUGTTbJtfuPi6JEPMMEnlp9p47ES42jFV/prrAU JOm4QjQiEAVREZF0LEmCeGpZ5EgoBeULb1mMtupwo81N6oF++rfD/Ss1j 1e83Lmjf84DwOPWGmX69mbyb04ktJduDFb80bK7u3i1Gmqmos+lK66+dP dSV5mVh/OBpwjC0QyHqRgsII7g8F5Aic90jFrRyPNRSr1B3F2t4Vet3Wi 5QuEImIJWi5C0JnyQoloI0wvV0P5uGqz1FItgA4e5Hm/wa0nUr+2gnhq7 u+2MzU9UuErOe6iNLPYwgWv3HNF/hMzAl2ru7i1yCJr0fEJlh0OYOT9nL A==; X-IronPort-AV: E=McAfee;i="6500,9779,10464"; a="323568616" X-IronPort-AV: E=Sophos;i="5.93,300,1654585200"; d="scan'208";a="323568616" Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 08 Sep 2022 18:07:11 -0700 X-IronPort-AV: E=Sophos;i="5.93,300,1654585200"; d="scan'208";a="676950457" Received: from nldesimo-desk1.amr.corp.intel.com ([10.24.80.62]) by fmsmga008-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 08 Sep 2022 18:07:10 -0700 From: "Nate DeSimone" To: devel@edk2.groups.io Cc: Chasel Chiu , Sai Chaganty , Isaac Oram , Eric Dong , Liming Gao , Benjamin Doron , Michael Kubacki , Jeremy Soller Subject: [edk2-platforms] [PATCH V3 0/6] KabylakeOpenBoardPkg: HDMI DDC I2C Bus Debugging Date: Thu, 8 Sep 2022 18:06:58 -0700 Message-Id: <20220909010704.7186-1-nathaniel.l.desimone@intel.com> X-Mailer: git-send-email 2.27.0.windows.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Changes in V3: - Added new HdmiDebugPchDetectionLib to eliminate duplicated code between I2cHdmiDebugSerialPortLib and HdmiDebugGpioInitLib. - Removed duplicated register definitions - Removed commented out code - Removed TODOs Changes in V2: - Use SerialPortTerminalLib from MinPlatformPkg. - Override SerialPortLib on StatusCodeHandlerSmm not StatusCodeRouter. Normally the HDMI DDC I2C bus is used to read the EDID data from the monitor. An unintended but useful property of this interface is that is does not require DMA to perform I/O. This means that this interface can be used to perform I/O before DRAM is initialized. HDMI video output is a common feature of many laptops. This makes the HDMI DDC bus the only I/O interface that is often exposed outside of the laptop chassis while simultaneously capable of being used in Pre-Memory. Oddly... this makes it an ideal and novel way to perform closed chassis debug. This patch series adds a implementation of using the HDMI DDC I2C Bus as a "poor man's" UART. This is accomplished using the GMBUS that is part of the Intel HD Graphics device. Accordingly, this implementation will only work on systems with the HDMI port routed to the Intel graphics device, and will not work on systems with HDMI routed to a discrete GPU. In order to use this implementation, one will also need a BusPirate programmed with Nate's Custom Bus Pirate FW for I2C Serial Debug. This firmware is available from: https://github.com/nate-desimone/Bus_Pirate This firmware interfaces with the I2C Bus and forwards traffic from it to the FTDI USB Serial Adapter that is integrated with the BusPirate. In combination, this results in the HDMI port appearing to function as a regular UART to the user. If you would like to see this in action, I made a short YouTube video: https://www.youtube.com/watch?v=xe7cFhRsr80 Cc: Chasel Chiu Cc: Sai Chaganty Cc: Isaac Oram Cc: Eric Dong Cc: Liming Gao Cc: Benjamin Doron Cc: Michael Kubacki Cc: Jeremy Soller Signed-off-by: Nate DeSimone Nate DeSimone (6): KabylakeOpenBoardPkg: Add HdmiDebugPchDetectionLib KabylakeOpenBoardPkg: Add I2cHdmiDebugSerialPortLib KabylakeOpenBoardPkg: Add HdmiDebugGpioInitLib KabylakeOpenBoardPkg: Add SecBoardInitLib MinPlatformPkg: Add PcdDefaultTerminalType support to SerialPortTerminalLib KabylakeOpenBoardPkg/GalagoPro3: Enable HDMI DDC Debug Port .../PeiAspireVn7Dash572GInitPreMemLib.c | 3 +- .../AspireVn7Dash572G/OpenBoardPkg.dsc | 12 +- .../AspireVn7Dash572G/OpenBoardPkgPcd.dsc | 9 + .../BoardInitLib/PeiGalagoPro3InitPreMemLib.c | 13 +- .../GalagoPro3/OpenBoardPkg.dsc | 85 +- .../GalagoPro3/OpenBoardPkgPcd.dsc | 40 + .../Include/Library/HdmiDebugGpioInitLib.h | 26 + .../Library/HdmiDebugPchDetectionLib.h | 34 + .../PeiKabylakeRvp3InitPreMemLib.c | 13 +- .../KabylakeRvp3/OpenBoardPkg.dsc | 12 +- .../KabylakeRvp3/OpenBoardPkgPcd.dsc | 9 + .../HdmiDebugGpioInitLib.c | 163 ++++ .../HdmiDebugGpioInitLib.inf | 43 + .../HdmiDebugPchDetectionLib.c | 67 ++ .../HdmiDebugPchDetectionLib.inf | 38 + .../DxeI2cHdmiDebugSerialPortLib.inf | 53 ++ .../DxeSmmI2cHdmiDebugSerialPortLib.c | 161 ++++ .../Library/I2cHdmiDebugSerialPortLib/Gmbus.c | 755 ++++++++++++++++++ .../Library/I2cHdmiDebugSerialPortLib/Gmbus.h | 319 ++++++++ .../I2cDebugPortProtocol.c | 189 +++++ .../I2cDebugPortProtocol.h | 77 ++ .../I2cDebugPortTplDxe.c | 44 + .../I2cDebugPortTplNull.c | 36 + .../I2cHdmiDebugSerialPortLib.c | 198 +++++ .../I2cHdmiDebugSerialPortLib/IgfxI2c.c | 79 ++ .../I2cHdmiDebugSerialPortLib/IgfxI2c.h | 99 +++ .../PeiI2cHdmiDebugSerialPortLib.c | 224 ++++++ .../PeiI2cHdmiDebugSerialPortLib.inf | 54 ++ .../SecI2cHdmiDebugSerialPortLib.c | 134 ++++ .../SecI2cHdmiDebugSerialPortLib.inf | 53 ++ .../SmmI2cHdmiDebugSerialPortLib.inf | 53 ++ .../SecBoardInitLib/Ia32/SecBoardInit.nasm | 18 + .../Library/SecBoardInitLib/SecBoardInitLib.c | 35 + .../SecBoardInitLib/SecBoardInitLib.inf | 39 + .../KabylakeOpenBoardPkg/OpenBoardPkg.dec | 33 +- .../SerialPortTerminalLib.c | 66 +- .../SerialPortTerminalLib.h | 11 +- .../SerialPortTerminalLib.inf | 17 +- 38 files changed, 3259 insertions(+), 55 deletions(-) create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Include/Library/HdmiDebugGpioInitLib.h create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Include/Library/HdmiDebugPchDetectionLib.h create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Library/HdmiDebugGpioInitLib/HdmiDebugGpioInitLib.c create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Library/HdmiDebugGpioInitLib/HdmiDebugGpioInitLib.inf create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Library/HdmiDebugPchDetectionLib/HdmiDebugPchDetectionLib.c create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Library/HdmiDebugPchDetectionLib/HdmiDebugPchDetectionLib.inf create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Library/I2cHdmiDebugSerialPortLib/DxeI2cHdmiDebugSerialPortLib.inf create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Library/I2cHdmiDebugSerialPortLib/DxeSmmI2cHdmiDebugSerialPortLib.c create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Library/I2cHdmiDebugSerialPortLib/Gmbus.c create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Library/I2cHdmiDebugSerialPortLib/Gmbus.h create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Library/I2cHdmiDebugSerialPortLib/I2cDebugPortProtocol.c create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Library/I2cHdmiDebugSerialPortLib/I2cDebugPortProtocol.h create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Library/I2cHdmiDebugSerialPortLib/I2cDebugPortTplDxe.c create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Library/I2cHdmiDebugSerialPortLib/I2cDebugPortTplNull.c create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Library/I2cHdmiDebugSerialPortLib/I2cHdmiDebugSerialPortLib.c create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Library/I2cHdmiDebugSerialPortLib/IgfxI2c.c create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Library/I2cHdmiDebugSerialPortLib/IgfxI2c.h create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Library/I2cHdmiDebugSerialPortLib/PeiI2cHdmiDebugSerialPortLib.c create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Library/I2cHdmiDebugSerialPortLib/PeiI2cHdmiDebugSerialPortLib.inf create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Library/I2cHdmiDebugSerialPortLib/SecI2cHdmiDebugSerialPortLib.c create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Library/I2cHdmiDebugSerialPortLib/SecI2cHdmiDebugSerialPortLib.inf create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Library/I2cHdmiDebugSerialPortLib/SmmI2cHdmiDebugSerialPortLib.inf create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Library/SecBoardInitLib/Ia32/SecBoardInit.nasm create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Library/SecBoardInitLib/SecBoardInitLib.c create mode 100644 Platform/Intel/KabylakeOpenBoardPkg/Library/SecBoardInitLib/SecBoardInitLib.inf -- 2.27.0.windows.1