Page MenuHomeFreeBSD
Differential D55985

ufshci: add uic powermode parameter to sysctl
ClosedPublic
Actions

Authored by jaeyoon on Mar 20 2026, 7:23 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Sep 22, 8:51 PM
Unknown Object (File)
Sat, Sep 12, 9:17 AM
Unknown Object (File)
Fri, Sep 11, 6:55 PM
Unknown Object (File)
Fri, Sep 11, 3:41 AM
Unknown Object (File)
Thu, Sep 10, 9:45 PM
Unknown Object (File)
Wed, Sep 9, 8:31 AM
Unknown Object (File)
Tue, Sep 8, 10:56 AM
Unknown Object (File)
Tue, Sep 8, 4:49 AM
View All Files
Subscribers
ziaee

Details

Reviewers
imp
Commits
rG085028463fa9: ufshci: add uic powermode parameter to sysctl
Summary

Adds parameters related to the performance of the UFS device.

Sponsored by: Samsung Electronics

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 71657
Build 68540: arc lint + arc unit

Event Timeline

jaeyoon requested review of this revision.Mar 20 2026, 7:23 AM
jaeyoon created this revision.
Harbormaster completed remote builds in B71577: Diff 173995.Mar 20 2026, 7:23 AM
jaeyoon added a parent revision: D55984: ufshci: fix bug in ufshci_req_sdb_enable.Mar 20 2026, 7:24 AM
jaeyoon added a child revision: D55986: ufshci: Support ACPI.
imp accepted this revision.Mar 20 2026, 1:39 PM

Is there documentation that needs to be updated?
The change itself looks good, though maybe the commit message is a little vague?

sys/dev/ufshci/ufshci_pci.c
37

I don't think you want this change...

This revision is now accepted and ready to land.Mar 20 2026, 1:39 PM
jaeyoon updated this revision to Diff 174145.Mar 23 2026, 2:36 AM

Modified the man page.

This revision now requires review to proceed.Mar 23 2026, 2:36 AM
Herald added a subscriber: ziaee. · View Herald TranscriptMar 23 2026, 2:36 AM
Harbormaster completed remote builds in B71657: Diff 174145.Mar 23 2026, 2:36 AM
jaeyoon added a comment.Mar 23 2026, 6:09 AM

Could you please approve this patch?

imp accepted this revision.Mar 23 2026, 2:25 PM
This revision is now accepted and ready to land.Mar 23 2026, 2:25 PM
jaeyoon marked an inline comment as done.Mar 24 2026, 12:18 AM
jaeyoon added inline comments.
sys/dev/ufshci/ufshci_pci.c
37

Clang-format enforces this awkward indentation.

imp added a comment.Mar 24 2026, 8:58 PM

OK. thanks for the info on clang-format. It's approved to commit.

Closed by commit rG085028463fa9: ufshci: add uic powermode parameter to sysctl (authored by jaeyoon). · Explain WhyMar 25 2026, 4:47 AM
This revision was automatically updated to reflect the committed changes.
jaeyoon marked an inline comment as done.
jaeyoon added a commit: rG085028463fa9: ufshci: add uic powermode parameter to sysctl.

Revision Contents
Changeset List

PathSize
share/
man/
man4/
34 lines
sys/
dev/
ufshci/
12 lines
3 lines
3 lines
35 lines

Diff 174145

View Options

share/man/man4/ufshci.4

Loading...
View Options

sys/dev/ufshci/ufshci_dev.c

Loading...
View Options

sys/dev/ufshci/ufshci_pci.c

Loading...
View Options

sys/dev/ufshci/ufshci_private.h

Loading...
View Options

sys/dev/ufshci/ufshci_sysctl.c

Loading...