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

[FL-1911] FuriHal: i2c refactoring #847

Merged
merged 5 commits into from
Nov 28, 2021
Merged

[FL-1911] FuriHal: i2c refactoring #847

merged 5 commits into from
Nov 28, 2021

Conversation

skotopes
Copy link
Member

@skotopes skotopes commented Nov 26, 2021

What's new

  • FuriHal: i2c refactoring

Verification

  • Compile and upload
  • Ensure that i2c devices functioning correctly

Checklist (do not modify)

  • PR has description of feature/bug or link to Confluence/Jira task
  • Description contains actions to verify feature/bugfix
  • I've built this code, uploaded it to the device and verified feature/bugfix

@github-actions
Copy link

github-actions bot commented Nov 26, 2021

Click here to flash the 76954597 version of this branch via WebUSB.

@skotopes skotopes merged commit cf591ef into dev Nov 28, 2021
@skotopes skotopes deleted the aku/1911_i2c_refactoring branch November 28, 2021 18:28

/** Perform I2C tx transfer
*
* @param instance I2C_TypeDef instance
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FuriHalI2cBusHandle

const uint8_t address,
const uint8_t* data,
const uint8_t size,
uint32_t timeout);

/** Perform I2C rx transfer
*
* @param instance I2C_TypeDef instance
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FuriHalI2cBusHandle

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep, I'll update it before hackathon.

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.

None yet

3 participants