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

err_t definition conflicts with nRF52 BSP #11

Closed
Kongduino opened this issue Nov 13, 2022 · 3 comments
Closed

err_t definition conflicts with nRF52 BSP #11

Kongduino opened this issue Nov 13, 2022 · 3 comments
Assignees
Labels
Seeed_SHT35 Label for Seeed_SHT35 UAY Unassigned yet

Comments

@Kongduino
Copy link

In the Arduino BSP for nRF52 – which Xiao BLE uses – err_t is defined as uint32_t, whereas in this library it is defined as:

typedef enum {
  NO_ERROR = 0,
  ERROR_PARAM = -1,
  ERROR_COMM = -2,
  ERROR_OTHERS = -128,
} err_t;

As a quickie, I changed the name of the enum, and all references to it, to err_tp. The example now compiles on Xiao BLE. I also had to add a #define for the I2C pins:

#define SDAPIN PIN_WIRE_SDA
#define SCLPIN PIN_WIRE_SCL
@latolukasz
Copy link

I have the same issue with xiao esp32 s3

@MatthewJeffson MatthewJeffson added UAY Unassigned yet Seeed_SHT35 Label for Seeed_SHT35 labels Oct 9, 2024
@Lesords Lesords self-assigned this Oct 15, 2024
@Lesords
Copy link
Contributor

Lesords commented Oct 16, 2024

Hello,

I'm very sorry to have kept you waiting so long.

Do you still have this problem now?

@Lesords
Copy link
Contributor

Lesords commented Nov 4, 2024

Hello,

I'm going to close this issue, feel free to re-open it if you have any other questions.

[Autocommentary]

@Lesords Lesords closed this as completed Nov 4, 2024
@github-project-automation github-project-automation bot moved this from Todo to Done in Issues and PR Assemble Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Seeed_SHT35 Label for Seeed_SHT35 UAY Unassigned yet
Projects
Status: Done
Development

No branches or pull requests

4 participants