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

rev.A board review #11

Closed
andreika-git opened this issue Dec 25, 2021 · 9 comments
Closed

rev.A board review #11

andreika-git opened this issue Dec 25, 2021 · 9 comments
Assignees

Comments

@andreika-git
Copy link
Collaborator

andreika-git commented Dec 25, 2021

@rusefillc @ElDominio Can you please review the schematics?

alphax_2ch.pdf

@ElDominio
Copy link
Collaborator

Looks fine to me!

@mck1117
Copy link

mck1117 commented Dec 26, 2021

We should go ahead and at least connect IN_CRANK- to U2 pin 3, and U2 pin 4 to IN_D3. This will get us at least the same impedance on both VR pins - important that it gets clipped symmetrically, or it can push the VR signal "off center".

If the area is available, we should add a second pullup so we get a nearly-free extra digital input channel as well.

@andreika-git
Copy link
Collaborator Author

Updated the schematics:

  • more stm32_stop friendly;
  • controlling the analog power supply (TLS115) and bluetooth module supply;
  • misc. minor improvements;
    alphax_2ch.pdf

@mck1117
Copy link

mck1117 commented Dec 27, 2021

Dunno if we care, but we could merge Q2 and Q15 and have only a single pullup control for both VR/hall switchable inputs, since there's no way to use only one as hall at a time.

@ElDominio
Copy link
Collaborator

Some late 90s/early 2000s cars are VR crank/Hall cam, would they still work?

@rusefillc
Copy link
Contributor

@ElDominio yes they would. the discussion is about VR+ and VR- both becoming separate hall inputs if you have hall crank.

@andreika-git
Copy link
Collaborator Author

@ElDominio @rusefillc The board is ready for review!
alphax_2ch-a

This was referenced Dec 28, 2021
Closed
@Qwerty-OFF
Copy link
Collaborator

Немного мелочей по печатной плате:

  1. У U1 включить сплошную заливку полигона к центральному паду
  2. Обратить немного внимания на DRC, там есть ругательства на шелкографию, которые возможно легко пофиксятся
  3. Инвертировать цвета QR кода на плате, иначе может плохо распознаваться (только если это не белая плата с черной шелкографией). Также стоит сделать его из картинки более высокого разрешения, чтобы не было видно артефактов конвертации (сглаживания углов). Как пример вот:
    image
  4. Добавить переходных отверстий GND, чтобы не было бесполезной висящей в воздухе меди в ситуациях подобных этой:
    image
    И стараться избегать появления таких отростков там где они не нужны
    image
    image
  5. Попутно где глаз зацепится можно исправить такие некрасивости
    image
  6. У транзисторов для чуть более лучшего теплоотвода можно сделать полигон к переходным отверстиям, или разместить их непосредственно в паде (только небольшого диаметра, чтобы припой не утекал)
    image

@andreika-git
Copy link
Collaborator Author

Update:

  • better QR codes and other minor improvements (thanks @Qwerty-OFF)
  • move VR_ANALOG from SENS1 to RES3 (both ADC3)
    P.S. The board files are ready for BOM review!

@andreika-git andreika-git changed the title rev.A schematics review rev.A board review Dec 31, 2021
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

5 participants