Page MenuHomeFreeBSD
Differential D41272

hpwmc_amd.c: style improvements
ClosedPublic
Actions

Authored by mhorne on Aug 1 2023, 4:23 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 13, 11:49 PM
Unknown Object (File)
Tue, Nov 25, 8:07 AM
Unknown Object (File)
Nov 12 2025, 2:18 AM
Unknown Object (File)
Oct 29 2025, 9:49 AM
Unknown Object (File)
Oct 29 2025, 8:43 AM
Unknown Object (File)
Oct 29 2025, 8:43 AM
Unknown Object (File)
Oct 29 2025, 8:22 AM
Unknown Object (File)
Oct 19 2025, 5:33 PM
View All 84 Files
Subscribers
imp

Details

Reviewers
jkoshy
gnn
Group Reviewers
pmc
Commits
rGd9e3fe322662: hpwmc_amd.c: style improvements
Summary
  • Return style
  • Explicit value checks
  • Whitespace formatting
  • Comment formatting
  • Local variable declaration order
  • __unused annotations

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 52930
Build 49821: arc lint + arc unit

Event Timeline

mhorne created this revision.Aug 1 2023, 4:23 PM
Herald added a reviewer: gnn. · View Herald TranscriptAug 1 2023, 4:23 PM
Herald added a subscriber: imp. · View Herald Transcript
mhorne requested review of this revision.Aug 1 2023, 4:23 PM
Harbormaster completed remote builds in B52930: Diff 125426.Aug 1 2023, 4:23 PM
mhorne added a parent revision: D41271: hwpmc_core: remove two non-existent function prototypes.Aug 1 2023, 4:23 PM
mhorne added a child revision: D41273: hwpmc_amd: simplify counter descriptor definitions.
mhorne added inline comments.Aug 1 2023, 4:27 PM
sys/dev/hwpmc/hwpmc_amd.c
888

This snuck in; fixed locally.

jkoshy accepted this revision.Aug 3 2023, 6:00 PM
jkoshy added inline comments.
sys/dev/hwpmc/hwpmc_amd.c
987

Add a comment about this being the __i386__ case?

This revision is now accepted and ready to land.Aug 3 2023, 6:00 PM
mhorne added inline comments.Aug 4 2023, 1:38 PM
sys/dev/hwpmc/hwpmc_amd.c
987

The ifdef gets removed in D41274, so I really have no idea why I bothered touching these lines. I'll drop this hunk of the diff before pushing.

Closed by commit rGd9e3fe322662: hpwmc_amd.c: style improvements (authored by mhorne). · Explain WhyOct 18 2023, 6:06 PM
This revision was automatically updated to reflect the committed changes.
mhorne added a commit: rGd9e3fe322662: hpwmc_amd.c: style improvements.

Revision Contents
Changeset List

PathSize
sys/
dev/
hwpmc/
280 lines

Diff 125426

View Options

sys/dev/hwpmc/hwpmc_amd.c

Loading...