From: Kevin Hilman <khilman@ti.com>
To: "Sawant, Anand" <sawant@ti.com>
Cc: linux-omap@vger.kernel.org
Subject: Re: [PATCH] omap2plus: smartreflex: remove sr debug directory in omap_sr_remove()
Date: Tue, 15 Feb 2011 08:06:56 -0800 [thread overview]
Message-ID: <87d3mtwacf.fsf@ti.com> (raw)
In-Reply-To: <AANLkTik3h6ovXAkcU0xMPGEi93L0TqN45s0hpHGwj6CG@mail.gmail.com> (Anand Sawant's message of "Tue, 15 Feb 2011 10:56:17 +0530")
"Sawant, Anand" <sawant@ti.com> writes:
> Kevin,
>
> On Sat, Feb 12, 2011 at 5:20 AM, Kevin Hilman <khilman@ti.com> wrote:
>> Hi Anand,
>>
>> Anand S Sawant <sawant@ti.com> writes:
>>
>>> omap_sr_probe() creates the smartreflex debug directory and its
>>> underlying nvalue debug directory. These directories are removed in
>>> omap_sr_remove().
>>>
>>> Basic smartreflex functionality tested on OMAP3630 Zoom3 & OMAP4430 SDP
>>>
>>> Signed-off-by: Anand S Sawant <sawant@ti.com>
>>
>> Looks good.
>>
>> Please repost with the linux-arm-kernel list in CC:
>> linux-arm-kernel@lists.infradead.org. Thend, unless there are other
>> comments/objections, I'll queue this for 2.6.39.
>>
>
> Thanks, will repost.
>
>> Also, can summarize on which platforms this was tested.
>
> I tested SR functionality at the default OPP on 3630 Zoom3 &
> OMAP4430 SDP.
What I was wondering is how you tested the functionality of the changes
you made, namely the sr-remove path.
Kevin
>> Just curious:
>> Were you able to build SR as a module to test this?
>>
>
> No.
>
> Best Regards,
>
> Anand S Sawant
>> Kevin
>>
>>> ---
>>> arch/arm/mach-omap2/smartreflex.c | 24 ++++++++++++++----------
>>> 1 files changed, 14 insertions(+), 10 deletions(-)
>>>
>>> diff --git a/arch/arm/mach-omap2/smartreflex.c b/arch/arm/mach-omap2/smartreflex.c
>>> index d7deadf..57df5cf 100644
>>> --- a/arch/arm/mach-omap2/smartreflex.c
>>> +++ b/arch/arm/mach-omap2/smartreflex.c
>>> @@ -54,6 +54,7 @@ struct omap_sr {
>>> struct list_head node;
>>> struct omap_sr_nvalue_table *nvalue_table;
>>> struct voltagedomain *voltdm;
>>> + struct dentry *dbg_dir;
>>> };
>>>
>>> /* sr_list contains all the instances of smartreflex module */
>>> @@ -822,7 +823,7 @@ static int __init omap_sr_probe(struct platform_device *pdev)
>>> struct omap_sr *sr_info = kzalloc(sizeof(struct omap_sr), GFP_KERNEL);
>>> struct omap_sr_data *pdata = pdev->dev.platform_data;
>>> struct resource *mem, *irq;
>>> - struct dentry *vdd_dbg_dir, *dbg_dir, *nvalue_dir;
>>> + struct dentry *vdd_dbg_dir, *nvalue_dir;
>>> struct omap_volt_data *volt_data;
>>> int i, ret = 0;
>>>
>>> @@ -894,23 +895,24 @@ static int __init omap_sr_probe(struct platform_device *pdev)
>>> if (!vdd_dbg_dir)
>>> return -EINVAL;
>>>
>>> - dbg_dir = debugfs_create_dir("smartreflex", vdd_dbg_dir);
>>> - if (IS_ERR(dbg_dir)) {
>>> + sr_info->dbg_dir = debugfs_create_dir("smartreflex", vdd_dbg_dir);
>>> + if (IS_ERR(sr_info->dbg_dir)) {
>>> dev_err(&pdev->dev, "%s: Unable to create debugfs directory\n",
>>> __func__);
>>> - return PTR_ERR(dbg_dir);
>>> + return PTR_ERR(sr_info->dbg_dir);
>>> }
>>>
>>> - (void) debugfs_create_file("autocomp", S_IRUGO | S_IWUGO, dbg_dir,
>>> - (void *)sr_info, &pm_sr_fops);
>>> - (void) debugfs_create_x32("errweight", S_IRUGO, dbg_dir,
>>> + /* Using S_IWUSR to fix checkpatch warning */
>>> + (void) debugfs_create_file("autocomp", S_IRUGO | S_IWUSR,
>>> + sr_info->dbg_dir, (void *)sr_info, &pm_sr_fops);
>>> + (void) debugfs_create_x32("errweight", S_IRUGO, sr_info->dbg_dir,
>>> &sr_info->err_weight);
>>> - (void) debugfs_create_x32("errmaxlimit", S_IRUGO, dbg_dir,
>>> + (void) debugfs_create_x32("errmaxlimit", S_IRUGO, sr_info->dbg_dir,
>>> &sr_info->err_maxlimit);
>>> - (void) debugfs_create_x32("errminlimit", S_IRUGO, dbg_dir,
>>> + (void) debugfs_create_x32("errminlimit", S_IRUGO, sr_info->dbg_dir,
>>> &sr_info->err_minlimit);
>>>
>>> - nvalue_dir = debugfs_create_dir("nvalue", dbg_dir);
>>> + nvalue_dir = debugfs_create_dir("nvalue", sr_info->dbg_dir);
>>> if (IS_ERR(nvalue_dir)) {
>>> dev_err(&pdev->dev, "%s: Unable to create debugfs directory"
>>> "for n-values\n", __func__);
>>> @@ -974,6 +976,8 @@ static int __devexit omap_sr_remove(struct platform_device *pdev)
>>>
>>> if (sr_info->autocomp_active)
>>> sr_stop_vddautocomp(sr_info);
>>> + if (sr_info->dbg_dir)
>>> + debugfs_remove_recursive(sr_info->dbg_dir);
>>>
>>> list_del(&sr_info->node);
>>> iounmap(sr_info->base);
>>
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2011-02-15 16:07 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-02-05 13:32 [PATCH] omap2plus: smartreflex: remove sr debug directory in omap_sr_remove() Anand S Sawant
2011-02-11 23:50 ` Kevin Hilman
2011-02-15 5:26 ` Sawant, Anand
2011-02-15 16:06 ` Kevin Hilman [this message]
2011-02-17 16:09 ` Sawant, Anand
-- strict thread matches above, loose matches on Subject: below --
2011-02-17 15:57 sawant
2011-03-09 23:42 ` Kevin Hilman
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=87d3mtwacf.fsf@ti.com \
--to=khilman@ti.com \
--cc=linux-omap@vger.kernel.org \
--cc=sawant@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