Skip to content

[201811][platform/cel]: Port fancontrol changes for dx010/e1031 to 201811 branch#4867

Merged
jleveque merged 7 commits intosonic-net:201811from
mudsut4ke:201811_dx010_fsc
Jul 4, 2020
Merged

[201811][platform/cel]: Port fancontrol changes for dx010/e1031 to 201811 branch#4867
jleveque merged 7 commits intosonic-net:201811from
mudsut4ke:201811_dx010_fsc

Conversation

@mudsut4ke
Copy link
Copy Markdown
Contributor

@mudsut4ke mudsut4ke commented Jun 29, 2020

- What I did

  • Update fancontrol service for Seastone-DX010/E1031 device to support hysteresis temperature threshold and difference config for each unit fan direction type (B2F/F2B) follow master branch

- How I did it

  • Add fancontrol config file fancontrol-B2F,fancontrol-F2B
  • Add fancontrol script fancontrol.sh
  • Modify fancontrol daemon to support i2c base config

- How to verify it

Signed-off-by: Wirut Getbamrung wgetbumr@celestica.com

@mudsut4ke
Copy link
Copy Markdown
Contributor Author

retest broadcom please

@yxieca yxieca requested a review from jleveque July 1, 2020 03:04
Copy link
Copy Markdown
Contributor

@jleveque jleveque left a comment

Choose a reason for hiding this comment

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

There is inconsistent alignment in a number of these files due to a combination of tabs and spaces. Please replace all tabs with four spaces throughout.

@jleveque jleveque changed the title [platform/cel]: Porting back dx010/e1031 FSC to 201811 branch [201811][platform/cel]: Port fancontrol changes for dx010/e1031 to 201811 branch Jul 1, 2020
@mudsut4ke mudsut4ke marked this pull request as ready for review July 1, 2020 07:35
@mudsut4ke
Copy link
Copy Markdown
Contributor Author

There is inconsistent alignment in a number of these files due to a combination of tabs and spaces. Please replace all tabs with four spaces throughout.

@jleveque , updated in a5d529d

@jleveque jleveque merged commit c9d86f0 into sonic-net:201811 Jul 4, 2020
yxieca added a commit that referenced this pull request Sep 29, 2020
yxieca added a commit that referenced this pull request Sep 30, 2020
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