* Re: sonypc with Sony Vaio VGN-SZ1VP @ 2006-09-27 11:51 stelian 2006-09-28 16:27 ` Yu Luming 2007-01-04 5:24 ` Len Brown 0 siblings, 2 replies; 33+ messages in thread From: stelian @ 2006-09-27 11:51 UTC (permalink / raw) To: Len Brown Cc: Andrew Morton, Ismail Donmez, Stelian Pop, Andrea Gelmini, linux-kernel, linux-acpi >> > Will sony_acpi ever make it to the mainline? Its very useful for new >> Vaio >> > models. > > Nope, not as it is. Useful != supportable. > > 1. It must not create any files under /proc/acpi > This is creating a machine-specific API, which > is exactly what we don't want Nobody can maintain > 50 machine specific APIs. > > These objects must appear generic and under sysfs > as if acpi were not involved in providing them. > > 2. its source code shall not live in drivers/acpi > it is not part of the ACPI implementation after all -- > it is a platform specific driver. In this case, would a patch ripping off asus_acpi, ibm_acpi and toshiba_acpi from the kernel be accepted ? I don't really care much about sony_acpi (since I'm not maintaining it anymore, even if I still answer support requests about it), but this is just silly. This has been going on for more than one and a half year now. Meanwhile (at least from what I've seen), the ACPI subsystem still doesn't provide this "generic" API which platform specific driver need to implement. drivers/acpi/{hotkey.c,video.c} are just rudimentary, and there is no indication that this is going forward: In March 2005 you (Len) said: > The goal is to DELETE ibm, toshiba, and asus drivers -- or at least the > duplicated functions in them. > > platform specific drivers make it harder, not easier, to support more > hardware -- there are a zillion vendors out there, implementing special > drivers for each of them is a strategy of last resort. and > I'd like to keep this driver out-of-tree > until we prove that we can't enhance the > generic code to handle this hardware > without the addition of a new driver. How long is this going to take ? Stelian. ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2006-09-27 11:51 sonypc with Sony Vaio VGN-SZ1VP stelian @ 2006-09-28 16:27 ` Yu Luming 2007-01-04 5:24 ` Len Brown 1 sibling, 0 replies; 33+ messages in thread From: Yu Luming @ 2006-09-28 16:27 UTC (permalink / raw) To: stelian Cc: Len Brown, Andrew Morton, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi On Wednesday 27 September 2006 19:51, stelian@popies.net wrote: > >> > Will sony_acpi ever make it to the mainline? Its very useful for new > >> > >> Vaio > >> > >> > models. > > > > Nope, not as it is. Useful != supportable. > > > > 1. It must not create any files under /proc/acpi > > This is creating a machine-specific API, which > > is exactly what we don't want Nobody can maintain > > 50 machine specific APIs. > > > > These objects must appear generic and under sysfs > > as if acpi were not involved in providing them. > > > > 2. its source code shall not live in drivers/acpi > > it is not part of the ACPI implementation after all -- > > it is a platform specific driver. > > In this case, would a patch ripping off asus_acpi, ibm_acpi and > toshiba_acpi from the kernel be accepted ? > > I don't really care much about sony_acpi (since I'm not maintaining it > anymore, even if I still answer support requests about it), but this is > just silly. This has been going on for more than one and a half year now. > > Meanwhile (at least from what I've seen), the ACPI subsystem still doesn't > provide this "generic" API which platform specific driver need to > implement. drivers/acpi/{hotkey.c,video.c} are just rudimentary, and there > is no indication that this is going forward: > > In March 2005 you (Len) said: > > The goal is to DELETE ibm, toshiba, and asus drivers -- or at least the > > duplicated functions in them. > > > > platform specific drivers make it harder, not easier, to support more > > hardware -- there are a zillion vendors out there, implementing special > > drivers for each of them is a strategy of last resort. hotkey.c was expected to replace all platform specific driver under acpi directory, and I have ever expected that ACPI spec would define standard device ID, and AML method name and event number for common keys such as brightness control, output switch. So, I was expecting the hotkey.c could become the generic driver when such spec was published and accepted by OEMs. But, I don't know if such kind of things will happen. > > and > > > I'd like to keep this driver out-of-tree > > until we prove that we can't enhance the > > generic code to handle this hardware > > without the addition of a new driver. So, if there are NO standards, and we don't want mess up user space tools with a dozen of totally different acpi proc interface for different platform drivers. We have to use generic code to create unified interface for the sake of clean user space tool. Some technique are: 1. use input layer to translate any hot-key event into key code defined in input.h 2. use backlight class (driver/video/backlight.c) to hook generic brightness control interface for brightness control under sysfs. 3. use output class to hook generic output switch control interface for display output switch control under sysfs. 4. other generic code. .. > > How long is this going to take ? > I think the maintainer of asus_acpi, toshiba_acpi, ibm_acpi, sony_acpi, panasonic_acpi, msi_acpi, ... should use the techniques mentioned above. for new platform, Please don't just fork a new driver from toshiba_acpi.c, or the existing ones in drivers/acpi. They also need to use generic code mentioned above. Then, the platform specific driver could be accepted into mainline. Otherwise, I don't know how these kind of platform specific driver can be maintained, and deployed by OSV. Thanks, Luming ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2006-09-27 11:51 sonypc with Sony Vaio VGN-SZ1VP stelian 2006-09-28 16:27 ` Yu Luming @ 2007-01-04 5:24 ` Len Brown 2007-01-04 10:09 ` Stelian Pop 2007-01-09 15:19 ` Luming Yu 1 sibling, 2 replies; 33+ messages in thread From: Len Brown @ 2007-01-04 5:24 UTC (permalink / raw) To: stelian Cc: Andrew Morton, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi On Wednesday 27 September 2006 07:51, stelian@popies.net wrote: > >> > Will sony_acpi ever make it to the mainline? Its very useful for new > > Nope, not as it is. Useful != supportable. > > > > 1. It must not create any files under /proc/acpi > > This is creating a machine-specific API, which > > is exactly what we don't want Nobody can maintain > > 50 machine specific APIs. > > > > These objects must appear generic and under sysfs > > as if acpi were not involved in providing them. > > > > 2. its source code shall not live in drivers/acpi > > it is not part of the ACPI implementation after all -- > > it is a platform specific driver. > >... > > I don't really care much about sony_acpi (since I'm not maintaining it > anymore, even if I still answer support requests about it), but this is > just silly. This has been going on for more than one and a half year now. > > Meanwhile (at least from what I've seen), the ACPI subsystem still doesn't > provide this "generic" API which platform specific driver need to > implement. drivers/acpi/{hotkey.c,video.c} are just rudimentary, and there > is no indication that this is going forward: You are right. And the reason is that platform specific drivers are not part of ACPI. They must either be vendor documented/supplied or reverse-engineered. Vendors have not been forthcoming with documentation or code to support Linux laptops, and our happy team here at Intel is not allowed to be in the reverse enginering business. So I concur that hotkey.c is a failed experiment, and I'm going to delete it. There is more different than common on these boxes, so it makes no sense. video.c, however, is standard, and stays for those machines that actually do follow the public specification. > In March 2005 you (Len) said: > > > The goal is to DELETE ibm, toshiba, and asus drivers -- or at least the > video.c handles the standard compliant machines.> duplicated functions in them. > > > > platform specific drivers make it harder, not easier, to support more > > hardware -- there are a zillion vendors out there, implementing special > > drivers for each of them is a strategy of last resort. Still true, though it is clear we'll never be able to delete platform specific parts -- just the parts that duplicate the generic standard functions.. > > I'd like to keep this driver out-of-tree > > until we prove that we can't enhance the > > generic code to handle this hardware > > without the addition of a new driver. > > How long is this going to take ? How about 2.6.21? What needs to happen is 1. a maintainer for sony_acpi.c needs to step forward I can't do this, I'm not allowed to be in the reverse engineering business. 2. /proc/acpi/sony API needs to be deleted 3. source needs to move out of drivers/acpi, and into drivers/misc along with msi. Luming has a sony laptop and can help with this, but he can't be the permanent maintainer any more than I can, for the same reason. If we can get past #1, then I recommend we apply the patch series in -mm to the acpi-test tree and go from there. -Len ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2007-01-04 5:24 ` Len Brown @ 2007-01-04 10:09 ` Stelian Pop 2007-01-04 19:15 ` Mattia Dongili 2007-01-09 15:19 ` Luming Yu 1 sibling, 1 reply; 33+ messages in thread From: Stelian Pop @ 2007-01-04 10:09 UTC (permalink / raw) To: Len Brown Cc: Andrew Morton, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi Le jeudi 04 janvier 2007 à 00:24 -0500, Len Brown a écrit : > > > I'd like to keep this driver out-of-tree > > > until we prove that we can't enhance the > > > generic code to handle this hardware > > > without the addition of a new driver. > > > > How long is this going to take ? > > How about 2.6.21? Good news ! > What needs to happen is > 1. a maintainer for sony_acpi.c needs to step forward > I can't do this, I'm not allowed to be in the reverse engineering business. Well, I can't do this either, because I just don't have the required hardware anymore. If someone want to step forward now it is a great time ! > > 2. /proc/acpi/sony API needs to be deleted > > 3. source needs to move out of drivers/acpi, and into drivers/misc along with msi. Sensible suggestions, the new maintainer will have to work on this. Thanks Len. -- Stelian Pop <stelian@popies.net> ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2007-01-04 10:09 ` Stelian Pop @ 2007-01-04 19:15 ` Mattia Dongili 2007-01-04 20:51 ` Andrew Morton ` (2 more replies) 0 siblings, 3 replies; 33+ messages in thread From: Mattia Dongili @ 2007-01-04 19:15 UTC (permalink / raw) To: Stelian Pop Cc: Len Brown, Andrew Morton, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney On Thu, Jan 04, 2007 at 11:09:44AM +0100, Stelian Pop wrote: > Le jeudi 04 janvier 2007 à 00:24 -0500, Len Brown a écrit : > > > > > I'd like to keep this driver out-of-tree > > > > until we prove that we can't enhance the > > > > generic code to handle this hardware > > > > without the addition of a new driver. > > > > > > How long is this going to take ? > > > > How about 2.6.21? > > Good news ! > > > What needs to happen is > > 1. a maintainer for sony_acpi.c needs to step forward > > I can't do this, I'm not allowed to be in the reverse engineering business. > > Well, I can't do this either, because I just don't have the required > hardware anymore. > > If someone want to step forward now it is a great time ! I have the hw and I'd be happy to do some basic working on the code but: - I'll probably need some help; - I'll have an almost-blackout between the end of February and the end of April as I'm moving to a different country and I'll need some time before I can be active again (I hope I'll have at least easy mail access for all the time though). Anyway if it is still ok I can maintain the thing, to months seems enough to give the driver a shape. > > 2. /proc/acpi/sony API needs to be deleted > > > > 3. source needs to move out of drivers/acpi, and into drivers/misc along with msi. And turn extra-backlight features into platform_device stuff? So 2 and 3 can come together. Moreover, I own an SZ72B and an older GR7 and have come to the same findings of Cacy, plus a patch to allow a smarter "debug" mode. So, how to proceed? (I've just cloned the linux-acpi-2.6 tree) Thanks Len, Thanks Stelian -- mattia :wq! ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2007-01-04 19:15 ` Mattia Dongili @ 2007-01-04 20:51 ` Andrew Morton 2007-01-04 21:18 ` Mattia Dongili 2007-01-04 23:36 ` Stelian Pop 2007-01-04 23:34 ` Stelian Pop 2007-01-05 10:02 ` Stelian Pop 2 siblings, 2 replies; 33+ messages in thread From: Andrew Morton @ 2007-01-04 20:51 UTC (permalink / raw) To: Mattia Dongili Cc: Stelian Pop, Len Brown, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney On Thu, 4 Jan 2007 20:15:12 +0100 Mattia Dongili <malattia@linux.it> wrote: > On Thu, Jan 04, 2007 at 11:09:44AM +0100, Stelian Pop wrote: > > Le jeudi 04 janvier 2007 __ 00:24 -0500, Len Brown a __crit : > > > > > > > I'd like to keep this driver out-of-tree > > > > > until we prove that we can't enhance the > > > > > generic code to handle this hardware > > > > > without the addition of a new driver. > > > > > > > > How long is this going to take ? > > > > > > How about 2.6.21? > > > > Good news ! > > > > > What needs to happen is > > > 1. a maintainer for sony_acpi.c needs to step forward > > > I can't do this, I'm not allowed to be in the reverse engineering business. > > > > Well, I can't do this either, because I just don't have the required > > hardware anymore. > > > > If someone want to step forward now it is a great time ! > > I have the hw and I'd be happy to do some basic working on the code Neato, thanks. > but: > - I'll probably need some help; > - I'll have an almost-blackout between the end of February and the end > of April as I'm moving to a different country and I'll need some time > before I can be active again (I hope I'll have at least easy mail > access for all the time though). > Anyway if it is still ok I can maintain the thing, to months seems > enough to give the driver a shape. > > > > 2. /proc/acpi/sony API needs to be deleted > > > > > > 3. source needs to move out of drivers/acpi, and into drivers/misc along with msi. > > And turn extra-backlight features into platform_device stuff? So 2 and 3 > can come together. > > Moreover, I own an SZ72B and an older GR7 and have come to the same > findings of Cacy, plus a patch to allow a smarter "debug" mode. > So, how to proceed? (I've just cloned the linux-acpi-2.6 tree) > I have a VGN-something-or-other notebook and I use this driver regularly. The place to start (please) is the patches in -mm: 2.6-sony_acpi4.patch sony_apci-resume.patch sony_apci-resume-fix.patch acpi-add-backlight-support-to-the-sony_acpi.patch acpi-add-backlight-support-to-the-sony_acpi-v2.patch video-sysfs-support-take-2-add-dev-argument-for-backlight_device_register-sony_acpi-fix.patch It presently has both the /proc and /sys/.../backlight/.. interfaces, so the first job would be to chop out the /proc stuff. ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2007-01-04 20:51 ` Andrew Morton @ 2007-01-04 21:18 ` Mattia Dongili 2007-01-04 21:28 ` Andrew Morton 2007-01-04 23:36 ` Stelian Pop 1 sibling, 1 reply; 33+ messages in thread From: Mattia Dongili @ 2007-01-04 21:18 UTC (permalink / raw) To: Andrew Morton Cc: Stelian Pop, Len Brown, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney On Thu, Jan 04, 2007 at 12:51:07PM -0800, Andrew Morton wrote: > On Thu, 4 Jan 2007 20:15:12 +0100 > Mattia Dongili <malattia@linux.it> wrote: [...] > > but: > > - I'll probably need some help; > > - I'll have an almost-blackout between the end of February and the end > > of April as I'm moving to a different country and I'll need some time > > before I can be active again (I hope I'll have at least easy mail > > access for all the time though). > > Anyway if it is still ok I can maintain the thing, to months seems > > enough to give the driver a shape. > > > > > > 2. /proc/acpi/sony API needs to be deleted > > > > > > > > 3. source needs to move out of drivers/acpi, and into drivers/misc along with msi. > > > > And turn extra-backlight features into platform_device stuff? So 2 and 3 > > can come together. > > > > Moreover, I own an SZ72B and an older GR7 and have come to the same > > findings of Cacy, plus a patch to allow a smarter "debug" mode. > > So, how to proceed? (I've just cloned the linux-acpi-2.6 tree) > > > > I have a VGN-something-or-other notebook and I use this driver regularly. Hehehe, I think all the sony lappies are VGN-something :) > The place to start (please) is the patches in -mm: > > 2.6-sony_acpi4.patch > sony_apci-resume.patch > sony_apci-resume-fix.patch > acpi-add-backlight-support-to-the-sony_acpi.patch > acpi-add-backlight-support-to-the-sony_acpi-v2.patch > video-sysfs-support-take-2-add-dev-argument-for-backlight_device_register-sony_acpi-fix.patch > > It presently has both the /proc and /sys/.../backlight/.. interfaces, so the first > job would be to chop out the /proc stuff. Ok, I'll import all of them and start from there. Is it ok to wipe all the /proc stuff without notice? -- mattia :wq! ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2007-01-04 21:18 ` Mattia Dongili @ 2007-01-04 21:28 ` Andrew Morton 2007-01-04 21:36 ` Timo Hoenig 2007-01-04 21:36 ` Richard Hughes 0 siblings, 2 replies; 33+ messages in thread From: Andrew Morton @ 2007-01-04 21:28 UTC (permalink / raw) To: Mattia Dongili Cc: Stelian Pop, Len Brown, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney On Thu, 4 Jan 2007 22:18:30 +0100 Mattia Dongili <malattia@linux.it> wrote: > > The place to start (please) is the patches in -mm: > > > > 2.6-sony_acpi4.patch > > sony_apci-resume.patch > > sony_apci-resume-fix.patch > > acpi-add-backlight-support-to-the-sony_acpi.patch > > acpi-add-backlight-support-to-the-sony_acpi-v2.patch > > video-sysfs-support-take-2-add-dev-argument-for-backlight_device_register-sony_acpi-fix.patch > > > > It presently has both the /proc and /sys/.../backlight/.. interfaces, so the first > > job would be to chop out the /proc stuff. > > Ok, I'll import all of them and start from there. > Is it ok to wipe all the /proc stuff without notice? spose so. I don't know if any apps are dependent upon the /proc file, but the driver isn't in mainline yet so it's unlikely that there's a mountain of software depending upon existing interfaces. ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2007-01-04 21:28 ` Andrew Morton @ 2007-01-04 21:36 ` Timo Hoenig 2007-01-04 21:36 ` Richard Hughes 1 sibling, 0 replies; 33+ messages in thread From: Timo Hoenig @ 2007-01-04 21:36 UTC (permalink / raw) To: Andrew Morton Cc: Mattia Dongili, Stelian Pop, Len Brown, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney Hi, On Thu, 2007-01-04 at 13:28 -0800, Andrew Morton wrote: > spose so. I don't know if any apps are dependent upon the /proc file, > but the driver isn't in mainline yet so it's unlikely that there's a > mountain of software depending upon existing interfaces. Not a mountain, but still a few applications support that interface. At least the powersave daemon 'powersaved' and HAL support the brightness interface of the Sony driver. The responsible developers are following the linux-acpi list. Timo ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2007-01-04 21:28 ` Andrew Morton 2007-01-04 21:36 ` Timo Hoenig @ 2007-01-04 21:36 ` Richard Hughes 2007-01-04 21:58 ` Mattia Dongili 1 sibling, 1 reply; 33+ messages in thread From: Richard Hughes @ 2007-01-04 21:36 UTC (permalink / raw) To: Andrew Morton Cc: Mattia Dongili, Stelian Pop, Len Brown, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney On Thu, 2007-01-04 at 13:28 -0800, Andrew Morton wrote: > On Thu, 4 Jan 2007 22:18:30 +0100 > Mattia Dongili <malattia@linux.it> wrote: > > > > The place to start (please) is the patches in -mm: > > > > > > 2.6-sony_acpi4.patch > > > sony_apci-resume.patch > > > sony_apci-resume-fix.patch > > > acpi-add-backlight-support-to-the-sony_acpi.patch > > > acpi-add-backlight-support-to-the-sony_acpi-v2.patch > > > video-sysfs-support-take-2-add-dev-argument-for-backlight_device_register-sony_acpi-fix.patch > > > > > > It presently has both the /proc and /sys/.../backlight/.. interfaces, so the first > > > job would be to chop out the /proc stuff. > > > > Ok, I'll import all of them and start from there. > > Is it ok to wipe all the /proc stuff without notice? > > spose so. I don't know if any apps are dependent upon the /proc file, > but the driver isn't in mainline yet so it's unlikely that there's a > mountain of software depending upon existing interfaces. Well, HAL has used it for changing the brightness for the last year or so: /proc/acpi/sony/brightness Although if you use a new enough HAL (CVS), the laptop will be supported via the shiny new backlight class. Richard. ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2007-01-04 21:36 ` Richard Hughes @ 2007-01-04 21:58 ` Mattia Dongili 2007-01-05 17:02 ` Len Brown 0 siblings, 1 reply; 33+ messages in thread From: Mattia Dongili @ 2007-01-04 21:58 UTC (permalink / raw) To: Richard Hughes Cc: Andrew Morton, Stelian Pop, Len Brown, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney On Thu, Jan 04, 2007 at 09:36:36PM +0000, Richard Hughes wrote: > On Thu, 2007-01-04 at 13:28 -0800, Andrew Morton wrote: > > On Thu, 4 Jan 2007 22:18:30 +0100 > > Mattia Dongili <malattia@linux.it> wrote: > > > > > > The place to start (please) is the patches in -mm: > > > > > > > > 2.6-sony_acpi4.patch > > > > sony_apci-resume.patch > > > > sony_apci-resume-fix.patch > > > > acpi-add-backlight-support-to-the-sony_acpi.patch > > > > acpi-add-backlight-support-to-the-sony_acpi-v2.patch > > > > video-sysfs-support-take-2-add-dev-argument-for-backlight_device_register-sony_acpi-fix.patch > > > > > > > > It presently has both the /proc and /sys/.../backlight/.. interfaces, so the first > > > > job would be to chop out the /proc stuff. > > > > > > Ok, I'll import all of them and start from there. > > > Is it ok to wipe all the /proc stuff without notice? > > > > spose so. I don't know if any apps are dependent upon the /proc file, > > but the driver isn't in mainline yet so it's unlikely that there's a > > mountain of software depending upon existing interfaces. > > Well, HAL has used it for changing the brightness for the last year or > so: /proc/acpi/sony/brightness > > Although if you use a new enough HAL (CVS), the laptop will be supported > via the shiny new backlight class. great, -mm already has the /sys/class/backlight in place for sony_acpi and I suppose the /proc entry can be kept until 2.6.20 is released, i.e. just before pushing things for .21. Len, would you allow it? -- mattia :wq! ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2007-01-04 21:58 ` Mattia Dongili @ 2007-01-05 17:02 ` Len Brown 2007-01-05 18:06 ` Mattia Dongili 0 siblings, 1 reply; 33+ messages in thread From: Len Brown @ 2007-01-05 17:02 UTC (permalink / raw) To: Mattia Dongili Cc: Richard Hughes, Andrew Morton, Stelian Pop, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney > > Well, HAL has used it for changing the brightness for the last year or > > so: /proc/acpi/sony/brightness > > > > Although if you use a new enough HAL (CVS), the laptop will be supported > > via the shiny new backlight class. > > great, -mm already has the /sys/class/backlight in place for sony_acpi > and I suppose the /proc entry can be kept until 2.6.20 is released, i.e. > just before pushing things for .21. > > Len, would you allow it? Sure, no problem. Checking it into my tree with /proc/acpi/sony is no different than what is in -mm today. When we push upstream, however, the /proc/acpi/sony part should be gone, or at least scheduled for removal. I suggest a sub CONFIG option under CONFIG_SONY_ACPI, say, CONFIG_SONY_ACPI_PROCFS so you can #ifdef the code that is going away. thanks for stepping forward Mattia, -Len ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2007-01-05 17:02 ` Len Brown @ 2007-01-05 18:06 ` Mattia Dongili 0 siblings, 0 replies; 33+ messages in thread From: Mattia Dongili @ 2007-01-05 18:06 UTC (permalink / raw) To: Len Brown Cc: Richard Hughes, Andrew Morton, Stelian Pop, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney On Fri, Jan 05, 2007 at 12:02:30PM -0500, Len Brown wrote: > > > Well, HAL has used it for changing the brightness for the last year or > > > so: /proc/acpi/sony/brightness > > > > > > Although if you use a new enough HAL (CVS), the laptop will be supported > > > via the shiny new backlight class. > > > > great, -mm already has the /sys/class/backlight in place for sony_acpi > > and I suppose the /proc entry can be kept until 2.6.20 is released, i.e. > > just before pushing things for .21. > > > > Len, would you allow it? > > Sure, no problem. > Checking it into my tree with /proc/acpi/sony is > no different than what is in -mm today. > > When we push upstream, however, the /proc/acpi/sony part should be gone, > or at least scheduled for removal. I'd rather avoid pushing mainline something already scheduled for removal. Also, a workaround can eventually be implemented in the userspace tools using /proc/acpi/sony [...] > thanks for stepping forward Mattia, It's me who should thank :) Thanks -- mattia :wq! ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2007-01-04 20:51 ` Andrew Morton 2007-01-04 21:18 ` Mattia Dongili @ 2007-01-04 23:36 ` Stelian Pop 2007-01-04 23:44 ` Andrew Morton 2007-01-05 0:11 ` sonypc with Sony Vaio VGN-SZ1VP Jan Engelhardt 1 sibling, 2 replies; 33+ messages in thread From: Stelian Pop @ 2007-01-04 23:36 UTC (permalink / raw) To: Andrew Morton Cc: Mattia Dongili, Len Brown, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney Le jeudi 04 janvier 2007 à 12:51 -0800, Andrew Morton a écrit : > The place to start (please) is the patches in -mm: > > 2.6-sony_acpi4.patch > sony_apci-resume.patch > sony_apci-resume-fix.patch > acpi-add-backlight-support-to-the-sony_acpi.patch > acpi-add-backlight-support-to-the-sony_acpi-v2.patch > video-sysfs-support-take-2-add-dev-argument-for-backlight_device_register-sony_acpi-fix.patch In addition to those, I also have the attached patch in my local tree. Stelian. --- Added acpi_bus_generate event for forwarding Fn-keys pressed to acpi subsystem, and made correspondent necessary changes for this to work. Signed-off-by: Nilton Volpato <nilton.volpato@gmail.com> --- drivers/acpi/sony_acpi.c | 55 ++++++++++++++++++++++++---------------------- 1 files changed, 29 insertions(+), 26 deletions(-) diff --git a/drivers/acpi/sony_acpi.c b/drivers/acpi/sony_acpi.c index e23522a..0c9367f 100644 --- a/drivers/acpi/sony_acpi.c +++ b/drivers/acpi/sony_acpi.c @@ -33,7 +33,7 @@ #define ACPI_SNC_CLASS "sony" #define ACPI_SNC_HID "SNY5001" -#define ACPI_SNC_DRIVER_NAME "ACPI Sony Notebook Control Driver v0.2" +#define ACPI_SNC_DRIVER_NAME "ACPI Sony Notebook Control Driver v0.3" #define LOG_PFX KERN_WARNING "sony_acpi: " @@ -61,6 +61,7 @@ static struct acpi_driver sony_acpi_driv static acpi_handle sony_acpi_handle; static struct proc_dir_entry *sony_acpi_dir; +static struct acpi_device *sony_acpi_acpi_device = NULL; static struct sony_acpi_value { char *name; /* name of the entry */ @@ -245,7 +246,9 @@ static int sony_acpi_write(struct file * static void sony_acpi_notify(acpi_handle handle, u32 event, void *data) { - printk(LOG_PFX "sony_acpi_notify\n"); + if (debug) + printk(LOG_PFX "sony_acpi_notify, event: %d\n", event); + acpi_bus_generate_event(sony_acpi_acpi_device, 1, event); } static acpi_status sony_walk_callback(acpi_handle handle, u32 level, @@ -269,6 +272,8 @@ static int sony_acpi_add(struct acpi_dev int result; struct sony_acpi_value *item; + sony_acpi_acpi_device = device; + sony_acpi_handle = device->handle; acpi_driver_data(device) = NULL; @@ -282,16 +287,16 @@ static int sony_acpi_add(struct acpi_dev result = -ENODEV; goto outwalk; } + } - status = acpi_install_notify_handler(sony_acpi_handle, - ACPI_DEVICE_NOTIFY, - sony_acpi_notify, - NULL); - if (ACPI_FAILURE(status)) { - printk(LOG_PFX "unable to install notify handler\n"); - result = -ENODEV; - goto outnotify; - } + status = acpi_install_notify_handler(sony_acpi_handle, + ACPI_DEVICE_NOTIFY, + sony_acpi_notify, + NULL); + if (ACPI_FAILURE(status)) { + printk(LOG_PFX "unable to install notify handler\n"); + result = -ENODEV; + goto outnotify; } for (item = sony_acpi_values; item->name; ++item) { @@ -310,7 +315,7 @@ static int sony_acpi_add(struct acpi_dev item->acpiset, &handle))) continue; - item->proc = create_proc_entry(item->name, 0600, + item->proc = create_proc_entry(item->name, 0666, acpi_device_dir(device)); if (!item->proc) { printk(LOG_PFX "unable to create proc entry\n"); @@ -329,13 +334,11 @@ static int sony_acpi_add(struct acpi_dev return 0; outproc: - if (debug) { - status = acpi_remove_notify_handler(sony_acpi_handle, - ACPI_DEVICE_NOTIFY, - sony_acpi_notify); - if (ACPI_FAILURE(status)) - printk(LOG_PFX "unable to remove notify handler\n"); - } + status = acpi_remove_notify_handler(sony_acpi_handle, + ACPI_DEVICE_NOTIFY, + sony_acpi_notify); + if (ACPI_FAILURE(status)) + printk(LOG_PFX "unable to remove notify handler\n"); outnotify: for (item = sony_acpi_values; item->name; ++item) if (item->proc) @@ -350,13 +353,13 @@ static int sony_acpi_remove(struct acpi_ acpi_status status; struct sony_acpi_value *item; - if (debug) { - status = acpi_remove_notify_handler(sony_acpi_handle, - ACPI_DEVICE_NOTIFY, - sony_acpi_notify); - if (ACPI_FAILURE(status)) - printk(LOG_PFX "unable to remove notify handler\n"); - } + sony_acpi_acpi_device = NULL; + + status = acpi_remove_notify_handler(sony_acpi_handle, + ACPI_DEVICE_NOTIFY, + sony_acpi_notify); + if (ACPI_FAILURE(status)) + printk(LOG_PFX "unable to remove notify handler\n"); for (item = sony_acpi_values; item->name; ++item) if (item->proc) -- Stelian Pop <stelian@popies.net> ^ permalink raw reply related [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2007-01-04 23:36 ` Stelian Pop @ 2007-01-04 23:44 ` Andrew Morton 2007-01-04 23:54 ` Stelian Pop 2007-01-05 2:20 ` MoRpHeUz 2007-01-05 0:11 ` sonypc with Sony Vaio VGN-SZ1VP Jan Engelhardt 1 sibling, 2 replies; 33+ messages in thread From: Andrew Morton @ 2007-01-04 23:44 UTC (permalink / raw) To: Stelian Pop Cc: Mattia Dongili, Len Brown, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney On Fri, 05 Jan 2007 00:36:23 +0100 Stelian Pop <stelian@popies.net> wrote: > Added acpi_bus_generate event for forwarding Fn-keys pressed to acpi subsystem, > and made correspondent necessary changes for this to work. neato. err, how does one use this? ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2007-01-04 23:44 ` Andrew Morton @ 2007-01-04 23:54 ` Stelian Pop 2007-01-05 4:16 ` Andrew Morton 2007-01-05 2:20 ` MoRpHeUz 1 sibling, 1 reply; 33+ messages in thread From: Stelian Pop @ 2007-01-04 23:54 UTC (permalink / raw) To: Andrew Morton Cc: Mattia Dongili, Len Brown, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney Le jeudi 04 janvier 2007 à 15:44 -0800, Andrew Morton a écrit : > On Fri, 05 Jan 2007 00:36:23 +0100 > Stelian Pop <stelian@popies.net> wrote: > > > Added acpi_bus_generate event for forwarding Fn-keys pressed to acpi subsystem, > > and made correspondent necessary changes for this to work. > > neato. > > err, how does one use this? :) Well, it seems that on some Vaios (including Nilton's pcg-frv26 but not only this one), the Fn key events aren't seen by sonypi or sony_acpi GHKE method, but do generate an ACPI notify event. For those laptops, the patch forwards the ACPI event to the ACPI system and can be later interpreted in userspace using acpid's /etc/acpi/default.sh (example directly from Nilton): > case "$group" in > button) > case "$action" in > power) /usr/sbin/hibernate > ;; > > lid) cat /proc/acpi/button/lid/LID/state > ;; > > *) logger "ACPI action $action is not defined ($@)" > ;; > esac > ;; > > SNC) echo "$@" > /dev/tcp/localhost/50007 > ;; > > *) logger "ACPI group $group / action $action is not defined" > ;; > esac > > In which I just forward the SNC event to another userspace application > listening on a TCP port. Stelian. -- Stelian Pop <stelian@popies.net> ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2007-01-04 23:54 ` Stelian Pop @ 2007-01-05 4:16 ` Andrew Morton 2007-01-05 9:58 ` Stelian Pop 0 siblings, 1 reply; 33+ messages in thread From: Andrew Morton @ 2007-01-05 4:16 UTC (permalink / raw) To: Stelian Pop Cc: Mattia Dongili, Len Brown, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney On Fri, 05 Jan 2007 00:54:32 +0100 Stelian Pop <stelian@popies.net> wrote: > Le jeudi 04 janvier 2007 à 15:44 -0800, Andrew Morton a écrit : > > On Fri, 05 Jan 2007 00:36:23 +0100 > > Stelian Pop <stelian@popies.net> wrote: > > > > > Added acpi_bus_generate event for forwarding Fn-keys pressed to acpi subsystem, > > > and made correspondent necessary changes for this to work. > > > > neato. > > > > err, how does one use this? > > :) > > Well, it seems that on some Vaios (including Nilton's pcg-frv26 but not > only this one), the Fn key events aren't seen by sonypi or sony_acpi > GHKE method, but do generate an ACPI notify event. Speak English ;) > For those laptops, the patch forwards the ACPI event to the ACPI system > and can be later interpreted in userspace using > acpid's /etc/acpi/default.sh (example directly from Nilton): The only things Mr Red Hat gave me are /etc/acpi/events/sample.conf and /etc/acpi/events/video.conf. > > case "$group" in > > button) > > case "$action" in > > power) /usr/sbin/hibernate > > ;; > > > > lid) cat /proc/acpi/button/lid/LID/state > > ;; > > > > *) logger "ACPI action $action is not defined ($@)" > > ;; > > esac > > ;; > > > > SNC) echo "$@" > /dev/tcp/localhost/50007 > > ;; > > > > *) logger "ACPI group $group / action $action is not defined" > > ;; > > esac > > > > In which I just forward the SNC event to another userspace application > > listening on a TCP port. > I pressed then released a button and dmesg said [ 76.961568] evbug.c: Event. Dev: <NULL>, Type: 1, Code: 148, Value: 1 [ 76.961576] evbug.c: Event. Dev: <NULL>, Type: 0, Code: 0, Value: 0 [ 76.963277] evbug.c: Event. Dev: <NULL>, Type: 1, Code: 148, Value: 0 [ 76.963284] evbug.c: Event. Dev: <NULL>, Type: 0, Code: 0, Value: 0 [ 76.967341] evbug.c: Event. Dev: <NULL>, Type: 1, Code: 148, Value: 1 [ 76.967349] evbug.c: Event. Dev: <NULL>, Type: 0, Code: 0, Value: 0 [ 76.968136] evbug.c: Event. Dev: <NULL>, Type: 1, Code: 148, Value: 0 [ 76.968143] evbug.c: Event. Dev: <NULL>, Type: 0, Code: 0, Value: 0 Nothing else happened. ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2007-01-05 4:16 ` Andrew Morton @ 2007-01-05 9:58 ` Stelian Pop 0 siblings, 0 replies; 33+ messages in thread From: Stelian Pop @ 2007-01-05 9:58 UTC (permalink / raw) To: Andrew Morton Cc: Mattia Dongili, Len Brown, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney Le jeudi 04 janvier 2007 à 20:16 -0800, Andrew Morton a écrit : > On Fri, 05 Jan 2007 00:54:32 +0100 > Stelian Pop <stelian@popies.net> wrote: > > > Le jeudi 04 janvier 2007 à 15:44 -0800, Andrew Morton a écrit : > > > On Fri, 05 Jan 2007 00:36:23 +0100 > > > Stelian Pop <stelian@popies.net> wrote: > > > > > > > Added acpi_bus_generate event for forwarding Fn-keys pressed to acpi subsystem, > > > > and made correspondent necessary changes for this to work. > > > > > > neato. > > > > > > err, how does one use this? > > > > :) > > > > Well, it seems that on some Vaios (including Nilton's pcg-frv26 but not > > only this one), the Fn key events aren't seen by sonypi or sony_acpi > > GHKE method, but do generate an ACPI notify event. > > Speak English ;) Oh, sorry, I'll try again :) There are several ways of detecting the Fn key events, depending on the Vaio series: - some Vaios report them using the SPIC device (driven by sonypi) - some Vaios let the driver poll for the Fn key status using the GHKE ACPI method of the SNC device (driven by sony_acpi) - some Vaios generate an ACPI notify event on the SNC device when a Fn key is pressed - this is what the latest patch is for. Unfortunately there are way too many different Vaio series, and no information about what series support what method. I should have maintained some sort of wiki to let the users build themselves a comprehensive list, but I never get around to do it. Maybe Mattia could do it if he has the time and will... > > For those laptops, the patch forwards the ACPI event to the ACPI system > > and can be later interpreted in userspace using > > acpid's /etc/acpi/default.sh (example directly from Nilton): > > The only things Mr Red Hat gave me are /etc/acpi/events/sample.conf and > /etc/acpi/events/video.conf. Well, there was an /etc/acpi/default.sh in an older version of acpid... I'm not sure what it looks like now on a recent Fedora but on my Ubuntu I still have an acpid package, which has some files in /etc/acpi/ looking familiar. > I pressed then released a button and dmesg said > > [ 76.961568] evbug.c: Event. Dev: <NULL>, Type: 1, Code: 148, Value: 1 > [ 76.961576] evbug.c: Event. Dev: <NULL>, Type: 0, Code: 0, Value: 0 > [ 76.963277] evbug.c: Event. Dev: <NULL>, Type: 1, Code: 148, Value: 0 > [ 76.963284] evbug.c: Event. Dev: <NULL>, Type: 0, Code: 0, Value: 0 > [ 76.967341] evbug.c: Event. Dev: <NULL>, Type: 1, Code: 148, Value: 1 > [ 76.967349] evbug.c: Event. Dev: <NULL>, Type: 0, Code: 0, Value: 0 > [ 76.968136] evbug.c: Event. Dev: <NULL>, Type: 1, Code: 148, Value: 0 > [ 76.968143] evbug.c: Event. Dev: <NULL>, Type: 0, Code: 0, Value: 0 > > Nothing else happened. Well, you got the events from evdev, which means you probably got them directly from sonypi (or the regular keyboard..) Unless your distribution does some neat tricks and somehow feeds the events coming from somewhere into the event subsystem (like Ubuntu's acpi_fakekey for example...). Stelian. -- Stelian Pop <stelian@popies.net> ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2007-01-04 23:44 ` Andrew Morton 2007-01-04 23:54 ` Stelian Pop @ 2007-01-05 2:20 ` MoRpHeUz 2007-01-05 17:11 ` Sony Vaio VGN-SZ340 (was Re: sonypc with Sony Vaio VGN-SZ1VP) Len Brown 1 sibling, 1 reply; 33+ messages in thread From: MoRpHeUz @ 2007-01-05 2:20 UTC (permalink / raw) To: Andrew Morton Cc: Stelian Pop, Mattia Dongili, Len Brown, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney Hi, I own a Sony Vaio VGN-SZ340 and I had problems regarding acpi + it's dual core processor. The guys from Intel gave me a workaround and now it recognises both cores. The problem is that it does not do cpu frequency scaling for both cores, just for cpu0...And when I boot with acpi the Nvidia graphic card doesnt work also. I don't know if you know about these problems regarding sony acpi. The guys from Intel said that this notebook have 2 dst's. So to detect both cores the workaround just uses the second dst (although frequency scaling does work for both.) I can help you to fix this bug...I have the machine where we can do some tests.. Best regards, On 1/4/07, Andrew Morton <akpm@osdl.org> wrote: > On Fri, 05 Jan 2007 00:36:23 +0100 > Stelian Pop <stelian@popies.net> wrote: > > > Added acpi_bus_generate event for forwarding Fn-keys pressed to acpi subsystem, > > and made correspondent necessary changes for this to work. > > neato. > > err, how does one use this? > - > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > ^ permalink raw reply [flat|nested] 33+ messages in thread
* Sony Vaio VGN-SZ340 (was Re: sonypc with Sony Vaio VGN-SZ1VP) 2007-01-05 2:20 ` MoRpHeUz @ 2007-01-05 17:11 ` Len Brown 2007-01-05 17:24 ` MoRpHeUz 0 siblings, 1 reply; 33+ messages in thread From: Len Brown @ 2007-01-05 17:11 UTC (permalink / raw) To: MoRpHeUz Cc: Andrew Morton, Stelian Pop, Mattia Dongili, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney On Thursday 04 January 2007 21:20, MoRpHeUz wrote: > Hi, > > I own a Sony Vaio VGN-SZ340 and I had problems regarding acpi + it's > dual core processor. The guys from Intel gave me a workaround and now > it recognises both cores. What workaround are you using? > The problem is that it does not do cpu frequency scaling for both > cores, just for cpu0...And when I boot with acpi the Nvidia graphic > card doesnt work also. > > I don't know if you know about these problems regarding sony acpi. > The guys from Intel said that this notebook have 2 dst's. So to detect > both cores the workaround just uses the second dst (although frequency > scaling does work for both.) > > I can help you to fix this bug...I have the machine where we can do > some tests.. The frequency scaling issue sounds like a BIOS/Linux incompatibility. Please open a bugzilla, if you haven't already, and include the output from acpidump. The nvidia issue sounds like an interrupt issue, so please reproduce it using the open source nvidia driver (not the nvidia binary), and include the lspci -vv output, dmesg, and /proc/interrupts. thanks, -Len ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: Sony Vaio VGN-SZ340 (was Re: sonypc with Sony Vaio VGN-SZ1VP) 2007-01-05 17:11 ` Sony Vaio VGN-SZ340 (was Re: sonypc with Sony Vaio VGN-SZ1VP) Len Brown @ 2007-01-05 17:24 ` MoRpHeUz 2007-01-05 18:10 ` Len Brown 0 siblings, 1 reply; 33+ messages in thread From: MoRpHeUz @ 2007-01-05 17:24 UTC (permalink / raw) To: Len Brown Cc: Andrew Morton, Stelian Pop, Mattia Dongili, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney > What workaround are you using? This one: http://bugzilla.kernel.org/show_bug.cgi?id=7465 > The frequency scaling issue sounds like a BIOS/Linux incompatibility. > Please open a bugzilla, if you haven't already, and include the > output from acpidump. I agree that it sound like a BIOS/Linux incompatibility. You can find my acpidump and DSDT inside the link above. That bug is still opened. > The nvidia issue sounds like an interrupt issue, so please reproduce > it using the open source nvidia driver (not the nvidia binary), > and include the lspci -vv output, dmesg, and /proc/interrupts. Will try that ! Thanks ! ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: Sony Vaio VGN-SZ340 (was Re: sonypc with Sony Vaio VGN-SZ1VP) 2007-01-05 17:24 ` MoRpHeUz @ 2007-01-05 18:10 ` Len Brown 2007-01-06 4:09 ` Bjorn Helgaas 0 siblings, 1 reply; 33+ messages in thread From: Len Brown @ 2007-01-05 18:10 UTC (permalink / raw) To: MoRpHeUz Cc: Andrew Morton, Stelian Pop, Mattia Dongili, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney On Friday 05 January 2007 12:24, MoRpHeUz wrote: > > What workaround are you using? > > This one: http://bugzilla.kernel.org/show_bug.cgi?id=7465 Ah yes, the duplicate MADT issue is clearly a BIOS bug. It is possible that we can tweak our Linux workaround for it to be more Microsoft Windows Bug Compatbile(TM). > > The frequency scaling issue sounds like a BIOS/Linux incompatibility. It looks like this issue results from that above, rather than being an additional problem. > > The nvidia issue sounds like an interrupt issue, so please reproduce > > it using the open source nvidia driver (not the nvidia binary), > > and include the lspci -vv output, dmesg, and /proc/interrupts. > > Will try that ! If interrupts fail using the open source nvidia driver, (and using the workaround from the bug above to use the right MADT, please open a new bug report as I think it would be an independent issue. thanks, -Len ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: Sony Vaio VGN-SZ340 (was Re: sonypc with Sony Vaio VGN-SZ1VP) 2007-01-05 18:10 ` Len Brown @ 2007-01-06 4:09 ` Bjorn Helgaas 2007-01-11 19:52 ` Len Brown 0 siblings, 1 reply; 33+ messages in thread From: Bjorn Helgaas @ 2007-01-06 4:09 UTC (permalink / raw) To: Len Brown Cc: MoRpHeUz, Andrew Morton, Stelian Pop, Mattia Dongili, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney On Friday 05 January 2007 11:10, Len Brown wrote: > On Friday 05 January 2007 12:24, MoRpHeUz wrote: > > > What workaround are you using? > > > > This one: http://bugzilla.kernel.org/show_bug.cgi?id=7465 > > Ah yes, the duplicate MADT issue is clearly a BIOS bug. > It is possible that we can tweak our Linux workaround for it to be more > Microsoft Windows Bug Compatbile(TM). Maybe Windows discovers processors using the namespace rather than the MADT. ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: Sony Vaio VGN-SZ340 (was Re: sonypc with Sony Vaio VGN-SZ1VP) 2007-01-06 4:09 ` Bjorn Helgaas @ 2007-01-11 19:52 ` Len Brown 2007-01-11 20:01 ` Alexey Starikovskiy 0 siblings, 1 reply; 33+ messages in thread From: Len Brown @ 2007-01-11 19:52 UTC (permalink / raw) To: Bjorn Helgaas Cc: MoRpHeUz, Andrew Morton, Stelian Pop, Mattia Dongili, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney On Friday 05 January 2007 23:09, Bjorn Helgaas wrote: > On Friday 05 January 2007 11:10, Len Brown wrote: > > On Friday 05 January 2007 12:24, MoRpHeUz wrote: > > > > What workaround are you using? > > > > > > This one: http://bugzilla.kernel.org/show_bug.cgi?id=7465 > > > > Ah yes, the duplicate MADT issue is clearly a BIOS bug. > > It is possible that we can tweak our Linux workaround for it to be more > > Microsoft Windows Bug Compatbile(TM). > > Maybe Windows discovers processors using the namespace rather > than the MADT. Nod. Based on the fact that the 1st MADT on this box is toast, they're not using that. If the last one also doesn't work universally, then they must be using the namespace. For us to do the same would be a relatively significant change -- as it means we either have to push SMP startup after the interpreter init, or move the interpreter init yet sooner. In general, over the last couple of years, we've been forced for compatibility with various systems to move ACPI initialization sooner and sooner. (I think the last issue was getting the HW into "ACPI mode" sooner because some stuff I don't recall didn't work if we didn't) It would probably make sense to experiment with what the soonest we can initialize ACPI, as I have a feeling we're going to have to head that way. -Len ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: Sony Vaio VGN-SZ340 (was Re: sonypc with Sony Vaio VGN-SZ1VP) 2007-01-11 19:52 ` Len Brown @ 2007-01-11 20:01 ` Alexey Starikovskiy 0 siblings, 0 replies; 33+ messages in thread From: Alexey Starikovskiy @ 2007-01-11 20:01 UTC (permalink / raw) To: Len Brown Cc: Bjorn Helgaas, MoRpHeUz, Andrew Morton, Stelian Pop, Mattia Dongili, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney Len Brown wrote: > On Friday 05 January 2007 23:09, Bjorn Helgaas wrote: > >> On Friday 05 January 2007 11:10, Len Brown wrote: >> >>> On Friday 05 January 2007 12:24, MoRpHeUz wrote: >>> >>>>> What workaround are you using? >>>>> >>>> This one: http://bugzilla.kernel.org/show_bug.cgi?id=7465 >>>> >>> Ah yes, the duplicate MADT issue is clearly a BIOS bug. >>> It is possible that we can tweak our Linux workaround for it to be more >>> Microsoft Windows Bug Compatbile(TM). >>> >> Maybe Windows discovers processors using the namespace rather >> than the MADT. >> > > Nod. > > Based on the fact that the 1st MADT on this box is toast, they're not using that. > If the last one also doesn't work universally, then they must be using the namespace. > > For us to do the same would be a relatively significant change -- as it means > we either have to push SMP startup after the interpreter init, or move the > interpreter init yet sooner. > > In general, over the last couple of years, we've been forced for compatibility > with various systems to move ACPI initialization sooner and sooner. > (I think the last issue was getting the HW into "ACPI mode" sooner > because some stuff I don't recall didn't work if we didn't) > It would probably make sense to experiment with what the soonest we > can initialize ACPI, as I have a feeling we're going to have to head that way. > > -Len > - > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > If any of the two tables does not work, may be we need both together? Regards, Alex. ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2007-01-04 23:36 ` Stelian Pop 2007-01-04 23:44 ` Andrew Morton @ 2007-01-05 0:11 ` Jan Engelhardt 2007-01-05 9:15 ` Mattia Dongili 2007-01-05 9:59 ` Stelian Pop 1 sibling, 2 replies; 33+ messages in thread From: Jan Engelhardt @ 2007-01-05 0:11 UTC (permalink / raw) To: Stelian Pop Cc: Andrew Morton, Mattia Dongili, Len Brown, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney On Jan 5 2007 00:36, Stelian Pop wrote: >@@ -61,6 +61,7 @@ static struct acpi_driver sony_acpi_driv > > static acpi_handle sony_acpi_handle; > static struct proc_dir_entry *sony_acpi_dir; >+static struct acpi_device *sony_acpi_acpi_device = NULL; acpi_acpi? >@@ -310,7 +315,7 @@ static int sony_acpi_add(struct acpi_dev > item->acpiset, &handle))) > continue; > >- item->proc = create_proc_entry(item->name, 0600, >+ item->proc = create_proc_entry(item->name, 0666, > acpi_device_dir(device)); > if (!item->proc) { > printk(LOG_PFX "unable to create proc entry\n"); Is this safe? I would not want normal users to poke on that. -`J' -- Himself owner of a VAIO U3. ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2007-01-05 0:11 ` sonypc with Sony Vaio VGN-SZ1VP Jan Engelhardt @ 2007-01-05 9:15 ` Mattia Dongili 2007-01-05 9:59 ` Stelian Pop 1 sibling, 0 replies; 33+ messages in thread From: Mattia Dongili @ 2007-01-05 9:15 UTC (permalink / raw) To: Jan Engelhardt Cc: Stelian Pop, Andrew Morton, Len Brown, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney On Fri, Jan 05, 2007 at 01:11:16AM +0100, Jan Engelhardt wrote: > > On Jan 5 2007 00:36, Stelian Pop wrote: > >@@ -61,6 +61,7 @@ static struct acpi_driver sony_acpi_driv > > > > static acpi_handle sony_acpi_handle; > > static struct proc_dir_entry *sony_acpi_dir; > >+static struct acpi_device *sony_acpi_acpi_device = NULL; > > acpi_acpi? > > >@@ -310,7 +315,7 @@ static int sony_acpi_add(struct acpi_dev > > item->acpiset, &handle))) > > continue; > > > >- item->proc = create_proc_entry(item->name, 0600, > >+ item->proc = create_proc_entry(item->name, 0666, > > acpi_device_dir(device)); > > if (!item->proc) { > > printk(LOG_PFX "unable to create proc entry\n"); > > Is this safe? I would not want normal users to poke on that. Hmmm, seconded. It also seems quite a gratuitous change and I have a different patch that takes care of permissions and the /proc stuff is going away in any case. -- mattia :wq! ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2007-01-05 0:11 ` sonypc with Sony Vaio VGN-SZ1VP Jan Engelhardt 2007-01-05 9:15 ` Mattia Dongili @ 2007-01-05 9:59 ` Stelian Pop 1 sibling, 0 replies; 33+ messages in thread From: Stelian Pop @ 2007-01-05 9:59 UTC (permalink / raw) To: Jan Engelhardt Cc: Andrew Morton, Mattia Dongili, Len Brown, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney Le vendredi 05 janvier 2007 à 01:11 +0100, Jan Engelhardt a écrit : > On Jan 5 2007 00:36, Stelian Pop wrote: > >@@ -61,6 +61,7 @@ static struct acpi_driver sony_acpi_driv > > > > static acpi_handle sony_acpi_handle; > > static struct proc_dir_entry *sony_acpi_dir; > >+static struct acpi_device *sony_acpi_acpi_device = NULL; > > acpi_acpi? Yeah, maybe Mattia will have more imagination than I at naming variables :) -- Stelian Pop <stelian@popies.net> ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2007-01-04 19:15 ` Mattia Dongili 2007-01-04 20:51 ` Andrew Morton @ 2007-01-04 23:34 ` Stelian Pop 2007-01-05 10:02 ` Stelian Pop 2 siblings, 0 replies; 33+ messages in thread From: Stelian Pop @ 2007-01-04 23:34 UTC (permalink / raw) To: Mattia Dongili Cc: Len Brown, Andrew Morton, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney Le jeudi 04 janvier 2007 à 20:15 +0100, Mattia Dongili a écrit : > > If someone want to step forward now it is a great time ! > > I have the hw and I'd be happy to do some basic working on the code Cool ! > but: > - I'll probably need some help; Feel free to ask... > - I'll have an almost-blackout between the end of February and the end > of April as I'm moving to a different country and I'll need some time > before I can be active again (I hope I'll have at least easy mail > access for all the time though). Well, I am the current "maintainer" for this and it has probably been a year since I left Sony's laptop world, so I guess it won't make much difference if you're not very active for a month or two :) Stelian. -- Stelian Pop <stelian@popies.net> ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2007-01-04 19:15 ` Mattia Dongili 2007-01-04 20:51 ` Andrew Morton 2007-01-04 23:34 ` Stelian Pop @ 2007-01-05 10:02 ` Stelian Pop 2007-01-05 12:13 ` Mattia Dongili 2 siblings, 1 reply; 33+ messages in thread From: Stelian Pop @ 2007-01-05 10:02 UTC (permalink / raw) To: Mattia Dongili Cc: Len Brown, Andrew Morton, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney Le jeudi 04 janvier 2007 à 20:15 +0100, Mattia Dongili a écrit : > > > What needs to happen is > > > 1. a maintainer for sony_acpi.c needs to step forward > > > I can't do this, I'm not allowed to be in the reverse engineering business. > > > > Well, I can't do this either, because I just don't have the required > > hardware anymore. > > > > If someone want to step forward now it is a great time ! > > I have the hw and I'd be happy to do some basic working on the code FYI, sonypi is also searching for a new maintainer, and it is quite closely related to sony_acpi... If you are interested by the job, it is all yours. :) Stelian. -- Stelian Pop <stelian@popies.net> ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2007-01-05 10:02 ` Stelian Pop @ 2007-01-05 12:13 ` Mattia Dongili 2007-01-05 14:23 ` Jan Engelhardt 0 siblings, 1 reply; 33+ messages in thread From: Mattia Dongili @ 2007-01-05 12:13 UTC (permalink / raw) To: Stelian Pop Cc: Len Brown, Andrew Morton, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney On Fri, Jan 05, 2007 at 11:02:03AM +0100, Stelian Pop wrote: > Le jeudi 04 janvier 2007 à 20:15 +0100, Mattia Dongili a écrit : > > > > > What needs to happen is > > > > 1. a maintainer for sony_acpi.c needs to step forward > > > > I can't do this, I'm not allowed to be in the reverse engineering business. > > > > > > Well, I can't do this either, because I just don't have the required > > > hardware anymore. > > > > > > If someone want to step forward now it is a great time ! > > > > I have the hw and I'd be happy to do some basic working on the code > > FYI, sonypi is also searching for a new maintainer, and it is quite > closely related to sony_acpi... Yes, probably the FnKeys stuff needs to worked out into a single driver to ease it usage (I still remember some very old discussion about making sonypi use the acpi methods to access the hw). > If you are interested by the job, it is all yours. :) Let's see if I can come up with something, I have also an ux50 that is not very happy with current sonypi Thanks -- mattia :wq! ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2007-01-05 12:13 ` Mattia Dongili @ 2007-01-05 14:23 ` Jan Engelhardt 0 siblings, 0 replies; 33+ messages in thread From: Jan Engelhardt @ 2007-01-05 14:23 UTC (permalink / raw) To: Mattia Dongili Cc: Stelian Pop, Len Brown, Andrew Morton, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi, Cacy Rodney On Jan 5 2007 13:13, Mattia Dongili wrote: > >> If you are interested by the job, it is all yours. :) > >Let's see if I can come up with something, I have also an ux50 that is >not very happy with current sonypi Feel free to contact me for testing on U3. FnKey is done through sonypi here, if I unload it, `showkey` does not give keycodes for them anymore. -`J' -- ^ permalink raw reply [flat|nested] 33+ messages in thread
* Re: sonypc with Sony Vaio VGN-SZ1VP 2007-01-04 5:24 ` Len Brown 2007-01-04 10:09 ` Stelian Pop @ 2007-01-09 15:19 ` Luming Yu 1 sibling, 0 replies; 33+ messages in thread From: Luming Yu @ 2007-01-09 15:19 UTC (permalink / raw) To: Len Brown Cc: stelian, Andrew Morton, Ismail Donmez, Andrea Gelmini, linux-kernel, linux-acpi > Luming has a sony laptop and can help with this, but > he can't be the permanent maintainer any more than I can, for the same reason. > If we can get past #1, then I recommend we apply the patch series in -mm to > the acpi-test tree and go from there. Yes, I happen to have a sony laptop. So, I can help the permanent maintainer of sony acpi driver on acpi related issues. --Luming ^ permalink raw reply [flat|nested] 33+ messages in thread
end of thread, other threads:[~2007-01-11 20:01 UTC | newest] Thread overview: 33+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2006-09-27 11:51 sonypc with Sony Vaio VGN-SZ1VP stelian 2006-09-28 16:27 ` Yu Luming 2007-01-04 5:24 ` Len Brown 2007-01-04 10:09 ` Stelian Pop 2007-01-04 19:15 ` Mattia Dongili 2007-01-04 20:51 ` Andrew Morton 2007-01-04 21:18 ` Mattia Dongili 2007-01-04 21:28 ` Andrew Morton 2007-01-04 21:36 ` Timo Hoenig 2007-01-04 21:36 ` Richard Hughes 2007-01-04 21:58 ` Mattia Dongili 2007-01-05 17:02 ` Len Brown 2007-01-05 18:06 ` Mattia Dongili 2007-01-04 23:36 ` Stelian Pop 2007-01-04 23:44 ` Andrew Morton 2007-01-04 23:54 ` Stelian Pop 2007-01-05 4:16 ` Andrew Morton 2007-01-05 9:58 ` Stelian Pop 2007-01-05 2:20 ` MoRpHeUz 2007-01-05 17:11 ` Sony Vaio VGN-SZ340 (was Re: sonypc with Sony Vaio VGN-SZ1VP) Len Brown 2007-01-05 17:24 ` MoRpHeUz 2007-01-05 18:10 ` Len Brown 2007-01-06 4:09 ` Bjorn Helgaas 2007-01-11 19:52 ` Len Brown 2007-01-11 20:01 ` Alexey Starikovskiy 2007-01-05 0:11 ` sonypc with Sony Vaio VGN-SZ1VP Jan Engelhardt 2007-01-05 9:15 ` Mattia Dongili 2007-01-05 9:59 ` Stelian Pop 2007-01-04 23:34 ` Stelian Pop 2007-01-05 10:02 ` Stelian Pop 2007-01-05 12:13 ` Mattia Dongili 2007-01-05 14:23 ` Jan Engelhardt 2007-01-09 15:19 ` Luming Yu
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).