Skip to content

In targets.json, add the "IPV4" setting for RZ_A1H #2018

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

Closed
wants to merge 2 commits into from
Closed

In targets.json, add the "IPV4" setting for RZ_A1H #2018

wants to merge 2 commits into from

Conversation

TomoYamanaka
Copy link
Contributor

Hi

We added "IPV4" of Compile Macro in "targets.json" file.
We failed in the reletion test of IPV4 because "IPV4" was not entered into our Build Target setting.

Regards,
Yamanaka

This is an update to RTX kernel for Cortex-A version to allow c++ libc array init to be called after kernel start.
Ref: #1730
We added "IPV4" of Compile Macro in "targets.json" file.
We failed in the reletion test of IPV4 because "IPV4" was not entered into our Build Target setting.
low_level_init_needed_local = __low_level_init();
if (low_level_init_needed_local) {
__iar_data_init3();
mbed_sdk_init();
Copy link
Contributor

Choose a reason for hiding this comment

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

please look at https://github.com/mbedmicro/mbed/pull/1998/files, mbed_main() should be called here?

Copy link
Contributor

Choose a reason for hiding this comment

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

Should be done in pre_main function

@sg-
Copy link
Contributor

sg- commented Jun 29, 2016

@TomoYamanaka can you rebase this? Looks like there is a duplicate commit that was merged already.

@TomoYamanaka
Copy link
Contributor Author

Thank you for your comments.

I will rebase this PR.
And, the changes to the "mbed_main()" which has been pointed out from 0xc0170 and bridadan will issue as a new PR.

@TomoYamanaka
Copy link
Contributor Author

In order to rebase, I close this PR.

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