PDA

View Full Version : Min. fan speed adjustment TOGETHER with automated temperature control!


kbaggen
2007-05-19, 21:56
Hi there,
would it (or is it?) possible to set the fan speed to a min. value togehther with running automatic temperature control!

Why?
As now when automaticed temperature control is running fan speed goes as low as 2% for 2 min later to go though the rougf....e.g. up to 100%.

If a minium could be set (20-30%), this going up and down in fan speed would likely be much lower with a gennerally lower noise-level, especially if you as I have installed low-noise fan, hence, running at 20-30% at all time, would cool the sustem more!

KBN

kbaggen
2007-05-19, 22:07
One remark to above........
........hence, running at MINIUM 20-30% at all time, would cool the sustem more!

C-Quel
2007-05-19, 22:59
If your fan speed is reaching 100% on auto i think you may find:

1, you have modified your fan and the cfm is not sufficient
2, you have a hardware issue
3, you have your xbox running in a very bad enviroment with a serious lack of ventilation.

Ossiostborn
2007-05-20, 09:54
I already thought about this feature before.
In my case it is exactly the same: The fan works a few minutes with 2%, goes up to 40% and comes down to 2% again. On the other hand, 10% cooling isn't safe enough for my scales und 20% is already too loud^^.

I would really like to see this feature :;): !

Alex

kbaggen
2007-05-20, 13:31
If your fan speed is reaching 100% on auto i think you may find:

1, you have modified your fan and the cfm is not sufficient
2, you have a hardware issue
3, you have your xbox running in a very bad enviroment with a serious lack of ventilation.

C-Quel,
I might have excavitagted a bit :-)

Gamester17
2007-05-20, 14:53
I agree with the suggestion that this option would be nice, and personaly *I would set my minimum to 10% ...the reason for that being that the harddrive temperature is not in the calculation for the current automated temperature control and the hotter it gets the shorter life-cycle it will have. So even if I would prefer to use the automated temperature control feature, I would still like to be able to set a minimum procentage parameter that the fan will not go under.

BoxFreak
2008-03-29, 19:44
I thought just this same idea (min fan speed for auto control) when I tested the auto temp control. Itīs just not working well (fan going from 2% to 100%).

Is that min. fan speed for auto adjustment impossible?

Gamester17
2008-03-29, 20:25
Is that min. fan speed for auto adjustment impossible?Not impossible, it is only that no programmer has volunteered to code and implement it into XBMC.

BoxFreak
2008-03-29, 21:45
Not impossible, it is only that no programmer has volunteered to code and implement it into XBMC.

So thereīs hope, thatīs nice.

Hopefully someone will volunteer. That automated temperature control would be much more usable with that feature.

DrDDT
2008-03-31, 09:37
I cannot test this, because my replacement fan is running inaudible on 12v, but here is a patch to hardcode a new minimum:


Index: utils/FanController.cpp

================================================== =================

--- utils/FanController.cpp (revision 12371)

+++ utils/FanController.cpp (working copy)

@@ -334,6 +334,6 @@

}
}

- if (calculatedFanSpeed < 1) {calculatedFanSpeed = 1;} // always keep the fan running
+ if (calculatedFanSpeed < 5) {calculatedFanSpeed = 5;} // always keep the fan running
if (calculatedFanSpeed > 50) {calculatedFanSpeed = 50;}
}


Fan speed is 1..50 (2%-100%) default, so chaning it to 5..50 would result in 10%..100%.

If this works, it's not too hard to add a configurable minimum.

DrDDT
2008-04-07, 11:46
Patch submitted: https://sourceforge.net/tracker/index.php?func=detail&aid=1936534&group_id=87054&atid=581840

abit_late
2008-04-22, 01:54
Hi DrDDT,
Would it be possible for the minimum to be lower than 10%, say 5%? 10% is way too loud for my tastes.. my boxes have been running without problems at 2% fan speed since way back!
What do you think?
Al

DrDDT
2008-04-22, 12:11
Hi DrDDT,
Would it be possible for the minimum to be lower than 10%, say 5%? 10% is way too loud for my tastes.. my boxes have been running without problems at 2% fan speed since way back!
What do you think?
Al

10% is just the default.You can change it in the GUI.

spiff
2008-04-22, 12:19
actually, unless i remember completely wrong it is 10% steps with 10 being the minimum atm. i did scratch my head a bit over it but i figured you had your reasons when i added the patch

navanman
2008-04-22, 12:26
I too have this query. I have had my fan running at 2% since the beginning of using XBMC even playing full screen visualisations in the summer.
With this new patch, the fan won't go below 10% which is quite alot more audible than running at 2%.
With the patch you submitted the fan can only be changed in 10% increments from 10% to 100%.

Is there a way to have 2% fan with temperature control or remove this patch?

DrDDT
2008-04-22, 12:47
actually, unless i remember completely wrong it is 10% steps with 10 being the minimum atm. i did scratch my head a bit over it but i figured you had your reasons when i added the patch

The patch contained:

AddInt(13, "system.minfanspeed", 13411, 5, 2, 1, 50, SPIN_CONTROL_TEXT);
(fan speed = 1-50 so multiply by 2 for percentage)

What I intended was a default of 10%, minimum of 2%, step of 2% and max 100%.

spiff
2008-04-22, 12:57
weirdness. that does indeed look correct. however it does not behave like that :/

DrDDT
2008-04-22, 13:07
weirdness. that does indeed look correct. however it does not behave like that :/

I think this code is the source of the problem.
I 'removed duplicate code' that wasn't duplicate :shocked:


else if (strSetting.Equals("system.fanspeed") || strSetting.Equals("system.minfanspeed"))
{
CSettingInt *pSettingInt = (CSettingInt*)pSetting;
CGUISpinControlEx *pControl = (CGUISpinControlEx *)GetControl(GetSetting(strSetting)->GetID());
CStdString strPercentMask = g_localizeStrings.Get(14047);
for (int i = 5; i <= pSettingInt->m_iMax; i += 5)
{
CStdString strLabel;
strLabel.Format(strPercentMask.c_str(), i*2);
pControl->AddLabel(strLabel, i);
}
pControl->SetValue(int(pSettingInt->GetData()));
}

spiff
2008-04-22, 13:41
indeed - that should say int i=pSettingInt->m_iMin; and pSettingInt->m_iStep

DrDDT
2008-04-22, 14:13
indeed - that should say int i=pSettingInt->m_iMin; and pSettingInt->m_iStep

I'll test this and submit a new patch.
I also want do doublecheck that the values 5, 2, 1, 50 are correct.
Maybe the 2 has to be a 1 for the minimum of 2%?

spiff
2008-04-22, 15:00
ah yeah, sorry, missed that one.

it has to be 1% as 1-50 is the range of the spinner. labels are 2x the range..

abit_late
2008-04-23, 00:45
thanks for your attention to this fellas, much appreciated <tips hat>

DrDDT
2008-04-23, 14:48
I just submitted a patch to fix the incorrect settings:

http://sourceforge.net/tracker/index.php?func=detail&aid=1949575&group_id=87054&atid=581840