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

[Minipack] Support API2.0 and fix drv in kernel5.10 #9776

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

jostar-yang
Copy link
Contributor

Signed-off-by: Jostar Yang jostar_yang@accton.com.tw

Why I did it

Support API2.0 and fix drv in kernel5.10

How I did it

  1. Implement sonic_platform needed code and files
  2. Fix drv build fail on kernel5.10

How to verify it

Test show cmd and sensors.
root@sonic:/home/admin# show platform fan
Drawer LED FAN Speed Direction Presence Status Timestamp


FanTray1 green FAN-1F 52% intake Present OK 20220117 08:00:14
FanTray1 green FAN-1R 52% intake Present OK 20220117 08:00:16
FanTray2 green FAN-2F 47% intake Present OK 20220117 08:00:20
FanTray2 green FAN-2R 50% intake Present OK 20220117 08:00:24
FanTray3 green FAN-3F 52% intake Present OK 20220117 08:00:26
FanTray3 green FAN-3R 52% intake Present OK 20220117 08:00:28
FanTray4 green FAN-4F 47% intake Present OK 20220117 08:00:30
FanTray4 green FAN-4R 50% intake Present OK 20220117 07:59:53
FanTray5 green FAN-5F 52% intake Present OK 20220117 07:59:56
FanTray5 green FAN-5R 50% intake Present OK 20220117 08:00:01
FanTray6 green FAN-6F 50% intake Present OK 20220117 08:00:03
FanTray6 green FAN-6R 50% intake Present OK 20220117 08:00:05
FanTray7 green FAN-7F 52% intake Present OK 20220117 08:00:05
FanTray7 green FAN-7R 52% intake Present OK 20220117 08:00:05
FanTray8 green FAN-8F 50% intake Present OK 20220117 08:00:05
FanTray8 green FAN-8R 50% intake Present OK 20220117 08:00:05
N/A red PSU-1 FAN-1 N/A N/A Not Present N/A 20220117 08:00:05
N/A red PSU-2 FAN-1 N/A N/A Not Present N/A 20220117 08:00:05
N/A red PSU-3 FAN-1 N/A N/A Not Present N/A 20220117 08:00:05
N/A green PSU-4 FAN-1 100% intake Present OK 20220117 08:00:05
root@sonic:/home/admin#

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

Description for the changelog

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

Signed-off-by: Jostar Yang <jostar_yang@accton.com.tw>
@lgtm-com
Copy link

lgtm-com bot commented Jan 17, 2022

This pull request introduces 8 alerts and fixes 1 when merging 4f9219f into f166b99 - view on LGTM.com

new alerts:

  • 4 for Unused local variable
  • 3 for Unused import
  • 1 for First parameter of a method is not named 'self'

fixed alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Jan 19, 2022

This pull request fixes 19 alerts when merging 86f1bad into d5b0261 - view on LGTM.com

fixed alerts:

  • 14 for Unused import
  • 4 for Unused local variable
  • 1 for First parameter of a method is not named 'self'

@jostar-yang jostar-yang requested a review from a team as a code owner June 10, 2022 02:01
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Oct 4, 2022

CLA Missing ID CLA Not Signed

@lgtm-com
Copy link

lgtm-com bot commented Oct 4, 2022

This pull request introduces 1 alert and fixes 19 when merging e355126 into 95f4af3 - view on LGTM.com

new alerts:

  • 1 for Except block handles 'BaseException'

fixed alerts:

  • 14 for Unused import
  • 4 for Unused local variable
  • 1 for First parameter of a method is not named 'self'

Signed-off-by: brandonchuang <brandon_chuang@edge-core.com>
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.

2 participants