From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web10.147.1634590291298582377 for ; Mon, 18 Oct 2021 13:51:31 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: arm.com, ip: 217.140.110.172, mailfrom: jeremy.linton@arm.com) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 111A3D6E; Mon, 18 Oct 2021 13:51:30 -0700 (PDT) Received: from u200856.usa.arm.com (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id B0D5A3F73D; Mon, 18 Oct 2021 13:51:29 -0700 (PDT) From: "Jeremy Linton" To: devel@edk2.groups.io Cc: pete@akeo.ie, ardb+tianocore@kernel.org, leif@nuviainc.com, awarkentin@vmware.com, Sunny.Wang@arm.com, samer.el-haj-mahmoud@arm.com, kettenis@openbsd.org, Jeremy Linton Subject: [PATCH v2 0/5] Platform/Rpi: Various cleanups + DT booting Date: Mon, 18 Oct 2021 15:51:22 -0500 Message-Id: <20211018205127.7099-1-jeremy.linton@arm.com> X-Mailer: git-send-email 2.26.3 MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: Jeremy Linton This set is a few patches I've been collecting to fix minor issues I've seen while debugging other problems, or just various things I think should probably be changed. It also includes the patch to adjust the PCIe/XHCI dma-attributes which fixes DT booting on recent Linux kernels as well as OpenBSD. I've been running all of them in some form or another for a while and generally nothing has broken because of them AFAIK. So its probably time to start getting a few of them out of my private tree. The first patch syncs the DT/PCIe translation to the way its being used in UEFI, and removes the XHCI reset node. The second is just a compiler warning. The third is mostly expanding the mailbox lock to cover the return data. Number 4 is an odd one because it just looks wrong, and I'm worried its causing random bugs. The final is a corrected shutdown sequence that was discussed months ago. It looks right. but didn't actually fix the data persistence problems that resulted in the couple second reboot delay that is currently in place. v1->v2: Include the DT patch to sync the PCIe/XHCI values Now that it also fixes a problem with OpenBSD. Rework the error handling in the DisconnectAll code borrowed from the UEFI spec. Adjust some DEBUG_XXX values so they make more sense. Some comment/changelog grammar. Jeremy Linton (5): Platform/RaspberryPi: Always use non translating DMA in DT mode Platform/RaspberryPi: Fix vfr warning caused by two defaults Platform/RaspberryPi: Expand locking to cover return data Platform/RaspberryPi: Normal memory should not be marked as uncached Platform/RaspberryPi: Disconnect/shutdown all drivers before reboot Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxe.c | 4 +- .../RaspberryPi/Drivers/ConfigDxe/ConfigDxeHii.vfr | 2 +- Platform/RaspberryPi/Drivers/FdtDxe/FdtDxe.c | 75 +++++++++++++++ .../Drivers/RpiFirmwareDxe/RpiFirmwareDxe.c | 102 ++++++++++++--------- Platform/RaspberryPi/Library/ResetLib/ResetLib.c | 42 +++++++++ 5 files changed, 179 insertions(+), 46 deletions(-) -- 2.13.7