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: <CAHp75VcmpgSJ6EZtYa-c3SjY5=QOhncYORrZexCJnkrO4eSfuQ@mail.gmail.com>
Date:   Thu, 30 Mar 2017 12:05:49 +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 11:39 AM, Chanwoo Choi <cw00.choi@...sung.com> wrote:
> The extcon core already provides the extcon_register_notifier() function
> in order to register the notifier block which is used to monitor
> the status change for the specific external connector such as EXTCON_USB,
> EXTCON_USB_HOST and so on. The extcon consumer uses the this function.
>
> The extcon consumer may need to monitor the all supported external
> connectors from the extcon device. In this case, The extcon consumer
> should have each notifier_block structure for each external connector.
>
> This patch adds the new extcon_register_notifier_all() function
> that extcon consumer is able to monitor the status change of all
> supported external connectors by using only one notifier_block structure.
>

> +/**
> + * 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.
> + */

Have you checked how it looks like in resulting document file (man /
html / ...) ?
My concern is multi-line short function description.

-- 
With Best Regards,
Andy Shevchenko