-
Notifications
You must be signed in to change notification settings - Fork 38
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
Problem with instructions for applying kernel patch #6
Comments
If you want to go the easy route the ubuntu ppa also has a kernel for 14.04, but I think it was last updated in May, so the package installer won't think it's the latest any more, and you'll probably have to pick it manually. It's due for an update, but that usually takes hours, and I unfortunately don't have that much time right now. As of right now you probably want linux-image-extra-3.13.0-53-generic and linux-libc-dev which has the include file. It also has in the linux source package the already patched files. I had to change the patches slightly to build the 3.13 kernel. I think I copied and pasted from my terminal everything I did to compile just the module, but I may have missed something. I tried to follow my own instructions, and it had similar errors. Compiling the full kernel might be a better option, but it takes hours with the config that Ubuntu uses. |
@teknotus thanks, appreciate the help! i think compiling the whole kernel maybe easiest. Will try someday soon. |
Just a question, have you submitted the patch for inclusion into the kernel? |
@floe nope sorry I haven't gotten to that yet. I got pulled in a bunch of different directions with do I work on a userspace library, a point cloud module, calibration tools, etc. I usually ended up going with whatever someone would pay me to do, and I got too burned out with that to get through the somewhat intimidating task of upstreaming to the Linux kernel. I'm tired of maintaining patches for various kernel versions, and ubuntu kernel packages so it's now the thing I want to get done the most. |
Actually, I think it should be pretty straightforward, your patch already meets the style guidelines; I think linux-media is the list to submit to. |
I'm getting back to working on the F200, would you mind if I just submit your patch to linux-media? |
Oh I finally got involved in the conversation on linux-media a few days ago. Intel submitted a patch for the R200 that looks almost exactly like mine, but with different formats. It got rejected, and thus mine would probably also be rejected at the moment. They are having some argument about if depth cameras should be treated differently than regular cameras, and having many specialty image formats for depth cameras. I've been a bit distracted figuring out the R200. I got a HP Spectre X2 which has a built in R200. It has a few bits of hardware that don't work in Linux yet, so I might end up sending multiple patches to the kernel. I got the LTE modem working well enough to get GPS data out of it from Linux. I also really want the IMU working so that I can use it with 3d reconstruction. |
Hi @mirzashah @teknotus
So as i have seen in your previous comments that you are also encountering with same problems.So can you please suggest some solution for above problem |
The usbtv tracker is here. |
Hi @teknotus
I had problems applying the patch to any kernel in Ubuntu 14.04 LTS. Perhaps you can see the mistake I'm making and can point me in the right direction:
Patch fails to automatically apply with git apply -- the patch failed to apply to 3.13, 3.16, and 4.0 kernel sources, both from Ubuntu sources as well as vanilla kernels from github.com/torvalds. The patch isn't that large so I was able to apply all the diffs by hand.
When I went to compile as it was looking at headers in /usr/source/
uname-r
versus the headers in the downloaded source where modified videodev2.h and uvc_camera.h reside. This results in compile errors because all the new macros in videodev2.h are not found.Now based on the step:
sudo cp include/uapi/linux/videodev2.h /usr/include/linux/videodev2.h
I'm guessing that that step was intended so that the code would build against
/usr/include/linux/videodev2.h
. So I tried taking the modified videodev2.h and copying it to/usr/source/
uname-r/include/linux/videodev2.h
as well. Once I did this I was able to compile the moduleLooks like a lot of v4l2 calls weren't linked into the module. If you have any insights into what I"m doing wrong, please let me know. Thanks!
The text was updated successfully, but these errors were encountered: