[PATCH] input: rotary: Init wake according to platform pm_wakeup flag.

Subsystems: input (keyboard, mouse, joystick, touchscreen) drivers, the rest

STALE5190d

6 messages, 2 authors, 2012-05-17 · open the first message on its own page

[PATCH] input: rotary: Init wake according to platform pm_wakeup flag.

From: <hidden>
Date: 2012-05-16 08:47:27

From: Sonic Zhang <redacted>

Rotary can't be used as a wakeup source in all Blackfin processors.

Signed-off-by: Sonic Zhang <redacted>
---
 drivers/input/misc/bfin_rotary.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/input/misc/bfin_rotary.c b/drivers/input/misc/bfin_rotary.c
index 1c4146f..bb1971c 100644
--- a/drivers/input/misc/bfin_rotary.c
+++ b/drivers/input/misc/bfin_rotary.c

@@ -182,7 +182,7 @@ static int __devinit bfin_rotary_probe(struct platform_device *pdev)
 	bfin_write_CNT_CONFIG(bfin_read_CNT_CONFIG() | CNTE);
 
 	platform_set_drvdata(pdev, rotary);
-	device_init_wakeup(&pdev->dev, 1);
+	device_init_wakeup(&pdev->dev, pdata->pm_wakeup);
 
 	return 0;

-- 
1.7.0.4

Re: [PATCH] input: rotary: Init wake according to platform pm_wakeup flag.

From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Date: 2012-05-16 18:30:50

Hi Sonic,

On Wed, May 16, 2012 at 04:36:23PM +0800, sonic.zhang@analog.com wrote:

quoted hunk

From: Sonic Zhang <redacted>

Rotary can't be used as a wakeup source in all Blackfin processors.

Signed-off-by: Sonic Zhang <redacted>
---
 drivers/input/misc/bfin_rotary.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/input/misc/bfin_rotary.c b/drivers/input/misc/bfin_rotary.c
index 1c4146f..bb1971c 100644
--- a/drivers/input/misc/bfin_rotary.c
+++ b/drivers/input/misc/bfin_rotary.c

@@ -182,7 +182,7 @@ static int __devinit bfin_rotary_probe(struct platform_device *pdev)
 	bfin_write_CNT_CONFIG(bfin_read_CNT_CONFIG() | CNTE);
 
 	platform_set_drvdata(pdev, rotary);
-	device_init_wakeup(&pdev->dev, 1);
+	device_init_wakeup(&pdev->dev, pdata->pm_wakeup);

I do not see pm_wakeup in bfin_rotary_platform_data.

Thanks.

-- 
Dmitry

RE: [PATCH] input: rotary: Init wake according to platform pm_wakeup flag.

From: Zhang, Sonic <hidden>
Date: 2012-05-17 02:57:38

Hi Dmitry,

-----Original Message-----
From: Dmitry Torokhov [mailto:dmitry.torokhov@gmail.com]
Sent: Thursday, May 17, 2012 2:31 AM
To: Zhang, Sonic
Cc: linux-input@vger.kernel.org; LKML; uclinux-dist-devel@blackfin.uclinux.org
Subject: Re: [PATCH] input: rotary: Init wake according to platform pm_wakeup
flag.

Hi Sonic,

On Wed, May 16, 2012 at 04:36:23PM +0800, sonic.zhang@analog.com wrote:

quoted

From: Sonic Zhang <redacted>

Rotary can't be used as a wakeup source in all Blackfin processors.

Signed-off-by: Sonic Zhang <redacted>
---
 drivers/input/misc/bfin_rotary.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/input/misc/bfin_rotary.c b/drivers/input/misc/bfin_rotary.c
index 1c4146f..bb1971c 100644
--- a/drivers/input/misc/bfin_rotary.c
+++ b/drivers/input/misc/bfin_rotary.c

@@ -182,7 +182,7 @@ static int __devinit bfin_rotary_probe(struct

platform_device *pdev)

quoted

     bfin_write_CNT_CONFIG(bfin_read_CNT_CONFIG() | CNTE);

     platform_set_drvdata(pdev, rotary);
-    device_init_wakeup(&pdev->dev, 1);
+    device_init_wakeup(&pdev->dev, pdata->pm_wakeup);

I do not see pm_wakeup in bfin_rotary_platform_data.

The patch against the bfin_rotary.h under blackfin architecture folder will be sent out soon by Bob Liu.

Regards,

Sonic

Thanks.

--
Dmitry

Re: [PATCH] input: rotary: Init wake according to platform pm_wakeup flag.

From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Date: 2012-05-17 06:08:11

On Wed, May 16, 2012 at 10:58:00PM -0400, Zhang, Sonic wrote:

Hi Dmitry,

quoted

-----Original Message-----
From: Dmitry Torokhov [mailto:dmitry.torokhov@gmail.com]
Sent: Thursday, May 17, 2012 2:31 AM
To: Zhang, Sonic
Cc: linux-input@vger.kernel.org; LKML; uclinux-dist-devel@blackfin.uclinux.org
Subject: Re: [PATCH] input: rotary: Init wake according to platform pm_wakeup
flag.

Hi Sonic,

On Wed, May 16, 2012 at 04:36:23PM +0800, sonic.zhang@analog.com wrote:

quoted

From: Sonic Zhang <redacted>

Rotary can't be used as a wakeup source in all Blackfin processors.

Signed-off-by: Sonic Zhang <redacted>
---
 drivers/input/misc/bfin_rotary.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/input/misc/bfin_rotary.c b/drivers/input/misc/bfin_rotary.c
index 1c4146f..bb1971c 100644
--- a/drivers/input/misc/bfin_rotary.c
+++ b/drivers/input/misc/bfin_rotary.c

@@ -182,7 +182,7 @@ static int __devinit bfin_rotary_probe(struct

platform_device *pdev)

quoted

     bfin_write_CNT_CONFIG(bfin_read_CNT_CONFIG() | CNTE);

     platform_set_drvdata(pdev, rotary);
-    device_init_wakeup(&pdev->dev, 1);
+    device_init_wakeup(&pdev->dev, pdata->pm_wakeup);

I do not see pm_wakeup in bfin_rotary_platform_data.

The patch against the bfin_rotary.h under blackfin architecture folder will be sent out soon by Bob Liu.

This split makes no sense - if I apply your patch to my tree it menas
that my tree will be broken on blackfin architecture until blackfin arch
gets pulled into mainline and I do next merge (around 3.5-rc3??). Please
combine both patches and get both pieces applied at the same time.

Thanks.

-- 
Dmitry

RE: [PATCH] input: rotary: Init wake according to platform pm_wakeup flag.

From: Zhang, Sonic <hidden>
Date: 2012-05-17 06:32:17

Hi Dmitry,

-----Original Message-----
From: Dmitry Torokhov [mailto:dmitry.torokhov@gmail.com]
Sent: Thursday, May 17, 2012 2:08 PM
To: Zhang, Sonic
Cc: linux-input@vger.kernel.org; LKML; uclinux-dist-devel@blackfin.uclinux.org;
Bob Liu
Subject: Re: [PATCH] input: rotary: Init wake according to platform pm_wakeup
flag.

On Wed, May 16, 2012 at 10:58:00PM -0400, Zhang, Sonic wrote:

quoted

Hi Dmitry,

quoted

-----Original Message-----
From: Dmitry Torokhov [mailto:dmitry.torokhov@gmail.com]
Sent: Thursday, May 17, 2012 2:31 AM
To: Zhang, Sonic
Cc: linux-input@vger.kernel.org; LKML; uclinux-dist-

devel@blackfin.uclinux.org

quoted

Subject: Re: [PATCH] input: rotary: Init wake according to platform pm_wakeup
flag.

Hi Sonic,

On Wed, May 16, 2012 at 04:36:23PM +0800, sonic.zhang@analog.com wrote:

quoted

From: Sonic Zhang <redacted>

Rotary can't be used as a wakeup source in all Blackfin processors.

Signed-off-by: Sonic Zhang <redacted>
---
 drivers/input/misc/bfin_rotary.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/input/misc/bfin_rotary.c b/drivers/input/misc/bfin_rotary.c
index 1c4146f..bb1971c 100644
--- a/drivers/input/misc/bfin_rotary.c
+++ b/drivers/input/misc/bfin_rotary.c

@@ -182,7 +182,7 @@ static int __devinit bfin_rotary_probe(struct

platform_device *pdev)

quoted

     bfin_write_CNT_CONFIG(bfin_read_CNT_CONFIG() | CNTE);

     platform_set_drvdata(pdev, rotary);
-    device_init_wakeup(&pdev->dev, 1);
+    device_init_wakeup(&pdev->dev, pdata->pm_wakeup);

I do not see pm_wakeup in bfin_rotary_platform_data.

The patch against the bfin_rotary.h under blackfin architecture folder will be sent

out soon by Bob Liu.

This split makes no sense - if I apply your patch to my tree it menas
that my tree will be broken on blackfin architecture until blackfin arch
gets pulled into mainline and I do next merge (around 3.5-rc3??). Please
combine both patches and get both pieces applied at the same time.

The rotary Blackfin architecture patch is on top of the new bf60x machine patch sets. You may fail to apply it before the bf60x patches are merged.
Is it better to send you this patch later after you do next merge?


Regards,

Sonic

Thanks.

--
Dmitry

Re: [PATCH] input: rotary: Init wake according to platform pm_wakeup flag.

From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Date: 2012-05-17 06:51:49

On Thu, May 17, 2012 at 02:32:37AM -0400, Zhang, Sonic wrote:

Hi Dmitry,

quoted

-----Original Message-----
From: Dmitry Torokhov [mailto:dmitry.torokhov@gmail.com]
Sent: Thursday, May 17, 2012 2:08 PM
To: Zhang, Sonic
Cc: linux-input@vger.kernel.org; LKML; uclinux-dist-devel@blackfin.uclinux.org;
Bob Liu
Subject: Re: [PATCH] input: rotary: Init wake according to platform pm_wakeup
flag.

On Wed, May 16, 2012 at 10:58:00PM -0400, Zhang, Sonic wrote:

quoted

Hi Dmitry,

quoted

-----Original Message-----
From: Dmitry Torokhov [mailto:dmitry.torokhov@gmail.com]
Sent: Thursday, May 17, 2012 2:31 AM
To: Zhang, Sonic
Cc: linux-input@vger.kernel.org; LKML; uclinux-dist-

devel@blackfin.uclinux.org

quoted

Subject: Re: [PATCH] input: rotary: Init wake according to platform pm_wakeup
flag.

Hi Sonic,

On Wed, May 16, 2012 at 04:36:23PM +0800, sonic.zhang@analog.com wrote:

quoted

From: Sonic Zhang <redacted>

Rotary can't be used as a wakeup source in all Blackfin processors.

Signed-off-by: Sonic Zhang <redacted>
---
 drivers/input/misc/bfin_rotary.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/input/misc/bfin_rotary.c b/drivers/input/misc/bfin_rotary.c
index 1c4146f..bb1971c 100644
--- a/drivers/input/misc/bfin_rotary.c
+++ b/drivers/input/misc/bfin_rotary.c

@@ -182,7 +182,7 @@ static int __devinit bfin_rotary_probe(struct

platform_device *pdev)

quoted

     bfin_write_CNT_CONFIG(bfin_read_CNT_CONFIG() | CNTE);

     platform_set_drvdata(pdev, rotary);
-    device_init_wakeup(&pdev->dev, 1);
+    device_init_wakeup(&pdev->dev, pdata->pm_wakeup);

I do not see pm_wakeup in bfin_rotary_platform_data.

The patch against the bfin_rotary.h under blackfin architecture folder will be sent

out soon by Bob Liu.

This split makes no sense - if I apply your patch to my tree it menas
that my tree will be broken on blackfin architecture until blackfin arch
gets pulled into mainline and I do next merge (around 3.5-rc3??). Please
combine both patches and get both pieces applied at the same time.

The rotary Blackfin architecture patch is on top of the new bf60x
machine patch sets. You may fail to apply it before the bf60x patches
are merged.  Is it better to send you this patch later after you do
next merge?

You may also add

	Acked-by: Dmitry Torokhov [off-list ref]

to this patch and merge it through blackfin tree.

-- 
Dmitry

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help