Page MenuHomeFreeBSD
Differential D41563

sff: Improve names and error messages
AcceptedPublic
Actions

Authored by dsl on Aug 23 2023, 12:28 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, May 4, 2:39 PM
Unknown Object (File)
Thu, Apr 30, 7:27 AM
Unknown Object (File)
Fri, Apr 17, 4:44 AM
Unknown Object (File)
Apr 11 2026, 10:28 PM
Unknown Object (File)
Apr 11 2026, 10:25 AM
Unknown Object (File)
Apr 9 2026, 5:48 AM
Unknown Object (File)
Apr 8 2026, 6:16 PM
Unknown Object (File)
Apr 7 2026, 6:33 AM
View All Files
Subscribers
imp
loos
arm64

Details

Reviewers
jrtc27
bz
Summary

Names of the GPIO handles and printed errors are improved for readability.

Test Plan

N/A

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

dsl created this revision.Aug 23 2023, 12:28 PM
Herald added subscribers: loos, imp. · View Herald TranscriptAug 23 2023, 12:28 PM
dsl requested review of this revision.Aug 23 2023, 12:28 PM
bz added a subscriber: arm64.Aug 23 2023, 10:09 PM
dsl added a comment.Aug 24 2023, 7:59 AM

@jrtc27 Could you comment or accept it if everything's according to your expectation?

dsl updated this revision to Diff 126534.Aug 25 2023, 10:28 PM

Added a comment about "mod_abs"

bz added a comment.Sep 5 2023, 6:52 PM

@jrtc27 given this seems to be partially triggered by feedback from you, any further comments?

jrtc27 added a comment.Sep 5 2023, 6:56 PM

One comment otherwise LGTM

sys/dev/sff/sfp_fdt.c
119–120
dsl updated this revision to Diff 127178.Sep 9 2023, 2:13 PM
dsl marked an inline comment as done.

re-based and updated according to the comment

dsl added a comment.Sep 19 2023, 10:39 AM

@jrtc27 @bz Any comments?

bz accepted this revision.Sep 21 2023, 5:02 PM

Approved by: bz (mentor)

This revision is now accepted and ready to land.Sep 21 2023, 5:02 PM

Revision Contents
Changeset List

PathSize
sys/
dev/
sff/
53 lines

Diff 127178

View Options

sys/dev/sff/sfp_fdt.c

Loading...