* Re: [PATCH] serial: omap: Remove unnecessary checks from suspend/resume
2012-09-18 11:35 [PATCH] serial: omap: Remove unnecessary checks from suspend/resume Sourav Poddar
@ 2012-09-18 11:34 ` Felipe Balbi
2012-09-18 11:44 ` Poddar, Sourav
0 siblings, 1 reply; 5+ messages in thread
From: Felipe Balbi @ 2012-09-18 11:34 UTC (permalink / raw)
To: Sourav Poddar
Cc: gregkh, alan, tony, khilman, linux-omap, linux-serial,
linux-arm-kernel, balbi, santosh.shilimkar
[-- Attachment #1: Type: text/plain, Size: 2321 bytes --]
On Tue, Sep 18, 2012 at 05:05:54PM +0530, Sourav Poddar wrote:
> Drop the check for "up" being valid on suspend/resume callbacks.
> It should be valid always. Get rid of the "pdata" check also as
> serial_omap_get_context_loss_count() checks for it.
>
> Tested on omap4 panda and 3630 based Beagle board.
you need a blank line here. Other than that:
Reviewed-by: Felipe Balbi <balbi@ti.com>
ps: what kind of tests did you run, btw ?
> Signed-off-by: Sourav Poddar <sourav.poddar@ti.com>
> ---
> drivers/tty/serial/omap-serial.c | 23 +++++++++--------------
> 1 files changed, 9 insertions(+), 14 deletions(-)
>
> diff --git a/drivers/tty/serial/omap-serial.c b/drivers/tty/serial/omap-serial.c
> index f175385..3c05c5e 100644
> --- a/drivers/tty/serial/omap-serial.c
> +++ b/drivers/tty/serial/omap-serial.c
> @@ -1222,10 +1222,8 @@ static int serial_omap_suspend(struct device *dev)
> {
> struct uart_omap_port *up = dev_get_drvdata(dev);
>
> - if (up) {
> - uart_suspend_port(&serial_omap_reg, &up->port);
> - flush_work_sync(&up->qos_work);
> - }
> + uart_suspend_port(&serial_omap_reg, &up->port);
> + flush_work_sync(&up->qos_work);
>
> return 0;
> }
> @@ -1234,8 +1232,8 @@ static int serial_omap_resume(struct device *dev)
> {
> struct uart_omap_port *up = dev_get_drvdata(dev);
>
> - if (up)
> - uart_resume_port(&serial_omap_reg, &up->port);
> + uart_resume_port(&serial_omap_reg, &up->port);
> +
> return 0;
> }
> #endif
> @@ -1553,17 +1551,14 @@ static int serial_omap_runtime_suspend(struct device *dev)
> static int serial_omap_runtime_resume(struct device *dev)
> {
> struct uart_omap_port *up = dev_get_drvdata(dev);
> - struct omap_uart_port_info *pdata = dev->platform_data;
>
> - if (up && pdata) {
> - u32 loss_cnt = serial_omap_get_context_loss_count(up);
> + u32 loss_cnt = serial_omap_get_context_loss_count(up);
>
> - if (up->context_loss_cnt != loss_cnt)
> - serial_omap_restore_context(up);
> + if (up->context_loss_cnt != loss_cnt)
> + serial_omap_restore_context(up);
>
> - up->latency = up->calc_latency;
> - schedule_work(&up->qos_work);
> - }
> + up->latency = up->calc_latency;
> + schedule_work(&up->qos_work);
>
> return 0;
> }
> --
> 1.7.1
>
--
balbi
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] serial: omap: Remove unnecessary checks from suspend/resume
@ 2012-09-18 11:35 Sourav Poddar
2012-09-18 11:34 ` Felipe Balbi
0 siblings, 1 reply; 5+ messages in thread
From: Sourav Poddar @ 2012-09-18 11:35 UTC (permalink / raw)
To: gregkh
Cc: alan, tony, khilman, linux-omap, linux-serial, linux-arm-kernel,
balbi, santosh.shilimkar, Sourav Poddar
Drop the check for "up" being valid on suspend/resume callbacks.
It should be valid always. Get rid of the "pdata" check also as
serial_omap_get_context_loss_count() checks for it.
Tested on omap4 panda and 3630 based Beagle board.
Signed-off-by: Sourav Poddar <sourav.poddar@ti.com>
---
drivers/tty/serial/omap-serial.c | 23 +++++++++--------------
1 files changed, 9 insertions(+), 14 deletions(-)
diff --git a/drivers/tty/serial/omap-serial.c b/drivers/tty/serial/omap-serial.c
index f175385..3c05c5e 100644
--- a/drivers/tty/serial/omap-serial.c
+++ b/drivers/tty/serial/omap-serial.c
@@ -1222,10 +1222,8 @@ static int serial_omap_suspend(struct device *dev)
{
struct uart_omap_port *up = dev_get_drvdata(dev);
- if (up) {
- uart_suspend_port(&serial_omap_reg, &up->port);
- flush_work_sync(&up->qos_work);
- }
+ uart_suspend_port(&serial_omap_reg, &up->port);
+ flush_work_sync(&up->qos_work);
return 0;
}
@@ -1234,8 +1232,8 @@ static int serial_omap_resume(struct device *dev)
{
struct uart_omap_port *up = dev_get_drvdata(dev);
- if (up)
- uart_resume_port(&serial_omap_reg, &up->port);
+ uart_resume_port(&serial_omap_reg, &up->port);
+
return 0;
}
#endif
@@ -1553,17 +1551,14 @@ static int serial_omap_runtime_suspend(struct device *dev)
static int serial_omap_runtime_resume(struct device *dev)
{
struct uart_omap_port *up = dev_get_drvdata(dev);
- struct omap_uart_port_info *pdata = dev->platform_data;
- if (up && pdata) {
- u32 loss_cnt = serial_omap_get_context_loss_count(up);
+ u32 loss_cnt = serial_omap_get_context_loss_count(up);
- if (up->context_loss_cnt != loss_cnt)
- serial_omap_restore_context(up);
+ if (up->context_loss_cnt != loss_cnt)
+ serial_omap_restore_context(up);
- up->latency = up->calc_latency;
- schedule_work(&up->qos_work);
- }
+ up->latency = up->calc_latency;
+ schedule_work(&up->qos_work);
return 0;
}
--
1.7.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] serial: omap: Remove unnecessary checks from suspend/resume
2012-09-18 11:34 ` Felipe Balbi
@ 2012-09-18 11:44 ` Poddar, Sourav
2012-09-19 21:43 ` Kevin Hilman
0 siblings, 1 reply; 5+ messages in thread
From: Poddar, Sourav @ 2012-09-18 11:44 UTC (permalink / raw)
To: balbi
Cc: gregkh, alan, tony, khilman, linux-omap, linux-serial,
linux-arm-kernel, santosh.shilimkar
Hi Felipe,
On Tue, Sep 18, 2012 at 5:04 PM, Felipe Balbi <balbi@ti.com> wrote:
> On Tue, Sep 18, 2012 at 05:05:54PM +0530, Sourav Poddar wrote:
>> Drop the check for "up" being valid on suspend/resume callbacks.
>> It should be valid always. Get rid of the "pdata" check also as
>> serial_omap_get_context_loss_count() checks for it.
>>
>> Tested on omap4 panda and 3630 based Beagle board.
>
> you need a blank line here. Other than that:
>
> Reviewed-by: Felipe Balbi <balbi@ti.com>
>
> ps: what kind of tests did you run, btw ?
>
Boot tested on Panda.
Boot and PM tested(hitiing Off) on Beagle.
>> Signed-off-by: Sourav Poddar <sourav.poddar@ti.com>
>> ---
>> drivers/tty/serial/omap-serial.c | 23 +++++++++--------------
>> 1 files changed, 9 insertions(+), 14 deletions(-)
>>
>> diff --git a/drivers/tty/serial/omap-serial.c b/drivers/tty/serial/omap-serial.c
>> index f175385..3c05c5e 100644
>> --- a/drivers/tty/serial/omap-serial.c
>> +++ b/drivers/tty/serial/omap-serial.c
>> @@ -1222,10 +1222,8 @@ static int serial_omap_suspend(struct device *dev)
>> {
>> struct uart_omap_port *up = dev_get_drvdata(dev);
>>
>> - if (up) {
>> - uart_suspend_port(&serial_omap_reg, &up->port);
>> - flush_work_sync(&up->qos_work);
>> - }
>> + uart_suspend_port(&serial_omap_reg, &up->port);
>> + flush_work_sync(&up->qos_work);
>>
>> return 0;
>> }
>> @@ -1234,8 +1232,8 @@ static int serial_omap_resume(struct device *dev)
>> {
>> struct uart_omap_port *up = dev_get_drvdata(dev);
>>
>> - if (up)
>> - uart_resume_port(&serial_omap_reg, &up->port);
>> + uart_resume_port(&serial_omap_reg, &up->port);
>> +
>> return 0;
>> }
>> #endif
>> @@ -1553,17 +1551,14 @@ static int serial_omap_runtime_suspend(struct device *dev)
>> static int serial_omap_runtime_resume(struct device *dev)
>> {
>> struct uart_omap_port *up = dev_get_drvdata(dev);
>> - struct omap_uart_port_info *pdata = dev->platform_data;
>>
>> - if (up && pdata) {
>> - u32 loss_cnt = serial_omap_get_context_loss_count(up);
>> + u32 loss_cnt = serial_omap_get_context_loss_count(up);
>>
>> - if (up->context_loss_cnt != loss_cnt)
>> - serial_omap_restore_context(up);
>> + if (up->context_loss_cnt != loss_cnt)
>> + serial_omap_restore_context(up);
>>
>> - up->latency = up->calc_latency;
>> - schedule_work(&up->qos_work);
>> - }
>> + up->latency = up->calc_latency;
>> + schedule_work(&up->qos_work);
>>
>> return 0;
>> }
>> --
>> 1.7.1
>>
>
> --
> balbi
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] serial: omap: Remove unnecessary checks from suspend/resume
2012-09-18 11:44 ` Poddar, Sourav
@ 2012-09-19 21:43 ` Kevin Hilman
2012-09-21 5:10 ` Poddar, Sourav
0 siblings, 1 reply; 5+ messages in thread
From: Kevin Hilman @ 2012-09-19 21:43 UTC (permalink / raw)
To: Poddar, Sourav
Cc: balbi, gregkh, alan, tony, linux-omap, linux-serial,
linux-arm-kernel, santosh.shilimkar
"Poddar, Sourav" <sourav.poddar@ti.com> writes:
> Hi Felipe,
>
> On Tue, Sep 18, 2012 at 5:04 PM, Felipe Balbi <balbi@ti.com> wrote:
>> On Tue, Sep 18, 2012 at 05:05:54PM +0530, Sourav Poddar wrote:
>>> Drop the check for "up" being valid on suspend/resume callbacks.
>>> It should be valid always. Get rid of the "pdata" check also as
>>> serial_omap_get_context_loss_count() checks for it.
>>>
>>> Tested on omap4 panda and 3630 based Beagle board.
>>
>> you need a blank line here. Other than that:
>>
>> Reviewed-by: Felipe Balbi <balbi@ti.com>
>>
>> ps: what kind of tests did you run, btw ?
>>
> Boot tested on Panda.
> Boot and PM tested(hitiing Off) on Beagle.
What does "PM tested" mean. hitting off in idle? hitting off in
suspend? both?
Please clarify.
Thanks,
Kevin
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] serial: omap: Remove unnecessary checks from suspend/resume
2012-09-19 21:43 ` Kevin Hilman
@ 2012-09-21 5:10 ` Poddar, Sourav
0 siblings, 0 replies; 5+ messages in thread
From: Poddar, Sourav @ 2012-09-21 5:10 UTC (permalink / raw)
To: Kevin Hilman
Cc: balbi, gregkh, alan, tony, linux-omap, linux-serial,
linux-arm-kernel, santosh.shilimkar
Hi Kevin,
On Thu, Sep 20, 2012 at 3:13 AM, Kevin Hilman
<khilman@deeprootsystems.com> wrote:
> "Poddar, Sourav" <sourav.poddar@ti.com> writes:
>
>> Hi Felipe,
>>
>> On Tue, Sep 18, 2012 at 5:04 PM, Felipe Balbi <balbi@ti.com> wrote:
>>> On Tue, Sep 18, 2012 at 05:05:54PM +0530, Sourav Poddar wrote:
>>>> Drop the check for "up" being valid on suspend/resume callbacks.
>>>> It should be valid always. Get rid of the "pdata" check also as
>>>> serial_omap_get_context_loss_count() checks for it.
>>>>
>>>> Tested on omap4 panda and 3630 based Beagle board.
>>>
>>> you need a blank line here. Other than that:
>>>
>>> Reviewed-by: Felipe Balbi <balbi@ti.com>
>>>
>>> ps: what kind of tests did you run, btw ?
>>>
>> Boot tested on Panda.
>> Boot and PM tested(hitiing Off) on Beagle.
>
> What does "PM tested" mean. hitting off in idle? hitting off in
> suspend? both?
>
Hitting off in both idle and suspend.
> Please clarify.
>
> Thanks,
>
> Kevin
~Sourav
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2012-09-21 5:10 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-09-18 11:35 [PATCH] serial: omap: Remove unnecessary checks from suspend/resume Sourav Poddar
2012-09-18 11:34 ` Felipe Balbi
2012-09-18 11:44 ` Poddar, Sourav
2012-09-19 21:43 ` Kevin Hilman
2012-09-21 5:10 ` Poddar, Sourav
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox