On Thu, Mar 30, 2017 at 1:56 PM, Chanwoo Choi <cw00.c...@samsung.com> wrote: > On 2017년 03월 30일 19:42, Andy Shevchenko wrote: >> On Thu, Mar 30, 2017 at 12:24 PM, Chanwoo Choi <cw00.c...@samsung.com> wrote: >>> On 2017년 03월 30일 18:05, Andy Shevchenko wrote: >>>> On Thu, Mar 30, 2017 at 11:39 AM, Chanwoo Choi <cw00.c...@samsung.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