Hi, Reza and Laszlo Sorry for late response due to PRC holiday.
For #1 (MdeModulePkg: Check D2H register status in AhciPioTransfer), I have refined the patch according to Sava's feedback. I will check it in after passing unit test. For #6 (MdeModulePkg; IdeMode needs to select master or slave before diagnostics command), I need make a bigger scope of test before marking it as review-passed. For #5 (MdeModulePkg: AhciMode calls GetChannelInfo when enumerating), I don't see why it's needed. So I am prone to reject this patch if no further comments. Thanks Feng -----Original Message----- From: Reza Jelveh [mailto:reza.jel...@tuhh.de] Sent: Monday, September 29, 2014 10:06 PM To: Laszlo Ersek Cc: Gabriel L. Somlo (CMU); edk2-devel@lists.sourceforge.net Subject: Re: [edk2] [PATCH v2 0/6] enable SataController in OVMF On 29/09/14 15:37, Laszlo Ersek wrote: > On 09/29/14 15:23, Reza Jelveh wrote: > > On 29/09/14 13:32, Laszlo Ersek wrote: > >> What is the review / development / resubmission status of this patchset? > >> In particular, have opinions converged on the non-OVMF packages, ie. > >> MdeModulePkg, DuetPkg, and PcAtChipsetPkg? > > The PcAtChipsetPkg/DuetPkg remains untouched. The copy of > > SataController is in OVMF as requested by the Intel guys. > > OK, that covers patches #2 and #4. > > What about MdeModulePkg (patches #1, #5, #6)? #5 good question #1 I have added the structural changes requested by Feng and Sava #6 A more detailed commit message was requested, I've fixed that. I guess I should have just resubmitted without waiting for the applesupport work. > > > I have changed it back to a Pxe module although I need to test the make > > flag. > > ENOPARSE... Nevermind, I was talking about the applesupport patches. > > also started playing around with the bsd based hfs driver, since > > gabriel said that some of the original refit code was actually bsd. > > > > The applepartitiontable is still missing. > What's that? PartitionDxe needs support for apple partition tables for the mounting the isos. More related to applesupport. I should have submitted a patch for this already. ------------------------------------------------------------------------------ Slashdot TV. Videos for Nerds. Stuff that Matters. http://pubads.g.doubleclick.net/gampad/clk?id=160591471&iu=/4140/ostg.clktrk _______________________________________________ edk2-devel mailing list edk2-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/edk2-devel ------------------------------------------------------------------------------ Meet PCI DSS 3.0 Compliance Requirements with EventLog Analyzer Achieve PCI DSS 3.0 Compliant Status with Out-of-the-box PCI DSS Reports Are you Audit-Ready for PCI DSS 3.0 Compliance? Download White paper Comply to PCI DSS 3.0 Requirement 10 and 11.5 with EventLog Analyzer http://pubads.g.doubleclick.net/gampad/clk?id=154622311&iu=/4140/ostg.clktrk _______________________________________________ edk2-devel mailing list edk2-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/edk2-devel
