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

Fix temp sensor issue #76

Merged
merged 5 commits into from
Nov 25, 2024
Merged

Conversation

sasataku
Copy link
Member

I tested the temperature sensor implements added in the 2f93c95 on the SC-Sat1 Payload board, but it did not work.
Therefore, fixes the issue in this PR.

The primary cause was the difference in the I2C slave address.

In addition, this PR includes other improvements as well.

The I2C settings related to the temperature sensor were added to
`uart1.overlay`, which is an optional overlay used for debugging
purposes when UART1 is in use. Since data acquisition from the
temperature sensor is required even when UART1 is not used, these
settings will be moved to `app.overlay`.

Signed-off-by: Takuya Sasaki <takuya.sasaki@spacecubics.com>
The temp sensor on the Payload Board of SC-Sat1 has a slave address
of `4E`, so fixes it.

Signed-off-by: Takuya Sasaki <takuya.sasaki@spacecubics.com>
If data acquisition from the temperature sensor fails, the CSP packet
is not sent. In such cases, the CSP packet that was read needs to be
released. If data acquisition succeeds, the packet is automatically
released within `csp_send()`, so manual release is not necessary.

Signed-off-by: Takuya Sasaki <takuya.sasaki@spacecubics.com>
Removes the unused defines.

Signed-off-by: Takuya Sasaki <takuya.sasaki@spacecubics.com>
Updates the version for Pico to v1.0.1.

Signed-off-by: Takuya Sasaki <takuya.sasaki@spacecubics.com>
@sasataku sasataku requested a review from yashi as a code owner November 25, 2024 13:49
@sasataku sasataku merged commit 177709f into spacecubics:main Nov 25, 2024
20 checks passed
@sasataku sasataku deleted the fix-temp-sensor-issue branch November 25, 2024 13:52
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.

1 participant