From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-15.1 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 7DA62C433E9 for ; Mon, 1 Feb 2021 09:57:48 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 1B1E364EA6 for ; Mon, 1 Feb 2021 09:57:48 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 1B1E364EA6 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=xs4all.nl Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=dri-devel-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 67C1E6E16F; Mon, 1 Feb 2021 09:57:47 +0000 (UTC) Received: from lb1-smtp-cloud7.xs4all.net (lb1-smtp-cloud7.xs4all.net [194.109.24.24]) by gabe.freedesktop.org (Postfix) with ESMTPS id 4C52D6E16F for ; Mon, 1 Feb 2021 09:57:45 +0000 (UTC) Received: from cust-b5b5937f ([IPv6:fc0c:c16d:66b8:757f:c639:739b:9d66:799d]) by smtp-cloud7.xs4all.net with ESMTPA id 6Vxgluj1Xefbk6Vxklg06Q; Mon, 01 Feb 2021 10:57:43 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=xs4all.nl; s=s2; t=1612173463; bh=5L0KAYINbMVLYN+e0G2cePDwOg1VIYpVmSB2dTWAzME=; h=Subject:From:To:Message-ID:Date:MIME-Version:Content-Type:From: Subject; b=qSK1hWb/oqziIFFXFuJWhVKqj65m7kOj1ae9COJH8emZvesJrJiQe+yu42JRqsRR7 dqBoWZQyrVO2beEXOhqUSd+5MSd2kHksxil5ksFPy48LO6FO8ENLtM8pGMP/IADVgC k25KuoCUXfzlNWlwmmBSXWxzkA1VZVe4+pptWT3nGoW2+aE+/jz/C6X8PbLbxi44EL CAkFZtlnamX2kj2YOaUPPEs0xeuzBbPYt1nJbRSaexsdZEYd+scl9VR99iVQZ8M4g8 2k0kUs6W8XOrf2DsPNFbCkRDpXIL/dLGC2/bS+PpH6iduKgf7GsVpbvHmuXeJMy2// 2Y/vKyUA+fZ8A== Subject: Re: [PATCH v3 1/4] dp/dp_mst: Add support for sink event notify messages From: Hans Verkuil To: Sam McNally , LKML , Lyude Paul References: <20200923121320.v3.1.I8693156f555875e5c8342e86ab37ce968dfdd277@changeid> <62e32c43-29b0-9818-a4dc-ba4fb3fd1a89@xs4all.nl> Message-ID: <8bea06b5-0cbd-a135-e09e-075318bdf054@xs4all.nl> Date: Mon, 1 Feb 2021 10:57:36 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.6.0 MIME-Version: 1.0 In-Reply-To: <62e32c43-29b0-9818-a4dc-ba4fb3fd1a89@xs4all.nl> Content-Language: en-US X-CMAE-Envelope: MS4xfIEqwsQAWZhnwLxqO0Qd65U2+xKP9iljo7e0POX3+htoyyjDuSTtIZ46us8nKb7SsW8DlhK0OoJx6ohT3uV9Abg7XHijMVKAHe4KnfMx0l5y+FUgHrx8 fYGcsffKvS98VoVGLRoTuB+3S4slSv0wF0Fy2eb15nhDDxAGyZuqqv4vf0w7/MrOFqHgHYMuKnZ3V6z9G4o+r3y4MWIYBWBFTCdwWv28MJWlQUtynaP5SXY1 Iem414xhK3lxZ8rOYQRFZLU9aMI4SBCa6Y43zYpi0gn91mqrc/cX1yKnGVR6bkrd4R8orjAAXo32ZBbLq7MjGcqCkvZUO7CTJIRealRifRpJHpsEbwpbso7U bebInyq3+HoFXGwFiJTTd1yGyQEIHjAOUlDzlry7/hxhG2wRjJ+XemmBuHAvdBCBnPGH/hiQ3RcyizUQaVZQHdQX0rFSCwfWXojzq3C2qcEgCnhIBsY= X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: David Airlie , dri-devel@lists.freedesktop.org, Thomas Zimmermann Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" Hi Lyude, Daniel referred me to you as the best person to review the MST parts of this series. I can commit this, but then I prefer to have a Reviewed-by or Acked-by from someone for the first 3 DP MST patches. Alternatively, you can take the whole series (I've reviewed the 4th CEC patch). Regards, Hans On 12/01/2021 10:24, Hans Verkuil wrote: > Hi Sam, > > This series still hasn't been merged. It still applies cleanly to v5.11-rc1. > > Daniel, can you merge this series for 5.12? Or Ack this series so I can merge it? > > The first three patches deal with DP MST support, and this needs review from > you or David. > > Regards, > > Hans > > On 23/09/2020 04:13, Sam McNally wrote: >> Sink event notify messages are used for MST CEC IRQs. Add parsing >> support for sink event notify messages in preparation for handling MST >> CEC IRQs. >> >> Signed-off-by: Sam McNally >> --- >> >> (no changes since v1) >> >> drivers/gpu/drm/drm_dp_mst_topology.c | 37 ++++++++++++++++++++++++++- >> include/drm/drm_dp_mst_helper.h | 14 ++++++++++ >> 2 files changed, 50 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/drm_dp_mst_topology.c b/drivers/gpu/drm/drm_dp_mst_topology.c >> index 17dbed0a9800..15b6cc39a754 100644 >> --- a/drivers/gpu/drm/drm_dp_mst_topology.c >> +++ b/drivers/gpu/drm/drm_dp_mst_topology.c >> @@ -1027,6 +1027,30 @@ static bool drm_dp_sideband_parse_resource_status_notify(struct drm_dp_sideband_ >> return false; >> } >> >> +static bool drm_dp_sideband_parse_sink_event_notify( >> + struct drm_dp_sideband_msg_rx *raw, >> + struct drm_dp_sideband_msg_req_body *msg) >> +{ >> + int idx = 1; >> + >> + msg->u.sink_event.port_number = (raw->msg[idx] & 0xf0) >> 4; >> + idx++; >> + if (idx > raw->curlen) >> + goto fail_len; >> + >> + memcpy(msg->u.sink_event.guid, &raw->msg[idx], 16); >> + idx += 16; >> + if (idx > raw->curlen) >> + goto fail_len; >> + >> + msg->u.sink_event.event_id = (raw->msg[idx] << 8) | (raw->msg[idx + 1]); >> + idx++; >> + return true; >> +fail_len: >> + DRM_DEBUG_KMS("sink event notify parse length fail %d %d\n", idx, raw->curlen); >> + return false; >> +} >> + >> static bool drm_dp_sideband_parse_req(struct drm_dp_sideband_msg_rx *raw, >> struct drm_dp_sideband_msg_req_body *msg) >> { >> @@ -1038,6 +1062,8 @@ static bool drm_dp_sideband_parse_req(struct drm_dp_sideband_msg_rx *raw, >> return drm_dp_sideband_parse_connection_status_notify(raw, msg); >> case DP_RESOURCE_STATUS_NOTIFY: >> return drm_dp_sideband_parse_resource_status_notify(raw, msg); >> + case DP_SINK_EVENT_NOTIFY: >> + return drm_dp_sideband_parse_sink_event_notify(raw, msg); >> default: >> DRM_ERROR("Got unknown request 0x%02x (%s)\n", msg->req_type, >> drm_dp_mst_req_type_str(msg->req_type)); >> @@ -3875,6 +3901,8 @@ drm_dp_mst_process_up_req(struct drm_dp_mst_topology_mgr *mgr, >> guid = msg->u.conn_stat.guid; >> else if (msg->req_type == DP_RESOURCE_STATUS_NOTIFY) >> guid = msg->u.resource_stat.guid; >> + else if (msg->req_type == DP_SINK_EVENT_NOTIFY) >> + guid = msg->u.sink_event.guid; >> >> if (guid) >> mstb = drm_dp_get_mst_branch_device_by_guid(mgr, guid); >> @@ -3948,7 +3976,8 @@ static int drm_dp_mst_handle_up_req(struct drm_dp_mst_topology_mgr *mgr) >> drm_dp_sideband_parse_req(&mgr->up_req_recv, &up_req->msg); >> >> if (up_req->msg.req_type != DP_CONNECTION_STATUS_NOTIFY && >> - up_req->msg.req_type != DP_RESOURCE_STATUS_NOTIFY) { >> + up_req->msg.req_type != DP_RESOURCE_STATUS_NOTIFY && >> + up_req->msg.req_type != DP_SINK_EVENT_NOTIFY) { >> DRM_DEBUG_KMS("Received unknown up req type, ignoring: %x\n", >> up_req->msg.req_type); >> kfree(up_req); >> @@ -3976,6 +4005,12 @@ static int drm_dp_mst_handle_up_req(struct drm_dp_mst_topology_mgr *mgr) >> DRM_DEBUG_KMS("Got RSN: pn: %d avail_pbn %d\n", >> res_stat->port_number, >> res_stat->available_pbn); >> + } else if (up_req->msg.req_type == DP_SINK_EVENT_NOTIFY) { >> + const struct drm_dp_sink_event_notify *sink_event = >> + &up_req->msg.u.sink_event; >> + >> + DRM_DEBUG_KMS("Got SEN: pn: %d event_id %d\n", >> + sink_event->port_number, sink_event->event_id); >> } >> >> up_req->hdr = mgr->up_req_recv.initial_hdr; >> diff --git a/include/drm/drm_dp_mst_helper.h b/include/drm/drm_dp_mst_helper.h >> index 6ae5860d8644..c7c79e0ced18 100644 >> --- a/include/drm/drm_dp_mst_helper.h >> +++ b/include/drm/drm_dp_mst_helper.h >> @@ -402,6 +402,19 @@ struct drm_dp_resource_status_notify { >> u16 available_pbn; >> }; >> >> +#define DP_SINK_EVENT_PANEL_REPLAY_ACTIVE_FRAME_CRC_ERROR BIT(0) >> +#define DP_SINK_EVENT_PANEL_REPLAY_RFB_STORAGE_ERROR BIT(1) >> +#define DP_SINK_EVENT_DSC_RC_BUFFER_UNDER_RUN BIT(2) >> +#define DP_SINK_EVENT_DSC_RC_BUFFER_OVERFLOW BIT(3) >> +#define DP_SINK_EVENT_DSC_CHUNK_LENGTH_ERROR BIT(4) >> +#define DP_SINK_EVENT_CEC_IRQ_EVENT BIT(5) >> + >> +struct drm_dp_sink_event_notify { >> + u8 port_number; >> + u8 guid[16]; >> + u16 event_id; >> +}; >> + >> struct drm_dp_query_payload_ack_reply { >> u8 port_number; >> u16 allocated_pbn; >> @@ -413,6 +426,7 @@ struct drm_dp_sideband_msg_req_body { >> struct drm_dp_connection_status_notify conn_stat; >> struct drm_dp_port_number_req port_num; >> struct drm_dp_resource_status_notify resource_stat; >> + struct drm_dp_sink_event_notify sink_event; >> >> struct drm_dp_query_payload query_payload; >> struct drm_dp_allocate_payload allocate_payload; >> > _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel