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

Can't compile under fedora with 6.3.4 kernel #20

Open
jlnhab opened this issue Jun 3, 2023 · 5 comments
Open

Can't compile under fedora with 6.3.4 kernel #20

jlnhab opened this issue Jun 3, 2023 · 5 comments

Comments

@jlnhab
Copy link

jlnhab commented Jun 3, 2023

CC [M] /usr/src/phones/dahdi-linux/drivers/dahdi/xpp/xbus-sysfs.o /usr/src/phones/dahdi-linux/drivers/dahdi/xpp/xbus-sysfs.c:467:19: error: initialization of Ç int (*)(const struct device *, struct kobj_uevent_env *) Ç from incompatible pointer type Ç int (*)(struct device *, struct kobj_uevent_env *) Ç [8;;https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html#index-Wincompatible-pointer-types-Werror=incompatible-pointer-types8;;] 467 | .uevent = astribank_uevent, | ^~~~~~~~~~~~~~~~
@jlnhab jlnhab changed the title Can't compile under fedora with 6.34 kernel Can't compile under fedora with 6.3.4 kernel Jun 3, 2023
@InterLinked1
Copy link
Contributor

What is the bitness of your system (32/64-bit)?
Does it work if you compile using PhreakScript? https://github.com/InterLinked1/phreakscript

@jlnhab
Copy link
Author

jlnhab commented Jun 3, 2023

64 bit
No, I didn't. I maintained a working source until now, but I can't find the definition to correct it.

@listout
Copy link
Contributor

listout commented Jun 22, 2023

@jlnhab This could be due to torvalds/linux@2a81ada. Can you please try my patch at #21.

Basically, struct bus_type.uevent() now takes a const *.

@jlnhab
Copy link
Author

jlnhab commented Jun 22, 2023

I applied it and I still have errors.
Thanks for your help.

@InterLinked1
Copy link
Contributor

I applied it and I still have errors. Thanks for your help.

Is this still an issue for you? I just tested with Fedora 40 using kernel 6.8.5, and I don't run into this build issue.

The fix that listout pointed out was also merged in d7bbc8a

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

3 participants