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

bump usb middleware to mcux v2.14.0 #316

Merged

Conversation

jsiverskog
Copy link
Contributor

No description provided.

@jsiverskog
Copy link
Contributor Author

corresponding zephyr pr: zephyrproject-rtos/zephyr#67483

@jsiverskog jsiverskog force-pushed the bump_sdk_middleware_usb branch from 3d6d12c to 1877e44 Compare January 11, 2024 15:19
@jsiverskog
Copy link
Contributor Author

@danieldegrasse : do you have time to review this? thanks

@jsiverskog
Copy link
Contributor Author

@danieldegrasse : what do i have to do in order for this to get merged?

@danieldegrasse
Copy link
Collaborator

@danieldegrasse : what do i have to do in order for this to get merged?

Let's get the PR on the Zephyr side approved. Once @hakehuang approves that PR, I think we are good to merge this, and move forwards. @mmahadevan108, do you agree with this approach?

@dleach02
Copy link
Member

I also have pulled in these files into my SDK 2.15.000 PR. We had a build issue that is being looked.

@danieldegrasse
Copy link
Collaborator

@jsiverskog can you rebase to resolve conflicts?

jsiverskog and others added 4 commits February 1, 2024 21:04
this is taken from
git@github.com:nxp-mcuxpresso/mcux-sdk-middleware-usb.git, tag
MCUX_2.14.0.

Signed-off-by: Jacob Siverskog <jacob@teenage.engineering>
re-apply patch after sdk refresh.

without this kStatus_USB_Error was always returned for
kUSB_DeviceControlRun and kUSB_DeviceControlStop control types. let's
do it as for ehci/khci.

this change fixes the tests/subsys/usb/device/usb.device test for
lpcxpresso55s69_cpu0.

this is from nxp-mcuxpresso/mcuxsdk-middleware-usb#6

Signed-off-by: Jacob Siverskog <jacob@teenage.engineering>
Submitted a PR to https://github.com/NXPmicro/mcux-sdk-middleware-usb
to fix this build issue

Signed-off-by: Mahesh Mahadevan <mahesh.mahadevan@nxp.com>
add a missing usb middleware fix.

Signed-off-by: Jacob Siverskog <jacob@teenage.engineering>
@jsiverskog jsiverskog force-pushed the bump_sdk_middleware_usb branch from 9a5eb18 to c331c4e Compare February 1, 2024 20:05
@jsiverskog
Copy link
Contributor Author

@jsiverskog can you rebase to resolve conflicts?

done.

@mmahadevan108 mmahadevan108 merged commit d45b14c into zephyrproject-rtos:master Feb 2, 2024
dleach02 pushed a commit to jsiverskog/zephyr that referenced this pull request Feb 2, 2024
this brings in a newer version of the usb middleware.

see zephyrproject-rtos/hal_nxp#316 for more
information.

Signed-off-by: Jacob Siverskog <jacob@teenage.engineering>
henrikbrixandersen pushed a commit to zephyrproject-rtos/zephyr that referenced this pull request Feb 2, 2024
this brings in a newer version of the usb middleware.

see zephyrproject-rtos/hal_nxp#316 for more
information.

Signed-off-by: Jacob Siverskog <jacob@teenage.engineering>
@jsiverskog jsiverskog deleted the bump_sdk_middleware_usb branch February 5, 2024 07:43
coreboot-org-bot pushed a commit to coreboot/zephyr-cros that referenced this pull request Feb 10, 2024
this brings in a newer version of the usb middleware.

see zephyrproject-rtos/hal_nxp#316 for more
information.

(cherry picked from commit 031f84d)

Original-Signed-off-by: Jacob Siverskog <jacob@teenage.engineering>
GitOrigin-RevId: 031f84d
Change-Id: I6c8c48b51c98dbc616f79a8468a2083d7262d916
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/zephyr/+/5280249
Tested-by: ChromeOS Prod (Robot) <chromeos-ci-prod@chromeos-bot.iam.gserviceaccount.com>
Commit-Queue: Al Semjonovs <asemjonovs@google.com>
Tested-by: Al Semjonovs <asemjonovs@google.com>
Reviewed-by: Al Semjonovs <asemjonovs@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants