[Doc] class_find_device: fix reference to argument "match"

[Doc] class_find_device: fix reference to argument "match"

[Rolf Eike Beer]

Signed-off-by: Rolf Eike Beer <eike-kernel@sf-tec.de>
---
 drivers/base/class.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/base/class.c b/drivers/base/class.c
index 4f1df2e..5bd577b 100644
--- a/drivers/base/class.c
+++ b/drivers/base/class.c
@@ -403,7 +403,7 @@ EXPORT_SYMBOL_GPL(class_for_each_device);
  *
  * Note, you will need to drop the reference with put_device() after use.
  *
- * @fn is allowed to do anything including calling back into class
+ * @match is allowed to do anything including calling back into class
  * code.  There's no locking restriction.
  */
 struct device *class_find_device(struct class *class, struct device *start,
-- 
1.7.6

Re: [Doc] class_find_device: fix reference to argument

[Wanlong Gao]

Rolf Eike Beer <eike <at> sf-mail.de> writes:

> 
> Signed-off-by: Rolf Eike Beer <eike-kernel <at> sf-tec.de>
> ---
>  drivers/base/class.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 

> - * @fn is allowed to do anything including calling back into class
> + * @match is allowed to do anything including calling back into class
>   * code.  There's no locking restriction.
>   */
>  struct device *class_find_device(struct class *class, struct device *start,

NACK, it means the function class_find_device but not the @match.

Thanks
-Wanlong Gao

Re: [Doc] class_find_device: fix reference to argument

[Randy Dunlap]

On Sun, 21 Aug 2011 14:46:04 +0000 (UTC) Wanlong Gao wrote:

> Rolf Eike Beer <eike <at> sf-mail.de> writes:
> 
> > 
> > Signed-off-by: Rolf Eike Beer <eike-kernel <at> sf-tec.de>
> > ---
> >  drivers/base/class.c |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> > 
> 
> > - * @fn is allowed to do anything including calling back into class
> > + * @match is allowed to do anything including calling back into class
> >   * code.  There's no locking restriction.
> >   */
> >  struct device *class_find_device(struct class *class, struct device *start,
> 
> NACK, it means the function class_find_device but not the @match.

Are you sure?  Looks to me like a comment that was copied from
class_for_each_device() but shouldn't have been copied.

Anyway, I'll cc: Greg to see if he knows.

---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***

Re: [Doc] class_find_device: fix reference to argument

[Greg KH]

On Sun, Aug 21, 2011 at 02:53:55PM -0700, Randy Dunlap wrote:
> On Sun, 21 Aug 2011 14:46:04 +0000 (UTC) Wanlong Gao wrote:
> 
> > Rolf Eike Beer <eike <at> sf-mail.de> writes:
> > 
> > > 
> > > Signed-off-by: Rolf Eike Beer <eike-kernel <at> sf-tec.de>
> > > ---
> > >  drivers/base/class.c |    2 +-
> > >  1 files changed, 1 insertions(+), 1 deletions(-)
> > > 
> > 
> > > - * @fn is allowed to do anything including calling back into class
> > > + * @match is allowed to do anything including calling back into class
> > >   * code.  There's no locking restriction.
> > >   */
> > >  struct device *class_find_device(struct class *class, struct device *start,
> > 
> > NACK, it means the function class_find_device but not the @match.
> 
> Are you sure?  Looks to me like a comment that was copied from
> class_for_each_device() but shouldn't have been copied.
> 
> Anyway, I'll cc: Greg to see if he knows.

I think the above patch is probably correct, as it can reenter if
needed.

thanks,

greg k-h

[Doc] class_find_device: fix reference to argument "match"

[Rolf Eike Beer]

There is no argument "fn".

Signed-off-by: Rolf Eike Beer <eike-kernel@sf-tec.de>
---
 drivers/base/class.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/base/class.c b/drivers/base/class.c
index 6e81088..71059e3 100644
--- a/drivers/base/class.c
+++ b/drivers/base/class.c
@@ -406,7 +406,7 @@ EXPORT_SYMBOL_GPL(class_for_each_device);
  *
  * Note, you will need to drop the reference with put_device() after use.
  *
- * @fn is allowed to do anything including calling back into class
+ * @match is allowed to do anything including calling back into class
  * code.  There's no locking restriction.
  */
 struct device *class_find_device(struct class *class, struct device *start,
-- 
2.5.1