Re: [edk2] MdeModulePkg/Universal/Network/SnpDxp/Station_address.c

2014-03-10 Thread Ari Zigler
Hi Ruth, Yes, I'm developing a UNDI driver for our device. The bellow code, if integrated into a BIOS FW, do cause me an issue as it implies that there is no need to pass PXE_OPFLAGS_STATION_ADDRESS_WRITE whereas my driver expects this kind of flag in order to perform SetAddress(). With the cur

Re: [edk2] MdeModulePkg/Universal/Network/SnpDxp/Station_address.c

2014-03-10 Thread Li, Ruth
Ari, Below piece of code has been here for many years, the purpose of the comment seems to remind the operation result for some UNDI implementation. Are you development UNDI driver for some device? Whether below code bring any issue to you? Thanks, Ruth From: Ari Zigler [mailto:a...@mellanox.c

Re: [edk2] MdeModulePkg: (at least) 4 writes to Flash to update a NV variable in UpdateVariable()

2014-03-10 Thread Andrew Fish
On Mar 10, 2014, at 7:04 PM, Zeng, Star wrote: > Hi Olivier, > > As we know, the BlockSize is the unit of FVB, FVB needs to depend on lower > protocol(SPI protocol) to do real write operation, SPI Protocol does not seem to be part of the Open source or industry standard. FirmwareVolumeBloc

Re: [edk2] MdeModulePkg: (at least) 4 writes to Flash to update a NV variable in UpdateVariable()

2014-03-10 Thread Zeng, Star
Hi Olivier, As we know, the BlockSize is the unit of FVB, FVB needs to depend on lower protocol(SPI protocol) to do real write operation, but the unit of write operation may be one byte[cid:image001.png@01CF3D11.5A7ACF50], or other bytes. Anyway, we should not assume the operation unit of real

[edk2] Success!

2014-03-10 Thread Bill Paul
The miniPCIe to PCIe adapter that I ordered arrived today, and I'm happy to report that I was able to use it in conjunction with my custom EFI option ROM driver loaded onto an Intel 82574 ethernet card to un-brick my ExoPC Slate tablet. I want to say thanks again for all those who answered my e

Re: [edk2] MdeModulePkg: (at least) 4 writes to Flash to update a NV variable in UpdateVariable()

2014-03-10 Thread Tim Lewis
Olivier - I don't have the whole code section memorized, but if I remember right there is the problem when there is a power failure as the variable header is being written. If VAR_HEADER_VALID_ONLY was set in this case, but the entire variable header was not written, wouldn't this be another in

[edk2] MdeModulePkg: (at least) 4 writes to Flash to update a NV variable in UpdateVariable()

2014-03-10 Thread Olivier Martin
Dear MdeModulePkg maintainers, We have seen on some platforms that flash writing/erasing counts for most of the boot time. We have also noticed UpdateVariable() (in MdeModulePkg/Universal/RuntimeDxe/Variable.c) might make 4 accesses to the same region of Flash to update a non-volatile variable

[edk2] Tianocore now supports Android FastBoot

2014-03-10 Thread Olivier Martin
Dear all, Android FastBoot support has been added to Tianocore into SVN 15317 (2014-03-05). Android Fastboot is a protocol to update the flash and/or boot filesystem on Android devices from a host over USB. The fastboot utility is part of the Android SDK. This support is architecture indep

Re: [edk2] [PATCH] ShellPkg/ShellCommands/SetVar: Make '-rt' imply '-bs'

2014-03-10 Thread Carsey, Jaben
Reviewed and commited. 15323 From: Olivier Martin [mailto:olivier.mar...@arm.com] Sent: Monday, March 10, 2014 10:56 AM To: Carsey, Jaben Cc: edk2-devel@lists.sourceforge.net; Brendan Jackman Subject: [PATCH] ShellPkg/ShellCommands/SetVar: Make '-rt' imply '-bs' Importance: High Dear ShellPkg mai

Re: [edk2] Pandaboard LCD driver

2014-03-10 Thread Olivier Martin
Hi, As you said Pandaboard support currently lives in Linaro repository. You can send your patch to linaro-u...@lists.linaro.org Thanks, Olivier From: vale...@inbox.ru [mailto:vale...@inbox.ru] Sent: 10 March 2014 16:11 To: edk2-devel@lists.sourceforge.net Subject: [edk2] Pandaboard LCD dr

[edk2] [PATCH] ShellPkg/ShellCommands/SetVar: Make '-rt' imply '-bs'

2014-03-10 Thread Olivier Martin
Dear ShellPkg maintainer, Please find the attached patch that makes '-rt' imply '-bs'. It's invalid to set a variable that's available from runtime services but not from boot services. Currently if you pass '-rt' without '-bs' you get a generic 'Invalid Parameter' message. We should eit

[edk2] Pandaboard LCD driver

2014-03-10 Thread valeri2
Hi all. I have developed a patch Omap44xxPkg/LcdGraphicsOutputDxe driver for it to work properly on the Pandaboard. Can I somehow publish it in Linaro UEFI tree?-- Learn Graph Databases - Download FREE O'Reilly Book "Gr

[edk2] [PATCH] MdeModulePkg/Core/PiSmmCore/PiSmmCore.c: Fix compiler warning about variable set but unused.

2014-03-10 Thread Kirkendall, Garrett
MdeModulePkg maintainers, please consider either one of these two patches to fix a GCC compiler warning about a variable being set but unused. One eliminates the Status variable in question, and the other returns the Status variable instead of EFI_SUCCESS directly. Garrett Kirkendall [Descrip