Hi Michael,

What module type should I use instead? The reason being this SerialPortLib 
implementation specifically fits into the class of UEFI implementations, where 
RAM is always available (initialized by something else prior to Tiano) and 
non-const globals can be used (e.g. PrePi).

A

-----Original Message-----
From: Kinney, Michael D <michael.d.kin...@intel.com> 
Sent: Wednesday, March 1, 2023 10:56 AM
To: devel@edk2.groups.io; kra...@redhat.com; Warkentin, Andrei 
<andrei.warken...@intel.com>
Cc: Daniel Schaefer <g...@danielschaefer.me>; Sunil V L 
<suni...@ventanamicro.com>; Kinney, Michael D <michael.d.kin...@intel.com>
Subject: RE: [edk2-devel] [edk2 2/2] MdePkg: add BaseSerialPortLibRiscVSbi

Using a global is not compatible with XIP code where only const globals are 
supported.

A module of type BASE is considered compatible with XIP components.


Mike

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Gerd Hoffmann
> Sent: Wednesday, March 1, 2023 1:07 AM
> To: Warkentin, Andrei <andrei.warken...@intel.com>
> Cc: devel@edk2.groups.io; Daniel Schaefer <g...@danielschaefer.me>; Sunil V L 
> <suni...@ventanamicro.com>
> Subject: Re: [edk2-devel] [edk2 2/2] MdePkg: add BaseSerialPortLibRiscVSbi
> 
> On Wed, Mar 01, 2023 at 08:50:38AM +0000, Warkentin, Andrei wrote:
> > The library caller could call Poll() first,
> 
> Ah, Poll is part of the library API, not just an internal helper.
> Ok, makes sense then.
> 
> series:
> Acked-by: Gerd Hoffmann <kra...@redhat.com>
> 
> take care,
>   Gerd
> 
> 
> 
> 
> 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#100601): https://edk2.groups.io/g/devel/message/100601
Mute This Topic: https://groups.io/mt/97309875/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to