ljalves / linux_media

TBS linux open source drivers
https://github.com/ljalves/linux_media/wiki
Other
89 stars 150 forks source link

Kernel build fixes #122

Closed bw35 closed 8 years ago

bw35 commented 8 years ago

Building the kernel from the linux_media repository w/o media_build requires some small modifications to the saa716x Kconfig and Makefile.

tkurbad commented 8 years ago

@bw35 These changes prevent the saa716x_budget module from being build w/ media_build. How can this be overcome? Has media_build to be changed somehow?

bw35 commented 8 years ago

@tkurbad In fact I didn't try with media_build. I've cross compiled the kernel in this repo. I'll try with media_build to see where it fails.

Maybe you have more info for me: For which kernel version did you try to build the drivers and what was the relevant output of the build process?

tkurbad commented 8 years ago

I tried to compile for kernel version 4.5.0. There weren't any build errors. Only after installing I noticed that the saa716x_budget module was missing (saa716x_core, saa716x_ff, and saa716x_hybrid were there, nonetheless)

I then checked out c9626a9e5e90cc9eac7d985159f0d67a6870146b and reran the build. Now the missing module is there again (and works).

HTH, Torsten

bw35 commented 8 years ago

@tkurbad I can't reproduce your problem. I've done the media_build method as described in the wiki and got all modules including saa716x_budget.ko in the media_build/v4l directory. Checked with Kernel 4.1.15 and 4.5.0, latter required a small fix in the cdx2820r_core.c because of a change introduced by kernel 4.5 in include/linux/gpio/driver.

tkurbad commented 8 years ago

@bw35 Strange. I'll look into it again after my next kernel update.