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

Fix Build failures when compiling the NXP USB driver #67459

Merged
merged 2 commits into from
Jan 12, 2024
Merged

Fix Build failures when compiling the NXP USB driver #67459

merged 2 commits into from
Jan 12, 2024

Conversation

mmahadevan108
Copy link
Collaborator

Failure introduced as part of PR #67261

Copy link
Contributor

@tmon-nordic tmon-nordic left a comment

Choose a reason for hiding this comment

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

Is it necessary to revert both commits? Isn't 21c9c4a the only culprit?

Copy link
Collaborator

@jfischer-no jfischer-no left a comment

Choose a reason for hiding this comment

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

If necessary, this should be done with
git revert -s 21c9c4a
git revert -s 2b91d62

@jsiverskog jsiverskog mentioned this pull request Jan 11, 2024
@mmahadevan108
Copy link
Collaborator Author

Is it necessary to revert both commits? Isn't 21c9c4a the only culprit?

Yes, both are related to the changes added as part of the PR to the USB HAL driver.

… mcux driver"

This reverts commit 21c9c4a.

Signed-off-by: Mahesh Mahadevan <[email protected]>
@mmahadevan108
Copy link
Collaborator Author

If necessary, this should be done with git revert -s 21c9c4a git revert -s 2b91d62

@jfischer-no, that is how I reverted these commits.

@dleach02 dleach02 requested a review from jfischer-no January 11, 2024 17:29
@dleach02 dleach02 added the Hotfix Fix for issues blocking development, i.e. CI issues, tests failing in CI, etc. label Jan 11, 2024
@jfischer-no
Copy link
Collaborator

If necessary, this should be done with git revert -s 21c9c4a git revert -s 2b91d62

@jfischer-no, that is how I reverted these commits.

Then the commit message would be

    Revert "drivers: usb: device: add start of frame notifications to nxp mcux driver"
    
    This reverts commit 21c9c4abfe5c858b1c5d6c45a2c919d81b12a02b.

easier to spot in the history and re-reverted.

This reverts commit 2b91d62.

Signed-off-by: Mahesh Mahadevan <[email protected]>
@mmahadevan108
Copy link
Collaborator Author

mmahadevan108 commented Jan 11, 2024

2b91d62

Updated the PR with the original revert commit messages.

@carlescufi carlescufi merged commit 0a26da9 into zephyrproject-rtos:main Jan 12, 2024
16 checks passed
@mmahadevan108 mmahadevan108 deleted the Fix_USB_Build branch March 5, 2024 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: USB Universal Serial Bus Hotfix Fix for issues blocking development, i.e. CI issues, tests failing in CI, etc. platform: NXP NXP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants