Skip to content

Support Flash iAP and Bootloader for GR-PEACH and GR-LYCHEE #6925

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 6 commits into from
Jun 22, 2018

Conversation

TomoYamanaka
Copy link
Contributor

@TomoYamanaka TomoYamanaka commented May 16, 2018

Description

Renesas added flash iap and bootloader feature to GR-PEACH and GR-LYCHEE.

Flash iAP was successful by tests-mbed_drivers-flashiap and tests-mbed_hal-flash.
Also Those was tested with https://github.com/ARMmbed/mbed-os-example-bootloader and was successful.

Pull request type

[ ] Fix
[ ] Refactor
[x] New target
[ ] Feature
[ ] Breaking change

0xc0170
0xc0170 previously approved these changes May 18, 2018
@0xc0170
Copy link
Contributor

0xc0170 commented May 20, 2018

/morph build

@mbed-ci
Copy link

mbed-ci commented May 20, 2018

Build : FAILURE

Build number : 2073
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/6925/

@0xc0170
Copy link
Contributor

0xc0170 commented May 20, 2018

@TomoYamanaka Please review the build failures

@TomoYamanaka
Copy link
Contributor Author

I rebased my commits due to address the build failure.

cmonr
cmonr previously approved these changes May 21, 2018
Copy link
Contributor

@cmonr cmonr left a comment

Choose a reason for hiding this comment

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

👍

@cmonr
Copy link
Contributor

cmonr commented May 21, 2018

/morph build

@mbed-ci
Copy link

mbed-ci commented May 21, 2018

Build : SUCCESS

Build number : 2086
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/6925/

Triggering tests

/morph test
/morph uvisor-test
/morph export-build
/morph mbed2-build

@mbed-ci
Copy link

mbed-ci commented May 22, 2018

@mbed-ci
Copy link

mbed-ci commented May 22, 2018

@0xc0170
Copy link
Contributor

0xc0170 commented May 22, 2018

Please review exporter failure. There are multiple warnings for redefinition (that would be nice to be fixed), but more importantly here - 3 failures reported for ARMCC

@TomoYamanaka
Copy link
Contributor Author

Exporter occurs failure only IAR according the above report, so I modified the IAR definition for export.

@TomoYamanaka
Copy link
Contributor Author

TomoYamanaka commented May 24, 2018

With respect to the error message of Exporter for UVISION,

assembling irq_ca.S...
irq_ca.S: Error:  #5: cannot open source input file "__UVISION_VERSION SETA 521": No such file or directory
irq_ca.S: __UVISION_VERSION SETA 521: 0 warnings, 1 errorirq_ca.S: Error: A1067E: Output file specified as 'mbed-os\rtos\TARGET_CORTEX\rtx5\RTX\Source\TOOLCHAIN_ARM\TARGET_CORTEX_A\irq_ca.S', but it has already been specified as '.\build\irq_ca.o'
irq_ca.S: error: A1905U: Pre-processor step failed for '__UVISION_VERSION SETA 521'

I don' know why the message is outputed.
The specified file is out of this time's changes renge, and I cannot find "__UVISION_VERSION SETA 521" file into mbed-os repo.
mbed-os\rtos\TARGET_CORTEX\rtx5\RTX\Source\TOOLCHAIN_ARM\TARGET_CORTEX_A\irq_ca.S

Previously, although I updated this file for RTX5 update by cherry-picking from CMSIS_5 repo, I did not change it since then.
this error did not outputed in the PR of its updating.
#5628 (comment)

@toyowata Please help me to go ahead with the above.

@TomoYamanaka
Copy link
Contributor Author

I will appreciate if you can accelerate the resolution because this is a blocker for Renesas user's activity.

@cmonr
Copy link
Contributor

cmonr commented Jun 19, 2018

@TomoYamanaka It is high in my queue of work to get a PR in that will allow this to pass CI. Once that PR is made, I'll link it here.

@cmonr
Copy link
Contributor

cmonr commented Jun 21, 2018

@TomoYamanaka #7295 has been created.

The mainly changes is below:
- Add flash_api.c
- Add the definition of SPI multi I/O Bus controller that is used for flash access
- Add "FLASH" as device feature
- Add the macro regarding information of the incorporated Flash
- Add the processing to expand code to RAM
The mainly changes is below:
- Update scatter file, linker file for bootloader support
- Update the file for RZ/A1 serial flash boot loader
- Add "device name" and "bootloader_supported" in targets.json
I added the information of GR-LYCHEE for supporting bootloader in index.json file of arm pack manager.
I added the process for Cortex-A in mbed_application.c because this process was for only Cortex-M.
Also I enabled the macro of MBED_APPLICATION_SUPPORT for Cortex-A.
I changed the information("sectors" and "memory") of GR-PEACH for supporting bootloader in index.json file of arm pack manager.
@TomoYamanaka TomoYamanaka dismissed stale reviews from theotherjimmy, 0xc0170, and cmonr via 72fd8ca June 22, 2018 01:55
@TomoYamanaka TomoYamanaka force-pushed the feature-flashiap_bootloader branch from bff07fd to 72fd8ca Compare June 22, 2018 01:55
@TomoYamanaka
Copy link
Contributor Author

The exporter's problem was resolved, and I rebased to resolve a conflict of "tools/arm_pack_manager/index.json".
So could you review and re-trigger CI test again?

@0xc0170
Copy link
Contributor

0xc0170 commented Jun 22, 2018

/morph build

@mbed-ci
Copy link

mbed-ci commented Jun 22, 2018

Build : SUCCESS

Build number : 2432
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/6925/

Triggering tests

/morph test
/morph uvisor-test
/morph export-build
/morph mbed2-build

@mbed-ci
Copy link

mbed-ci commented Jun 22, 2018

@mbed-ci
Copy link

mbed-ci commented Jun 22, 2018

@mbed-ci
Copy link

mbed-ci commented Aug 2, 2018

Exporter Build : FAILURE

Build number : 2355
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/exporter/6925/

@cmonr
Copy link
Contributor

cmonr commented Aug 2, 2018

#ghostinthemachine

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.

8 participants