[PATCH v2 2/4] drm/i915/gt: reconcile Excess struct member kernel-doc warnings
Randy Dunlap
rdunlap at infradead.org
Thu Dec 28 23:42:52 UTC 2023
Hi Andi,
On 12/28/23 08:25, Andi Shyti wrote:
> Hi Randy,
>
> ...
>
>> /**
>> * struct intel_gsc - graphics security controller
>> *
>> - * @gem_obj: scratch memory GSC operations
>> - * @intf : gsc interface
>> + * @intf : gsc interface
>> + * @intf.adev : MEI aux. device for this @intf
>> + * @intf.gem_obj : scratch memory GSC operations
>> + * @intf.irq : IRQ for this device (%-1 for no IRQ)
>> + * @intf.id : this interface's id number/index
>> */
>
> I don't really like the tabs here and the space before the ':',
> which makes it a bit harder to read. Besides it doesn't look
> consistent with the rest of the documentation.
>
> Otherwise, looks good.
Yes, one existing line had " : " and somehow I managed to use that on
all of the new lines. Not good. I'll redo patch 2/4.
Thanks.
--
#Randy
More information about the Intel-gfx
mailing list