<div dir="ltr"><div>Hi all, especially Ethin.</div><div> </div><div>Apologies for radio silence - last week I was off on holiday, and before that ... let's just say corporate acquisitions generate some distractions.</div><div>Anyway, I'm back, and since I'm down as the mentor for this task, feel free to spam me with any remaining/new questions.</div><div> </div><div>/</div><div> Leif </div></div> <div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Apr 6, 2021 at 4:17 PM Ethin Probst <<a href="mailto:harlydavidsen@gmail.com">harlydavidsen@gmail.com</a>> wrote: </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I'll attach the bug for the build tools to the BZ shortly. Laszlo, thanks for that. I don't know their email addresses though. And yes, I was going to make it device independent, as the majority (if not all) of UEFI protocols are. On 4/6/21, Laszlo Ersek <<a href="mailto:lersek@redhat.com" target="_blank">lersek@redhat.com</a>> wrote: > On 03/31/21 08:41, Nate DeSimone wrote: >> Another option is to put the protocol definition in MdeModulePkg and >> mark it with the EDKII_ prefix. For my last “code first” UEFI spec >> contribution I did this with the PPI that added up getting added. > > The new audio protocol should be generic, only its implementation in > question should be virtio specific. > > Please include Gerd Hoffmann (CC'd) in the protocol design, as well as > the developers of the virtio-sound device model in QEMU. > > Thanks > Laszlo > > >> >> >> >> Thanks, >> >> Nate >> >> >> >> *From: *<<a href="mailto:devel@edk2.groups.io" target="_blank">devel@edk2.groups.io</a>> on behalf of "Andrew Fish via <a href="http://groups.io" rel="noreferrer" target="_blank">groups.io</a>" >> <afish=<a href="mailto:apple.com@groups.io" target="_blank">apple.com@groups.io</a>> >> *Reply-To: *"<a href="mailto:devel@edk2.groups.io" target="_blank">devel@edk2.groups.io</a>" <<a href="mailto:devel@edk2.groups.io" target="_blank">devel@edk2.groups.io</a>>, >> "<a href="mailto:afish@apple.com" target="_blank">afish@apple.com</a>" <<a href="mailto:afish@apple.com" target="_blank">afish@apple.com</a>> >> *Date: *Tuesday, March 30, 2021 at 10:54 PM >> *To: *edk2-devel-groups-io <<a href="mailto:devel@edk2.groups.io" target="_blank">devel@edk2.groups.io</a>>, >> "<a href="mailto:harlydavidsen@gmail.com" target="_blank">harlydavidsen@gmail.com</a>" <<a href="mailto:harlydavidsen@gmail.com" target="_blank">harlydavidsen@gmail.com</a>> >> *Cc: *Rafael Rodrigues Machado <<a href="mailto:rafaelrodrigues.machado@gmail.com" target="_blank">rafaelrodrigues.machado@gmail.com</a>> >> *Subject: *Re: [edk2-devel] VirtIO Sound Driver (GSoC 2021) >> >> >> >> >> >> On Mar 30, 2021, at 5:01 PM, Ethin Probst <<a href="mailto:harlydavidsen@gmail.com" target="_blank">harlydavidsen@gmail.com</a> >> <mailto:<a href="mailto:harlydavidsen@gmail.com" target="_blank">harlydavidsen@gmail.com</a>>> wrote: >> >> >> >> I'm wondering where exactly I should add the VirtIO sound protocol. I >> just familiarized myself with the build system and am about to test >> it >> by building OVMF if possible, but I'm wondering where I should >> actually put the protocol and all that stuff. Maybe there's >> documentation I've missed as well. >> >> >> >> Ethin, >> >> >> >> For the driver I’d match the patter of OVMF [1] and use >> OvmfPkg/VirtioSoundDxe/. Maybe even use one of the other drivers as a >> template. >> >> >> >> The protocol is more of a public thing. I think eventually we would like >> to publish the protocol in the UEFI Spec (I can help with that part) and >> that would mean we put the Protocol definition in >> MdePkg/Include/Protocol, but we don’t want to do that before it is >> standardized as that causes compatibility issues. So this is a “code >> first project” (code prototype and then contribute to the UEFI Forum for >> inclusion in the specification) so we need to follow some code first >> rules that I don’t remember of the top of my head? So why not start out >> the protocol definition OvmfPkg/Include/Protocol. You can also add a >> test application looks like you can just use the root [2] of OVMF for >> that. That way the project is not blocked. >> >> >> >> We can have a conversation on the mailing list about better places to >> put stuff, and it should be easy enough to move stuff around if >> everything else is working. >> >> >> >> [1] find OvmfPkg -iname '*Virtio*.inf' >> >> OvmfPkg/VirtioPciDeviceDxe/VirtioPciDeviceDxe.inf >> >> OvmfPkg/VirtioScsiDxe/VirtioScsi.inf >> >> OvmfPkg/Library/VirtioMmioDeviceLib/VirtioMmioDeviceLib.inf >> >> OvmfPkg/Library/VirtioLib/VirtioLib.inf >> >> OvmfPkg/VirtioGpuDxe/VirtioGpu.inf >> >> OvmfPkg/VirtioBlkDxe/VirtioBlk.inf >> >> OvmfPkg/Virtio10Dxe/Virtio10.inf >> >> OvmfPkg/VirtioNetDxe/VirtioNet.inf >> >> OvmfPkg/VirtioRngDxe/VirtioRng.inf >> >> >> >> [2] /Volumes/Case/edk2-github/OvmfPkg>git grep APPLICATION -- *.inf | >> grep MODULE_TYPE >> >> EnrollDefaultKeys/EnrollDefaultKeys.inf:13: MODULE_TYPE >> = UEFI_APPLICATION >> >> >> >> Thanks, >> >> >> >> Andrew Fish >> >> >> >> >> >> >> On 3/30/21, Ethin Probst via <a href="http://groups.io" rel="noreferrer" target="_blank">groups.io</a> <<a href="http://groups.io/" rel="noreferrer" target="_blank">http://groups.io/</a>> >> <harlydavidsen=<a href="mailto:gmail.com@groups.io" target="_blank">gmail.com@groups.io</a> >> <mailto:<a href="mailto:harlydavidsen" target="_blank">harlydavidsen</a>=<a href="mailto:gmail.com@groups.io" target="_blank">gmail.com@groups.io</a>>> wrote: >> >> I agree. Plus, it gives me a chance to finally learn the EDK2 >> build >> system and how it works! I've been working on a hobby OS as a >> side >> project and, though learning from other code examples from OSes >> is >> fun, I have to say that learning from the firmware code like from >> SeaBIOS has been some of the most enlightening and interesting >> times >> thus far. >> Thanks for the link to your code, Rafael; once I get virtIO >> support >> in, I can work on HDA support, though I might tackle USB support >> second and HDA third. We'll see, but VirtIO definitely is coming >> first. >> >> As I said before, I look forward to working with all of you >> wonderful >> people! >> >> On 3/30/21, Rafael Rodrigues Machado >> <<a href="mailto:rafaelrodrigues.machado@gmail.com" target="_blank">rafaelrodrigues.machado@gmail.com</a> >> <mailto:<a href="mailto:rafaelrodrigues.machado@gmail.com" target="_blank">rafaelrodrigues.machado@gmail.com</a>>> >> wrote: >> >> This would be amazing so people can continue my work related >> to >> accessibility at BIOS. Something desired by the blind people >> since the >> 90's >> Just for reference, this is what I have done: >> >> >> <a href="https://github.com/RafaelRMachado/Msc_UefiHda_PreOs_Accessibility" rel="noreferrer" target="_blank">https://github.com/RafaelRMachado/Msc_UefiHda_PreOs_Accessibility</a> >> >> Thanks >> Rafael >> >> Em seg, 29 de mar de 2021 20:24, Ethin Probst >> <<a href="mailto:harlydavidsen@gmail.com" target="_blank">harlydavidsen@gmail.com</a>> >> escreveu: >> >> >> Hello everyone, >> >> This is the first time I've ever contributed to EDK2. As >> part of GSoC >> 2021, I have submitted a proposal to implement a UEFI >> audio output >> protocol that will utilize the VirtIO sound driver. I've >> already >> submitted a draft proposal, and apologize if I've done >> things out of >> order. This is my first time doing GSoC 2021, and >> contributing to EDK2 >> felt like a really fun thing to do! >> >> I look forward to working with you guys on this and any >> future projects! >> :-) >> >> -- >> Signed, >> Ethin D. Probst >> >> >> >> >> >> >> >> >> >> -- >> Signed, >> Ethin D. Probst >> >> >> >> >> >> >> >> -- >> Signed, >> Ethin D. Probst >> >> >> >> >> >> > > -- Signed, Ethin D. Probst </blockquote></div> <div width="1" style="color:white;clear:both">_._,_._,_</div> <hr> Groups.io Links: You receive all messages sent to this group. <a target="_blank" href="https://edk2.groups.io/g/devel/message/74038">View/Reply Online (#74038)</a> | | <a target="_blank" href="https://groups.io/mt/81710286/1813853">Mute This Topic</a> | <a href="https://edk2.groups.io/g/devel/post">New Topic</a> <a href="https://edk2.groups.io/g/devel/editsub/1813853">Your Subscription</a> | <a href="mailto:devel+owner@edk2.groups.io">Contact Group Owner</a> | <a href="https://edk2.groups.io/g/devel/unsub">Unsubscribe</a> [edk2-devel-archive@redhat.com] <div width="1" style="color:white;clear:both">_._,_._,_</div>