public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] Documentation/hwmon: Fix description of devm_hwmon_device_unregister()
@ 2023-06-09  7:55 Yongsheng Yang
  2023-06-09 13:44 ` Bagas Sanjaya
  2023-06-10 14:39 ` Guenter Roeck
  0 siblings, 2 replies; 4+ messages in thread
From: Yongsheng Yang @ 2023-06-09  7:55 UTC (permalink / raw)
  To: jdelvare, linux, corbet
  Cc: linux-hwmon, linux-doc, linux-kernel, Yongsheng Yang

Use devm_hwmon_device_register_with_info to replace
hwmon_device_register_with_info in description of
devm_hwmon_device_unregister.

Signed-off-by: Yongsheng Yang <iyysheng@gmail.com>
---
 Documentation/hwmon/hwmon-kernel-api.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/hwmon/hwmon-kernel-api.rst b/Documentation/hwmon/hwmon-kernel-api.rst
index c2d1e0299d8d..6cacf7daf25c 100644
--- a/Documentation/hwmon/hwmon-kernel-api.rst
+++ b/Documentation/hwmon/hwmon-kernel-api.rst
@@ -66,7 +66,7 @@ hwmon_device_register_with_info.
 
 devm_hwmon_device_unregister does not normally have to be called. It is only
 needed for error handling, and only needed if the driver probe fails after
-the call to hwmon_device_register_with_info and if the automatic (device
+the call to devm_hwmon_device_register_with_info and if the automatic (device
 managed) removal would be too late.
 
 All supported hwmon device registration functions only accept valid device
-- 
2.37.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH v2] Documentation/hwmon: Fix description of devm_hwmon_device_unregister()
  2023-06-09  7:55 [PATCH v2] Documentation/hwmon: Fix description of devm_hwmon_device_unregister() Yongsheng Yang
@ 2023-06-09 13:44 ` Bagas Sanjaya
  2023-06-10 14:39 ` Guenter Roeck
  1 sibling, 0 replies; 4+ messages in thread
From: Bagas Sanjaya @ 2023-06-09 13:44 UTC (permalink / raw)
  To: Yongsheng Yang, jdelvare, linux, corbet
  Cc: linux-hwmon, linux-doc, linux-kernel

On 6/9/23 14:55, Yongsheng Yang wrote:
> diff --git a/Documentation/hwmon/hwmon-kernel-api.rst b/Documentation/hwmon/hwmon-kernel-api.rst
> index c2d1e0299d8d..6cacf7daf25c 100644
> --- a/Documentation/hwmon/hwmon-kernel-api.rst
> +++ b/Documentation/hwmon/hwmon-kernel-api.rst
> @@ -66,7 +66,7 @@ hwmon_device_register_with_info.
>  
>  devm_hwmon_device_unregister does not normally have to be called. It is only
>  needed for error handling, and only needed if the driver probe fails after
> -the call to hwmon_device_register_with_info and if the automatic (device
> +the call to devm_hwmon_device_register_with_info and if the automatic (device
>  managed) removal would be too late.
>  
>  All supported hwmon device registration functions only accept valid device

LGTM, thanks!

Reviewed-by: Bagas Sanjaya <bagasdotme@gmail.com>

-- 
An old man doll... just what I always wanted! - Clara


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v2] Documentation/hwmon: Fix description of devm_hwmon_device_unregister()
  2023-06-09  7:55 [PATCH v2] Documentation/hwmon: Fix description of devm_hwmon_device_unregister() Yongsheng Yang
  2023-06-09 13:44 ` Bagas Sanjaya
@ 2023-06-10 14:39 ` Guenter Roeck
  2023-06-12  1:42   ` Yongsheng Yang
  1 sibling, 1 reply; 4+ messages in thread
From: Guenter Roeck @ 2023-06-10 14:39 UTC (permalink / raw)
  To: Yongsheng Yang; +Cc: jdelvare, corbet, linux-hwmon, linux-doc, linux-kernel

On Fri, Jun 09, 2023 at 11:55:10AM +0400, Yongsheng Yang wrote:
> Use devm_hwmon_device_register_with_info to replace
> hwmon_device_register_with_info in description of
> devm_hwmon_device_unregister.
> 

You didn't add () to function names, but I guess it is better to do that
in a separate patch anyway.

Applied.

Thanks,
Guenter

> Signed-off-by: Yongsheng Yang <iyysheng@gmail.com>
> Reviewed-by: Bagas Sanjaya <bagasdotme@gmail.com>
> ---
>  Documentation/hwmon/hwmon-kernel-api.rst | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/Documentation/hwmon/hwmon-kernel-api.rst b/Documentation/hwmon/hwmon-kernel-api.rst
> index c2d1e0299d8d..6cacf7daf25c 100644
> --- a/Documentation/hwmon/hwmon-kernel-api.rst
> +++ b/Documentation/hwmon/hwmon-kernel-api.rst
> @@ -66,7 +66,7 @@ hwmon_device_register_with_info.
>  
>  devm_hwmon_device_unregister does not normally have to be called. It is only
>  needed for error handling, and only needed if the driver probe fails after
> -the call to hwmon_device_register_with_info and if the automatic (device
> +the call to devm_hwmon_device_register_with_info and if the automatic (device
>  managed) removal would be too late.
>  
>  All supported hwmon device registration functions only accept valid device

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH v2] Documentation/hwmon: Fix description of devm_hwmon_device_unregister()
  2023-06-10 14:39 ` Guenter Roeck
@ 2023-06-12  1:42   ` Yongsheng Yang
  0 siblings, 0 replies; 4+ messages in thread
From: Yongsheng Yang @ 2023-06-12  1:42 UTC (permalink / raw)
  To: linux; +Cc: corbet, iyysheng, jdelvare, linux-doc, linux-hwmon, linux-kernel

> On Fri, Jun 09, 2023 at 11:55:10AM +0400, Yongsheng Yang wrote:
> > Use devm_hwmon_device_register_with_info to replace
> > hwmon_device_register_with_info in description of
> > devm_hwmon_device_unregister.
> > 
> 
> You didn't add () to function names, but I guess it is better to do that
> in a separate patch anyway.
> 
> Applied.
> 
> Thanks,
> Guenter
> 
Ok, I just add () to function names in commit message title. I will add
() to function names releate in the whole commit message. Then I will
post the updated version.

Thanks

Yongsheng
> > Signed-off-by: Yongsheng Yang <iyysheng@gmail.com>
> > Reviewed-by: Bagas Sanjaya <bagasdotme@gmail.com>
> > ---
> >  Documentation/hwmon/hwmon-kernel-api.rst | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/Documentation/hwmon/hwmon-kernel-api.rst b/Documentation/hwmon/hwmon-kernel-api.rst
> > index c2d1e0299d8d..6cacf7daf25c 100644
> > --- a/Documentation/hwmon/hwmon-kernel-api.rst
> > +++ b/Documentation/hwmon/hwmon-kernel-api.rst
> > @@ -66,7 +66,7 @@ hwmon_device_register_with_info.
> >  
> >  devm_hwmon_device_unregister does not normally have to be called. It is only
> >  needed for error handling, and only needed if the driver probe fails after
> > -the call to hwmon_device_register_with_info and if the automatic (device
> > +the call to devm_hwmon_device_register_with_info and if the automatic (device
> >  managed) removal would be too late.
> >  
> >  All supported hwmon device registration functions only accept valid device


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2023-06-12  1:43 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-06-09  7:55 [PATCH v2] Documentation/hwmon: Fix description of devm_hwmon_device_unregister() Yongsheng Yang
2023-06-09 13:44 ` Bagas Sanjaya
2023-06-10 14:39 ` Guenter Roeck
2023-06-12  1:42   ` Yongsheng Yang

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox