linux-kernel - Re: [PATCH 2/2] extcon: Add new extcon_register_notifier

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAHp75VfZ8YURJBjMC6v5660T=AKX8_Fom+1Wht3HXCREsGOGtg@mail.gmail.com>
Date:   Thu, 30 Mar 2017 14:09:32 +0300
From:   Andy Shevchenko <andy.shevchenko@...il.com>
To:     Chanwoo Choi <cw00.choi@...sung.com>
Cc:     "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        Hans de Goede <hdegoede@...hat.com>, chanwoo@...nel.org,
        MyungJoo Ham <myungjoo.ham@...sung.com>
Subject: Re: [PATCH 2/2] extcon: Add new extcon_register_notifier_all() to
 monitor all external connectors

On Thu, Mar 30, 2017 at 1:56 PM, Chanwoo Choi <cw00.choi@...sung.com> wrote:
> On 2017년 03월 30일 19:42, Andy Shevchenko wrote:
>> On Thu, Mar 30, 2017 at 12:24 PM, Chanwoo Choi <cw00.choi@...sung.com> wrote:
>>> On 2017년 03월 30일 18:05, Andy Shevchenko wrote:
>>>> On Thu, Mar 30, 2017 at 11:39 AM, Chanwoo Choi <cw00.choi@...sung.com> wrote:

>>>>> +/**
>>>>> + * extcon_register_notifier_all() - Register a notifier block to get the noti
>>>>> + *                             of the status change for all supported external
>>>>> + *                             connectors from extcon.
>>>>> + * @edev:      the extcon device that has the external connecotr.
>>>>> + * @nb:                a notifier block to be registered.
>>>>> + *
>>>>> + * Note that the second parameter given to the callback of nb (val) is
>>>>> + * the current state and third parameter is the edev pointer.
>>>>> + */

> If you metioned the following guide on Line 111
> in the Documentation/doc-guide/kernel-doc.rst,

Yes, that's one.

> OK. I'll modify it.

Thanks.

>
> Example kernel-doc function comment::
>
>   /**
>    * foobar() - Brief description of foobar.
>    * @arg: Description of argument of foobar.
>    *
>    * Longer description of foobar.
>    *
>    * Return: Description of return value of foobar.
>    */
>   int foobar(int arg)
>
> --
> Best Regards,
> Chanwoo Choi
> Samsung Electronics



-- 
With Best Regards,
Andy Shevchenko