2009-11-22 08:16:18

by Jaswinder Singh Rajput

[permalink] [raw]
Subject: [PATCH] PM: Fix kernel-doc notation


Fix the following htmldocs warnings:

Warning(drivers/base/power/main.c:249): No description found for parameter 'master'
Warning(drivers/base/power/main.c:249): Excess function parameter 'slave' description in 'device_pm_wait_for_slaves'
Warning(drivers/base/power/main.c:292): No description found for parameter 'master'
Warning(drivers/base/power/main.c:292): Excess function parameter 'slave' description in 'device_pm_check_slaves'

Signed-off-by: Jaswinder Singh Rajput <[email protected]>
Cc: Rafael J. Wysocki <[email protected]>
Cc: Pavel Machek <[email protected]>
Cc: Randy Dunlap <[email protected]>
---
drivers/base/power/main.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/base/power/main.c b/drivers/base/power/main.c
index e8d1ea6..5f13084 100644
--- a/drivers/base/power/main.c
+++ b/drivers/base/power/main.c
@@ -243,7 +243,7 @@ static void device_pm_wait_for_masters(struct device *slave)

/**
* device_pm_wait_for_slaves - Wait for all slaves of given device.
- * @slave: Device to wait for the slaves of.
+ * @master: Device to wait for the slaves of.
*/
static void device_pm_wait_for_slaves(struct device *master)
{
@@ -286,7 +286,7 @@ static int device_pm_check_masters(struct device *slave)

/**
* device_pm_check_slaves - Check power.op_complete for slaves of a device.
- * @slave: Device to check the slaves of.
+ * @master: Device to check the slaves of.
*/
static int device_pm_check_slaves(struct device *master)
{
--
1.6.5.3


2009-11-22 17:52:58

by Pavel Machek

[permalink] [raw]
Subject: Re: [PATCH] PM: Fix kernel-doc notation

On Sun 2009-11-22 13:43:57, Jaswinder Singh Rajput wrote:
>
> Fix the following htmldocs warnings:
>
> Warning(drivers/base/power/main.c:249): No description found for parameter 'master'
> Warning(drivers/base/power/main.c:249): Excess function parameter 'slave' description in 'device_pm_wait_for_slaves'
> Warning(drivers/base/power/main.c:292): No description found for parameter 'master'
> Warning(drivers/base/power/main.c:292): Excess function parameter 'slave' description in 'device_pm_check_slaves'
>
> Signed-off-by: Jaswinder Singh Rajput <[email protected]>
> Cc: Rafael J. Wysocki <[email protected]>

Acked-by: Pavel Machek <[email protected]>

--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

2009-11-25 00:22:18

by Rafael J. Wysocki

[permalink] [raw]
Subject: Re: [PATCH] PM: Fix kernel-doc notation

On Sunday 22 November 2009, Jaswinder Singh Rajput wrote:
>
> Fix the following htmldocs warnings:
>
> Warning(drivers/base/power/main.c:249): No description found for parameter 'master'
> Warning(drivers/base/power/main.c:249): Excess function parameter 'slave' description in 'device_pm_wait_for_slaves'
> Warning(drivers/base/power/main.c:292): No description found for parameter 'master'
> Warning(drivers/base/power/main.c:292): Excess function parameter 'slave' description in 'device_pm_check_slaves'
>
> Signed-off-by: Jaswinder Singh Rajput <[email protected]>
> Cc: Rafael J. Wysocki <[email protected]>
> Cc: Pavel Machek <[email protected]>
> Cc: Randy Dunlap <[email protected]>

Applied to suspend-2.6/linux-next.

Thanks,
Rafael

2009-12-11 20:05:01

by Randy Dunlap

[permalink] [raw]
Subject: Re: [PATCH] PM: Fix kernel-doc notation

On Sun, 22 Nov 2009 13:43:57 +0530 Jaswinder Singh Rajput wrote:

>
> Fix the following htmldocs warnings:
>
> Warning(drivers/base/power/main.c:249): No description found for parameter 'master'
> Warning(drivers/base/power/main.c:249): Excess function parameter 'slave' description in 'device_pm_wait_for_slaves'
> Warning(drivers/base/power/main.c:292): No description found for parameter 'master'
> Warning(drivers/base/power/main.c:292): Excess function parameter 'slave' description in 'device_pm_check_slaves'
>
> Signed-off-by: Jaswinder Singh Rajput <[email protected]>
> Cc: Rafael J. Wysocki <[email protected]>
> Cc: Pavel Machek <[email protected]>
> Cc: Randy Dunlap <[email protected]>

What tree was this for?
I guess it's not needed now, I can't find these functions..

Thanks.

> ---
> drivers/base/power/main.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/base/power/main.c b/drivers/base/power/main.c
> index e8d1ea6..5f13084 100644
> --- a/drivers/base/power/main.c
> +++ b/drivers/base/power/main.c
> @@ -243,7 +243,7 @@ static void device_pm_wait_for_masters(struct device *slave)
>
> /**
> * device_pm_wait_for_slaves - Wait for all slaves of given device.
> - * @slave: Device to wait for the slaves of.
> + * @master: Device to wait for the slaves of.
> */
> static void device_pm_wait_for_slaves(struct device *master)
> {
> @@ -286,7 +286,7 @@ static int device_pm_check_masters(struct device *slave)
>
> /**
> * device_pm_check_slaves - Check power.op_complete for slaves of a device.
> - * @slave: Device to check the slaves of.
> + * @master: Device to check the slaves of.
> */
> static int device_pm_check_slaves(struct device *master)
> {
> --

---
~Randy

2009-12-11 21:07:47

by Rafael J. Wysocki

[permalink] [raw]
Subject: Re: [PATCH] PM: Fix kernel-doc notation

On Friday 11 December 2009, Randy Dunlap wrote:
> On Sun, 22 Nov 2009 13:43:57 +0530 Jaswinder Singh Rajput wrote:
>
> >
> > Fix the following htmldocs warnings:
> >
> > Warning(drivers/base/power/main.c:249): No description found for parameter 'master'
> > Warning(drivers/base/power/main.c:249): Excess function parameter 'slave' description in 'device_pm_wait_for_slaves'
> > Warning(drivers/base/power/main.c:292): No description found for parameter 'master'
> > Warning(drivers/base/power/main.c:292): Excess function parameter 'slave' description in 'device_pm_check_slaves'
> >
> > Signed-off-by: Jaswinder Singh Rajput <[email protected]>
> > Cc: Rafael J. Wysocki <[email protected]>
> > Cc: Pavel Machek <[email protected]>
> > Cc: Randy Dunlap <[email protected]>
>
> What tree was this for?
> I guess it's not needed now, I can't find these functions..

It was on top of the async suspend/resume patches that Linus has rejected.

Rafael