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

ec/system76/ec/system76_ec.c: Rework EC FW sync #577

Merged
merged 6 commits into from
Dec 3, 2024
Merged

Conversation

miczyg1
Copy link
Contributor

@miczyg1 miczyg1 commented Oct 23, 2024

No description provided.

@miczyg1 miczyg1 marked this pull request as draft October 23, 2024 13:40
@miczyg1 miczyg1 force-pushed the ec_fw_sync_fix branch 2 times, most recently from 4afe654 to 085a4b0 Compare October 28, 2024 16:08
@miczyg1 miczyg1 changed the title src/ec/system76/ec/system76_ec.c: WIP fix flash update ec/system76/ec/system76_ec.c: Rework EC FW sync Nov 5, 2024
@miczyg1 miczyg1 requested a review from mkopec November 5, 2024 12:10
@miczyg1 miczyg1 marked this pull request as ready for review November 5, 2024 12:10
@miczyg1
Copy link
Contributor Author

miczyg1 commented Nov 5, 2024

@mkopec it seems to work now. Made a few update attempts in a row and none failed with AAI programming. Though I tested only a couple unused sectors... When I will be at office, I can test full flash update, but to be safe I would like to have a EC kit at hand. You may already test it on your end. No EC FW changes introduced.

@mkopec
Copy link
Member

mkopec commented Nov 12, 2024

I've been flashing back & forth between v0.9.0-rc10 and final with no issues so far 👍

src/lib/hexdump.c Outdated Show resolved Hide resolved
@mkopec
Copy link
Member

mkopec commented Nov 12, 2024

I also think we may want to load the EC update file uncodnitionally, to make the TPM measurements consistent between AC and DC, but that doesn't need to go in this PR

@mkopec mkopec force-pushed the mtl-h-dgpu branch 4 times, most recently from 9b25a46 to 0678fea Compare November 19, 2024 13:00
Base automatically changed from mtl-h-dgpu to dasharo November 19, 2024 13:01
mkopec
mkopec previously approved these changes Dec 3, 2024
Make theupdate  process more reliable and limit the flash operations
to sector that actually need it.

Also fix the AAi WP operation which sometimes got interrupted after
first 2 bytes.

Signed-off-by: Michał Żygowski <[email protected]>
Fix incorrect native function of PCH UART0 pads.

Signed-off-by: Michał Żygowski <[email protected]>
@mkopec mkopec force-pushed the ec_fw_sync_fix branch 2 times, most recently from 34f6dd4 to 55a896a Compare December 3, 2024 12:59
@mkopec mkopec merged commit 9385f24 into dasharo Dec 3, 2024
54 checks passed
@mkopec mkopec deleted the ec_fw_sync_fix branch December 3, 2024 14:02
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