Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 195891 - app-laptop/acpi4asus should not compile a kernel module on 2.6.23 kernels
Summary: app-laptop/acpi4asus should not compile a kernel module on 2.6.23 kernels
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Mobile Herd (OBSOLETE)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: kernel-2.6.23
  Show dependency tree
 
Reported: 2007-10-14 20:53 UTC by thanahauser
Modified: 2009-02-14 21:38 UTC (History)
8 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
acpi4asus-0.41-2.6.23.patch (acpi4asus-0.41-2.6.23.patch,1.54 KB, patch)
2007-10-14 20:54 UTC, thanahauser
Details | Diff
updated ebuild using patch above (acpi4asus-0.41-r1.ebuild,1.44 KB, text/plain)
2007-12-19 19:01 UTC, Tomáš Chvátal (RETIRED)
Details
Better patch to fix the issue (acpi4asus-0.41-2.6.23.1.patch,1.03 KB, patch)
2007-12-21 13:09 UTC, Daniele Varrazzo
Details | Diff
"fixed" version of ebuild (acpi4asus-0.41-r1.ebuild,1.70 KB, text/plain)
2008-01-12 11:06 UTC, Tomáš Chvátal (RETIRED)
Details
acpi4asus-0.41.ebuild.diff (acpi4asus-0.41.ebuild.diff,1.99 KB, patch)
2008-01-12 11:41 UTC, Jakub Moc (RETIRED)
Details | Diff
acpi4asus-0.41.ebuild.diff (acpi4asus-0.41.ebuild.diff,2.04 KB, patch)
2008-01-12 11:57 UTC, Jakub Moc (RETIRED)
Details | Diff
Patch adding module USE flag (acpi4asus-0.41.ebuild.patch,1.48 KB, patch)
2009-01-11 22:04 UTC, Pavel Bazika
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description thanahauser 2007-10-14 20:53:09 UTC
>>> Compiling source in /var/tmp/portage/app-laptop/acpi4asus-0.41/work/acpi4asus-0.41 ...
 * Preparing asus-laptop module
Kernel configuration : 
CONFIG_BACKLIGHT_CLASS_DEVICE : [Y]
CONFIG_LEDS_CLASS : [Y]
make -C /usr/src/linux SUBDIRS=/var/tmp/portage/app-laptop/acpi4asus-0.41/work/acpi4asus-0.41/driver modules
make[1]: Entering directory `/usr/src/linux-2.6.23-gentoo'
  CC [M]  /var/tmp/portage/app-laptop/acpi4asus-0.41/work/acpi4asus-0.41/driver/asus-laptop.o
/var/tmp/portage/app-laptop/acpi4asus-0.41/work/acpi4asus-0.41/driver/asus-laptop.c:210: warning: initialization from incompatible pointer type
/var/tmp/portage/app-laptop/acpi4asus-0.41/work/acpi4asus-0.41/driver/asus-laptop.c: In function 'asus_hotk_notify':
/var/tmp/portage/app-laptop/acpi4asus-0.41/work/acpi4asus-0.41/driver/asus-laptop.c:734: error: implicit declaration of function 'acpi_bus_generate_event'
/var/tmp/portage/app-laptop/acpi4asus-0.41/work/acpi4asus-0.41/driver/asus-laptop.c: In function 'asus_led_exit':
/var/tmp/portage/app-laptop/acpi4asus-0.41/work/acpi4asus-0.41/driver/asus-laptop.c:1083: error: 'struct led_classdev' has no member named 'class_dev'
/var/tmp/portage/app-laptop/acpi4asus-0.41/work/acpi4asus-0.41/driver/asus-laptop.c:1083: error: 'struct led_classdev' has no member named 'class_dev'
/var/tmp/portage/app-laptop/acpi4asus-0.41/work/acpi4asus-0.41/driver/asus-laptop.c:1084: error: 'struct led_classdev' has no member named 'class_dev'
/var/tmp/portage/app-laptop/acpi4asus-0.41/work/acpi4asus-0.41/driver/asus-laptop.c:1084: error: 'struct led_classdev' has no member named 'class_dev'
/var/tmp/portage/app-laptop/acpi4asus-0.41/work/acpi4asus-0.41/driver/asus-laptop.c:1085: error: 'struct led_classdev' has no member named 'class_dev'
/var/tmp/portage/app-laptop/acpi4asus-0.41/work/acpi4asus-0.41/driver/asus-laptop.c:1085: error: 'struct led_classdev' has no member named 'class_dev'
/var/tmp/portage/app-laptop/acpi4asus-0.41/work/acpi4asus-0.41/driver/asus-laptop.c:1086: error: 'struct led_classdev' has no member named 'class_dev'
/var/tmp/portage/app-laptop/acpi4asus-0.41/work/acpi4asus-0.41/driver/asus-laptop.c:1086: error: 'struct led_classdev' has no member named 'class_dev'
/var/tmp/portage/app-laptop/acpi4asus-0.41/work/acpi4asus-0.41/driver/asus-laptop.c:1087: error: 'struct led_classdev' has no member named 'class_dev'
/var/tmp/portage/app-laptop/acpi4asus-0.41/work/acpi4asus-0.41/driver/asus-laptop.c:1087: error: 'struct led_classdev' has no member named 'class_dev'
make[2]: *** [/var/tmp/portage/app-laptop/acpi4asus-0.41/work/acpi4asus-0.41/driver/asus-laptop.o] Error 1
make[1]: *** [_module_/var/tmp/portage/app-laptop/acpi4asus-0.41/work/acpi4asus-0.41/driver] Error 2
make[1]: Leaving directory `/usr/src/linux-2.6.23-gentoo'
make: *** [default] Error 2

Reproducible: Always




These problems may be solved with :

-> from 2.6.23 Changelog

ACPI: Schedule /proc/acpi/event for removal

Schedule /proc/acpi/event for removal in 6 months.

Re-name acpi_bus_generate_event() to acpi_bus_generate_proc_event()
to make sure there is no confusion that it is for /proc/acpi/event only.

Add CONFIG_ACPI_PROC_EVENT to allow removal of /proc/acpi/event.
There is no functional change if CONFIG_ACPI_PROC_EVENT=y

-> http://www.mail-archive.com/git-commits-head@vger.kernel.org/msg19616.html
Comment 1 thanahauser 2007-10-14 20:54:23 UTC
Created attachment 133473 [details, diff]
acpi4asus-0.41-2.6.23.patch
Comment 2 Pacho Ramos gentoo-dev 2007-11-09 19:20:53 UTC
This doesn't work for me, after merging it I get the following:

# modprobe asus-laptop 
FATAL: Error inserting asus_laptop (/lib/modules/2.6.23-gentoo-r1/acpi/asus-laptop.ko): No such device

But previous version with kernel-2.6.22 works ok :-/
Comment 3 Tomáš Chvátal (RETIRED) gentoo-dev 2007-12-19 19:01:31 UTC
Created attachment 138902 [details]
updated ebuild using patch above

for easier usage creating new ebuild
(In reply to comment #2)
you should emerge this when you're on 2.6.23 kernel, this ussualy happens when you did that on prevous version of your kernel
Comment 4 Daniele Varrazzo 2007-12-21 13:09:25 UTC
Created attachment 139036 [details, diff]
Better patch to fix the issue

I fixed the issue before trying to search the bugs and prepared a patch on my own. :-\

My patch seems better anyway. I fixed the missing 'class_dev' as suggested in http://lkml.org/lkml/2007/7/23/151

Regards
Comment 5 Tomáš Chvátal (RETIRED) gentoo-dev 2007-12-22 16:02:12 UTC
Well this patch is not working for me, I tried it throught LKML suggestion when .23 was new and it compiles fine but fn keys didn't work :(
Comment 6 Pacho Ramos gentoo-dev 2008-01-04 17:15:29 UTC
Maybe acpi4asus is no more longer needed. I am now using asus-latop module provided by kernel. You can find it in:
Device Drivers  --->
	Misc devices  --->
		Asus Laptop Extras (EXPERIMENTAL)

Its description is:
CONFIG_ASUS_LAPTOP

  │ This is the new Linux driver for Asus laptops. It may also support some                                 
  │ MEDION, JVC or VICTOR laptops. It makes all the extra buttons generate                                           
  │ standard ACPI events that go through /proc/acpi/events. It also adds       
  │ support for video output switching, LCD backlight control, Bluetooth and           
  │ Wlan control, and most importantly, allows you to blink those fancy LEDs.              
  │                                                                                                                                             
  │ For more information and a userspace daemon for handling the extra                                                                           
  │ buttons see <http://acpi4asus.sf.net/>.                                                                                  
  │                                                                                                                                               
  │ If you have an ACPI-compatible ASUS laptop, say Y or M here.   
Comment 7 Tomáš Chvátal (RETIRED) gentoo-dev 2008-01-12 11:06:14 UTC
Created attachment 140777 [details]
"fixed" version of ebuild

Ok I agree. This ebuild should be blocked for 2.6.23 kernels and above. 
Why to bother with it when we have complete functional mainstream resolution of all problems with this :)
Comment 8 Jakub Moc (RETIRED) gentoo-dev 2008-01-12 11:41:30 UTC
Created attachment 140789 [details, diff]
acpi4asus-0.41.ebuild.diff

- do not compile the kernel module on 2.6.23+ kernels, and check for CONFIG_ASUS_LAPTOP instead
- some cleanup (redundant dodir, einfo -> elog), quoting
Comment 9 Jakub Moc (RETIRED) gentoo-dev 2008-01-12 11:57:19 UTC
Created attachment 140793 [details, diff]
acpi4asus-0.41.ebuild.diff

fix pkg_setup (can't use KV_DIR before it's set).
Comment 10 Cheba 2008-01-16 11:35:50 UTC
I have no CONFIG_ASUS_LAPTOP in my kernel (tuxonice-2.6.23-r6). There is only CONFIG_ACPI_ASUS.
Comment 11 Tomáš Chvátal (RETIRED) gentoo-dev 2008-01-18 13:21:11 UTC
(In reply to comment #10)
> I have no CONFIG_ASUS_LAPTOP in my kernel (tuxonice-2.6.23-r6). There is only
> CONFIG_ACPI_ASUS.
> 
scarab@Ugly-Elf: /usr/src/linux > uname -a
Linux Ugly-Elf 2.6.23-gentoo-r2 #24 Wed Jan 9 18:53:59 CET 2008 i686 Intel(R) Pentium(R) M processor 1.86GHz GenuineIntel GNU/Linux
scarab@Ugly-Elf: /usr/src/linux > cat .config |grep CONFIG_ASUS
CONFIG_ASUS_LAPTOP=y
scarab@Ugly-Elf: /usr/src/linux > cat .config |grep CONFIG_ACPI_ASUS
# CONFIG_ACPI_ASUS is not set


I patch suspend2 myself rather than using that ebuild, cause it ussualy have some issues, but it should not remove other kernel options.

Try to look into
Device Drivers  --->
        Misc devices  --->
                Asus Laptop Extras (EXPERIMENTAL)

if it's not there report bug (it'll be clearly bug of tuxonice-sources)
Comment 12 sambartle 2008-05-20 11:24:11 UTC
(In reply to comment #11)
I also have no CONFIG_ASUS_LAPTOP in my kernel (Clean un-patched Gentoo-Sources-2.6.24-r8)

I have been searching for hours for this option!

> (In reply to comment #10)
> > I have no CONFIG_ASUS_LAPTOP in my kernel (tuxonice-2.6.23-r6). There is only
> > CONFIG_ACPI_ASUS.
> > 
> scarab@Ugly-Elf: /usr/src/linux > uname -a
> Linux Ugly-Elf 2.6.23-gentoo-r2 #24 Wed Jan 9 18:53:59 CET 2008 i686 Intel(R)
> Pentium(R) M processor 1.86GHz GenuineIntel GNU/Linux
> scarab@Ugly-Elf: /usr/src/linux > cat .config |grep CONFIG_ASUS
> CONFIG_ASUS_LAPTOP=y
> scarab@Ugly-Elf: /usr/src/linux > cat .config |grep CONFIG_ACPI_ASUS
> # CONFIG_ACPI_ASUS is not set
> 
> 
> I patch suspend2 myself rather than using that ebuild, cause it ussualy have
> some issues, but it should not remove other kernel options.
> 
> Try to look into
> Device Drivers  --->
>         Misc devices  --->
>                 Asus Laptop Extras (EXPERIMENTAL)
> 
> if it's not there report bug (it'll be clearly bug of tuxonice-sources)
> 

(In reply to comment #11)
> (In reply to comment #10)
> > I have no CONFIG_ASUS_LAPTOP in my kernel (tuxonice-2.6.23-r6). There is only
> > CONFIG_ACPI_ASUS.
> > 
> scarab@Ugly-Elf: /usr/src/linux > uname -a
> Linux Ugly-Elf 2.6.23-gentoo-r2 #24 Wed Jan 9 18:53:59 CET 2008 i686 Intel(R)
> Pentium(R) M processor 1.86GHz GenuineIntel GNU/Linux
> scarab@Ugly-Elf: /usr/src/linux > cat .config |grep CONFIG_ASUS
> CONFIG_ASUS_LAPTOP=y
> scarab@Ugly-Elf: /usr/src/linux > cat .config |grep CONFIG_ACPI_ASUS
> # CONFIG_ACPI_ASUS is not set
> 
> 
> I patch suspend2 myself rather than using that ebuild, cause it ussualy have
> some issues, but it should not remove other kernel options.
> 
> Try to look into
> Device Drivers  --->
>         Misc devices  --->
>                 Asus Laptop Extras (EXPERIMENTAL)
> 
> if it's not there report bug (it'll be clearly bug of tuxonice-sources)
> 

Comment 13 Richard C 2008-06-21 06:20:48 UTC
Check out 
http://forums.gentoo.org/viewtopic-t-693153-highlight-configacpiasus.html

If the option is not available, check that the dependencies are set :
It was Device Drivers->LED Support->LED Class Support for me.

Full Dependencies: MISC_DEVICES && X86 && ACPI && EXPERIMENTAL && !ACPI_ASUS && LEDS_CLASS && BACKLIGHT_CLASS_DEVICE
Comment 14 Pavel Bazika 2009-01-11 22:04:35 UTC
Created attachment 178123 [details, diff]
Patch adding module USE flag

This is a 0.41 ebuild patch that adds 'module' USE flag functionality to this ebuild. Kernel module is built only, if this USE flag is enabled.
I don't know, how to put inherit statement into USE flag condition (it shows me illegal inherit error), so it's leaved there independent on USE flag setting.
Comment 15 Tomáš Chvátal (RETIRED) gentoo-dev 2009-02-14 21:38:25 UTC
Fix commited to the tree.
(i really needed it working)