* [PATCH] Documentation/hwmon: Fix description of devm_hwmon_device_unregister
@ 2023-06-08 22:42 YYang
2023-06-09 1:31 ` Bagas Sanjaya
2023-06-09 6:45 ` Jonathan Corbet
0 siblings, 2 replies; 6+ messages in thread
From: YYang @ 2023-06-08 22:42 UTC (permalink / raw)
To: jdelvare, linux, corbet; +Cc: linux-hwmon, linux-doc, linux-kernel, YYang
From: YYang <iyysheng@gmai.com>
Use devm_hwmon_device_register_with_info to replace
hwmon_device_register_with_info in description of
devm_hwmon_device_unregister.
Signed-off-by: YYang <iyysheng@gmai.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] 6+ messages in thread
* Re: [PATCH] Documentation/hwmon: Fix description of devm_hwmon_device_unregister
2023-06-08 22:42 [PATCH] Documentation/hwmon: Fix description of devm_hwmon_device_unregister YYang
@ 2023-06-09 1:31 ` Bagas Sanjaya
2023-06-09 1:51 ` Guenter Roeck
2023-06-09 6:45 ` Jonathan Corbet
1 sibling, 1 reply; 6+ messages in thread
From: Bagas Sanjaya @ 2023-06-09 1:31 UTC (permalink / raw)
To: YYang, jdelvare, linux, corbet
Cc: linux-hwmon, linux-doc, linux-kernel, YYang
[-- Attachment #1: Type: text/plain, Size: 399 bytes --]
On Fri, Jun 09, 2023 at 02:42:30AM +0400, YYang wrote:
> From: YYang <iyysheng@gmai.com>
>
> Use devm_hwmon_device_register_with_info to replace
> hwmon_device_register_with_info in description of
> devm_hwmon_device_unregister.
>
> Signed-off-by: YYang <iyysheng@gmai.com>
Is your SoB address gmail one (sounds typo here)?
--
An old man doll... just what I always wanted! - Clara
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Documentation/hwmon: Fix description of devm_hwmon_device_unregister
2023-06-09 1:31 ` Bagas Sanjaya
@ 2023-06-09 1:51 ` Guenter Roeck
2023-06-09 7:47 ` Yongsheng Yang
0 siblings, 1 reply; 6+ messages in thread
From: Guenter Roeck @ 2023-06-09 1:51 UTC (permalink / raw)
To: Bagas Sanjaya, YYang, jdelvare, corbet
Cc: linux-hwmon, linux-doc, linux-kernel, YYang
On 6/8/23 18:31, Bagas Sanjaya wrote:
> On Fri, Jun 09, 2023 at 02:42:30AM +0400, YYang wrote:
>> From: YYang <iyysheng@gmai.com>
>>
>> Use devm_hwmon_device_register_with_info to replace
>> hwmon_device_register_with_info in description of
>> devm_hwmon_device_unregister.
>>
>> Signed-off-by: YYang <iyysheng@gmai.com>
>
> Is your SoB address gmail one (sounds typo here)?
>
Looks like it. YYang - please resubmit with correct SoB.
Thanks,
Guenter
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Documentation/hwmon: Fix description of devm_hwmon_device_unregister
2023-06-08 22:42 [PATCH] Documentation/hwmon: Fix description of devm_hwmon_device_unregister YYang
2023-06-09 1:31 ` Bagas Sanjaya
@ 2023-06-09 6:45 ` Jonathan Corbet
2023-06-09 8:39 ` Yongsheng Yang
1 sibling, 1 reply; 6+ messages in thread
From: Jonathan Corbet @ 2023-06-09 6:45 UTC (permalink / raw)
To: YYang, jdelvare, linux; +Cc: linux-hwmon, linux-doc, linux-kernel, YYang
YYang <iyysheng@gmail.com> writes:
> From: YYang <iyysheng@gmai.com>
>
> Use devm_hwmon_device_register_with_info to replace
> hwmon_device_register_with_info in description of
> devm_hwmon_device_unregister.
>
> Signed-off-by: YYang <iyysheng@gmai.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.
If, while you're at it, you add the trailing parentheses() to the
function name, then the docs build will automatically make a cross-link
to the documentation.
Thanks,
jon
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Documentation/hwmon: Fix description of devm_hwmon_device_unregister
2023-06-09 1:51 ` Guenter Roeck
@ 2023-06-09 7:47 ` Yongsheng Yang
0 siblings, 0 replies; 6+ messages in thread
From: Yongsheng Yang @ 2023-06-09 7:47 UTC (permalink / raw)
To: linux
Cc: bagasdotme, corbet, iyysheng, jdelvare, linux-doc, linux-hwmon,
linux-kernel
> On 6/8/23 18:31, Bagas Sanjaya wrote:
> > On Fri, Jun 09, 2023 at 02:42:30AM +0400, YYang wrote:
> >> From: YYang <iyysheng@gmai.com>
> >>
> >> Use devm_hwmon_device_register_with_info to replace
> >> hwmon_device_register_with_info in description of
> >> devm_hwmon_device_unregister.
> >>
> >> Signed-off-by: YYang <iyysheng@gmai.com>
> >
> > Is your SoB address gmail one (sounds typo here)?
> >
>
> Looks like it. YYang - please resubmit with correct SoB.
>
> Thanks,
> Guenter
Sorry for my mistake, I fixed the misspelling. Will post the updated version.
Thanks
Yongsheng
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Documentation/hwmon: Fix description of devm_hwmon_device_unregister
2023-06-09 6:45 ` Jonathan Corbet
@ 2023-06-09 8:39 ` Yongsheng Yang
0 siblings, 0 replies; 6+ messages in thread
From: Yongsheng Yang @ 2023-06-09 8:39 UTC (permalink / raw)
To: corbet
Cc: iyysheng, iyysheng, jdelvare, linux-doc, linux-hwmon,
linux-kernel, linux
> YYang <iyysheng@gmail.com> writes:
>
> > From: YYang <iyysheng@gmai.com>
> >
> > Use devm_hwmon_device_register_with_info to replace
> > hwmon_device_register_with_info in description of
> > devm_hwmon_device_unregister.
> >
> > Signed-off-by: YYang <iyysheng@gmai.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.
>
> If, while you're at it, you add the trailing parentheses() to the
> function name, then the docs build will automatically make a cross-link
> to the documentation.
>
> Thanks,
>
> jon
Sure, I have modfied the commit message and post the updated version. Thanks
for your advice.
Thanks
Yongsheng
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-06-09 8:40 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-06-08 22:42 [PATCH] Documentation/hwmon: Fix description of devm_hwmon_device_unregister YYang
2023-06-09 1:31 ` Bagas Sanjaya
2023-06-09 1:51 ` Guenter Roeck
2023-06-09 7:47 ` Yongsheng Yang
2023-06-09 6:45 ` Jonathan Corbet
2023-06-09 8:39 ` Yongsheng Yang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox