Skip to content

Conversation

dbestm
Copy link
Contributor

@dbestm dbestm commented Mar 8, 2016

No description provided.

@0xc0170
Copy link
Contributor

0xc0170 commented Mar 9, 2016

How this differs from #17 ? There were some problems identified by that PR, thus was put on hold.

@dbestm
Copy link
Contributor Author

dbestm commented Mar 9, 2016

DMA is used and the stop management feature is available.

@dbestm
Copy link
Contributor Author

dbestm commented Mar 9, 2016

the possibility to send frame with no data also.

@betzw
Copy link
Contributor

betzw commented Mar 21, 2016

Pls. refer to my comments in ARMmbed/mbed-hal-st-stm32f401re@2504124!

betzw referenced this pull request in ARMmbed/mbed-hal-st-stm32f401re Mar 21, 2016
@bcostm
Copy link
Contributor

bcostm commented Mar 22, 2016

No objection to merge this PR ? The F401 build failed without it.

@betzw
Copy link
Contributor

betzw commented Mar 22, 2016

I think there is a problem around the macro MODULE_SIZE_I2C which should be written with an additional "S" like MODULES_SIZE_I2C in order to not break the compilation of mbed-drivers anymore (for more information pls. refer to ARMmbed/mbed-hal-st-stm32f401re@2504124)!!!
Furthermore, the definition of this macro must be made in some header file which gets also included by mbed-drivers e.g. target_config.h.

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.

4 participants