Dear Girish, I hope this message finds you well.
I would like to check if it would be possible for you to submit pull requests (PRs) for the SMBIOS generators that you had previously posted to the mailing list. Additionally, if you have developed any new SMBIOS generators since then, it would be beneficial to include those as well. It would be valuable to have this functionality supported in the upstream EDK2 mainline. Could you please also provide a list of the SMBIOS generators that are currently implemented on your end? This will help us identify the remaining ones and plan for their upstreaming accordingly. Regards, Sami Mujawar From: Girish Mahadevan <[email protected]> Date: Saturday, 26 July 2025 at 00:07 To: Jeff Brasen ([email protected]) <[email protected]>, Pierre Gondois <[email protected]>, Sarah Walker <[email protected]>, edk2-devel-groups-io <[email protected]>, Sami Mujawar <[email protected]> Subject: Re: Merging Dynamic SMBIOS Patches in edk2 Mainline Hi Sami I wanted to update you that we have merged the SMBIOS Type4 and Type7 table generators into the ‘dynamictables-reorg’ branch of the ‘edk2-staging’ repository. That is great, we will try to use this (we're currently using our own version) As we now prepare to upstream the initial Dynamic SMBIOS support to the edk2 mainline, we’d like to include the following patches authored by you as part of that PR: • DynamicTablesPkg: Add SMBIOS table generation<https://github.com/tianocore/edk2-staging/commit/2d1c8e7b46eaca9e9ed54406a82759ee91d04f2c> • DynamicTablesPkg: Move ACPI building & change DEPEX on protocol<https://github.com/tianocore/edk2-staging/commit/0e2d3e17c1c54ff06b94228e5617141c7040d209> • MdePkg/SmBios.h: Add New definition for Invalid Handle<https://github.com/tianocore/edk2-staging/commit/c5ba14126e7338010bcc0c8885b26d4ac6cfdb9d> Could you please confirm if you're okay with us including these patches in the PR? Once this initial support is merged upstream, we’ll be able to retire the staging branch. In parallel, it would also be great if you could follow up with a PR to upstream the SMBIOS table generators you had initially posted. Sure, go ahead ! I will start setting up PRs for the generators we've got. Best Regards Girish ________________________________ From: [email protected] <[email protected]> on behalf of Sami Mujawar via groups.io <[email protected]> Sent: Friday, July 25, 2025 9:02 AM To: Girish Mahadevan <[email protected]>; Jeff Brasen <[email protected]>; Pierre Gondois <[email protected]>; Sarah Walker <[email protected]>; edk2-devel-groups-io <[email protected]> Subject: [edk2-devel] Merging Dynamic SMBIOS Patches in edk2 Mainline External email: Use caution opening links or attachments Hi Girish, I hope you're doing well. I wanted to update you that we have merged the SMBIOS Type4 and Type7 table generators into the ‘dynamictables-reorg’ branch of the ‘edk2-staging’ repository. As we now prepare to upstream the initial Dynamic SMBIOS support to the edk2 mainline, we’d like to include the following patches authored by you as part of that PR: • DynamicTablesPkg: Add SMBIOS table generation<https://github.com/tianocore/edk2-staging/commit/2d1c8e7b46eaca9e9ed54406a82759ee91d04f2c> • DynamicTablesPkg: Move ACPI building & change DEPEX on protocol<https://github.com/tianocore/edk2-staging/commit/0e2d3e17c1c54ff06b94228e5617141c7040d209> • MdePkg/SmBios.h: Add New definition for Invalid Handle<https://github.com/tianocore/edk2-staging/commit/c5ba14126e7338010bcc0c8885b26d4ac6cfdb9d> Could you please confirm if you're okay with us including these patches in the PR? Once this initial support is merged upstream, we’ll be able to retire the staging branch. In parallel, it would also be great if you could follow up with a PR to upstream the SMBIOS table generators you had initially posted. Looking forward to your approval. Warm regards, Sami Mujawar IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#121699): https://edk2.groups.io/g/devel/message/121699 Mute This Topic: https://groups.io/mt/114338962/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
