From patchwork Wed Mar 7 00:24:40 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Laurent Pinchart X-Patchwork-Id: 10263227 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 82FA960211 for ; Wed, 7 Mar 2018 00:27:28 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 73A842922F for ; Wed, 7 Mar 2018 00:27:28 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 682F429231; Wed, 7 Mar 2018 00:27:28 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.1 required=2.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_MED,T_DKIM_INVALID autolearn=ham version=3.3.1 Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id EE9BE2922F for ; Wed, 7 Mar 2018 00:27:27 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 2BB786E592; Wed, 7 Mar 2018 00:24:50 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from galahad.ideasonboard.com (galahad.ideasonboard.com [185.26.127.97]) by gabe.freedesktop.org (Postfix) with ESMTPS id 8C3296E504 for ; Wed, 7 Mar 2018 00:24:45 +0000 (UTC) Received: from avalon.bb.dnainternet.fi (dfj612ybrt5fhg77mgycy-3.rev.dnainternet.fi [IPv6:2001:14ba:21f5:5b00:2e86:4862:ef6a:2804]) by galahad.ideasonboard.com (Postfix) with ESMTPSA id 20B9C22AFC; Wed, 7 Mar 2018 01:22:24 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1520382144; bh=zBemhvaeVjuB5ClmA7aSEQwxlBA+hOb73erc1l2yc+M=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=fFpStTThGDNsigk8fXNcyMFjxSxhcfejRRGsGbTJI8+iVX+kV7QDKy4snw3REIP/5 eAIk4Eu7KRFlefulNygwK3/3eQMMSCce8t24bf0USoHLbBKRf+c4v3PRylAUejhMZE xiK6SnABqYoNaFPQ3qfgcdXIvwq2sdioHfrm+IUY= From: Laurent Pinchart To: dri-devel@lists.freedesktop.org Subject: [PATCH/RFC 37/60] drm/omap: displays: Don't cast dssdev to panel data unnecessarily Date: Wed, 7 Mar 2018 02:24:40 +0200 Message-Id: <20180307002503.19455-38-laurent.pinchart@ideasonboard.com> X-Mailer: git-send-email 2.16.1 In-Reply-To: <20180307002503.19455-1-laurent.pinchart@ideasonboard.com> References: <20180307002503.19455-1-laurent.pinchart@ideasonboard.com> X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Tomi Valkeinen MIME-Version: 1.0 Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" X-Virus-Scanned: ClamAV using ClamSMTP The connect handle of the analog TV and HDMI connectors casts the dssdev to panel data only to then access fields of the panel data that are also present in the dssdev. Remove the cast and use dssdev directly. Signed-off-by: Laurent Pinchart --- drivers/gpu/drm/omapdrm/displays/connector-analog-tv.c | 5 ++--- drivers/gpu/drm/omapdrm/displays/connector-hdmi.c | 5 ++--- 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/omapdrm/displays/connector-analog-tv.c b/drivers/gpu/drm/omapdrm/displays/connector-analog-tv.c index 4f32c8ddcfbd..eab898b5bf0f 100644 --- a/drivers/gpu/drm/omapdrm/displays/connector-analog-tv.c +++ b/drivers/gpu/drm/omapdrm/displays/connector-analog-tv.c @@ -43,13 +43,12 @@ static const struct videomode tvc_pal_vm = { static int tvc_connect(struct omap_dss_device *dssdev) { - struct panel_drv_data *ddata = to_panel_data(dssdev); struct omap_dss_device *src; int r; - src = omapdss_of_find_connected_device(ddata->dev->of_node, 0); + src = omapdss_of_find_connected_device(dssdev->dev->of_node, 0); if (IS_ERR_OR_NULL(src)) { - dev_err(ddata->dev, "failed to find video source\n"); + dev_err(dssdev->dev, "failed to find video source\n"); return src ? PTR_ERR(src) : -EINVAL; } diff --git a/drivers/gpu/drm/omapdrm/displays/connector-hdmi.c b/drivers/gpu/drm/omapdrm/displays/connector-hdmi.c index 1df7ffd8b97d..360bc9f501a2 100644 --- a/drivers/gpu/drm/omapdrm/displays/connector-hdmi.c +++ b/drivers/gpu/drm/omapdrm/displays/connector-hdmi.c @@ -53,13 +53,12 @@ struct panel_drv_data { static int hdmic_connect(struct omap_dss_device *dssdev) { - struct panel_drv_data *ddata = to_panel_data(dssdev); struct omap_dss_device *src; int r; - src = omapdss_of_find_connected_device(ddata->dev->of_node, 0); + src = omapdss_of_find_connected_device(dssdev->dev->of_node, 0); if (IS_ERR_OR_NULL(src)) { - dev_err(ddata->dev, "failed to find video source\n"); + dev_err(dssdev->dev, "failed to find video source\n"); return src ? PTR_ERR(src) : -EINVAL; }