On Thu, Mar 30, 2017 at 11:39 AM, Chanwoo Choi <[email protected]> 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
