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
