28.10.2021 12:59, Rafael J. Wysocki пишет: >> +/** >> + * struct power_handler - Machine power-off + restart handler >> + * >> + * Describes power-off and restart handlers which are invoked by kernel >> + * to power off or restart this machine. Supports prioritized chaining for >> + * both restart and power-off handlers. Callback's priority must be unique. >> + * Intended to be used by device drivers that are responsible for restarting >> + * and powering off hardware which kernel is running on. >> + * >> + * Struct power_handler can be static. Members of this structure must not be >> + * altered while handler is registered. >> + * >> + * Fill the structure members and pass it to register_power_handler(). >> + */ >> +struct power_handler { >> + /** >> + * @cb_data: >> + * >> + * User data included in callback's argument. >> + */ > And here I would document the structure fields in the main kerneldoc > comment above. > > As is, it is a bit hard to grasp the whole definition. > I'll move the comments in v3, thanks.