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

[Disco_F769NI] adding new target #2504

Merged
merged 7 commits into from
Sep 10, 2016
Merged

Conversation

ohagendorf
Copy link
Contributor

new target DISCO_F769NI

  • adding target
  • adding to rtos libs
  • adding to travis build
  • exporters to GCC_ARM and Keil5 (uvision5) tested with MBED_BLINKY: OK

test results for GCC_ARM, ARM and IAR are all OK:
disco_f767ni.zip

@ciarmcom
Copy link
Member

Can one of the admins verify this patch?

@sandeepach
Copy link

Hello Ohangendorf
Should the DIS_F769I not use hard FP for better results? Why is that we still use softfp here.
Especially since in my C code i am using CMSIS Instrinsics where ever possible, i see a high penalty with using the CMSIS Library with no FP. CMSIS does not compile with SOFTFP. It has only 3 options viz NO FP, SINGLE- precision FP and Double precision FP for M7.

Regards
Sandeep

D4 = PJ_0,
D5 = PC_8,
D6 = PF_7,
D7 = PI_3,
Copy link
Member

Choose a reason for hiding this comment

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

Hello, I don't have the board with me, but I can read from the schematics of the board that D7 is connected on PJ3, not PI3. Could you please check ?
Cheers

Copy link
Contributor Author

Choose a reason for hiding this comment

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

corrected

@adustm
Copy link
Member

adustm commented Aug 22, 2016

Hello @ohagendorf ,
I have reviewed the code as much as I could without the board.
You did an excellent work, thank you so much.
Some details should be changed in the peripheralpin.c, but the rest looks good.

Cheers
Armelle

@sandeepach
Copy link

May i please know if there is any progress on this?

@0xc0170
Copy link
Contributor

0xc0170 commented Aug 25, 2016

May i please know if there is any progress on this?

@ohagendorf Will you find time to address those comments?

@ohagendorf
Copy link
Contributor Author

I just came back from an Internet free week. I'll do the corrections during the next days.

@sandeepach
Copy link

@ohagendorf Any updates on this?
Is there any plan to support Audio APIs with mbed for 769I and 746G Discovery boards? I know they are supported with Cube, but would be good to integrate them with mbed.

@ohagendorf
Copy link
Contributor Author

I did the rework as suggested by @adustm .
Test results are the same as before but now additonal with MBED_A28 (CAN loopback test).
And I rebased it.

But there will be a conflict with PR #2571 : when this PR is merged earlier than there is some work necessary and vice versa ( @svastm )

disco_f769ni.zip

@sandeepach
Copy link

Any further updates on this? Is this good to merge?

@ohagendorf
Copy link
Contributor Author

I added the missing ethernet initialisation.

And I run the following tests:

mbed test -m DISCO_F769NI -t GCC_ARM

and

mbed test -m DISCO_F769NI -t ARM

The tests mainly run through. But maybe network test have a timing, network or similar problem at my PC. I repeated the failed tests as single test runs. Sometimes I've got a timeout sometimes they run through.

disco_f769ni.zip

@ohagendorf
Copy link
Contributor Author

When these tests run through does that mean that a "5" can be added to release_versions in targets.json?

@bridadan
Copy link
Contributor

bridadan commented Sep 8, 2016

@ohagendorf Those test results look good to me. Do you have results for the ARM and IAR toolchain?

The two network tests that had issues are based on UDP, which sometimes does fail due to the nature of UDP. If you see that they pass when you run them again individually then they should be ok.

@ohagendorf
Copy link
Contributor Author

ARM and GCC_ARM both are included in the zip file above.
IAR is missing. I'll submit the definitions and hand in the results here.
The failed network tests are OK when run them again manually.

@ohagendorf
Copy link
Contributor Author

ohagendorf commented Sep 9, 2016

I submitted the IAR definitions for the project generator.

Unfortunately I have only a Keil license but no IAR. And several tests are bigger than 32k. I attached the results of possibles test:

disco_f769ni_iar.zip

The following tests couldn't be tested (>32k):

features-frameworks-utest-tests-unit_tests-control_type
features-net-feature_ipv4-tests-mbedmicro-net-nist_internet_time_service
features-net-feature_ipv4-tests-mbedmicro-net-tcp_client_echo
features-net-feature_ipv4-tests-mbedmicro-net-tcp_client_hello_world
features-net-feature_ipv4-tests-mbedmicro-net-udp_echo_client
tests-mbed_drivers-callback
tests-mbed_drivers-dev_null
tests-mbed_drivers-rtc
tests-mbed_drivers-stl_features

as suggested by adustm
the IPV4 feature was already enabled in an earlier commit but the mbed 5 ethernet initialisation was missing
@ohagendorf
Copy link
Contributor Author

rebased - test results are the same

@sg-
Copy link
Contributor

sg- commented Sep 9, 2016

@mbed-bot: TEST

HOST_OSES=ALL
BUILD_TOOLCHAINS=ALL
TARGETS=ALL

@sg- sg- removed the needs: work label Sep 9, 2016
@sg-
Copy link
Contributor

sg- commented Sep 9, 2016

/morph test

@mbed-bot
Copy link

[Build 887]
FAILURE: Something went wrong when building and testing.

@mbed-bot
Copy link

Result: SUCCESS

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

/morph test

Output

mbed Build Number: 807

All builds and test passed!

@sg- sg- merged commit c1ee7fb into ARMmbed:master Sep 10, 2016
ohagendorf added a commit to ohagendorf/mbed-os that referenced this pull request Sep 13, 2016
bugfixing the can loop test which failed after PR ARMmbed#2504
theotherjimmy added a commit that referenced this pull request Sep 19, 2016
Release mbed-os-5.1.4

Changes:

New Targets:
2504: [Disco_F769NI] adding new target [#2504]
2654: DELTA_DFBM_NQ620 platform porting [#2654]
2615: [MTM_MTCONNECT04S] Added support for MTM_MTCONNECT04S [#2615]
2548: Nucleof303ze [#2548]

Fixes:

2678: Fixing NCS36510 compile on Linux #2678
2657: [MAX326xx] Removed echoing of characters and carriage return. #2657
2651: Use lp_timer to count time in the deepsleep tests #2651
2645: NUCLEO_F446ZE - Enable mbed5 release version #2645
2643: Fix thread self termination #2643
2634: Updated USBHost for library changes #2634
2633: Updated USBDevice to use Callback #2633
2630: Test names not dependent on disk location of root #2630
2624: CFSTORE Bugfix for realloc() moving KV area and cfstore_file_t data structures not being updated correctly #2624
2623: DISCO_L476VG - Add Serial Flow Control pins + add SERIAL_FC macro #2623
2617: STM32F2xx - Enable Serial Flow Control #2617
2613: Correctly providing directories to build_apis #2613
2607: Fix uvisor memory tracing #2607
2604: Tools - Fix fill section size variation #2604
2601: Adding ON Semiconductor copyright notice to source and header files. #2601
2597: [HAL] Fixed "intrinsic is deprecated" warnings #2597
2596: [HAL] Improve memory tracer #2596
2594: Fix TCPServer constructor #2594
2593: Add app config command line switch for test and make #2593
2589: [NUC472] Fix heap configuration error with armcc #2589
2588: Timing tests drift refactor #2588
2587: add PTEx pins as option for SPI on Hexiwear - for SD Card Interface #2587
2584: Set size of callback irq array to IrqCnt #2584
2583: github issue and PR templates #2583
2582: [GCC_CR] fix runtime hang for baremetal build #2582
2580: lwip - Add check for previously-bound socket #2580
2579: lwip - Fix handling of max sockets in socket_accept #2579
2578: Fix double free in NanostackInterface #2578
2576: Add smoke test that builds example programs with mbed-cli #2576
2575: tools-config! -  Allow an empty or mal-formed config to be passed to the config system #2575
2562: Fix GCC lazy init race condition and add test #2562
2559: [utest]: Allow the linker to remove any part of utest if not used #2559
2545: Added define guards for SEQUENTIAL_FLASH_JOURNAL_MAX_LOGGED_BLOBS so  #2545
2538: STM32F4xx - Add support of ADC internal channels (Temp, VRef, VBat) #2538
2521: [NUCLEO_F207ZG] Add MBED5 capability #2521
2514: Updated FlexCan and SAI SDK drivers #2514
2487: Runtime dynamic memory tracing #2487
2442: Malloc heap info #2442
2419: [STM32F1] Add asynchronous serial #2419
2393: [tools] Prevent trace-backs from incomplete args #2393
2245: Refactor export subsystem #2245
2130: stm32 : reduce number of device.h files #2130
adbridge pushed a commit that referenced this pull request Sep 28, 2016
bugfixing the can loop test which failed after PR #2504
aisair pushed a commit to aisair/mbed that referenced this pull request Apr 30, 2024
Changes:

New Targets:
#2504: [Disco_F769NI] adding new target [ARMmbed/mbed-os#2504]
#2654: DELTA_DFBM_NQ620 platform porting [ARMmbed/mbed-os#2654]
#2615: [MTM_MTCONNECT04S] Added support for MTM_MTCONNECT04S [ARMmbed/mbed-os#2615]
#2548: Nucleof303ze [ARMmbed/mbed-os#2548]

Fixes:

#2657: [MAX326xx] Removed echoing of characters and carriage return. [ARMmbed/mbed-os#2657]
#2651: Use lp_timer to count time in the deepsleep tests [ARMmbed/mbed-os#2651]
#2643: Fix thread self termination [ARMmbed/mbed-os#2643]
#2623: DISCO_L476VG - Add Serial Flow Control pins + add SERIAL_FC macro [ARMmbed/mbed-os#2623]
#2617: STM32F2xx - Enable Serial Flow Control [ARMmbed/mbed-os#2617]
#2601: Adding ON Semiconductor copyright notice to source and header files. [ARMmbed/mbed-os#2601]
#2597: [HAL] Fixed "intrinsic is deprecated" warnings [ARMmbed/mbed-os#2597]
#2589: [NUC472] Fix heap configuration error with armcc [ARMmbed/mbed-os#2589]
#2587: add PTEx pins as option for SPI on Hexiwear - for SD Card Interface [ARMmbed/mbed-os#2587]
#2584: Set size of callback irq array to IrqCnt [ARMmbed/mbed-os#2584]
#2582: [GCC_CR] fix runtime hang for baremetal build [ARMmbed/mbed-os#2582]
#2562: Fix GCC lazy init race condition and add test [ARMmbed/mbed-os#2562]
#2538: STM32F4xx - Add support of ADC internal channels (Temp, VRef, VBat) [ARMmbed/mbed-os#2538]
#2514: Updated FlexCan and SAI SDK drivers [ARMmbed/mbed-os#2514]
#2442: Malloc heap info [ARMmbed/mbed-os#2442]
#2419: [STM32F1] Add asynchronous serial [ARMmbed/mbed-os#2419]
#2130: stm32 : reduce number of device.h files [ARMmbed/mbed-os#2130]
#2678: Fixing NCS36510 compile on Linux [ARMmbed/mbed-os#2678]
#2607: Fix uvisor memory tracing [ARMmbed/mbed-os#2607]
#2596: [HAL] Improve memory tracer [ARMmbed/mbed-os#2596]
#2487: Runtime dynamic memory tracing [ARMmbed/mbed-os#2487]
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.

9 participants