Feature request: engineConfig field for secondary fan.

It's all about the code!
Post Reply
spags
contributor
contributor
Posts: 136
Joined: Mon Oct 13, 2014 1:04 am

Feature request: engineConfig field for secondary fan.

Post by spags »

Hi,

My car as many other has two radiator fans, controlled separately. One for the cooling system and the other one for the a/c system.
Could we add a second fan relay pin field to the engine config struct?
User avatar
AndreyB
Site Admin
Posts: 14727
Joined: Wed Aug 28, 2013 1:28 am
Location: Jersey City
Github Username: rusefillc
Slack: Andrey B

Re: Feature request: engineConfig field for secondary fan.

Post by AndreyB »

spags wrote:Could we add a second fan relay pin field to the engine config struct?
I would say that's exactly the use-case for FSIO?

You might want something like

Code: Select all

	boardConfiguration->fsioPins[0] = GPIOD_11;
	strcpy(boardConfiguration->le_formulas[0], "coolant 80 >");
This case rusEfi does not have to know about second fan relay, and you can customize the logic of that as if it was FSIO output #1? See http://rusefi.com/wiki/index.php?title=Manual:Flexible_Logic see http://rusefi.com/forum/viewtopic.php?f=5&t=612
Very limited telepathic abilities - please post logs & tunes where appropriate - http://rusefi.com/s/questions

Always looking for C/C++/Java/PHP developers! Please help us see https://rusefi.com/s/howtocontribute
User avatar
AndreyB
Site Admin
Posts: 14727
Joined: Wed Aug 28, 2013 1:28 am
Location: Jersey City
Github Username: rusefillc
Slack: Andrey B

Re: Feature request: engineConfig field for secondary fan.

Post by AndreyB »

PS: fsioPins are already available via TunerStudio
there would be 'fsio_setting' - user-accessible settings - available on TS pretty soon
Very limited telepathic abilities - please post logs & tunes where appropriate - http://rusefi.com/s/questions

Always looking for C/C++/Java/PHP developers! Please help us see https://rusefi.com/s/howtocontribute
spags
contributor
contributor
Posts: 136
Joined: Mon Oct 13, 2014 1:04 am

Re: Feature request: engineConfig field for secondary fan.

Post by spags »

New fsio feature seems to cause a HardFault.

Steps to reproduce:
Flash with 5698.
Reboot board.
set_engine_type 24
Hardfault


If i remove:
setFsio(engineConfiguration, 0, GPIOD_11, "coolant 80 >");
from my config, everything works fine.
User avatar
AndreyB
Site Admin
Posts: 14727
Joined: Wed Aug 28, 2013 1:28 am
Location: Jersey City
Github Username: rusefillc
Slack: Andrey B

Re: Feature request: engineConfig field for secondary fan.

Post by AndreyB »

spags wrote:New fsio feature seems to cause a HardFault.
Yep, there were issues :(

5708 should be better.
Very limited telepathic abilities - please post logs & tunes where appropriate - http://rusefi.com/s/questions

Always looking for C/C++/Java/PHP developers! Please help us see https://rusefi.com/s/howtocontribute
spags
contributor
contributor
Posts: 136
Joined: Mon Oct 13, 2014 1:04 am

Re: Feature request: engineConfig field for secondary fan.

Post by spags »

Ok, not crashing the board this time. Will do functional tests later today.
User avatar
AndreyB
Site Admin
Posts: 14727
Joined: Wed Aug 28, 2013 1:28 am
Location: Jersey City
Github Username: rusefillc
Slack: Andrey B

Re: Feature request: engineConfig field for secondary fan.

Post by AndreyB »

spags wrote:Ok, not crashing the board this time. Will do functional tests later today.
fsioinfo should be showing the state of configuration in not-yet-very-readable form
Very limited telepathic abilities - please post logs & tunes where appropriate - http://rusefi.com/s/questions

Always looking for C/C++/Java/PHP developers! Please help us see https://rusefi.com/s/howtocontribute
Post Reply