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

[DellEMC S5232f] Updates and bug fixes for platform #5887

Merged
merged 1 commit into from
Nov 11, 2020

Conversation

srideepDell
Copy link
Contributor

@srideepDell srideepDell commented Nov 10, 2020

  • Fix platform sensors
  • fpga driver update for bug fixes
  • Update fixes for API 2.0 code

- Why I did it
Fixes identified in S5232 platform code are fixed in community code
- How I did it
Change to platform specific code under platform directory specific to s5232
- How to verify it
verified on s5232 platform logs attached
- Which release branch to backport (provide reason below if selected)

- Description for the changelog

  • Fix platform sensors
  • Fix issues reported in fpga driver
  • Update fixes for API 2.0 platform code

- A picture of a cute animal (not mandatory but encouraged)

  * Fix platform sensors
  * fpga driver update for bug fixes
  * Update fixes for API 2.0 code
@srideepDell
Copy link
Contributor Author

Ut_logs_s5232.txt

@lgtm-com
Copy link

lgtm-com bot commented Nov 10, 2020

This pull request introduces 4 alerts and fixes 1 when merging d472dd2 into aee389e - view on LGTM.com

new alerts:

  • 2 for Wrong number of arguments in a class instantiation
  • 1 for Module imports itself
  • 1 for 'import *' may pollute namespace

fixed alerts:

  • 1 for Wrong number of arguments in a class instantiation

@srideepDell srideepDell marked this pull request as ready for review November 11, 2020 05:24
@jleveque jleveque merged commit 89d9471 into sonic-net:master Nov 11, 2020
santhosh-kt pushed a commit to santhosh-kt/sonic-buildimage that referenced this pull request Feb 25, 2021
 * Fix platform sensors
 * Fix issues reported in fpga driver
 * Update fixes for API 2.0 platform code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants