Re: [Zaurus-devel] latest angstrom image + linux-2.6.38.4 report

2011-05-21 Thread Pavel Machek
Hi! > Stanislav Brabec wrote: > > Andrea Adami wrote: > > > I can confirm that touchscreen 'works' again on corgi(C860) and poodle. > > > > Well, I cannot. My OE image is broken again. That is why I copied my old > > good 2.6.26RP image to the new OE image (plus odev-compat stuff), and I > > was

[Zaurus-devel] Disable CF during suspend

2011-05-21 Thread Pavel Machek
Disable SPITZ_SCP_CF_POWER during suspend. Otherwise it eats battery, fast. Thanks to Stanislav Brabec. Signed-off-by: Pavel Machek diff --git a/arch/arm/mach-pxa/include/mach/spitz.h b/arch/arm/mach-pxa/include/mach/spitz.h index 685749a..06b7cbc 100644 --- a/arch/arm/mach-pxa/include

Re: [Zaurus-devel] latest angstrom image + linux-2.6.38.4 report

2011-05-21 Thread Pavel Machek
On Mon 2011-05-16 15:55:21, Stanislav Brabec wrote: > Pavel Machek wrote: > > Hi! > > > > > > > In general, defconfig in OE is very poor (missing support for some > > > > > parts > > > > > of the hardware and many important USB devices). > > > > > > > > > > Here are my configs: > > > > > http://w

Re: [Zaurus-devel] latest angstrom image + linux-2.6.38.4 report

2011-05-21 Thread Stanislav Brabec
Pavel Machek wrote: > Umm, can you raise the issue on linux-input? New kernel should not > break old tslib... They probably know, why they increased EV_VERSION. If we are sure that that change does not break tslib, we should patch tslib to accept both versions. -- _

[Zaurus-devel] spitz pm: move definitions to header file

2011-05-21 Thread Pavel Machek
This will allow me to use the definitions from spitz_battery.c module. Signed-off-by: Pavel Machek diff --git a/arch/arm/mach-pxa/include/mach/sharpsl_pm.h b/arch/arm/mach-pxa/include/mach/sharpsl_pm.h index 905be67..7407717 100644 --- a/arch/arm/mach-pxa/include/mach/sharpsl_pm.h +++ b/arch/ar

[Zaurus-devel] spitz pm: adjust messages

2011-05-21 Thread Pavel Machek
Charger error is something that should be reported to the syslog; do it. If temperature reading fails, we do not want to charge the battery, anyway. Signed-off-by: Pavel Machek diff --git a/arch/arm/mach-pxa/sharpsl_pm.c b/arch/arm/mach-pxa/sharpsl_pm.c index 785880f..513588c 100644 --- a/arch/a

[Zaurus-devel] spitz pm: add power monitoring driver

2011-05-21 Thread Pavel Machek
This adds battery/charge monitoring driver for spitz. This is still not 100% done; spitz.c parts need small cleanups to fit with the rest of code, and "limit" interface needs to be done. Question is... who merges this when its done? Would it be ok to go through PXA tree? Signed-off-by: Pavel M

Re: [Zaurus-devel] Disable CF during suspend

2011-05-21 Thread Stanislav Brabec
Pavel Machek wrote: > Disable SPITZ_SCP_CF_POWER during suspend. Otherwise it eats battery, > fast. Thanks to Stanislav Brabec. If you can verify that your card turns off on suspend, then you have my: Acked-by: Stanislav Brabec But I would line to see more aggressive power control, something lik

Re: [Zaurus-devel] Disable CF during suspend

2011-05-21 Thread Pavel Machek
Hi! > > Disable SPITZ_SCP_CF_POWER during suspend. Otherwise it eats battery, > > fast. Thanks to Stanislav Brabec. > > If you can verify that your card turns off on suspend, then you have my: > Acked-by: Stanislav Brabec Yes, I checked the LEDs to make sure it is off. It is. > But I would lin

Re: [Zaurus-devel] Zaurus GPIO buttons with matrix keypad drivers

2011-05-21 Thread Pavel Machek
Hi! > following comments from a Stanislav patch > > "Spitz switched to generic matrix-gpio keyboard driver in 2.6.32-rc0, > but that means that support for power button and lid switches was > lost. This restores it, using button-gpio code. > " > in the 2.6.38 source I only find in spitz.c: > > #

Re: [Zaurus-devel] spitz pm: add power monitoring driver

2011-05-21 Thread Pavel Machek
This adds battery/charge monitoring driver for spitz. This is version I'd like to see merged. Is it ok for Marek to merge this? Signed-off-by: Pavel Machek diff --git a/arch/arm/mach-pxa/sharpsl_pm.c b/arch/arm/mach-pxa/sharpsl_pm.c index 785880f..513588c 100644 --- a/arch/arm/mach-pxa/s

Re: [Zaurus-devel] spitz pm: adjust messages

2011-05-21 Thread Marek Vasut
On Saturday, May 21, 2011 02:24:50 PM Pavel Machek wrote: > Charger error is something that should be reported to the syslog; do > it. If temperature reading fails, we do not want to charge the > battery, anyway. > > Signed-off-by: Pavel Machek > > diff --git a/arch/arm/mach-pxa/sharpsl_pm.c > b

Re: [Zaurus-devel] spitz pm: add power monitoring driver

2011-05-21 Thread Marek Vasut
On Saturday, May 21, 2011 07:01:29 PM Pavel Machek wrote: > This adds battery/charge monitoring driver for spitz. > > This is version I'd like to see merged. > > Is it ok for Marek to merge this? No, I believe Anton should merge this. > > Signed-off-by: Pavel Machek > > diff --git a/arch/arm/

Re: [Zaurus-devel] [PATCH] MAX1111: Fix race condition causing NULL pointer exception

2011-05-21 Thread Pavel Machek
Hi! > > I think you want to hold the mutex at the point you setup the data to be > > transferred, do the transfer, and then release the mutex once you've read > > the results of the transfer. > > oh no, not again... > this was the earliest version, not the cleaned-up one (notice the lock in > se

Re: [Zaurus-devel] [PATCH] MAX1111: Fix race condition causing NULL pointer exception

2011-05-21 Thread Pavel Herrmann
On Saturday, May 21, 2011 10:28:07 PM Pavel Machek wrote: > Unfortunately that one does not apply, due to > > > @@ -213,6 +224,7 @@ static int __devexit max_remove(struct spi_device > > *spi) > > Whitespace damage. (but if you just delete this extra newline, its > fine). sorry for that one,

Re: [Zaurus-devel] latest angstrom image + linux-2.6.38.4 report

2011-05-21 Thread Andrea Adami
On Thu, May 19, 2011 at 1:27 AM, Andrea Adami wrote: > On Wed, May 18, 2011 at 12:31 AM, Andrea Adami wrote: >>> Well, again - snd-soc-spitz-not-autoloaded is a regression outside >>> kernel. The old kernel with the new OE image has the same problem. (It >>> means, that the kernel never provided