[PATCH] drm: mst: Fix query_payload ack reply struct

Lyude Paul lyude at redhat.com
Thu Aug 29 17:06:58 UTC 2019


Is it worth actually CCing stable on this? This patch is certainly correct but
I don't think we use this struct for anything quite yet.

Otherwise: Reviewed-by: Lyude Paul <lyude at redhat.com>

On Thu, 2019-08-29 at 12:52 -0400, Sean Paul wrote:
> From: Sean Paul <seanpaul at chromium.org>
> 
> Spec says[1] Allocated_PBN is 16 bits
> 
> [1]- DisplayPort 1.2 Spec, Section 2.11.9.8, Table 2-98
> 
> Fixes: ad7f8a1f9ced ("drm/helper: add Displayport multi-stream helper
> (v0.6)")
> Cc: Lyude Paul <lyude at redhat.com>
> Cc: Todd Previte <tprevite at gmail.com>
> Cc: Dave Airlie <airlied at redhat.com>
> Cc: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
> Cc: Maxime Ripard <maxime.ripard at bootlin.com>
> Cc: Sean Paul <sean at poorly.run>
> Cc: David Airlie <airlied at linux.ie>
> Cc: Daniel Vetter <daniel at ffwll.ch>
> Cc: dri-devel at lists.freedesktop.org
> Cc: <stable at vger.kernel.org> # v3.17+
> Signed-off-by: Sean Paul <seanpaul at chromium.org>
> ---
>  include/drm/drm_dp_mst_helper.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/drm/drm_dp_mst_helper.h
> b/include/drm/drm_dp_mst_helper.h
> index 2ba6253ea6d3..fc349204a71b 100644
> --- a/include/drm/drm_dp_mst_helper.h
> +++ b/include/drm/drm_dp_mst_helper.h
> @@ -334,7 +334,7 @@ struct drm_dp_resource_status_notify {
>  
>  struct drm_dp_query_payload_ack_reply {
>  	u8 port_number;
> -	u8 allocated_pbn;
> +	u16 allocated_pbn;
>  };
>  
>  struct drm_dp_sideband_msg_req_body {
-- 
Cheers,
	Lyude Paul



More information about the dri-devel mailing list