Skip to content

Feat: allow updating of OB FW from Programmer app#470

Open
kylebonnici wants to merge 7 commits into
mainfrom
feat/auto-update-ob-fw
Open

Feat: allow updating of OB FW from Programmer app#470
kylebonnici wants to merge 7 commits into
mainfrom
feat/auto-update-ob-fw

Conversation

@kylebonnici

@kylebonnici kylebonnici commented May 29, 2024

Copy link
Copy Markdown
Contributor

This PR add the ability in programme to update the J-Link OB Firmware.

This can be done in two ways:

  • Auto with every read, write and erase (will effect performance) default is for this option is off @bihanssen any suggestions?
  • Manually with a button

Auto UI (image updated)

image

Manual UI

image

Result in Logger

image image

@greg-fer if this is approve then Docs will need updating
@ketile let me know if UI is OK

@github-actions

Copy link
Copy Markdown

Add labels:

  • 'doc required' or 'doc not required'
  • 'ui required' or 'ui not required'

For UI review:

  • describe the changes
  • add pictures
  • add the relevant UI responsible(s) as a reviewer

@kylebonnici kylebonnici requested a review from greg-fer May 29, 2024 21:26
@kylebonnici kylebonnici added doc required This PR includes user-facing changes that need to be documented. ui required All PRs either need "ui required" or "ui not required". labels May 29, 2024
@kylebonnici kylebonnici requested a review from ketile May 29, 2024 21:26
Comment thread src/actions/jlinkTargetActions.ts Outdated
Comment thread src/components/ControlPanel.tsx Outdated
Comment thread src/components/ControlPanel.tsx Outdated
@@ -430,6 +454,16 @@ Are you sure you want to continue?`,
)}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

For line 451:

Use this setting with care for devices with modem: resetting it too many times might cause it to lock up.

@@ -430,6 +454,16 @@ Are you sure you want to continue?`,
)}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

For line 394:

The write operation is not supported for J-Link devices. Use Erase & write instead.

Comment thread Changelog.md Outdated
kylebonnici and others added 2 commits May 30, 2024 09:20
Co-authored-by: Grzegorz Ferenc <41291385+greg-fer@users.noreply.github.com>
@greg-fer

Copy link
Copy Markdown
Contributor

@greg-fer

Copy link
Copy Markdown
Contributor

When is this going to be released?

@greg-fer

greg-fer commented Sep 2, 2025

Copy link
Copy Markdown
Contributor

Any info when this is going to be released?

@CLAassistant

CLAassistant commented Apr 21, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@greg-fer

greg-fer commented May 6, 2026

Copy link
Copy Markdown
Contributor

Any updates about the release?
Is this PR abandoned and should be dropped?
@chunfantasy , @kylebonnici ?

@kylebonnici

Copy link
Copy Markdown
Contributor Author

This was last discussed with @bihanssen some to two + years ago and he had some reservations about adding this. Perhaps we should discuss again and either close commit to doing it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Do Not Merge doc required This PR includes user-facing changes that need to be documented. ui required All PRs either need "ui required" or "ui not required".

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants