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)
Sun, Nov 24, 1:46 AM
Unknown Object (File)
Fri, Nov 22, 2:31 AM
Unknown Object (File)
Mon, Nov 18, 7:46 PM
Unknown Object (File)
Oct 17 2024, 4:34 AM
Unknown Object (File)
Oct 17 2024, 4:29 AM
Unknown Object (File)
Oct 11 2024, 9:19 AM
Unknown Object (File)
Oct 4 2024, 3:24 PM
Unknown Object (File)
Sep 26 2024, 4:36 AM
View All 32 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...