Skip to content

Conversation

d-kato
Copy link
Contributor

@d-kato d-kato commented Jul 29, 2016

  • Add the process for the Cortex-A
    It branches in processing for Cortex-A by macro TARGET_MCU_CORTEX_A.
    A "swd_host.c" for Cortex-A is created as "swd_host_ca.c".
    The binary inspection was made comparison in 3 bytes of head of the data because there was no NVIC in Cortex-A.
    Comparative data can be set for each board by macro BINARY_DETECTION.
  • Add the partial erase processing for the chip erase is slow device.
    Our Flash will take more than 20 seconds to chip erase.
    Therefore, we have added the process of writing a program with partial erase.
    Selective erase becomes effective by macro FLASH_PARTIAL_ERASE.
  • Add target Renesas GR-PEACH
    A flash of rza1h is outside the chip, and the flash and port may be different from every board.
    So it was made the following path.
    "/source/target/renesas/rza1h/gr-peach/"

d-kato added 3 commits July 29, 2016 16:21
It branches in processing for Cortex-A by macro TARGET_MCU_CORTEX_A.
A "swd_host.c" for Cortex-A is created as "swd_host_ca.c".
The binary inspection was made comparison in 3 bytes of head of the data because there was no NVIC in Cortex-A.
Comparative data can be set for each board by macro BINARY_DETECTION.
Our Flash will take more than 20 seconds to chip erase.
Therefore, we have added the process of writing a program with partial erase.
Selective erase becomes effective by macro FLASH_PARTIAL_ERASE.
A flash of rza1h is outside the chip, and the flash and port may be different from every board.
So it was made the following path.
"/source/target/renesas/rza1h/gr-peach/"
@c1728p9
Copy link
Contributor

c1728p9 commented Aug 12, 2016

Hi @d-kato thanks for the PR. Would it be possible to move the partial erase into a second PR to make testing and merging easier?

#ifdef FLASH_PARTIAL_ERASE
// Processing for low-speed devices of chip erasure.
flash_manager_printf(" skip intf->erase_chip\r\n");
#else
Copy link
Contributor

Choose a reason for hiding this comment

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

FLASH_PARTIAL_ERASE is a very useful feature. This would be nice as a run-time configurable value. Maybe a function to set the mode?
flash_manager_init(const flash_intf_t *flash_intf, flash_mode_t)
or maybe a seperate function
flash_manager_mode(CHIP_ERASE) / flash_manager_mode(SECTOR_ERASE)

What do you think @d-kato?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think the "flash_manager_init()" is preferred.
In the case of "flash_manager_mode()", it might appear to be able to change at any time.

@d-kato
Copy link
Contributor Author

d-kato commented Aug 12, 2016

Hi @c1728p9 ,
Thank you very much for your comment.
Please check the change.

@c1728p9
Copy link
Contributor

c1728p9 commented Sep 16, 2016

Hi @d-kato unfortunately, I haven't had time to test this so it won't make it into 0242. I'll try to get to this by the next release. I very much want Cortex A / Peach support in DAPLink so thanks for the PR.

@d-kato
Copy link
Contributor Author

d-kato commented Sep 29, 2016

I close this PR for rebase.

@d-kato d-kato closed this Sep 29, 2016
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.

2 participants