From: "Jeremy Linton" <jeremy.linton@arm.com>
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 <lintonrjeremy@gmail.com>
Subject: [PATCH v2 0/5] Platform/Rpi: Various cleanups + DT booting
Date: Mon, 18 Oct 2021 15:51:22 -0500 [thread overview]
Message-ID: <20211018205127.7099-1-jeremy.linton@arm.com> (raw)
From: Jeremy Linton <lintonrjeremy@gmail.com>
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
next reply other threads:[~2021-10-18 20:51 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-10-18 20:51 Jeremy Linton [this message]
2021-10-18 20:51 ` [PATCH v2 1/5] Platform/RaspberryPi: Always use non translating DMA in DT mode Jeremy Linton
2021-10-18 20:51 ` [PATCH v2 2/5] Platform/RaspberryPi: Fix vfr warning caused by two defaults Jeremy Linton
2021-10-18 20:51 ` [PATCH v2 3/5] Platform/RaspberryPi: Expand locking to cover return data Jeremy Linton
2021-10-18 20:51 ` [PATCH v2 4/5] Platform/RaspberryPi: Normal memory should not be marked as uncached Jeremy Linton
2021-10-18 20:51 ` [PATCH v2 5/5] Platform/RaspberryPi: Disconnect/shutdown all drivers before reboot Jeremy Linton
2021-10-19 7:23 ` [PATCH v2 0/5] Platform/Rpi: Various cleanups + DT booting Ard Biesheuvel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20211018205127.7099-1-jeremy.linton@arm.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox