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=-6.9 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED 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 133ECC2BA18 for ; Fri, 3 Apr 2020 22:23:05 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id CC4CB20659 for ; Fri, 3 Apr 2020 22:23:04 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="fchoumnb" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728692AbgDCWXD (ORCPT ); Fri, 3 Apr 2020 18:23:03 -0400 Received: from us-smtp-1.mimecast.com ([207.211.31.81]:56448 "EHLO us-smtp-delivery-1.mimecast.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1728416AbgDCWXD (ORCPT ); Fri, 3 Apr 2020 18:23:03 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1585952582; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding; bh=eeGpbZMZxtLmrqNZB2P89d1keB123ujHn4Gr2i5vxGY=; b=fchoumnbmFW4mcFiTjvnMlnn6vNBaPCOIiXptktHMlV1RWEFB+s33rAJZJwNX5+sHvEmn3 dk4QNViQ5rIEqYg9wloCRuVMdoCf+ExebRhMEtDjeMwOPbLfACNH4k/zZUXdBiBoriDTUp VH+m12CQPxt8CrRaabEfIxu5wkUT7y0= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-27-pEFRls_hPHC28_QihEwNUA-1; Fri, 03 Apr 2020 18:22:58 -0400 X-MC-Unique: pEFRls_hPHC28_QihEwNUA-1 Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 85D328017CE; Fri, 3 Apr 2020 22:22:55 +0000 (UTC) Received: from Ruby.redhat.com (ovpn-117-203.rdu2.redhat.com [10.10.117.203]) by smtp.corp.redhat.com (Postfix) with ESMTP id D0F0626DE0; Fri, 3 Apr 2020 22:22:52 +0000 (UTC) From: Lyude Paul To: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org Cc: "Lee, Shawn C" , Sean Paul , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Daniel Vetter , Jani Nikula , Joonas Lahtinen , Rodrigo Vivi , =?UTF-8?q?Ville=20Syrj=C3=A4l=C3=A4?= , Imre Deak , Wambui Karuga , Chris Wilson , =?UTF-8?q?Jos=C3=A9=20Roberto=20de=20Souza?= , Lucas De Marchi , linux-kernel@vger.kernel.org Subject: [PATCH] drm/dp_mst: Remove drm_dp_mst_has_audio() Date: Fri, 3 Apr 2020 18:22:41 -0400 Message-Id: <20200403222251.937740-1-lyude@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 Content-Transfer-Encoding: quoted-printable Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Drive-by fix I noticed the other day - drm_dp_mst_has_audio() only ever made sense back when we still had to validate ports before accessing them in order to (attempt to) avoid NULL dereferences. Since we have proper reference counting that guarantees we always can safely access the MST port, there's no use in keeping this function around as all it does is validate the port pointer before checking the audio status. Note - drm_dp_mst_port->has_audio is technically protected by drm_device->mode_config.connection_mutex, since it's only ever updated from drm_dp_mst_get_edid(). Additionally, we change the declaration for port in struct intel_connector to be properly typed, so we can directly access it. Cc: "Lee, Shawn C" Cc: Sean Paul Signed-off-by: Lyude Paul --- drivers/gpu/drm/drm_dp_mst_topology.c | 21 ------------------- .../drm/i915/display/intel_display_debugfs.c | 10 ++------- .../drm/i915/display/intel_display_types.h | 2 +- drivers/gpu/drm/i915/display/intel_dp_mst.c | 3 +-- include/drm/drm_dp_mst_helper.h | 2 -- 5 files changed, 4 insertions(+), 34 deletions(-) diff --git a/drivers/gpu/drm/drm_dp_mst_topology.c b/drivers/gpu/drm/drm_= dp_mst_topology.c index 1ff49547b2e8..129126091e90 100644 --- a/drivers/gpu/drm/drm_dp_mst_topology.c +++ b/drivers/gpu/drm/drm_dp_mst_topology.c @@ -4063,27 +4063,6 @@ drm_dp_mst_detect_port(struct drm_connector *conne= ctor, } EXPORT_SYMBOL(drm_dp_mst_detect_port); =20 -/** - * 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 =3D false; - - port =3D drm_dp_mst_topology_get_port_validated(mgr, port); - if (!port) - return ret; - ret =3D port->has_audio; - drm_dp_mst_topology_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 diff --git a/drivers/gpu/drm/i915/display/intel_display_debugfs.c b/drive= rs/gpu/drm/i915/display/intel_display_debugfs.c index 424f4e52f783..9f736420d83f 100644 --- a/drivers/gpu/drm/i915/display/intel_display_debugfs.c +++ b/drivers/gpu/drm/i915/display/intel_display_debugfs.c @@ -631,15 +631,9 @@ static void intel_dp_info(struct seq_file *m, } =20 static void intel_dp_mst_info(struct seq_file *m, - struct intel_connector *intel_connector) + struct intel_connector *intel_connector) { - struct intel_encoder *intel_encoder =3D intel_attached_encoder(intel_co= nnector); - struct intel_dp_mst_encoder *intel_mst =3D - enc_to_mst(intel_encoder); - struct intel_digital_port *intel_dig_port =3D intel_mst->primary; - struct intel_dp *intel_dp =3D &intel_dig_port->dp; - bool has_audio =3D drm_dp_mst_port_has_audio(&intel_dp->mst_mgr, - intel_connector->port); + bool has_audio =3D intel_connector->port->has_audio; =20 seq_printf(m, "\taudio support: %s\n", yesno(has_audio)); } diff --git a/drivers/gpu/drm/i915/display/intel_display_types.h b/drivers= /gpu/drm/i915/display/intel_display_types.h index 2bedd626c686..1de7bef0a49b 100644 --- a/drivers/gpu/drm/i915/display/intel_display_types.h +++ b/drivers/gpu/drm/i915/display/intel_display_types.h @@ -436,7 +436,7 @@ struct intel_connector { state of connector->polled in case hotplug storm detection changes i= t */ u8 polled; =20 - void *port; /* store this opaque as its illegal to dereference it */ + struct drm_dp_mst_port *port; =20 struct intel_dp *mst_port; =20 diff --git a/drivers/gpu/drm/i915/display/intel_dp_mst.c b/drivers/gpu/dr= m/i915/display/intel_dp_mst.c index 61605eb8c2af..c35efc9e628d 100644 --- a/drivers/gpu/drm/i915/display/intel_dp_mst.c +++ b/drivers/gpu/drm/i915/display/intel_dp_mst.c @@ -114,8 +114,7 @@ static int intel_dp_mst_compute_config(struct intel_e= ncoder *encoder, =20 if (intel_conn_state->force_audio =3D=3D HDMI_AUDIO_AUTO) pipe_config->has_audio =3D - drm_dp_mst_port_has_audio(&intel_dp->mst_mgr, - connector->port); + connector->port->has_audio; else pipe_config->has_audio =3D intel_conn_state->force_audio =3D=3D HDMI_AUDIO_ON; diff --git a/include/drm/drm_dp_mst_helper.h b/include/drm/drm_dp_mst_hel= per.h index 7af51c947b81..2d7c26592c05 100644 --- a/include/drm/drm_dp_mst_helper.h +++ b/include/drm/drm_dp_mst_helper.h @@ -732,8 +732,6 @@ drm_dp_mst_detect_port(struct drm_connector *connecto= r, struct drm_dp_mst_topology_mgr *mgr, struct drm_dp_mst_port *port); =20 -bool drm_dp_mst_port_has_audio(struct drm_dp_mst_topology_mgr *mgr, - struct drm_dp_mst_port *port); struct edid *drm_dp_mst_get_edid(struct drm_connector *connector, struct= drm_dp_mst_topology_mgr *mgr, struct drm_dp_mst_port *port); =20 =20 --=20 2.25.1