Page 2 of 2

Re: ETB with single TPS input - 2016+ GM/Daewoo

Posted: Tue Mar 07, 2023 7:48 pm
by MadGoatCustoms
It works. The engine is running on SENT TPS.
One suggestion, remove the other pin options from the dropdown for SENT input. Why would you give the user an option if there's only one pin that can possibly work?

Re: ETB with single TPS input - 2016+ GM/Daewoo

Posted: Tue Mar 07, 2023 10:02 pm
by AndreyB

Re: ETB with single TPS input - 2016+ GM/Daewoo

Posted: Wed Mar 08, 2023 1:00 pm
by Dron_Gus
MadGoatCustoms wrote:
Tue Mar 07, 2023 7:48 pm
Why would you give the user an option if there's only one pin that can possibly work?
Because you are the first real user of SENT! Congratulations! And thank you for testing!

Re: ETB with single TPS input - 2016+ GM/Daewoo

Posted: Thu Jul 06, 2023 12:15 am
by tmbryhn
I got a 12675730AA throttle body on the bench planned to be fitted to a new engine project running Discovery_F407 FW on some custom hardware.
I just discovered it has what appears to be SENT TPS instead of regular dual analog output. I opened the gearbox cover and scoped the output; clean 5V square waves.

Would it be possible to have SENT implemented on other builds than MRE?

Re: ETB with single TPS input - 2016+ GM/Daewoo

Posted: Thu Jul 06, 2023 12:28 am
by AndreyB
tmbryhn wrote:
Thu Jul 06, 2023 12:15 am
Would it be possible to have SENT implemented on other builds than MRE?
Sure let's try that on discovery firmware https://github.com/rusefi/rusefi/commit/e457d3f543070a726d332571bd79998e904ec9f9

Re: ETB with single TPS input - 2016+ GM/Daewoo

Posted: Thu Jul 06, 2023 5:29 am
by tmbryhn
Thanks. Just downloaded nightly built discovery (updated 5 hours ago), but can't find SENT settings in TPS menu.
Is it updated yet?

Re: ETB with single TPS input - 2016+ GM/Daewoo

Posted: Thu Jul 06, 2023 5:48 am
by AndreyB

Re: ETB with single TPS input - 2016+ GM/Daewoo

Posted: Sun Jul 30, 2023 12:34 pm
by tmbryhn
Finally got to test SENT on custom hardware running latest release candidate of Discovery_F407.

I've removed Capacitor on RC filter asscosiated with analog input goint to PC4 pin for a clean square wave signal.
If I configure the TPS as a regular analog input, I can see the jitter from the SENT square wave signal coming in on the analog chanel.

I proceed to disable analog inputs, and define TPS input as shown in attached photo. I get "No ICU" error upon burning changes.
Screenshot_20230730_143121_com.android.gallery3d.jpg
Screenshot_20230730_143121_com.android.gallery3d.jpg (523.76 KiB) Viewed 6382 times
Is there a problem with using arbitrary pins on the MCU for SENT input that deviates from what's standard in the MRE build?

Re: ETB with single TPS input - 2016+ GM/Daewoo

Posted: Sun Jul 30, 2023 12:45 pm
by AndreyB
Current implementation requires ICU which stands for "input capture unit" which allows for precision timing capture. No support for arbitrary pins not sure if that's even possible.

So you have to use one of the (few) pins with ICU.

Re: ETB with single TPS input - 2016+ GM/Daewoo

Posted: Sun Jul 30, 2023 12:57 pm
by tmbryhn
Understood. I'll switch over to one of my analog temp inputs connected to PA1 and remove RC filter capacitor on that channel 👍🏼
Thank you for swift response! 🙏🏼

Re: ETB with single TPS input - 2016+ GM/Daewoo

Posted: Tue Aug 01, 2023 8:17 am
by tmbryhn
Tested yesterday. SENT w. GM-TB works like a charm on Discovery_F407 when signal is routed to MCU pin PA1 :D