Skip to content

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

Open
jostar-yang wants to merge 4 commits intosonic-net:masterfrom
jostar-yang:minipack_20220117_api2
Open

[Minipack] Support API2.0 and fix drv in kernel5.10#9776
jostar-yang wants to merge 4 commits intosonic-net:masterfrom
jostar-yang:minipack_20220117_api2

Conversation

@jostar-yang
Copy link
Copy Markdown
Contributor

Signed-off-by: Jostar Yang [email protected]

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)

@lgtm-com
Copy link
Copy Markdown

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
Copy Markdown

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
Copy Markdown

linux-foundation-easycla bot commented Oct 4, 2022

CLA Missing ID CLA Not Signed

@lgtm-com
Copy link
Copy Markdown

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'

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