From: Kevin Hilman <khilman@ti.com>
To: Nishanth Menon <nm@ti.com>
Cc: Jarkko Nikula <jhnikula@gmail.com>,
linux-omap@vger.kernel.org, Thara Gopinath <thara@ti.com>
Subject: Re: [RFC 3/3] omap3+: sr: Reuse sr_[start|stop]_vddautocomp functions
Date: Wed, 02 Mar 2011 16:48:38 -0800 [thread overview]
Message-ID: <87vd01t4ex.fsf@ti.com> (raw)
In-Reply-To: <4D6E775D.7040107@ti.com> (Nishanth Menon's message of "Wed, 02 Mar 2011 22:29:09 +0530")
Nishanth Menon <nm@ti.com> writes:
> Jarkko Nikula wrote, on 03/02/2011 09:27 PM:
>> sr_start_vddautocomp and sr_stop_autocomp functions can be reused from
>> omap_sr_enable, omap_sr_disable and omap_sr_disable_reset_volt and by
>> adding one additional argument sr_stop_autocomp.
>>
>> Signed-off-by: Jarkko Nikula<jhnikula@gmail.com>
>> ---
>> arch/arm/mach-omap2/smartreflex.c | 41 ++++++------------------------------
>> 1 files changed, 7 insertions(+), 34 deletions(-)
>>
>> diff --git a/arch/arm/mach-omap2/smartreflex.c b/arch/arm/mach-omap2/smartreflex.c
>> index 11741d8..7e6002f 100644
>> --- a/arch/arm/mach-omap2/smartreflex.c
>> +++ b/arch/arm/mach-omap2/smartreflex.c
>> @@ -227,7 +227,7 @@ static void sr_start_vddautocomp(struct omap_sr *sr)
>> sr->autocomp_active = true;
>> }
>>
>> -static void sr_stop_vddautocomp(struct omap_sr *sr)
>> +static void sr_stop_vddautocomp(struct omap_sr *sr, int is_volt_reset)
>> {
>> if (!sr->autocomp_active)
>> return;
>> @@ -239,7 +239,7 @@ static void sr_stop_vddautocomp(struct omap_sr *sr)
>> return;
>> }
>>
>> - if (!sr_class->disable(sr->voltdm, 1))
>> + if (!sr_class->disable(sr->voltdm, is_volt_reset))
>> sr->autocomp_active = false;
>> }
>>
>> @@ -681,16 +681,7 @@ void omap_sr_enable(struct voltagedomain *voltdm)
>> return;
>> }
>>
>> - if (!sr->autocomp_active)
>> - return;
>> -
>> - if (!sr_class || !(sr_class->enable) || !(sr_class->configure)) {
>> - dev_warn(&sr->pdev->dev, "%s: smartreflex class driver not"
>> - "registered\n", __func__);
>> - return;
>> - }
>> -
>> - sr_class->enable(voltdm);
>> + sr_start_vddautocomp(sr);
>> }
>>
>> /**
>> @@ -714,16 +705,7 @@ void omap_sr_disable(struct voltagedomain *voltdm)
>> return;
>> }
>>
>> - if (!sr->autocomp_active)
>> - return;
>> -
>> - if (!sr_class || !(sr_class->disable)) {
>> - dev_warn(&sr->pdev->dev, "%s: smartreflex class driver not"
>> - "registered\n", __func__);
>> - return;
>> - }
>> -
>> - sr_class->disable(voltdm, 0);
>> + sr_stop_vddautocomp(sr, 0);
>> }
>>
>> /**
>> @@ -747,16 +729,7 @@ void omap_sr_disable_reset_volt(struct voltagedomain *voltdm)
>> return;
>> }
>>
>> - if (!sr->autocomp_active)
>> - return;
>> -
>> - if (!sr_class || !(sr_class->disable)) {
>> - dev_warn(&sr->pdev->dev, "%s: smartreflex class driver not"
>> - "registered\n", __func__);
>> - return;
>> - }
>> -
>> - sr_class->disable(voltdm, 1);
>> + sr_stop_vddautocomp(sr, 1);
>> }
>>
>> /**
>> @@ -809,7 +782,7 @@ static int omap_sr_autocomp_store(void *data, u64 val)
>> }
>>
>> if (!val)
>> - sr_stop_vddautocomp(sr_info);
>> + sr_stop_vddautocomp(sr_info, 1);
>> else
>> sr_start_vddautocomp(sr_info);
>>
>> @@ -976,7 +949,7 @@ static int __devexit omap_sr_remove(struct platform_device *pdev)
>> }
>>
>> if (sr_info->autocomp_active)
>> - sr_stop_vddautocomp(sr_info);
>> + sr_stop_vddautocomp(sr_info, 1);
>>
>> list_del(&sr_info->node);
>> iounmap(sr_info->base);
>
> Looks like a nice cleanup to me.
>
> if the motivation of patch 1/3 is to do this cleanup, I am altogether
> for it (lesser code == lesser bugs ;) ). btw, this should not impact
> class1.5 either - core sr.c sequencing has not been modified in sr1.5
> :)
>
> Depending on Kevin's views, either you or I or both of us can rebase
> depending on whose ever series gets pulled in first.. (either should
> be a very minimal effort).
Nishanth,
I'd prefer if you handle these. You can add them to the fixup/cleanup
part of your SR1.5 series.
Thanks,
Kevin
next prev parent reply other threads:[~2011-03-03 0:48 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-03-02 15:57 [RFC 0/3] omap3+: sr: Minor fix and cleanups Jarkko Nikula
2011-03-02 15:57 ` [RFC 1/3] omap3+: sr: Prevent multiple smartreflex class driver enable calls Jarkko Nikula
2011-03-02 16:52 ` Nishanth Menon
2011-03-02 15:57 ` [RFC 2/3] omap3+: sr: Sync sr_stop_vddautocomp implementation with sr_start_vddautocomp Jarkko Nikula
2011-03-02 15:57 ` [RFC 3/3] omap3+: sr: Reuse sr_[start|stop]_vddautocomp functions Jarkko Nikula
2011-03-02 16:59 ` Nishanth Menon
2011-03-03 0:48 ` Kevin Hilman [this message]
2011-03-03 0:52 ` Nishanth Menon
2011-03-03 6:57 ` Jarkko Nikula
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87vd01t4ex.fsf@ti.com \
--to=khilman@ti.com \
--cc=jhnikula@gmail.com \
--cc=linux-omap@vger.kernel.org \
--cc=nm@ti.com \
--cc=thara@ti.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox