On Thu, 08 Jan 2026, "Gote, Nitin R" <[email protected]> wrote:
>> -----Original Message-----
>> From: Intel-gfx <[email protected]> On Behalf Of Jani
>> Nikula
>> Sent: Wednesday, January 7, 2026 9:32 PM
>> To: [email protected]
>> Cc: Nikula, Jani <[email protected]>; Usyskin, Alexander
>> <[email protected]>
>> Subject: [PATCH] mei: late_bind: fix struct intel_lb_component_ops kernel-doc
>> 
>> Fix kernel-doc warnings on struct intel_lb_component_ops:
>> 
>> Warning: include/drm/intel/intel_lb_mei_interface.h:55 Incorrect use of
>>   kernel-doc format: * push_payload - Sends a payload to the
>>   authentication firmware
>> 
>> And a bunch more. There isn't really support for documenting function pointer
>> struct members in kernel-doc, but at least reference the member properly.
>> 
>> Fixes: 741eeabb7c78 ("mei: late_bind: add late binding component driver")
>> Cc: Alexander Usyskin <[email protected]>
>> Signed-off-by: Jani Nikula <[email protected]>
>> ---
>>  include/drm/intel/intel_lb_mei_interface.h | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>> 
>> diff --git a/include/drm/intel/intel_lb_mei_interface.h
>> b/include/drm/intel/intel_lb_mei_interface.h
>> index d65be2cba2ab..0850738a30fc 100644
>> --- a/include/drm/intel/intel_lb_mei_interface.h
>> +++ b/include/drm/intel/intel_lb_mei_interface.h
>> @@ -53,7 +53,8 @@ enum intel_lb_status {
>>   */
>>  struct intel_lb_component_ops {
>>      /**
>> -     * push_payload - Sends a payload to the authentication firmware
>> +     * @push_payload: Sends a payload to the authentication firmware
>> +     *
> LGTM
> Reviewed-by: Nitin Gote <[email protected]>

Thanks, pushed to drm-intel-next.

BR,
Jani.


>
>>       * @dev: Device struct corresponding to the mei device
>>       * @type: Payload type (see &enum intel_lb_type)
>>       * @flags: Payload flags bitmap (e.g. %INTEL_LB_FLAGS_IS_PERSISTENT)
>> --
>> 2.47.3
>

-- 
Jani Nikula, Intel

Reply via email to