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

Have the linux kernel patches upstreamed #48

Closed
severin-lemaignan opened this issue Mar 1, 2016 · 5 comments
Closed

Have the linux kernel patches upstreamed #48

severin-lemaignan opened this issue Mar 1, 2016 · 5 comments

Comments

@severin-lemaignan
Copy link
Contributor

If the kernel patch required to use the camera on Linux have been upstreamed, it would be useful to mention it in the documentation, with the name of the first kernel release including these patches.

Otherwise, this issue can serve as a tracking issue for the upstreaming of these patches (links to the possible upstream discussions welcome!)

@teknotus
Copy link

teknotus commented Mar 1, 2016

If the kernel patch required to use the camera on Linux have been
upstreamed, it would be useful to mention it in the documentation, with the
name of the first kernel release including these patches.

There have been at least two patches sent upstream months ago. At least one
was rejected. I'm not sure of the current status. The kernel has extremely
high standards, and it can take some back and forth discussion to work out
the details. All of the discussion is in the linux-media mailing list.
Search for ds4 which is a code name for R200.

@severin-lemaignan
Copy link
Contributor Author

For reference, the last attempt of getting the patch upstream is discussed here (with previous reviews here)

@ddiakopoulos
Copy link
Contributor

Should note that the librealsense patches also contain pixel formats for F200 and SR300, which will likely never be submitted upstream.

@severin-lemaignan
Copy link
Contributor Author

@ddiakopoulos :-( Why that? it is a pity if we can not hope to have soon enough a 'plug and play' user experience with those nice little cameras!

@ddiakopoulos
Copy link
Contributor

The R200 (and future generations) were intended to be compatible with Android while the F/SR series was not. Note that first discussion of the DS4 (R200) on the kernel mailing list dates back over 2 years -- this is typical for upstream patches like this. In the case of F200 and SR300, we don't have enough resources to guide yet another patch through review. Furthermore, even if those patches were accepted, newer generations of cameras would have been released in the meanwhile with potentially even more new pixel formats. So, while it's 100% preferable to have everything upstream, the reality is that it's not a priority.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants