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

Some issues with compiling this on linux kernel 4.4 mainline #2

Open
fenrig opened this issue Feb 4, 2016 · 5 comments
Open

Some issues with compiling this on linux kernel 4.4 mainline #2

fenrig opened this issue Feb 4, 2016 · 5 comments

Comments

@fenrig
Copy link

fenrig commented Feb 4, 2016

I'm compiling your remote proc linux kernel driver using the yocto project for beagleboneblack, this uses linux kernel 4.4 mainline and your code and my kernel source arent fully matching because Im getting a few errors:

Issue 1:
I can't seem to find ".handle_custom_rsc" in rproc_ops which doesn't exist in the mainline kernel code, so my compiler doesnt compile this code. What kernel source did you use for developing this code?

Issue 2:
seq_puts and seq_printf are void functions and you use them for adding the return value (which is void!) to add to a ret int. The compiler on yocto marks these as errors, so I'm kind of wondering how your compiler didnt warn you for these

@shubhi1407
Copy link
Owner

  1. I used kernel 3.14. Building for 4.x would fail as the mainline has some files missing or changed.
  2. seq_printf returns void. My compiler may have thrown warnings but I didn't encounter errors.

I suggest you try for 3.14. Fixing it for 4.4 would require considerable effort i patching the kernel code.

@fenrig
Copy link
Author

fenrig commented Feb 4, 2016

mainline 3.14 or did you use the one from beaglebone?

@RobertCNelson
Copy link

Oh, if this is based on ti's 3.14.x remote_proc, @fenrig you should use the "4.1" branch here: https://github.com/beagleboard/linux/tree/4.1

as that remote_proc driver isn't mainline yet..

Regards,

@fenrig
Copy link
Author

fenrig commented Feb 4, 2016

yeah I was making a recipe for this kernel, thx for the info ;)

@klekkala
Copy link

Hey!! @fenrig !! Did you try out this driver for 4.4 mainstream?

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

No branches or pull requests

4 participants