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

Dev/add secure rom bl host example #316

Merged
merged 3 commits into from
Jan 19, 2023

Conversation

ozersa
Copy link
Contributor

@ozersa ozersa commented Dec 28, 2022

MSDK-975:
Added secure ROM bootloader host example on MAX32660/MAX32670/MAX32570 MCU
This example demonstrate how to communication with secure ROM BL

MAX32660/MAX32670/MAX32570 FW tested with

  • MAX32520-EvKIT
  • MAX32651 EvKIT

Related PR#7

Signed-off-by: Sadik.Ozer sadik.ozer@analog.com

Copy link
Contributor

@Jake-Carter Jake-Carter left a comment

Choose a reason for hiding this comment

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

Looks good!

Copy link
Contributor

@sihyung-maxim sihyung-maxim left a comment

Choose a reason for hiding this comment

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

Looks good. Just found a minor typo.

Comment on lines 3 to 4
The Analog Devices Inc. (ADI) secure microcontroller has special bootloader, it is called as "Secure ROM Bootloader"
this bootloaders provide firmware update function by secure way. The security related staff is handle by SDK and SDK tools.
Copy link
Contributor

Choose a reason for hiding this comment

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

Period and capitalization typo, each line is a sentence.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks updated.

Comment on lines 3 to 4
The Analog Devices Inc. (ADI) secure microcontroller has special bootloader, it is called as "Secure ROM Bootloader"
this bootloaders provide firmware update function by secure way. The security related staff is handle by SDK and SDK tools.
Copy link
Contributor

Choose a reason for hiding this comment

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

Period and capitalization typo, each line is a sentence.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks updated.

@sihyung-maxim
Copy link
Contributor

I remember around early December or so, the "BLE Examples Test" action was getting updated. I'm not sure what the status of that action is now.

@EdwinFairchild
Copy link
Contributor

EdwinFairchild commented Jan 3, 2023

I merged main into this so it can have the latest changes of CI. which will only test BLE devices if changes where made to physical layer, or its library or its examples.

ozersa added 3 commits January 4, 2023 08:44
on MAX32660 and MAX32670 MCU
This example demonstrate how to communication with secure ROM BL

Signed-off-by: Sadik.Ozer <sadik.ozer@analog.com>
Signed-off-by: Sadik.Ozer <sadik.ozer@analog.com>
#316

Signed-off-by: Sadik.Ozer <sadik.ozer@analog.com>
@ozersa ozersa force-pushed the dev/add_secure_rom_bl_host_example branch from c6d7cf2 to ffea95a Compare January 4, 2023 08:27
@ozersa
Copy link
Contributor Author

ozersa commented Jan 5, 2023

@Jake-Carter @sihyung-maxim may it to be merged (if there is not any open item)?

@lorne-maxim lorne-maxim merged commit 811f330 into main Jan 19, 2023
@lorne-maxim lorne-maxim deleted the dev/add_secure_rom_bl_host_example branch January 19, 2023 13:52
tim-magnusen pushed a commit to tim-magnusen/msdk that referenced this pull request Jan 19, 2023
MSDK-975: Add secure ROM bootloader host example on MAX32660 and MAX32670 MCU
This example demonstrates how to communicate with a secure ROM BL.
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.

5 participants