Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ArmPlatformPkg: Fix various typos #144

Closed
wants to merge 1 commit into from

Conversation

Coeur
Copy link
Contributor

@Coeur Coeur commented Jun 27, 2019

Meta

This is an experimental pull request, as discussed many times on the edk2 mailing-list.
It is:

  • an easy pull request
  • a non-urgent pull request

It fits the experiment of using a GitHub pull request over a mailing-list patch.

Once reviewed and approved, the maintainer of this repo (edk2) has multiple possibilities to handle it:

  • either use command line to rebase/merge/etc. this pull request (git pull https://github.com/coeur/edk2.git Typo-ArmPlatformPkg)
  • either use the GitHub interface and choose to rebase (or merge or squash): Screen Shot 2019-06-27 at 12 24 32 PM

Pull Request

This is actually a cherry pick for ArmPlatformPkg only from a larger change updated since 2018 at Coeur@1ee272d.

Fix various typos in ArmPlatformPkg.

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Coeur <coeur@gmx.fr>
@lersek
Copy link
Member

lersek commented Jun 27, 2019

Please see my comment at:

#143 (comment)

@lersek lersek closed this Jun 27, 2019
kuqin12 pushed a commit to kuqin12/edk2 that referenced this pull request Nov 29, 2022
## Description

To reduce the number of calls to SetMemoryAttributes(), merge contiguous
memory map entries by only attribute instead of by attribute and type.

## Breaking change?
No

## How This Was Tested

Inspecting the merge and booting with added protections

## Integration Instructions

N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants