Page MenuHomeFreeBSD
Differential D54002

ufshci: Support suspend/resume
ClosedPublic
Actions

Authored by jaeyoon on Dec 1 2025, 6:47 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Mar 23, 8:37 AM
Unknown Object (File)
Sun, Mar 22, 4:21 AM
Unknown Object (File)
Sat, Mar 21, 8:09 PM
Unknown Object (File)
Thu, Mar 19, 4:49 AM
Unknown Object (File)
Mon, Mar 16, 2:15 AM
Unknown Object (File)
Mon, Mar 16, 2:13 AM
Unknown Object (File)
Mon, Mar 16, 2:12 AM
Unknown Object (File)
Feb 15 2026, 10:02 PM
View All 28 Files
Subscribers
None

Details

Reviewers
imp
Commits
rG3a99f31fdb6c: ufshci: Support suspend/resume
Summary

Handle system power events and issue START STOP UNIT (SSU) to the
UFS Device WLUN (0x50).

Sponsored by: Samsung Electronics

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jaeyoon requested review of this revision.Dec 1 2025, 6:47 AM
jaeyoon created this revision.
Harbormaster completed remote builds in B68947: Diff 167345.Dec 1 2025, 6:47 AM
jaeyoon added a child revision: D54003: ufshci: Support UIC hibernation enter/exit.Dec 1 2025, 6:47 AM
imp added a comment.Dec 2 2025, 4:27 AM

Looks generally good, but I did have one question...

sys/dev/ufshci/ufshci.c
54 ↗(On Diff #167345)

Is there a reason that you didn't do this in a device_suspend/device_resume function?

jaeyoon added inline comments.Dec 2 2025, 5:16 AM
sys/dev/ufshci/ufshci.c
54 ↗(On Diff #167345)

Oh... I realize I misunderstood this.
You’re right! This should be handled via device_suspend/device_resume.
I’ll fix the implementation and send a revised patch for review.

jaeyoon updated this revision to Diff 167413.Dec 2 2025, 7:41 AM

Use device_suspend/device_resume

Harbormaster completed remote builds in B68990: Diff 167413.Dec 2 2025, 7:42 AM
jaeyoon added a comment.Dec 2 2025, 7:49 AM

After this fix, the following two reviews have been rebased and are ready for approval:
https://reviews.freebsd.org/D54003
https://reviews.freebsd.org/D54004

imp accepted this revision.Dec 3 2025, 12:44 AM
This revision is now accepted and ready to land.Dec 3 2025, 12:44 AM
Closed by commit rG3a99f31fdb6c: ufshci: Support suspend/resume (authored by jaeyoon). · Explain WhyDec 3 2025, 4:06 AM
This revision was automatically updated to reflect the committed changes.
jaeyoon added a commit: rG3a99f31fdb6c: ufshci: Support suspend/resume.

Revision Contents
Changeset List

PathSize
sys/
dev/
ufshci/
50 lines
19 lines
23 lines
35 lines
3 lines
7 lines
3 lines

Diff 167457

View Options

sys/dev/ufshci/ufshci_ctrlr.c

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_req_queue.c

Loading...
View Options

sys/dev/ufshci/ufshci_sim.c

Loading...
View Options

sys/dev/ufshci/ufshci_sysctl.c

Loading...