Re: [alsa-devel] [Intel-gfx] [PATCH V4 1/2] dp/mst: add SDP stream support
Add Takashi and ALSA mailing list
Best Regards, Libin
On 12/02/2015 02:09 PM, libin.yang@linux.intel.com wrote:
From: Libin Yang libin.yang@linux.intel.com
This adds code to initialise the SDP streams for a sink in the simplest ordering.
I've no idea how you'd want to control the ordering at this level, so don't bother until someone comes up with a use case.
Reviewed-by: Ander Conselvan de Oliveira conselvan2@gmail.com Signed-off-by: Libin Yang libin.yang@linux.intel.com Signed-off-by: Dave Airlie airlied@redhat.com
drivers/gpu/drm/drm_dp_mst_topology.c | 38 ++++++++++++++++++++++++++++++++--- include/drm/drm_dp_mst_helper.h | 7 +++++-- 2 files changed, 40 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/drm_dp_mst_topology.c b/drivers/gpu/drm/drm_dp_mst_topology.c index 809959d..f50eb7b 100644 --- a/drivers/gpu/drm/drm_dp_mst_topology.c +++ b/drivers/gpu/drm/drm_dp_mst_topology.c @@ -666,7 +666,9 @@ static int build_enum_path_resources(struct drm_dp_sideband_msg_tx *msg, int por }
static int build_allocate_payload(struct drm_dp_sideband_msg_tx *msg, int port_num,
u8 vcpi, uint16_t pbn)
u8 vcpi, uint16_t pbn,
u8 number_sdp_streams,
{ struct drm_dp_sideband_msg_req_body req; memset(&req, 0, sizeof(req));u8 *sdp_stream_sink)
@@ -674,6 +676,9 @@ static int build_allocate_payload(struct drm_dp_sideband_msg_tx *msg, int port_n req.u.allocate_payload.port_number = port_num; req.u.allocate_payload.vcpi = vcpi; req.u.allocate_payload.pbn = pbn;
- req.u.allocate_payload.number_sdp_streams = number_sdp_streams;
- memcpy(req.u.allocate_payload.sdp_stream_sink, sdp_stream_sink,
drm_dp_encode_sideband_req(&req, msg); msg->path_msg = true; return 0;number_sdp_streams);
@@ -1562,6 +1567,8 @@ static int drm_dp_payload_send_msg(struct drm_dp_mst_topology_mgr *mgr, struct drm_dp_sideband_msg_tx *txmsg; struct drm_dp_mst_branch *mstb; int len, ret;
u8 sinks[DRM_DP_MAX_SDP_STREAMS];
int i;
mstb = drm_dp_get_validated_mstb_ref(mgr, port->parent); if (!mstb)
@@ -1573,10 +1580,13 @@ static int drm_dp_payload_send_msg(struct drm_dp_mst_topology_mgr *mgr, goto fail_put; }
- for (i = 0; i < port->num_sdp_streams; i++)
sinks[i] = i;
- txmsg->dst = mstb; len = build_allocate_payload(txmsg, port->port_num, id,
pbn);
pbn, port->num_sdp_streams, sinks);
drm_dp_queue_down_tx(mgr, txmsg);
@@ -2259,6 +2269,27 @@ out: EXPORT_SYMBOL(drm_dp_mst_detect_port);
/**
- drm_dp_mst_port_has_audio() - Check whether port has audio capability or not
- @mgr: manager for this port
- @port: unverified pointer to a port.
- This returns whether the port supports audio or not.
- */
+bool drm_dp_mst_port_has_audio(struct drm_dp_mst_topology_mgr *mgr,
struct drm_dp_mst_port *port)
+{
- bool ret = false;
- port = drm_dp_get_validated_port_ref(mgr, port);
- if (!port)
return ret;
- ret = port->has_audio;
- drm_dp_put_port(port);
- return ret;
+} +EXPORT_SYMBOL(drm_dp_mst_port_has_audio);
+/**
- drm_dp_mst_get_edid() - get EDID for an MST port
- @connector: toplevel connector to get EDID for
- @mgr: manager for this port
@@ -2283,6 +2314,7 @@ struct edid *drm_dp_mst_get_edid(struct drm_connector *connector, struct drm_dp_ edid = drm_get_edid(connector, &port->aux.ddc); drm_mode_connector_set_tile_property(connector); }
- port->has_audio = drm_detect_monitor_audio(edid); drm_dp_put_port(port); return edid; }
@@ -2566,7 +2598,7 @@ static void drm_dp_mst_dump_mstb(struct seq_file *m,
seq_printf(m, "%smst: %p, %d\n", prefix, mstb, mstb->num_ports); list_for_each_entry(port, &mstb->ports, next) {
seq_printf(m, "%sport: %d: ddps: %d ldps: %d, %p, conn: %p\n", prefix, port->port_num, port->ddps, port->ldps, port, port->connector);
if (port->mstb) drm_dp_mst_dump_mstb(m, port->mstb); }seq_printf(m, "%sport: %d: ddps: %d ldps: %d, sdp: %d/%d, %p, conn: %p\n", prefix, port->port_num, port->ddps, port->ldps, port->num_sdp_streams, port->num_sdp_stream_sinks, port, port->connector);
diff --git a/include/drm/drm_dp_mst_helper.h b/include/drm/drm_dp_mst_helper.h index 5340099..74b5888 100644 --- a/include/drm/drm_dp_mst_helper.h +++ b/include/drm/drm_dp_mst_helper.h @@ -94,6 +94,7 @@ struct drm_dp_mst_port { struct drm_dp_mst_topology_mgr *mgr;
struct edid *cached_edid; /* for DP logical ports - make tiling work */
bool has_audio; };
/**
@@ -215,13 +216,13 @@ struct drm_dp_sideband_msg_rx { struct drm_dp_sideband_msg_hdr initial_hdr; };
+#define DRM_DP_MAX_SDP_STREAMS 16 struct drm_dp_allocate_payload { u8 port_number; u8 number_sdp_streams; u8 vcpi; u16 pbn;
- u8 sdp_stream_sink[8];
u8 sdp_stream_sink[DRM_DP_MAX_SDP_STREAMS]; };
struct drm_dp_allocate_payload_ack_reply {
@@ -484,6 +485,8 @@ int drm_dp_mst_hpd_irq(struct drm_dp_mst_topology_mgr *mgr, u8 *esi, bool *handl
enum drm_connector_status drm_dp_mst_detect_port(struct drm_connector *connector, struct drm_dp_mst_topology_mgr *mgr, struct drm_dp_mst_port *port);
+bool drm_dp_mst_port_has_audio(struct drm_dp_mst_topology_mgr *mgr,
struct edid *drm_dp_mst_get_edid(struct drm_connector *connector, struct drm_dp_mst_topology_mgr *mgr, struct drm_dp_mst_port *port);struct drm_dp_mst_port *port);
participants (1)
-
Libin Yang