Skip to content

[platform-celestica]: Update fancontrol service for Seastone-DX010 device#3690

Merged
lguohan merged 2 commits intosonic-net:masterfrom
mudsut4ke:master-update-fancontrol
Dec 16, 2019
Merged

[platform-celestica]: Update fancontrol service for Seastone-DX010 device#3690
lguohan merged 2 commits intosonic-net:masterfrom
mudsut4ke:master-update-fancontrol

Conversation

@mudsut4ke
Copy link
Contributor

- What I did

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

- 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

  • Run thermal test under

Signed-off-by: Wirut Getbamrung [email protected]

@yxieca
Copy link
Contributor

yxieca commented Jun 25, 2020

201811 branch needs this change but this PR cannot be cherry-picked cleanly.

abdosi pushed a commit that referenced this pull request Jun 28, 2020
…vice (#3690)

* [platform/cel]: add fancontrol service support for dx010

* [device/celestica]: add hysteresis temp to dx010 fancontrol configuration
praveen-li pushed a commit to praveen-li/sonic-buildimage that referenced this pull request May 28, 2021
…vice (sonic-net#3690)

* [platform/cel]: add fancontrol service support for dx010

* [device/celestica]: add hysteresis temp to dx010 fancontrol configuration

 Conflicts:
	device/celestica/x86_64-cel_seastone-r0/fancontrol

Signed-off-by: Zhenggen Xu <[email protected]>
mssonicbld added a commit that referenced this pull request Jun 8, 2025
…lly (#22886)

#### Why I did it
src/sonic-swss
```
* 70bf85f3 - (HEAD -> 202505, origin/202505) Fix indentation in template file (#3693) (22 hours ago) [mssonicbld]
* 8f714010 - Merge pull request #3690 from dgsudharsan/fpm_fix_2505 (23 hours ago) [Jianquan Ye]
* 06004451 - [202505][fpmsyncd]Fixing fpmsyncd to handle routes without protocol (35 hours ago) [Sudharsan Dhamal Gopalarathnam]
```
#### How I did it
#### How to verify it
#### Description for the changelog
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.

5 participants