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
F150433713: D41563.id127178.diff
Wed, Apr 1, 5:00 AM
Unknown Object (File)
Tue, Mar 24, 9:34 PM
Unknown Object (File)
Wed, Mar 11, 10:14 AM
Unknown Object (File)
Wed, Mar 11, 8:54 AM
Unknown Object (File)
Wed, Mar 11, 5:10 AM
Unknown Object (File)
Mon, Mar 9, 4:34 PM
Unknown Object (File)
Mar 1 2026, 10:24 AM
Unknown Object (File)
Mar 1 2026, 5:53 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...