Skip to content

Vacuum sensing 100hz signal fix#2

Closed
brainstorm wants to merge 9 commits into
markmaker:feature/best-for-pnpfrom
brainstorm:vacuum_sensing_100hz_signal_fix
Closed

Vacuum sensing 100hz signal fix#2
brainstorm wants to merge 9 commits into
markmaker:feature/best-for-pnpfrom
brainstorm:vacuum_sensing_100hz_signal_fix

Conversation

@brainstorm
Copy link
Copy Markdown

@brainstorm brainstorm commented Feb 7, 2023

Remove the 100Hz signal on the SS2/PB3 pin of the ATxmega TinyG MCU.

This PR is dependant on PR #1 .

@brainstorm brainstorm marked this pull request as ready for review February 7, 2023 10:29
@ian-arkver
Copy link
Copy Markdown

The last commit (ad38088) does not do what the comment and commit message claim it does.

The DIR register is a bitfield with one bit per pin. If a bit is '1' the pin is an output.

The effect of the statement is to clear bits 0 and 1 while leaving all the other bits unchanged.

@ian-arkver
Copy link
Copy Markdown

I'm not sure why configuring the ADC to differential input using PORTB.0 as the low input would make things more stable. This pin is the BOOT jumper. Is this jumper normally fitted to tie this pin to ground?

@brainstorm
Copy link
Copy Markdown
Author

There's an insane amount of nuance in Atmel ADC configurations and TinyG, hang in there, we are going deep in the rabbit hole and troubleshooting this, check this out if you are interested ;)

For instance, there's another gotcha Anthony found out today:

Screenshot 2023-02-24 at 10 10 29 pm

That 2Mhz signal issue from yesterday? Gone today after removing the capacitor.

Unfortunately our sensor stops responding as soon as it's connected to the TinyG, so we do know it's a software configuration detail, but we are trying a bunch of things and commit messages might not reflect exactly what we are actually doing (i.e we were discussing about making sure that PB3 is NOT an output, I might have gotten distracted when writing the commit message and discussing the topic with the others), sorry ;)

@brainstorm
Copy link
Copy Markdown
Author

After reflashing the "stock" firmware from Mark with ADC and the 100Hz signal and fine tuning the cap and resistor the vacuum sensing works fine, closing.

@brainstorm brainstorm closed this May 8, 2023
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

Successfully merging this pull request may close these issues.

2 participants