diff options
author | mirko <mirko@3c298f89-4303-0410-b956-a3cf2f4a3e73> | 2008-12-12 11:58:53 +0000 |
---|---|---|
committer | mirko <mirko@3c298f89-4303-0410-b956-a3cf2f4a3e73> | 2008-12-12 11:58:53 +0000 |
commit | fc54b9bf158eea9cae647ce2c58f7d9989af173a (patch) | |
tree | 54644c1229434d7ee13c5872bda4129e34337fc0 /target/linux/s3c24xx/patches-2.6.24/1195-fix-pcf50633-kill-white-splash-of-death-on-suspend.p.patch | |
parent | d9f49b62b28ebd245587854efa484974d90debbf (diff) |
changed Makefile and profiles, added patches for kernel 2.6.24
(stable-branch of Openmoko)
git-svn-id: svn://svn.openwrt.org/openwrt/trunk@13613 3c298f89-4303-0410-b956-a3cf2f4a3e73
Diffstat (limited to 'target/linux/s3c24xx/patches-2.6.24/1195-fix-pcf50633-kill-white-splash-of-death-on-suspend.p.patch')
-rw-r--r-- | target/linux/s3c24xx/patches-2.6.24/1195-fix-pcf50633-kill-white-splash-of-death-on-suspend.p.patch | 263 |
1 files changed, 263 insertions, 0 deletions
diff --git a/target/linux/s3c24xx/patches-2.6.24/1195-fix-pcf50633-kill-white-splash-of-death-on-suspend.p.patch b/target/linux/s3c24xx/patches-2.6.24/1195-fix-pcf50633-kill-white-splash-of-death-on-suspend.p.patch new file mode 100644 index 000000000..9b66ef1d1 --- /dev/null +++ b/target/linux/s3c24xx/patches-2.6.24/1195-fix-pcf50633-kill-white-splash-of-death-on-suspend.p.patch @@ -0,0 +1,263 @@ +From 358429b39b6801f4856e7c80d2c58ac7be4bd0bc Mon Sep 17 00:00:00 2001 +From: Andy Green <andy@openmoko.com> +Date: Wed, 2 Jul 2008 22:41:14 +0100 +Subject: [PATCH] fix-pcf50633-kill-white-splash-of-death-on-suspend.patch + +mach-gta02 meddles with the regulator platform struct after +it is defined, leading to LCM power getting lost in suspend +despite I set it to be left up. Fixing this finally removes +the incredibly stubborn white LCM on suspend "flash". + +This is also going to be implicated in Sean McNeil's +experience of monochromatic LCM after resume, which was +previously attacked by resetting and re-initing the LCM +from scratch. + +In addition, I realized that we take down core_1v3 in +pcf50633 suspend action, this is happening near the +start of suspend, so we are in a meta-race to finish +suspend in a controlled way before the caps on core_1v3 +run out (I only saw 23.3uF total). If it's true, this +is where the weirdo sensitivity to timing during +suspend is coming from. + +Therefore in this patch we also remove sleeps and +dev_info() etc (which have to flush on serial console) +from the pc50633 isr workqueue if we are in pcf50633 +driver suspend state 1, ie, suspending... because we +don't have time for it. + +Signed-off-by: Andy Green <andy@openmoko.com> +--- + arch/arm/mach-s3c2440/mach-gta02.c | 43 ++++++++++++++++++++++++++---------- + drivers/i2c/chips/pcf50633.c | 29 ++++++++++++----------- + drivers/mfd/glamo/glamo-core.c | 2 + + drivers/video/display/jbt6k74.c | 1 + + 4 files changed, 49 insertions(+), 26 deletions(-) + +diff --git a/arch/arm/mach-s3c2440/mach-gta02.c b/arch/arm/mach-s3c2440/mach-gta02.c +index 654dc8f..0bacafa 100644 +--- a/arch/arm/mach-s3c2440/mach-gta02.c ++++ b/arch/arm/mach-s3c2440/mach-gta02.c +@@ -485,6 +485,9 @@ static struct pcf50633_platform_data gta02_pcf_pdata = { + [1] = PCF50633_INT2_ONKEYF, + [2] = PCF50633_INT3_ONKEY1S + }, ++ /* warning: these get rewritten during machine init below ++ * depending on pcb variant ++ */ + .rails = { + [PCF50633_REGULATOR_AUTO] = { + .name = "io_3v3", +@@ -496,6 +499,12 @@ static struct pcf50633_platform_data gta02_pcf_pdata = { + }, + [PCF50633_REGULATOR_DOWN1] = { + .name = "core_1v3", ++ /* Wow, when we are going into suspend, after pcf50633 ++ * runs its suspend (which happens real early since it ++ * is an i2c device) we are running out of the 22uF cap ++ * on core_1v3 rail !!!! ++ */ ++ .flags = PMU_VRAIL_F_SUSPEND_ON, + .voltage = { + .init = 1300, + .max = 1600, +@@ -503,6 +512,7 @@ static struct pcf50633_platform_data gta02_pcf_pdata = { + }, + [PCF50633_REGULATOR_DOWN2] = { + .name = "core_1v8", ++ .flags = PMU_VRAIL_F_SUSPEND_ON, + .voltage = { + .init = 1800, + .max = 1800, +@@ -516,8 +526,7 @@ static struct pcf50633_platform_data gta02_pcf_pdata = { + }, + }, + [PCF50633_REGULATOR_LDO1] = { +- .name = "stby_1v3", +- .flags = PMU_VRAIL_F_SUSPEND_ON, ++ .name = "gsensor_3v3", + .voltage = { + .init = 1300, + .max = 1330, +@@ -531,7 +540,7 @@ static struct pcf50633_platform_data gta02_pcf_pdata = { + }, + }, + [PCF50633_REGULATOR_LDO3] = { +- .name = "lcm_3v", ++ .name = "unused3", + .voltage = { + .init = 3000, + .max = 3000, +@@ -545,20 +554,28 @@ static struct pcf50633_platform_data gta02_pcf_pdata = { + }, + }, + [PCF50633_REGULATOR_LDO5] = { +- .name = "gl_1v5", ++ .name = "rf3v", + .voltage = { + .init = 1500, + .max = 1500, + }, + }, + [PCF50633_REGULATOR_LDO6] = { +- .name = "user1", ++ .name = "lcm_3v", + .flags = PMU_VRAIL_F_SUSPEND_ON, + .voltage = { + .init = 0, + .max = 3300, + }, + }, ++ [PCF50633_REGULATOR_MEMLDO] = { ++ .name = "memldo", ++ .flags = PMU_VRAIL_F_SUSPEND_ON, ++ .voltage = { ++ .init = 1800, ++ .max = 1800, ++ }, ++ }, + }, + .defer_resume_backlight = 1, + }; +@@ -611,13 +628,15 @@ static void mangle_pmu_pdata_by_system_rev(void) + .max = 3000, + } + }); +- gta02_pcf_pdata.rails[PCF50633_REGULATOR_LDO6] = ((struct pmu_voltage_rail) { +- .name = "lcm_3v", +- .voltage = { +- .init = 3000, +- .max = 3000, +- } +- }); ++ gta02_pcf_pdata.rails[PCF50633_REGULATOR_LDO6] = ++ ((struct pmu_voltage_rail) { ++ .name = "lcm_3v", ++ .flags = PMU_VRAIL_F_SUSPEND_ON, ++ .voltage = { ++ .init = 3000, ++ .max = 3000, ++ } ++ }); + break; + default: + break; +diff --git a/drivers/i2c/chips/pcf50633.c b/drivers/i2c/chips/pcf50633.c +index 2f144bc..2e5981d 100644 +--- a/drivers/i2c/chips/pcf50633.c ++++ b/drivers/i2c/chips/pcf50633.c +@@ -750,9 +750,6 @@ static void pcf50633_work(struct work_struct *work) + mutex_lock(&pcf->working_lock); + pcf->working = 1; + +- dev_info(&pcf->client.dev, "pcf50633_work called with suspended = %d\n", +- pcf->have_been_suspended); +- + /* + * If we are inside suspend -> resume completion time we don't attempt + * service until we have fully resumed. Although we could talk to the +@@ -763,11 +760,8 @@ static void pcf50633_work(struct work_struct *work) + * completed. + */ + +- if (pcf->have_been_suspended && (pcf->have_been_suspended < 3)) { +- dev_info(&pcf->client.dev, "rescheduling, suspended = %d\n", +- pcf->have_been_suspended); ++ if (pcf->have_been_suspended && (pcf->have_been_suspended < 3)) + goto reschedule; +- } + + /* + * datasheet says we have to read the five IRQ +@@ -1158,9 +1152,13 @@ static void pcf50633_work(struct work_struct *work) + + reschedule: + /* don't spew, delaying whatever else is happening */ +- msleep(100); +- +- dev_info(&pcf->client.dev, "rescheduling interrupt service\n"); ++ /* EXCEPTION: if we are in the middle of suspending, we don't have ++ * time to hang around since we may be turned off core 1V3 already ++ */ ++ if (pcf->have_been_suspended != 1) { ++ msleep(50); ++ dev_info(&pcf->client.dev, "rescheduling interrupt service\n"); ++ } + if (!schedule_work(&pcf->work)) + dev_err(&pcf->client.dev, "int service reschedule failed\n"); + +@@ -2340,7 +2338,7 @@ static int pcf50633_suspend(struct device *dev, pm_message_t state) + PCF50633_REG_AUTOOUT, + sizeof(pcf->standby_regs.misc), + &pcf->standby_regs.misc[0]); +- if (ret != 18) ++ if (ret != sizeof(pcf->standby_regs.misc)) + dev_err(dev, "Failed to save misc levels and enables :-(\n"); + + /* regulator voltages and enable states */ +@@ -2348,7 +2346,7 @@ static int pcf50633_suspend(struct device *dev, pm_message_t state) + PCF50633_REG_LDO1OUT, + sizeof(pcf->standby_regs.ldo), + &pcf->standby_regs.ldo[0]); +- if (ret != 14) ++ if (ret != sizeof(pcf->standby_regs.ldo)) + dev_err(dev, "Failed to save LDO levels and enables :-(\n"); + + /* switch off power supplies that are not needed during suspend */ +@@ -2356,8 +2354,6 @@ static int pcf50633_suspend(struct device *dev, pm_message_t state) + if ((pcf->pdata->rails[i].flags & PMU_VRAIL_F_SUSPEND_ON)) + continue; + +- dev_dbg(dev, "disabling regulator %u\n", i); +- + /* we can save ourselves the read part of a read-modify-write + * here because we captured all these already + */ +@@ -2366,6 +2362,11 @@ static int pcf50633_suspend(struct device *dev, pm_message_t state) + else + tmp = pcf->standby_regs.ldo[(i - 4) * 2 + 1]; + ++ dev_info(dev, "disabling reg %s by setting ENA %d to 0x%02X\n", ++ pcf->pdata->rails[i].name, ++ regulator_registers[i] + 1, tmp & 0xfe); ++ ++ /* associated enable is always +1 from OUT reg */ + __reg_write(pcf, regulator_registers[i] + 1, tmp & 0xfe); + } + +diff --git a/drivers/mfd/glamo/glamo-core.c b/drivers/mfd/glamo/glamo-core.c +index 2a02f9c..2412b7d 100644 +--- a/drivers/mfd/glamo/glamo-core.c ++++ b/drivers/mfd/glamo/glamo-core.c +@@ -848,6 +848,7 @@ static void glamo_power(struct glamo_core *glamo, + ARRAY_SIZE(glamo_resume_script), 0); + + break; ++ + case GLAMO_POWER_STANDBY: + /* enable memory self-refresh */ + __reg_set_bit_mask(glamo, GLAMO_REG_MEM_DRAM1, +@@ -859,6 +860,7 @@ static void glamo_power(struct glamo_core *glamo, + __reg_set_bit_mask(glamo, GLAMO_REG_PLL_GEN3, 0x2000, 0xffff); + __reg_set_bit_mask(glamo, GLAMO_REG_DFT_GEN5, 0x0001, 0xffff); + break; ++ + case GLAMO_POWER_SUSPEND: + __reg_set_bit_mask(glamo, GLAMO_REG_MEM_DRAM2, + GLAMO_MEM_DRAM2_DEEP_PWRDOWN, 0xffff); +diff --git a/drivers/video/display/jbt6k74.c b/drivers/video/display/jbt6k74.c +index 6fa1fe7..b406298 100644 +--- a/drivers/video/display/jbt6k74.c ++++ b/drivers/video/display/jbt6k74.c +@@ -634,6 +634,7 @@ static int jbt_suspend(struct spi_device *spi, pm_message_t state) + + /* Save mode for resume */ + jbt->last_state = jbt->state; ++ + jbt6k74_enter_state(jbt, JBT_STATE_DEEP_STANDBY); + + jbt->have_resumed = 0; +-- +1.5.6.5 + |