RPi4 fixes to 3GB RAM limit logic

[edk2-devel] [edk2-platforms][PATCH 0/2] RPi4 fixes to 3GB RAM limit logic

Posted by Andrei Warkentin via Groups.Io 4 years, 1 month ago

Dear all,

Here are two minor fixes to the recently checked-in runtime logic
for enabling/disabling 3GB limit.

It's been tested on 2GB and 4GB boards.

Andrei Warkentin (2):
  Platform/RaspberryPi/Drivers/ConfigDxe: fix bug in 3GB RAM logic
  Platform/RaspberryPi/Drivers/PlatformSmbiosDxe: improve UX with 3GB
    limit

 Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxe.c         | 3 ++-
 .../Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c          | 7 ++++++-
 .../Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.inf        | 2 ++
 3 files changed, 10 insertions(+), 2 deletions(-)

-- 
2.17.1


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#55458): https://edk2.groups.io/g/devel/message/55458
Mute This Topic: https://groups.io/mt/71737116/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [edk2-platforms][PATCH 0/2] RPi4 fixes to 3GB RAM limit logic

Posted by Ard Biesheuvel 4 years, 1 month ago

On Wed, 4 Mar 2020 at 23:31, Andrei Warkentin <awarkentin@vmware.com> wrote:
>
> Dear all,
>
> Here are two minor fixes to the recently checked-in runtime logic
> for enabling/disabling 3GB limit.
>
> It's been tested on 2GB and 4GB boards.
>

Thanks Andrei,

For future postings, could you please use git send-email or some other
tool that doesn't mangle patches and require me to go through each one
and fix it up by hand? Diff surgery is not the best use of my time.

Thanks,

> Andrei Warkentin (2):
>   Platform/RaspberryPi/Drivers/ConfigDxe: fix bug in 3GB RAM logic
>   Platform/RaspberryPi/Drivers/PlatformSmbiosDxe: improve UX with 3GB
>     limit
>
>  Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxe.c         | 3 ++-
>  .../Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.c          | 7 ++++++-
>  .../Drivers/PlatformSmbiosDxe/PlatformSmbiosDxe.inf        | 2 ++
>  3 files changed, 10 insertions(+), 2 deletions(-)
>
> --
> 2.17.1
>

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#55538): https://edk2.groups.io/g/devel/message/55538
Mute This Topic: https://groups.io/mt/71737116/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [edk2-platforms][PATCH 0/2] RPi4 fixes to 3GB RAM limit logic

Posted by Ard Biesheuvel 4 years, 1 month ago

On Thu, 5 Mar 2020 at 14:52, Ard Biesheuvel <ard.biesheuvel@linaro.org> wrote:
>
> On Wed, 4 Mar 2020 at 23:31, Andrei Warkentin <awarkentin@vmware.com> wrote:
> >
> > Dear all,
> >
> > Here are two minor fixes to the recently checked-in runtime logic
> > for enabling/disabling 3GB limit.
> >
> > It's been tested on 2GB and 4GB boards.
> >
>
> Thanks Andrei,
>
> For future postings, could you please use git send-email or some other
> tool that doesn't mangle patches and require me to go through each one
> and fix it up by hand? Diff surgery is not the best use of my time.
>
> Thanks,
>
> > Andrei Warkentin (2):
> >   Platform/RaspberryPi/Drivers/ConfigDxe: fix bug in 3GB RAM logic
> >   Platform/RaspberryPi/Drivers/PlatformSmbiosDxe: improve UX with 3GB
> >     limit
> >

Pushed as dac18ca51781..28d2f4ecd87d

(with the suggested modification, and acks applied)

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#55539): https://edk2.groups.io/g/devel/message/55539
Mute This Topic: https://groups.io/mt/71737116/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-