On Wed, Dec 15, 2010 at 11:01 AM, Varadarajan, Charulatha <charu@xxxxxx> wrote: > * Shubhrajyoti D <a0393217@xxxxxxxxxxxx> [2010-12-15 09:39:58 +0530]: > >> From: Abraham Arce <x0066660@xxxxxx> >> >> Enable Runtime PM functionality in OMAP4 driver based on the following assumptions >> >> A minimal pm runtime get/put approach is implemented in probe/remove calls >> respectively. >> >> - Keyboard controller in wakeup domain so it is always on and >> power impact may be minimal >> - In OMAP4 the device control is at module/device level and ick/fclk level control is >> difficult so cutting of clocks will prevent interrupts. >> >> Signed-off-by: Abraham Arce <x0066660@xxxxxx> >> Cc: Kevin Hilman <khilman@xxxxxxxxxxxxxxxxxxx> > > This patch is sent thrice (once with a different subject) but the > version numbers are the same. It is not clear what is the intention of this > patch without hwmod database update. Am I missing any more patch here? Yes missed the version. I have updated the change logs to be more descriptive and the subject line. Regarding the hwmod database update. That I deffered for reasons - The clock changes need to be there as the update would mean reset of clocks. - Thought of completing the drivers/input before the hwmod database and the board changes. - Also currently the driver relies on the uboot settings for clock this might remove that dependency. > >> --- >> Updating the changelogs as per Kevin's suggestion >> drivers/input/keyboard/omap4-keypad.c | 10 +++++++++- >> 1 files changed, 9 insertions(+), 1 deletions(-) >> >> diff --git a/drivers/input/keyboard/omap4-keypad.c b/drivers/input/keyboard/omap4-keypad.c >> index 45bd097..3d35774 100644 >> --- a/drivers/input/keyboard/omap4-keypad.c >> +++ b/drivers/input/keyboard/omap4-keypad.c >> @@ -29,6 +29,7 @@ >> #include <linux/io.h> >> #include <linux/input.h> >> #include <linux/slab.h> >> +#include <linux/pm_runtime.h> >> >> #include <plat/omap4-keypad.h> >> >> @@ -239,8 +240,11 @@ static int __devinit omap4_keypad_probe(struct platform_device *pdev) >> matrix_keypad_build_keymap(pdata->keymap_data, row_shift, >> input_dev->keycode, input_dev->keybit); >> >> - omap4_keypad_config(keypad_data); >> + pm_runtime_enable(&pdev->dev); >> + pm_runtime_get_sync(&pdev->dev); >> >> + omap4_keypad_config(keypad_data); >> + >> error = request_irq(keypad_data->irq, omap4_keypad_interrupt, >> IRQF_TRIGGER_RISING, >> "omap4-keypad", keypad_data); >> @@ -278,6 +282,10 @@ static int __devexit omap4_keypad_remove(struct platform_device *pdev) >> struct resource *res; >> >> free_irq(keypad_data->irq, keypad_data); >> + >> + pm_runtime_put_sync(&pdev->dev); >> + pm_runtime_disable(&pdev->dev); >> + >> input_unregister_device(keypad_data->input); >> >> iounmap(keypad_data->base); >> -- >> 1.7.0.4 >> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-omap" in >> the body of a message to majordomo@xxxxxxxxxxxxxxx >> More majordomo info at http://vger.kernel.org/majordomo-info.html > -- > To unsubscribe from this list: send the line "unsubscribe linux-input" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
