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

Add uvcvideo version update patch #12905

Merged
merged 2 commits into from May 12, 2024

Conversation

Nir-Az
Copy link
Collaborator

@Nir-Az Nir-Az commented May 7, 2024

We wish to distinguish between a native uvc driver and a realsense manual patched uvc driver
This patch will add a "realsense-patch" suffix to the uvcvideo driver

image

Tracked on [LRS-1098]

@Nir-Az Nir-Az requested a review from dmipx May 7, 2024 18:18
@Nir-Az
Copy link
Collaborator Author

Nir-Az commented May 7, 2024

@dmipx can we assume that in all kernel version we support (and all distros) the driver version will be stated at the same path?
drivers/media/usb/uvc/uvcvideo.h ?

I think that currently this is the case.
I will test on U20 too

@Nir-Az Nir-Az requested a review from ev-mp May 7, 2024 18:22
@Nir-Az
Copy link
Collaborator Author

Nir-Az commented May 7, 2024

@dmipx can we assume that in all kernel version we support (and all distros) the driver version will be stated at the same path? drivers/media/usb/uvc/uvcvideo.h ?

I think that currently this is the case. I will test on U20 too

Tested on U20 too
image

/* ------------------------------------------------------------------------
* Driver specific constants.
*/
-
Copy link
Contributor

Choose a reason for hiding this comment

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

you should leave this new line here

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done

Copy link
Contributor

@dmipx dmipx left a comment

Choose a reason for hiding this comment

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

LGTM

@Nir-Az Nir-Az merged commit 6702e3d into IntelRealSense:development May 12, 2024
17 checks passed
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.

None yet

2 participants