Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

MdePkg:Added support for Smbios 3.7.0 Spec #5666

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

MdePkg:Added support for Smbios 3.7.0 Spec #5666

wants to merge 1 commit into from

Conversation

shenbagadevir
Copy link

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4563

As per Smbios 3.7.0 spec, add changes in Type9 and Type17 structures.

Signed-off-by: Shenbagadevi R shenbagadevir@ami.com

Cc: Gaoliming gaoliming@byosoft.com.cn
Cc: Sainadh N sainadhn@ami.com
Cc: Sundaresan S sundaresans@ami.com
Cc: Srinivasan M srinivasanm@ami.com
Cc: Ramesh R rameshr@ami.com

Reviewed-by: Gaoliming gaoliming@byosoft.com.cn

@shenbagadevir
MdePkg:Added support for Smbios 3.7.0 Spec
70dd572 
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4563

As per Smbios 3.7.0 spec, add changes in Type9 and Type17 structures.

Signed-off-by: Shenbagadevi R <shenbagadevir@ami.com>

Cc: Gaoliming <gaoliming@byosoft.com.cn>
Cc: Sainadh N <sainadhn@ami.com>
Cc: Sundaresan S <sundaresans@ami.com>
Cc: Srinivasan M <srinivasanm@ami.com>
Cc: Ramesh R <rameshr@ami.com>

Reviewed-by: Gaoliming <gaoliming@byosoft.com.cn>
@shenbagadevir
Copy link
Author

Hi @lgao4 ,

I have committed changes from only one file from MdePkg, but the PatchCheck is failed showing commits in multiple Packages error. When I run Patch check locally before pushing, the patch check was passed. Please review this and help to resolve this issue.

commiterr

Thanks,
Shenba

@shenbagadevir
Copy link
Author

Hi @lgao4 ,

I have committed changes from only one file from MdePkg, but the PatchCheck is failed showing commits in multiple Packages error. When I run Patch check locally before pushing, the patch check was passed. Please review this and help to resolve this issue.

commiterr

Thanks,
Shenba

@shenbagadevir
Copy link
Author

I have committed changes from only one file from MdePkg, but the PatchCheck is failed showing commits in multiple Packages error. When I run Patch check locally before pushing, the patch check was passed. Please review this and help to resolve this issue.

@lgao4 Please help regarding this issue.

@shenbagadevir
Copy link
Author

Hi @lgao4 ,

I have committed changes from only one file from MdePkg, but the PatchCheck is failed showing commits in multiple Packages error. When I run Patch check locally before pushing, the patch check was passed. Please review this and help to resolve this issue.

commiterr Thanks, Shenba

Hi @lgao4 ,

Please help regarding this issue.

@lgao4 lgao4 self-requested a review May 31, 2024 06:12
@lgao4 lgao4 added the push Auto push patch series in PR if all checks pass label Jun 2, 2024
@lgao4
Copy link
Contributor

lgao4 commented Jun 2, 2024

I don't see the error in the commit message. Please upload this patch based on the latest tree, and try again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lgao4 lgao4 lgao4 approved these changes

Assignees
No one assigned
Labels
push Auto push patch series in PR if all checks pass
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@shenbagadevir @lgao4