Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create new VR module using MAX9924 #301

Closed
ElDominio opened this issue Jun 14, 2023 · 4 comments
Closed

Create new VR module using MAX9924 #301

ElDominio opened this issue Jun 14, 2023 · 4 comments
Assignees

Comments

@ElDominio
Copy link

@Qwerty-OFF

Original design considerations for the current VR module were, in order of importance:

  1. MAX9924 was not assembled by JLCPCB, and usually not stocked
  2. Problems with compatibility with low-count triggers

I propose there be a VR module that uses MAX9924 since now problem (1) is not a factor, and (2) can still be handled by using this module in conjunction with the already-existing module for low count patterns (by having both modules on one board)

In my experience, MAX992x chips are very good at conditioning/producing a useable output with almost all triggers without the fuss of manual thresholds, and without the pains that we have been experiencing with the current module (sometimes needs caps? sometimes doesn't?)

If it has the same module-size as the current VR solution, would it not be a plug-and-play affair to use on the current line of Hellen ECU's? If so, can it be done?

@andreika-git

@rusefillc
Copy link
Collaborator

@Qwerty-OFF can we have vr-max9924 module completely footprint compatible with vr-discrete?

@rusefillc
Copy link
Collaborator

@Qwerty-OFF see interesting note at https://rusefi.com/forum/viewtopic.php?p=47921#p47921 mostly FYI

@rusefillc
Copy link
Collaborator

rusefillc commented Jul 23, 2023

@Qwerty-OFF we've realized that we forgot about something:

vr-descrete allowed us to have VR or Hall using two pinout pins and one MCU pin. The current max9924 is a loss of functionality since it's ONLY VR. At least for smaller universal boards we require VR/Hall 2 pinout pins solution like brought up on https://github.com/rusefi/alphax-8chan/issues/113

This is all related to #285

@ElDominio
Copy link
Author

This was done! And I love 9924

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants