Skip to content

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

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
uditkumarti merged 1 commit into from
Jul 13, 2025
Merged

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

uditkumarti merged 1 commit into from
Jul 13, 2025

Conversation

@uditkumarti
Copy link
Collaborator

@uditkumarti uditkumarti commented Jun 25, 2025

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
Copy link
Collaborator

@praneethbajjuri praneethbajjuri left a comment

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
Copy link
Member

StaticRocket commented Jun 25, 2025

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

@uditkumarti
Copy link
Collaborator Author

uditkumarti commented Jul 7, 2025

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
Copy link
Member

StaticRocket commented Jul 7, 2025

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
Copy link
Collaborator Author

uditkumarti commented Jul 8, 2025

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
@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 <[email protected]>
@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