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

Dev stm i2c f1 #3442

Merged
merged 3 commits into from
Dec 19, 2016
Merged

Dev stm i2c f1 #3442

merged 3 commits into from
Dec 19, 2016

Conversation

LMESTM
Copy link
Contributor

@LMESTM LMESTM commented Dec 14, 2016

Description

As agreed in discussion of PR #3416 this PR is the move of F1 to common i2c_api.c implementation.
So it removes the i2c_api_stm32f1.c file which was used temporarily.

The F1 HAL has been updated in order to get the needed I2C STM32 HAL API and has been aligned for btf and rxne use cases with other families that use the same I2C IP version (F2, F4, L1).

Finally the move to the common I2C file allows to enable I2C ASYNCH API for F1.

Status

READY

Related PRs

It is recommended to first integrate the below PR before this one:
#3429
#3430
They are marked as 'ready for merge' at the time I submit this PR.

Tests

MBED_A29: i2c master slave asynch OK
MBED_A20: i2c master slave OK
I2C ci test shiled test OK

This is a partial update of HAL drivers, especially for I2C driver update
Applying the same fix as in L1 and F4.
This is an alignement to F4 HAL as the same IP is used.
Now that F1 HAL has been updated to support required APIs,
the F1 family can also be moved to common code.
@sg-
Copy link
Contributor

sg- commented Dec 15, 2016

/morph test

@mbed-bot
Copy link

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 1274

Test failed!

@bridadan
Copy link
Contributor

Looks like the Public Test Result VM went down for some reason with no warning. It's back up now, restarting the job.

/morph test

@mbed-bot
Copy link

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 1277

All builds and test passed!

@bridadan
Copy link
Contributor

Initial results look good, though per @LMESTM's recommendation we should merge #3430 first before bringing this in. We should also run CI again on this after #3430 is merged.

@adbridge adbridge merged commit f2ce7eb into ARMmbed:master Dec 19, 2016
@LMESTM LMESTM deleted the dev_stm_i2c_f1 branch February 27, 2017 10:21
aisair pushed a commit to aisair/mbed that referenced this pull request Apr 30, 2024
Ports for Upcoming Targets

3459: Target: Add new target UBLOX_EVK_NINA_B1 ARMmbed/mbed-os#3459

Fixes and Changes

3430: Fix ci shield eeprom test ARMmbed/mbed-os#3430
3381: STM32F1 : map ST HAL assert into MBED assert ARMmbed/mbed-os#3381
3389: STM32F2 : map ST HAL assert into MBED assert ARMmbed/mbed-os#3389
3390: STM32F3 : map ST HAL assert into MBED assert ARMmbed/mbed-os#3390
3410: STM32L4 : map ST HAL assert into MBED assert ARMmbed/mbed-os#3410
3422: Enable CAN on DISCO_F303VC ARMmbed/mbed-os#3422
3442: Dev stm i2c f1 ARMmbed/mbed-os#3442
3460: KSDK I2C: Update the return value to match the API documentation change ARMmbed/mbed-os#3460
3472: [RZ/A1H]Fix TTB setting of RO_DATA area ARMmbed/mbed-os#3472
3451: Rename KSDK2 to MCUXpresso. This is the new name of this package ARMmbed/mbed-os#3451
3391: STM32F4 : map ST HAL assert into MBED assert ARMmbed/mbed-os#3391
3454: STM32: Refactor lp_ticker.c + rtc_api.c + sleep.c + rtc_api_hal.h files  ARMmbed/mbed-os#3454
3489: NUCLEO_F103RB - Correct CAN and PWM alternate-functions ARMmbed/mbed-os#3489
3405: Repair the transmit mailbox (0,1,2) empty interrupt flag not clear BUG ARMmbed/mbed-os#3405
3502: MCUXpresso I2C: Handle 0 byte write ARMmbed/mbed-os#3502
3365: [NUC472/M453] Support USB device ARMmbed/mbed-os#3365
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants