Re: [PATCH 00/10] [Intrepid] SRU: LP#257827 -- acpi brightness control

On Fri, Nov 21, 2008 at 05:16:10PM +0000, Andy Whitcroft wrote:
> On Thu, Nov 20, 2008 at 11:22:55AM -0700, Tim Gardner wrote:
> > Andy Whitcroft wrote:
> > > This series represents a backport of a change to the ACPI backlight
> > > support.  This change was original done to sort out brightness control
> > > problems for a number of different laptop models triggered by non-existant
> > > devices being created (as reported on this bug).  These changes also
> > > are reported to fix boot crashes on a number of Samsung laptops (as
> > > reported on LP#146692).  All of these changes are direct back ports from
> > > mainline, no changes were needed to apply them.  They are all upstream
> > > as of v2.6.28-rc5.
> > > 
> > > I have built test kernels with these applied and a number of community
> > > members have tested these and confirm that they do indeed allow those
> > > laptops to boot and brightness control is improved there.
> > > 
> > > Proposing these for SRU to Intrepid.
> > > 
> > > -apw
> > 
> > Disregard previous transmission wrt to '[PATCH 01/10] ACPI: video:
> > Ignore devices that aren't present in hardware'
> > 
> > Provisional ACK pending test results.
> 
> I have rebased these onto the top of the 2.6.27-10.20 and pushed those
> up for any intrested testers.  So far we have three positive tests
> results from the boot crashing Samsung laptops, with some reports of
> improved brightness handling from them.  That seems pretty positive.
> Will see if anyone from the primary report gives us any feedback this

Problem with missing files in the kernel headers (2.6.28) in Jaunty

Hi all,

I have noticed that if I try to build the Nvidia driver in Jaunty with the 
latest kernel, Nvidia's conftest.sh fails. After making this test more verbose 
I found out that there can be something wrong with the headers.

Nvidia's conftest.sh tries to build a file (see the attached .c file) and 
aborts the operation if this compilation test fails. In the attached make.log 
you will see the compilation command which is used and the compilation errors.

Put simply it looks like some header files are included by a few files even 
though they don't exist (e.g. asm/param.h is included in linux/sched.h).

Thanks in advance for your help (I hope to upload the new nvidia drivers 
soon).

Regards,

Alberto (aka tseliot)

Hi all,

I have noticed that if I try to build the Nvidia driver in Jaunty with the 
latest kernel, Nvidia's conftest.sh fails. After making this test more verbose 
I found out that there can be something wrong with the headers.

Re: [PATCH 00/10] [Intrepid] SRU: LP#257827 -- acpi brightness control

Andy Whitcroft wrote:
> On Fri, Nov 21, 2008 at 05:16:10PM +0000, Andy Whitcroft wrote:
>> On Thu, Nov 20, 2008 at 11:22:55AM -0700, Tim Gardner wrote:
>>> Andy Whitcroft wrote:
>>>> This series represents a backport of a change to the ACPI backlight
>>>> support.  This change was original done to sort out brightness control
>>>> problems for a number of different laptop models triggered by non-existant
>>>> devices being created (as reported on this bug).  These changes also
>>>> are reported to fix boot crashes on a number of Samsung laptops (as
>>>> reported on LP#146692).  All of these changes are direct back ports from
>>>> mainline, no changes were needed to apply them.  They are all upstream
>>>> as of v2.6.28-rc5.
>>>>
>>>> I have built test kernels with these applied and a number of community
>>>> members have tested these and confirm that they do indeed allow those
>>>> laptops to boot and brightness control is improved there.
>>>>
>>>> Proposing these for SRU to Intrepid.
>>>>
>>>> -apw
>>> Disregard previous transmission wrt to '[PATCH 01/10] ACPI: video:
>>> Ignore devices that aren't present in hardware'
>>>
>>> Provisional ACK pending test results.
>> I have rebased these onto the top of the 2.6.27-10.20 and pushed those
>> up for any intrested testers.  So far we have three positive tests
>> results from the boot crashing Samsung laptops, with some reports of
>> improved brightness handling from them.  That seems pretty positive.
>> Will see if anyone from the primary report gives us any feedback this
>> weekend.

Re: [PATCH 1/1] UBUNTU: SAUCE: (no-up) version: Implement version_signature proc file.

Andy Whitcroft wrote:
> It seems that the /proc/version_signature support in Jaunty was miss-merged.
> A significant rejig of the fs/proc support has occured and we need to
> redo this support basically.  The patch below looks like it would do the
> job, though I have only compile tested it at this time.  This applies once
> the miss-merge is reverted, commit 20391c6e76c6992bdb7027f8b6bdbe35845e909a.
> 
> Signed-off-by: Andy Whitcroft <apw <at> canonical.com>
> ---
>  fs/proc/Makefile            |    1 +
>  fs/proc/version_signature.c |   31 +++++++++++++++++++++++++++++++
>  init/Kconfig                |    9 +++++++++
>  init/version.c              |    6 +++++-
>  4 files changed, 46 insertions(+), 1 deletions(-)
>  create mode 100644 fs/proc/version_signature.c
> 
> diff --git a/fs/proc/Makefile b/fs/proc/Makefile
> index 63d9651..8ab67f1 100644
> --- a/fs/proc/Makefile
> +++ b/fs/proc/Makefile
>  <at>  <at>  -25,3 +25,4  <at>  <at>  proc-$(CONFIG_PROC_VMCORE)	+= vmcore.o
>  proc-$(CONFIG_PROC_DEVICETREE)	+= proc_devtree.o
>  proc-$(CONFIG_PRINTK)	+= kmsg.o
>  proc-$(CONFIG_PROC_PAGE_MONITOR)	+= page.o
> +proc-$(CONFIG_VERSION_SIGNATURE)	+= version_signature.o
> diff --git a/fs/proc/version_signature.c b/fs/proc/version_signature.c
> new file mode 100644
> index 0000000..534303e
> --- /dev/null
> +++ b/fs/proc/version_signature.c

[PATCH 0/3] SRU LP69925 Intrepid - CPU fan stops at kernel load

From 2fb41f56964fc3489a9b7c93fdf66bffed26e075 Mon Sep 17 00:00:00 2001
From: Steve Conklin <sconklin <at> canonical.com>
Date: Mon, 1 Dec 2008 09:28:24 -0600

On the notebook fujitsu amilo l1310g, based on ATI chipset with xpress 
200m video card, a few seconds after the splash screen, the fan stops 
and the cpu overheats.

These patches resolve the problem.

Zhao Yakui (3):
  UBUNTU: ACPI: Add DMI check to disable power state check in power
    transition
  UBUNTU: ACPI: Add "acpi.power_nocheck=1" to disable power state check
    in power transition
  UBUNTU: ACPI: Attach the ACPI device to the ACPI handle as early as
    possible

 Documentation/kernel-parameters.txt |    8 ++++++
 drivers/acpi/bus.c                  |   37 +++++++++++++++++++++++++++++-
 drivers/acpi/power.c                |   41 ++++++++++++++++++++++++++--------
 drivers/acpi/scan.c                 |   12 ++++++++-
 include/acpi/acpi_drivers.h         |    1 +
 5 files changed, 85 insertions(+), 14 deletions(-)

[PATCH 1/3] SRU LP69925 Intrepid - CPU fan stops at kernel load

From e59158ecd62da28dae3aee6df5d6f727b4266ec8 Mon Sep 17 00:00:00 2001
From: Zhao Yakui <yakui.zhao <at> intel.com>
Date: Mon, 11 Aug 2008 14:59:59 +0800
Subject: [PATCH 1/3] UBUNTU: ACPI: Add DMI check to disable power state check in power transition
 Bug: #69925

Cherry-picked from upstream
ACPI: Add DMI check to disable power state check in power transition

Add the DMI check to disable power check in the course of device power
transistion.

http://bugzilla.kernel.org/show_bug.cgi?id=11000

Signed-off-by: Zhao Yakui <yakui.zhao <at> intel.com>
Signed-off-by: Li Shaohua <shaohua.li <at> intel.com>
Signed-off-by: Andi Kleen <ak <at> linux.intel.com>
Signed-off-by: Len Brown <len.brown <at> intel.com>
Signed-off-by: Steve Conklin <sconklin <at> canonical.com>
---
 drivers/acpi/bus.c |   23 ++++++++++++++++++++++-
 1 files changed, 22 insertions(+), 1 deletions(-)

diff --git a/drivers/acpi/bus.c b/drivers/acpi/bus.c
index c54f38e..5fb89d9 100644
--- a/drivers/acpi/bus.c
+++ b/drivers/acpi/bus.c
 <at>  <at>  -48,6 +48,23  <at>  <at>  EXPORT_SYMBOL(acpi_root_dir);

 #define STRUCT_TO_INT(s)	(*((int*)&s))

[PATCH 2/3] SRU LP69925 Intrepid - CPU fan stops at kernel load

From 8ecd6c7171e1d90eedd9cd9d04d91a18ee4924cb Mon Sep 17 00:00:00 2001
From: Zhao Yakui <yakui.zhao <at> intel.com>
Date: Mon, 11 Aug 2008 14:57:50 +0800
Subject: [PATCH 2/3] UBUNTU: ACPI: Add "acpi.power_nocheck=1" to disable power state check in power transition

Bug: #69925

ACPI: Add "acpi.power_nocheck=1" to disable power state check in power transition

   Maybe the incorrect power state is returned on the bogus bios, which
is different with the real power state. For example: the bios returns D0
state and the real power state is D3. OS expects to set the device to D0
state. In  such case if OS uses the power state returned by the BIOS and
checks the device power state very strictly in power transition, the device
can't be transited to the correct power state.

   So the boot option of "acpi.power_nocheck=1" is added to avoid checking
the device power in the course of device power transition.

http://bugzilla.kernel.org/show_bug.cgi?id=8049
http://bugzilla.kernel.org/show_bug.cgi?id=11000

Signed-off-by: Zhao Yakui <yakui.zhao <at> intel.com>
Signed-off-by: Zhang Rui <rui.zhang <at> intel.com>
Signed-off-by: Li Shaohua <shaohua.li <at> intel.com>
Signed-off-by: Andi Kleen <ak <at> linux.intel.com>
Signed-off-by: Len Brown <len.brown <at> intel.com>
Signed-off-by: Steve Conklin <sconklin <at> canonical.com>
---
 Documentation/kernel-parameters.txt |    8 ++++++

[PATCH 3/3] SRU LP69925 Intrepid - CPU fan stops at kernel load

From 2fb41f56964fc3489a9b7c93fdf66bffed26e075 Mon Sep 17 00:00:00 2001
From: Zhao Yakui <yakui.zhao <at> intel.com>
Date: Mon, 11 Aug 2008 14:54:16 +0800
Subject: [PATCH 3/3] UBUNTU: ACPI: Attach the ACPI device to the ACPI handle as early as possible
 Bug: #69925

ACPI: Attach the ACPI device to the ACPI handle as early as possible

Attach the ACPI device to the ACPI handle as early as possible so that OS
can get the corresponding ACPI device by the acpi handle in the course
of getting the power/wakeup/performance flags.

http://bugzilla.kernel.org/show_bug.cgi?id=8049
http://bugzilla.kernel.org/show_bug.cgi?id=11000

Signed-off-by: Zhao Yakui <yakui.zhao <at> intel.com>
Signed-off-by: Zhang Rui <rui.zhang <at> intel.com>
Signed-off-by: Andi Kleen <ak <at> linux.intel.com>
Signed-off-by: Len Brown <len.brown <at> intel.com>
Signed-off-by: Steve Conklin <sconklin <at> canonical.com>
---
 drivers/acpi/scan.c |   12 ++++++++++--
 1 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/drivers/acpi/scan.c b/drivers/acpi/scan.c
index 0450761..afa66f4 100644
--- a/drivers/acpi/scan.c
+++ b/drivers/acpi/scan.c
 <at>  <at>  -1262,6 +1262,16  <at>  <at>  acpi_add_single_object(struct acpi_device **child,
 	acpi_device_set_id(device, parent, handle, type);

Re: Problem with missing files in the kernel headers (2.6.28) in Jaunty

Alberto Milone wrote:
> Hi all,
> 
> I have noticed that if I try to build the Nvidia driver in Jaunty
> with the latest kernel, Nvidia's conftest.sh fails. After making this
> test more verbose I found out that there can be something wrong with
> the headers.
> 
> Nvidia's conftest.sh tries to build a file (see the attached .c file)
> and aborts the operation if this compilation test fails. In the
> attached make.log you will see the compilation command which is used
> and the compilation errors.
> 
> Put simply it looks like some header files are included by a few
> files even though they don't exist (e.g. asm/param.h is included in
> linux/sched.h).
> 
> Thanks in advance for your help (I hope to upload the new nvidia
> drivers soon).
> 
> Regards,
> 
> Alberto (aka tseliot)
> 
> 

The results of an IRC conversation, notably a missing include path:

<tseliot> BenC: I'm using Nvidia's makefile as usual. Nvidia's
conftest.sh tries to build the test file with cc -D__KERNEL__

[PATCH] UBUNTU: SAUCE: SRU LP246952 Add options to support Force Feedback interfaces

From 1feddea24c69d21384771048b5526134425a626b Mon Sep 17 00:00:00 2001
From: Steve Conklin <sconklin <at> canonical.com>
Date: Fri, 28 Nov 2008 13:59:33 -0600
Subject: [PATCH] UBUNTU: SAUCE: Add options to support Force Feedback interfaces
 Bug: 246952

Changed only to generic kernel, not server. This has been tested by a 
community member and confirmed to work for his FF device.

Signed-off-by: Steve Conklin <sconklin <at> canonical.com>
---
 debian/config/amd64/config.generic |    7 +++++++
 debian/config/i386/config.generic  |    7 +++++++
 2 files changed, 14 insertions(+), 0 deletions(-)

diff --git a/debian/config/amd64/config.generic b/debian/config/amd64/config.generic
index 832185a..9cad764 100644
--- a/debian/config/amd64/config.generic
+++ b/debian/config/amd64/config.generic
 <at>  <at>  -9,3 +9,10  <at>  <at>  CONFIG_HZ=250
 CONFIG_HZ_250=y
 # CONFIG_PREEMPT_NONE is not set
 CONFIG_PREEMPT_VOLUNTARY=y
+CONFIG_HID_FF=y
+CONFIG_HID_PID=y
+CONFIG_LOGITECH_FF=y
+CONFIG_LOGIRUMBLEPAD2_FF=y
+CONFIG_PANTHERLORD_FF=y
+CONFIG_THRUSTMASTER_FF=y
+CONFIG_ZEROPLUS_FF=y