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 S6000 tmp75 temperature update #3870

Merged
merged 1 commit into from
Dec 13, 2019

Conversation

paavaanan
Copy link
Contributor

- What I did

  • Change tmp75 sensors default temperature value from 25/50 to 70/80 degree.

- How I did it

  • Updated sensors.conf for DellEMC S6000.

- How to verify it

- Description for the changelog

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

@paavaanan
Copy link
Contributor Author

This need to be cherry-picked for 201911 branch

@sujinmkang
Copy link
Collaborator

what does 70/80 means? what will be the fan speed for each temperature range?

@paavaanan
Copy link
Contributor Author

  • 80 is max temperature and 70 is max_hyesteris value. Switch temperature is safe between these values. You can read more about these (max, max_hyst) sensor.conf attributes in
    https://linux.die.net/man/5/sensors.conf

what will be the fan speed for each temperature range?

  • fan-speed/temperature corrleation is handled when we add fancontrol support.

@jleveque
Copy link
Contributor

jleveque commented May 8, 2020

Removed "Request for 201911 Branch" label. We should not cherry-pick a PR forward. PR's should be cherry-picked backward to prevent regressions. I expect Dell to open a separate PR against the master branch, and if it cannot be cherry-picked into the 201911 branch, a separate PR should be opened against that branch.

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.

4 participants