Skip to content

Fwupd - #2082

Open
joebonrichie wants to merge 3 commits into
mainfrom
fwupd
Open

Fwupd#2082
joebonrichie wants to merge 3 commits into
mainfrom
fwupd

Conversation

@joebonrichie

Copy link
Copy Markdown
Contributor

Summary

Add fwupd to update supported firmware device

Test Plan

Update UEFI firmware on Framework Laptop 16

Checklist

  • Recipe was built and tested against the volatile stream
  • This change could gainfully be highlighted in the Stream Update notes once merged

@joebonrichie
joebonrichie requested a review from a team as a code owner July 27, 2026 17:32
@joebonrichie
joebonrichie marked this pull request as draft July 27, 2026 17:32
@joebonrichie

Copy link
Copy Markdown
Contributor Author

need to add statelessness support to fwupd

@khaga87

khaga87 commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Hey @joebonrichie,

Just wondering if this is on (or fallen off) your radar? No pressure either way but just trying to figure out if fwupd might land in the repo anytime soon?

@K1ngfish3r K1ngfish3r left a comment

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.

I went through fwupd source and found this https://github.com/fwupd/fwupd/blob/94fec70e96ce92c28b0a5853b7846786ca4dd63d/src/fu-remote-list.c#L607-L622
so it first reads from /var/etc/fwupd (it writes to here), if it doesnt exist check /etc then finally fallback to /usr/share (DATADIR), so this pretty stateless already if you ask me

Comment thread f/fwupd-efi/stone.yaml Outdated
Comment thread f/fwupd/stone.yaml Outdated
Comment thread f/fwupd/stone.yaml Outdated
Comment thread f/fwupd/stone.yaml
Comment thread e/efivar/stone.yaml Outdated
K1ngfish3r
K1ngfish3r previously approved these changes Aug 16, 2026

@K1ngfish3r K1ngfish3r left a comment

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.

LGTM, leaving for other ppl to review/test

@khaga87

khaga87 commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

This is marked as draft? Is it ready for final review or is there still more work required?

@K1ngfish3r

Copy link
Copy Markdown
Contributor

I have updated my GPU firmware using this recipe

@joebonrichie
joebonrichie marked this pull request as ready for review September 11, 2026 09:51

@K1ngfish3r K1ngfish3r left a comment

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.

(hopefully) fixing CI

Comment thread f/fwupd/stone.yaml Outdated
@joebonrichie
joebonrichie force-pushed the fwupd branch 2 times, most recently from fb085a8 to cb4d7fa Compare September 11, 2026 19:39
@joebonrichie
joebonrichie marked this pull request as draft September 11, 2026 19:41
@joebonrichie
joebonrichie marked this pull request as ready for review September 11, 2026 19:47

@K1ngfish3r K1ngfish3r left a comment

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.

LGTM

Comment thread f/fwupd-efi/stone.yaml
license :
- LGPL-2.1-or-later WITH GPL-CC-1.0
builddeps :
- binary(objcopy) # llvm-objcopy does not support EFI targets

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can you explain this comment a bit more? Because we build both efifs, systemd-boot, and edk2 all with clang and they all work fine.

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.

it was pulled directly from

- binary(objcopy) # llvm-objcopy does not support EFI targets

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.

4 participants