rusefi / hellen-NB2-issues

0 stars 1 forks source link

AFR input changed #35

Closed BeerMoneyMotorsports closed 2 years ago

BeerMoneyMotorsports commented 2 years ago

from Rev D to E, the input for AFR has changed which causes a conflict in the yaml.

Rev D- Pin- 4w Input- IN_O2S STM- PA0

Rev E- Pin- 4w Input- IN_SENS1 STM- PF3

rusefillc commented 2 years ago

https://github.com/andreika-git/hellen-one/wiki/module-input says that IN_SENS1 is NOT an analog input

andreika-git commented 2 years ago

I thought 3L is the AFR input: image

4W and 4AA are OEM narrow-band sensors (currently not used in the firmware?).

BeerMoneyMotorsports commented 2 years ago

3L is not present on the OEM harness. We use the factory front o2 input wire, 4W, for wideband. This is a common industry practice for the pnp setups.

All the hellens using external sensor are wired to the factory o2 input wire.

andreika-git commented 2 years ago

I need Andrey's @rusefillc help for this one. Since the first rev.A, he told me that 3L should be used for AFR, and not OEM, because adding a new pin is better that cutting the existing OEM harness.

I can return 3L back to life as it was in rev.D but in that case we would have a new problem with IN_TPS2 (needed by ETB). We would need to find a free ADC channel for TPS2.

BeerMoneyMotorsports commented 2 years ago

After some deliberation, lets do the below.

Apply jumper to 4W connect output 4w to 3L with jumper

This will allow me to tie them together so that either can be used. Factory o2 is never used.

BeerMoneyMotorsports commented 2 years ago

image

andreika-git commented 2 years ago

What's the default configuration of these jumpers? Which one is DNP?

andreika-git commented 2 years ago

image

BeerMoneyMotorsports commented 2 years ago

DNP r39 Place r38

BeerMoneyMotorsports commented 2 years ago

@andreika-git do you need anything more from my end before creating the new files?

BeerMoneyMotorsports commented 2 years ago

fixed with F