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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxx>

On Thu, 2019-08-29 at 12:52 -0400, Sean Paul wrote:
> From: Sean Paul <seanpaul@xxxxxxxxxxxx>
> 
> 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@xxxxxxxxxx>
> Cc: Todd Previte <tprevite@xxxxxxxxx>
> Cc: Dave Airlie <airlied@xxxxxxxxxx>
> Cc: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx>
> Cc: Maxime Ripard <maxime.ripard@xxxxxxxxxxx>
> Cc: Sean Paul <sean@xxxxxxxxxx>
> Cc: David Airlie <airlied@xxxxxxxx>
> Cc: Daniel Vetter <daniel@xxxxxxxx>
> Cc: dri-devel@xxxxxxxxxxxxxxxxxxxxx
> Cc: <stable@xxxxxxxxxxxxxxx> # v3.17+
> Signed-off-by: Sean Paul <seanpaul@xxxxxxxxxxxx>
> ---
>  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

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux