Page MenuHomeFreeBSD
Differential D55763

Pouria's new driver report for Status/2026Q
ClosedPublic
Actions

Authored by pouria on Mar 9 2026, 10:16 AM.
Tags
None
Referenced Files
F160288731: D55763.id173564.diff
Mon, Jun 22, 11:27 PM
Unknown Object (File)
Mon, Jun 22, 6:33 AM
Unknown Object (File)
Tue, Jun 16, 11:43 PM
Unknown Object (File)
Tue, Jun 16, 9:27 AM
Unknown Object (File)
Tue, Jun 16, 9:22 AM
Unknown Object (File)
Mon, Jun 15, 10:23 PM
Unknown Object (File)
Mon, Jun 15, 6:08 PM
Unknown Object (File)
Sun, Jun 14, 7:58 PM
View All 82 Files
Subscribers
gperciva_tarsnap.com

Details

Reviewers
salvadore
Group Reviewers
status
Commits
R9:041299443fee: Status/2026Q1/acpi_system76.adoc: Add report
Summary

please review my status report for the new acpi driver.

Diff Detail

Repository
R9 FreeBSD doc repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 71279
Build 68162: arc lint + arc unit

Event Timeline

pouria created this revision.Mar 9 2026, 10:16 AM
Herald added 1 blocking reviewer(s): salvadore. · View Herald TranscriptMar 9 2026, 10:16 AM
pouria requested review of this revision.Mar 9 2026, 10:16 AM
Harbormaster completed remote builds in B71279: Diff 173411.Mar 9 2026, 10:16 AM
pouria retitled this revision from Pouria's geneve report for Status/2026Q to Pouria's new driver report for Status/2026Q.Mar 9 2026, 10:17 AM
gperciva_tarsnap.com added a subscriber: gperciva_tarsnap.com.Mar 9 2026, 5:35 PM
gperciva_tarsnap.com added inline comments.
website/content/en/status/report-2026-01-2026-03/acpi_system76.adoc
9

I'm seen other status reports including links to the git commits or Phabricator reviews for new features. I don't think this is strictly required for status reports, but it's worth considering.

14

lower-case

pouria updated this revision to Diff 173444.Mar 9 2026, 5:48 PM

I added the cgit links to their related commits
Is that good?

Harbormaster completed remote builds in B71294: Diff 173444.Mar 9 2026, 5:48 PM
gperciva_tarsnap.com added a comment.Mar 9 2026, 7:17 PM

Looks good to me! I'm new to reviewing status reports, though, so a more experienced reviewer might notice other issues.

salvadore added a comment.Mar 11 2026, 2:41 PM

Only a few contractions to remove and it is ready to merge, thanks!

website/content/en/status/report-2026-01-2026-03/acpi_system76.adoc
6–8

Please do not use contractions

15

Please do not use contractions

pouria updated this revision to Diff 173564.Mar 11 2026, 2:44 PM
pouria marked 4 inline comments as done.

Address @salvadore comments.

Harbormaster completed remote builds in B71337: Diff 173564.Mar 11 2026, 2:45 PM
This revision was not accepted when it landed; it landed in state Needs Review.Apr 10 2026, 1:58 PM
Closed by commit R9:041299443fee: Status/2026Q1/acpi_system76.adoc: Add report (authored by pouria, committed by salvadore). · Explain Why
This revision was automatically updated to reflect the committed changes.
salvadore added a commit: R9:041299443fee: Status/2026Q1/acpi_system76.adoc: Add report.

Revision Contents
Changeset List

PathSize
website/
content/
en/
status/
report-2026-01-2026-03/
14 lines

Diff 173411

View Options

website/content/en/status/report-2026-01-2026-03/acpi_system76.adoc

Loading...