Skip to content
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.

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

linux-yocto-dev: fix Upstream-Status for the UFS voltage patch #73

Merged
merged 2 commits into from
Nov 19, 2024

Conversation

lumag
Copy link
Contributor

@lumag lumag commented Nov 15, 2024

The change described in the patch isn't pending, but it has been rejected upstream. Use correct Upstream-Status for the patch.

Copy link
Contributor

@ricardosalveti ricardosalveti left a comment

Choose a reason for hiding this comment

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

LGTM

@ndechesne
Copy link
Contributor

thanks for spotting that. For the first patch (UFS voltage), what's the alternative? I don't think we want to carry for too long un-upstreamable patches in here.

@lumag
Copy link
Contributor Author

lumag commented Nov 18, 2024

For the first patch (UFS voltage), what's the alternative?

I explained my suggestions on the ML: add separate DT targeting boards with UFS 3.x (?). Each of board files should properly describe the HW / regulators.

@ricardosalveti
Copy link
Contributor

We need to check internally the multiple IDP revisions we have and how to handle the need for multiple DTs there.

The change described in the patch isn't pending, but it has been
rejected upstream. Use correct Upstream-Status for the patch.

Signed-off-by: Dmitry Baryshkov <[email protected]>
The patch in this layer that enables UFS nodes on the QCM6490-IDP board
corresponds to the v2 of the patchset which was rejected upstream
(instead of being Pending, as it was incorrectly stated in the
Upstream-Status tag).

Replace it with the version 3 of the patch, which is has been accepted
by Bjorn as is pending for the 6.13 release.

Signed-off-by: Dmitry Baryshkov <[email protected]>
@ndechesne ndechesne merged commit 9031d10 into qualcomm-linux:main Nov 19, 2024
7 checks passed
@lumag lumag deleted the patch-1 branch November 19, 2024 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

3 participants