Skip to content

Changed config for the K501UX#155

Merged
hirschmann merged 5 commits intohirschmann:masterfrom
ragekit:master
Mar 2, 2017
Merged

Changed config for the K501UX#155
hirschmann merged 5 commits intohirschmann:masterfrom
ragekit:master

Conversation

@ragekit
Copy link
Copy Markdown
Contributor

@ragekit ragekit commented Dec 20, 2016

Check first commit down there for explanation

Left fan max speed should be 8
Right fan seems to have only on and off settings, and no fine grained speed control so any value is good. put also 8 .
Also, register write config seem to be borowed from the LX which does not work like the UX, config from the Zenbook UX32LN seems to work better from what I checked while probing the EC.
ecpoll is kept at 100ms because im not sure why more or less ? cpu load maybe ? and crit temp keeped to 80 bc 90 seem very high

I fiddled around, i've never done this before but this seem correct. If anyone has better info, let me know
@hirschmann
Copy link
Copy Markdown
Owner

Thanks for the config!

I have some questions before I merge this pull request:

  1. If the right fan only has 2 possible states (on/off), why don't you set MaxSpeedValue to 1, so that min = 0, max = 1?
  2. Does the config still work if you increase the poll interval? This would result in less CPU load.
  3. Could you please add a description to the RegisterWriteRequests?
  4. Why is FanSpeedResetValue equal to MaxSpeedValue? This means the fan will be reset when set to 100% speed. Is this the intended behavior?

@ragekit
Copy link
Copy Markdown
Contributor Author

ragekit commented Dec 27, 2016

Inestigating all those.

for 1) totally, it can/should be 1 I think
2) Im going to make some test for that
3) I copied it from another config that apparently works for the UX, Im not sure how to describe them. All I know it that I need to write to them. If not, the EC seems to auto reset (if that makes sense ??)
4) this is an error, it shouldnt reset. Even reset on exit should be unchecked since it seems the uc reset anyhow.

Does that makes sense to you ?

Im a bit unsure why I need to write onto the register 160 and 166 the value 10 to make it work.

Im going to do further test

@ragekit
Copy link
Copy Markdown
Contributor Author

ragekit commented Dec 28, 2016

Ok i checked : for 3) Any value written to those register seem to work. If I dont write anything, then it doesnt work. Still a mystery as to why.

Still testing what max value I can set to polling

@hirschmann
Copy link
Copy Markdown
Owner

Great!
When you've finished testing, just update the pull request.

@ragekit
Copy link
Copy Markdown
Contributor Author

ragekit commented Jan 1, 2017

im around 200ms and still works. What would be the max you think ? 500ms ?

Still don't know how to describe those register btw, not a problem ?

@hirschmann
Copy link
Copy Markdown
Owner

I suggest to start at 3000ms and then decrease by 250 - 500ms until it works without problems.
A description would just be nice to have. I will accept the pull request without a description, as long as the config works properly.

@ragekit
Copy link
Copy Markdown
Contributor Author

ragekit commented Feb 2, 2017

I didn't forget this. I'm just swamped by work now. Ill try to find a way to properly finish this config

@keresztesizoli
Copy link
Copy Markdown

Hi ragekit,

any progress on it? When can we get your final work?
I'm waiting for it! :-)

Zoli

@ragekit
Copy link
Copy Markdown
Contributor Author

ragekit commented Feb 24, 2017

Ok, new update. Ive been able to put the polling up to 300ms but more it does not work

@ragekit
Copy link
Copy Markdown
Contributor Author

ragekit commented Feb 24, 2017

urgh no in fact 200ms is safer

@keresztesizoli
Copy link
Copy Markdown

I don't know why, but on "enabled" mode tha fan won't turn on. Even on very high temperatures!
Is it normal at this point of your config or not?

Thx,
Zoli

@ragekit
Copy link
Copy Markdown
Contributor Author

ragekit commented Feb 25, 2017

The fan should turn on starting at 65deg. Its taken from a computer with similar spec.
Im changing the pol again it seems 200ms is juuuuuuuuust almoooost not enought

@DJWassink
Copy link
Copy Markdown

DJWassink commented Feb 25, 2017

Just tested this config on my laptop, works pretty good.

Fun fact: Downloaded Prime95 today and my CPU wasn't able to get higher then ~70degrees with the fan spinning on 25%

@hirschmann
Copy link
Copy Markdown
Owner

Thanks @ragekit for your effort and thanks @DJWassink for testing!

@hirschmann hirschmann merged commit 03dc889 into hirschmann:master Mar 2, 2017
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.

4 participants