Skip to content

[STM32 NUCLEO] Init MAC address #2812

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

Merged
merged 3 commits into from
Sep 30, 2016
Merged

Conversation

jeromecoutant
Copy link
Collaborator

Description

New Pull Request after #2808 and #2743

@andreaslarssonublox

Status

READY

Copy link
Contributor

@sg- sg- left a comment

Choose a reason for hiding this comment

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

@0xc0170 @geky I think its time mac address gets looked at. This seems ok for now but leaves a lot to be desired


void mbed_mac_address(char *mac) {
if (mbed_otp_mac_address(mac)) {
return;
Copy link
Contributor

Choose a reason for hiding this comment

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

looks like too many spaces here

if (mbed_otp_mac_address(mac)) {
return;
} else {
mbed_default_mac_address(mac);
Copy link
Contributor

Choose a reason for hiding this comment

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

same comment

@sg-
Copy link
Contributor

sg- commented Sep 28, 2016

@andreaslarssonublox does this work for you?

@andreaslarssonublox
Copy link

I think it looks ok.
@sg-

@sg-
Copy link
Contributor

sg- commented Sep 28, 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: null

Examples Build failed!

@bridadan
Copy link
Contributor

Oops this was my fault! Messing with CI again, retriggering now.

/morph test

@mbed-bot
Copy link

Result: SUCCESS

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

/morph test

Output

mbed Build Number: 992

All builds and test passed!

@sg- sg- merged commit d8e484d into ARMmbed:master Sep 30, 2016
@jeromecoutant jeromecoutant deleted the PR_Add_MAC branch October 3, 2016 07:58
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.

6 participants