On 10/09/14 13:57, Tian, Feng wrote: > 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.
As far as I can see, this is now SVN r16225. Thanks. Laszlo > 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 > ------------------------------------------------------------------------------ Comprehensive Server Monitoring with Site24x7. Monitor 10 servers for $9/Month. Get alerted through email, SMS, voice calls or mobile push notifications. Take corrective actions from your mobile device. http://p.sf.net/sfu/Zoho _______________________________________________ edk2-devel mailing list edk2-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/edk2-devel
