2024-03-08 02:52:09

by R Sundar

Subject: [PATCH] Removed vcpi field description to fix kernel doc warning

For linux-next repo

/include/drm/display/drm_dp_mst_helper.h:156: warning: Excess struct member 'vcpi' description in 'drm_dp_mst_port'

Signed-off-by: R SUNDAR <[email protected]>
---
include/drm/display/drm_dp_mst_helper.h | 1 -
1 file changed, 1 deletion(-)

diff --git a/include/drm/display/drm_dp_mst_helper.h b/include/drm/display/drm_dp_mst_helper.h
index 9b19d8bd520a..3ae88a383a41 100644
--- a/include/drm/display/drm_dp_mst_helper.h
+++ b/include/drm/display/drm_dp_mst_helper.h
@@ -83,7 +83,6 @@ struct drm_dp_mst_branch;
* @passthrough_aux: parent aux to which DSC pass-through requests should be
* sent, only set if DSC pass-through is possible.
* @parent: branch device parent of this port
- * @vcpi: Virtual Channel Payload info for this port.
* @connector: DRM connector this port is connected to. Protected by
* &drm_dp_mst_topology_mgr.base.lock.
* @mgr: topology manager this port lives under.
--
2.34.1

2024-03-08 05:10:40

by Randy Dunlap

Subject: Re: [PATCH] Removed vcpi field description to fix kernel doc warning

On 3/7/24 18:51, R SUNDAR wrote:
> For linux-next repo
>
> /include/drm/display/drm_dp_mst_helper.h:156: warning: Excess struct member 'vcpi' description in 'drm_dp_mst_port'
>
> Signed-off-by: R SUNDAR <[email protected]>

or
https://lore.kernel.org/lkml/[email protected]/

but I would be happy to see either of them merged.

Thanks.

> ---
> include/drm/display/drm_dp_mst_helper.h | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/include/drm/display/drm_dp_mst_helper.h b/include/drm/display/drm_dp_mst_helper.h
> index 9b19d8bd520a..3ae88a383a41 100644
> --- a/include/drm/display/drm_dp_mst_helper.h
> +++ b/include/drm/display/drm_dp_mst_helper.h
> @@ -83,7 +83,6 @@ struct drm_dp_mst_branch;
> * @passthrough_aux: parent aux to which DSC pass-through requests should be
> * sent, only set if DSC pass-through is possible.
> * @parent: branch device parent of this port
> - * @vcpi: Virtual Channel Payload info for this port.
> * @connector: DRM connector this port is connected to. Protected by
> * &drm_dp_mst_topology_mgr.base.lock.
> * @mgr: topology manager this port lives under.

--
#Randy

2024-03-08 07:54:54

by Jani Nikula

Subject: Re: [PATCH] Removed vcpi field description to fix kernel doc warning

On Thu, 07 Mar 2024, Randy Dunlap <[email protected]> wrote:
> On 3/7/24 18:51, R SUNDAR wrote:
>> For linux-next repo
>>
>> /include/drm/display/drm_dp_mst_helper.h:156: warning: Excess struct member 'vcpi' description in 'drm_dp_mst_port'
>>
>> Signed-off-by: R SUNDAR <[email protected]>
>
> or
> https://lore.kernel.org/lkml/[email protected]/
>
> but I would be happy to see either of them merged.

That has fallen through the cracks, apologies, but I merged this
yesterday to drm-misc-next:

ca892057e03e ("drm/dp_mst: fix drm_dp_mst_helper.h kernel-doc")

BR,
Jani.

>
> Thanks.
>
>> ---
>> include/drm/display/drm_dp_mst_helper.h | 1 -
>> 1 file changed, 1 deletion(-)
>>
>> diff --git a/include/drm/display/drm_dp_mst_helper.h b/include/drm/display/drm_dp_mst_helper.h
>> index 9b19d8bd520a..3ae88a383a41 100644
>> --- a/include/drm/display/drm_dp_mst_helper.h
>> +++ b/include/drm/display/drm_dp_mst_helper.h
>> @@ -83,7 +83,6 @@ struct drm_dp_mst_branch;
>> * @passthrough_aux: parent aux to which DSC pass-through requests should be
>> * sent, only set if DSC pass-through is possible.
>> * @parent: branch device parent of this port
>> - * @vcpi: Virtual Channel Payload info for this port.
>> * @connector: DRM connector this port is connected to. Protected by
>> * &drm_dp_mst_topology_mgr.base.lock.
>> * @mgr: topology manager this port lives under.

--
Jani Nikula, Intel