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

can_write(): return error code when no tx mailboxes are available. #3829

Merged
merged 3 commits into from
Mar 9, 2017

Conversation

svogl
Copy link
Contributor

@svogl svogl commented Feb 23, 2017

Notes:

  • Pull requests will not be accepted until the submitter has agreed to the contributer agreement.
  • This is just a template, so feel free to use/remove the unnecessary things

Description

When no tx mailbox is available, can_write should return 0. This patch also removes the subsequent if() which is not needed any more and fixes a hard-coded value.

@0xc0170
Copy link
Contributor

0xc0170 commented Feb 23, 2017

Can you please sign https://developer.mbed.org/contributor_agreement/ ?

@0xc0170
Copy link
Contributor

0xc0170 commented Feb 23, 2017

@svogl
Copy link
Contributor Author

svogl commented Feb 23, 2017

@0xc0170 - did this already (my mbed username is svogl - how does this link into here?)
"You accepted the Contributor Agreement on Mon 20 Feb 2017."

@bcostm
Copy link
Contributor

bcostm commented Feb 23, 2017

LGTM

@0xc0170
Copy link
Contributor

0xc0170 commented Feb 27, 2017

/morph test

@0xc0170
Copy link
Contributor

0xc0170 commented Feb 27, 2017

Merge remote-tracking branch 'upstream/master'

Prefer rebase to bring it up to date. We will squash this patch, as its just one commit.

@mbed-bot
Copy link

Result: SUCCESS

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

/morph test

Output

mbed Build Number: 1601

All builds and test passed!

@svogl
Copy link
Contributor Author

svogl commented Mar 1, 2017

@0xc0170 - fine with me as long as it gets fixed :) anything I need to do?

@0xc0170
Copy link
Contributor

0xc0170 commented Mar 6, 2017

initial support for the STM32F303CB

Please remove this commit from this PR. It should be separate PR. Let us know once done !

@svogl
Copy link
Contributor Author

svogl commented Mar 7, 2017

Ok, did not know that the PR would got for HEAD revision; I dropped the commit and merged in the current upstream state.

@svogl
Copy link
Contributor Author

svogl commented Mar 7, 2017

@0xc0170 -- checks passed, should be ok now.

@0xc0170
Copy link
Contributor

0xc0170 commented Mar 7, 2017

Thanks for fixing the commits. We will run CI

/morph test

@mbed-bot
Copy link

mbed-bot commented Mar 7, 2017

Result: SUCCESS

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

/morph test

Output

mbed Build Number: 1652

All builds and test passed!

@0xc0170
Copy link
Contributor

0xc0170 commented Mar 9, 2017

Please rebase instead of merging (updating your branch with master), we will squash this as its just one commit. Thanks

@0xc0170 0xc0170 merged commit 1c4e0d7 into ARMmbed:master Mar 9, 2017
aisair pushed a commit to aisair/mbed that referenced this pull request Apr 30, 2024
Ports for Upcoming Targets


Fixes and Changes

3716: fix for issue #3715: correction in startup files for ARM and IAR, alignment of system_stm32f429xx.c files ARMmbed/mbed-os#3716
3741: STM32 remove warning in hal_tick_32b.c file ARMmbed/mbed-os#3741
3780: STM32L4 : Fix GPIO G port compatibility ARMmbed/mbed-os#3780
3831: NCS36510: SPISLAVE enabled (Conflict resolved) ARMmbed/mbed-os#3831
3836: Allow to redefine nRF's PSTORAGE_NUM_OF_PAGES outside of the mbed-os ARMmbed/mbed-os#3836
3840: STM32: gpio SPEED - always set High Speed by default ARMmbed/mbed-os#3840
3844: STM32 GPIO: Typo correction. Update comment (GPIO_IP_WITHOUT_BRR) ARMmbed/mbed-os#3844
3850: STM32: change spi error to debug warning ARMmbed/mbed-os#3850
3860: Define GPIO_IP_WITHOUT_BRR for xDot platform ARMmbed/mbed-os#3860
3880: DISCO_F469NI: allow the use of CAN2 instance when CAN1 is not activated ARMmbed/mbed-os#3880
3795: Fix pwm period calc ARMmbed/mbed-os#3795
3828: STM32 CAN API: correct format and type ARMmbed/mbed-os#3828
3842: TARGET_NRF: corrected spi_init() to properly handle re-initialization ARMmbed/mbed-os#3842
3843: STM32L476xG: set APB2 clock to 80MHz (instead of 40MHz) ARMmbed/mbed-os#3843
3879: NUCLEO_F446ZE: Add missing AnalogIn pins on PF_3, PF_5 and PF_10. ARMmbed/mbed-os#3879
3902: Fix heap and stack size for NUCLEO_F746ZG ARMmbed/mbed-os#3902
3829: can_write(): return error code when no tx mailboxes are available ARMmbed/mbed-os#3829
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