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.

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

[202012]Add config3 register which is used to get COMEX CPU board version #285

Merged
merged 1 commit into from
Jul 16, 2022

Conversation

keboliu
Copy link
Collaborator

@keboliu keboliu commented Jul 12, 2022

Signed-off-by: Kebo Liu kebol@nvidia.com

Add a new patch comes with the hw-mgmt version V.7.0010.2348

Add config3 register which is used to get COMEX CPU board version

Currently, this patch is under work: https://patchwork.kernel.org/project/platform-driver-x86/patch/20220711084559.62447-7-vadimp@nvidia.com/

when backporting to 202012 branch (kernel version 4.19), the patch was reshaped accordingly.

Signed-off-by: Kebo Liu <kebol@nvidia.com>
@saiarcot895 saiarcot895 merged commit 01e5508 into sonic-net:202012 Jul 16, 2022
@paulmenzel
Copy link
Contributor

Thank you for the patch. Next time, it’d be great, if you added a space after the tag [202012].

Subject: [PATCH] platform: mellanox: Add config3 register support

Add config3 register which is using to get COMEX CPU board version

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not see this change in Linus’ master branch. Please mention the upstream status.

dprital added a commit to dprital/sonic-buildimage that referenced this pull request Jul 16, 2022
Update sonic-linux-kernel submodule pointer to include the following:
* add new kernel patch come with hw-mgmt 7.0010.2348 ([sonic-net#285](sonic-net/sonic-linux-kernel#285))

Signed-off-by: dprital <drorp@nvidia.com>
@keboliu keboliu deleted the 202012_comex_respin branch July 18, 2022 01:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants