[edk2-devel] [PATCH v4 edk2-platforms 1/1] Silicon/DesignWare/Driver: DwEmacSnpDxe: Add DesignWare EMAC driver

[Ooi, Tzy Way]

Thanks Biesheuvel.

I would like to confirm with you on regards of one of the comment. There is one comment where you mentioned that:
"I am aware that TX is usually the hot path in UEFI, but we should still try to map a buffer directly rather than copy the contents into uncached memory"

I would like to confirm with you if do you mean I should do the DmaMap here instead of doing it at the beginning of the code while DmaAllocate is executed?
Or do you mean I shouldn’t copy it into a uncached memory region?

Thank you
-----Original Message-----
From: Ard Biesheuvel <ard.biesheuvel at linaro.org> 
Sent: Wednesday, May 22, 2019 3:22 PM
To: edk2-devel-groups-io <devel at edk2.groups.io>; Ooi, Tzy Way <tzy.way.ooi at intel.com>
Cc: Kinney, Michael D <michael.d.kinney at intel.com>; Loh, Tien Hock <tien.hock.loh at intel.com>; leif.lindholm at linaro.org
Subject: Re: [edk2-devel] [PATCH v4 edk2-platforms 1/1] Silicon/DesignWare/Driver: DwEmacSnpDxe: Add DesignWare EMAC driver

On Wed, 22 May 2019 at 03:37, Ooi, Tzy Way <tzy.way.ooi at intel.com> wrote:
>
> Hi Maintainers,
>
> Please could you help to review this patch? Thanks.
>

I already did, but my email response got rejected twice

https://edk2.groups.io/g/devel/message/40829?p=,,,20,0,0,0::Created,,emac,20,2,0,31541649

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

View/Reply Online (#41725): https://edk2.groups.io/g/devel/message/41725
Mute This Topic: https://groups.io/mt/31541649/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-