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

__devinit problem in base/arch/x86/calibration/smi-module.c #11

Closed
steffenmauch opened this issue Aug 2, 2013 · 5 comments
Closed

__devinit problem in base/arch/x86/calibration/smi-module.c #11

steffenmauch opened this issue Aug 2, 2013 · 5 comments
Labels

Comments

@steffenmauch
Copy link
Contributor

I think through the latest merge, __devinit has appeared inside base/arch/x86/calibration/smi-module.c again.

With __devinit it does not compile with Ubuntu 12.04!

@ShabbyX
Copy link
Owner

ShabbyX commented Aug 2, 2013

@steffenmauch, I compiled under 12.04 just a couple days ago (after the merge) and it was alright. Can you let me know more information from your setup? For example the output of the following commands:

$ gcc --version
$ uname -a

@steffenmauch
Copy link
Contributor Author

Linux rtai-fpga 3.8.13-RTAI #4 SMP PREEMPT Sat Aug 3 12:04:41 CEST 2013 i686 i686 i386 GNU/Linux

gcc (Ubuntu/Linaro 4.6.3-1ubuntu5) 4.6.3
Copyright (C) 2011 Free Software Foundation, Inc.
This is free software; see the source for copying conditions. There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

tried with x86 and x86_64 Ubuntu 12.04.2 LTS

@ShabbyX
Copy link
Owner

ShabbyX commented Aug 3, 2013

Fixed in bd08e27

Thanks for the report.

@ShabbyX ShabbyX closed this as completed Aug 3, 2013
@NTULINUX
Copy link
Collaborator

NTULINUX commented Aug 4, 2013

Should line 36 have a space after pound?

@ShabbyX
Copy link
Owner

ShabbyX commented Aug 4, 2013

That's how I indent directives inside #if. I don't recall right now how that's done in RTAI. Feel free to update it to RTAI's style

@ShabbyX ShabbyX added the fixed label Jun 27, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants