Skip to content

feat: Add UFS support for J784S4 and J742S2 EVM#332

Merged
uditkumarti merged 1 commit into
TexasInstruments:masterfrom
uditkumarti:ufs_support_j784s4
Jul 13, 2025
Merged

feat: Add UFS support for J784S4 and J742S2 EVM#332
uditkumarti merged 1 commit into
TexasInstruments:masterfrom
uditkumarti:ufs_support_j784s4

Conversation

@uditkumarti

@uditkumarti uditkumarti commented Jun 25, 2025

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions github-actions Bot requested review from devarsht and r-vignesh June 25, 2025 08:18
praneethbajjuri
praneethbajjuri reviewed Jun 25, 2025
View reviewed changes

@praneethbajjuri praneethbajjuri left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a short commit msg

@StaticRocket

StaticRocket commented Jun 25, 2025

Copy link
Copy Markdown
Member

Please try addressing the vale comments. You may ignore the ones about abbreviation definitions.

@uditkumarti

uditkumarti commented Jul 7, 2025

Copy link
Copy Markdown
Collaborator Author

Please try addressing the vale comments. You may ignore the ones about abbreviation definitions.

vale review said tick
vale / vale (pull_request)Successful in 30s

is something else , I am missing

@uditkumarti uditkumarti force-pushed the ufs_support_j784s4 branch from 67a58b8 to bfafd02 Compare July 7, 2025 13:48
praneethbajjuri
praneethbajjuri previously approved these changes Jul 7, 2025
View reviewed changes
@StaticRocket

StaticRocket commented Jul 7, 2025

Copy link
Copy Markdown
Member

The vale workflow is non-blocking. It only fails if there are so many warnings that it breaks the Github API comment limits. It will still leave comments over in the review section. Those should still be addressed.

@uditkumarti

uditkumarti commented Jul 8, 2025

Copy link
Copy Markdown
Collaborator Author

The vale workflow is non-blocking. It only fails if there are so many warnings that it breaks the Github API comment limits. It will still leave comments over in the review section. Those should still be addressed.

I haven't added anything new.. but anyway will fix other warnings

@uditkumarti uditkumarti force-pushed the ufs_support_j784s4 branch from bfafd02 to 56884ac Compare July 8, 2025 17:18
praneethbajjuri
praneethbajjuri previously approved these changes Jul 8, 2025
View reviewed changes
cshilwant
cshilwant previously approved these changes Jul 10, 2025
View reviewed changes
StaticRocket
StaticRocket requested changes Jul 10, 2025
View reviewed changes
Comment thread source/linux/Foundational_Components/Kernel/Kernel_Drivers/UFS.rst Outdated
@uditkumarti uditkumarti dismissed stale reviews from cshilwant and praneethbajjuri via 863ccdf July 11, 2025 03:57
@uditkumarti
feat: Add UFS support for J784S4 and J742S2 EVM
31251ed 
J784S4 and J742S2 EVM support UFS, so add in doc.

Signed-off-by: Udit Kumar <u-kumar1@ti.com>
@uditkumarti uditkumarti merged commit 649350e into TexasInstruments:master Jul 13, 2025
6 checks passed
@uditkumarti uditkumarti deleted the ufs_support_j784s4 branch August 5, 2025 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@praneethbajjuri praneethbajjuri praneethbajjuri approved these changes

@StaticRocket StaticRocket StaticRocket approved these changes

@cshilwant cshilwant cshilwant approved these changes

@aniket-l aniket-l Awaiting requested review from aniket-l

@VeeruPrudhvi VeeruPrudhvi Awaiting requested review from VeeruPrudhvi VeeruPrudhvi is a code owner

@devarsht devarsht Awaiting requested review from devarsht

@r-vignesh r-vignesh Awaiting requested review from r-vignesh

Assignees

@praneethbajjuri praneethbajjuri

@devarsht devarsht

@r-vignesh r-vignesh

@uditkumarti uditkumarti

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@uditkumarti @StaticRocket @praneethbajjuri @cshilwant @devarsht @r-vignesh