Hi,

Thank you for the review.

On 2/7/22 12:43, Simon Ser wrote:
> On Wednesday, January 26th, 2022 at 16:11, Hans de Goede 
> <hdego...@redhat.com> wrote:
> 
>> - * A pointer to the drm_privacy_screen's struct is passed as the void *data
>> + * A pointer to the drm_privacy_screen's struct is passed as the void \*data
> 
> Maybe we can use @data here instead? It's used to refer to arguments or struct
> members.

It is not an argument to the function being described; nor is it a struct 
member,
it is the void *data in:

typedef int (*notifier_fn_t)(struct notifier_block *nb,
                        unsigned long action, void *data);


> Alternatively, use double backquotes to format it as inline code blocks:
> 
>    ``void *data``

So I'll prepare a v2 using this.

Regards,

Hans

Reply via email to