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=-14.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,INCLUDES_CR_TRAILER,INCLUDES_PATCH,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=unavailable 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 F16A8C433DB for ; Mon, 21 Dec 2020 22:31:38 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (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 A89EE225AA for ; Mon, 21 Dec 2020 22:31:38 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org A89EE225AA Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=wvrI1IK8G/3Z5o2tT0lrJRXMfS5C8ApEluWRehuHJwI=; b=KI8VhNRPHJ5PFTrTymLO/7G99 e5+AXc01ESTrxDXdInAi6MYBRQsA0K3uCeI+3+pPZSPS6/ipvl4ZfZCf7HWq/vfh/tacEZBmGy05k rH6kLYzawGMNBisNud07eN6TLCjkEXoPOAd2/liiyokea0WNPa98FPwu2b7l75eBcaa7Bc0AvYWRY j7Np4hmokR9bLWoeK6uo3KDEft9Rd6Hi4ocW9OweD5AzngDWLwWIYB8oGckaXH/Y4UdhADWawdybq qezY8JCuqUKUfjxzRCnp34NHz/38pADFjpFU2CHeCD8hM9xjs4oYdcw/2jh+ZfvJWOZRaRFQNB7qC ca0cQrTVw==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1krTgr-0008Hl-0w; Mon, 21 Dec 2020 22:30:05 +0000 Received: from mail-oi1-f179.google.com ([209.85.167.179]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1krTgn-0008Gi-R7 for linux-arm-kernel@lists.infradead.org; Mon, 21 Dec 2020 22:30:03 +0000 Received: by mail-oi1-f179.google.com with SMTP id 9so12861220oiq.3 for ; Mon, 21 Dec 2020 14:30:01 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=sisPDAs9i+2uMX2z7wDDD4p5KFYrHxtdzjTUQaGvK4Y=; b=oL2QY6hmA8gaS96Hadpot5wimMICF9LP6fGxPDFACGk8LVe3qWfuL4VIziD+X0DHo1 ib+nM5HKmP4v3oHuz2UdioxCpr5BqKlxpJ22iGBPh7vPj+lHzs8RKEE2Z5gILXMJzFYL lM/qidlU4FhmB535SuevJY4Zcv3S/wII1eVx2sNcgQbWu1Y/bXxJ4E/41EbHFLL4zdjk V1n4wRJvpkZ+caba0V79UiOO1wbm/ORh8tTi2gWSZp1CV6pYM9oHoEU9thdeDR9AWEiX 4lsBpJpdLLLwHXbwTq9EtRIaKUaErPitW0YRLOnOzua7X7j8ierCjwISp/XWgmUwYZZ6 z4bw== X-Gm-Message-State: AOAM533+yApRj8Sh7Sf28JS1zwdEdh29D2N8luvO/JthdgGFNqXLfV46 MCrJtv9+o4SIF7TGJ5pZvg== X-Google-Smtp-Source: ABdhPJxjLw2QGOVawHNevZU/xGhb5ustZhRIXB3BEerBKxzlV1lf8sqU/vPE+py1Q1soeVfWVpic4Q== X-Received: by 2002:aca:ec43:: with SMTP id k64mr12714690oih.43.1608589800845; Mon, 21 Dec 2020 14:30:00 -0800 (PST) Received: from robh.at.kernel.org ([64.188.179.253]) by smtp.gmail.com with ESMTPSA id p18sm3914243ood.48.2020.12.21.14.29.57 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 21 Dec 2020 14:29:59 -0800 (PST) Received: (nullmailer pid 687179 invoked by uid 1000); Mon, 21 Dec 2020 22:29:55 -0000 Date: Mon, 21 Dec 2020 15:29:55 -0700 From: Rob Herring To: Liu Ying Subject: Re: [PATCH 07/14] drm/bridge: imx: Add i.MX8qm/qxp display pixel link support Message-ID: <20201221222955.GA652934@robh.at.kernel.org> References: <1608199173-28760-1-git-send-email-victor.liu@nxp.com> <1608199173-28760-8-git-send-email-victor.liu@nxp.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1608199173-28760-8-git-send-email-victor.liu@nxp.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20201221_173002_068663_53E888F1 X-CRM114-Status: GOOD ( 35.31 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: devicetree@vger.kernel.org, jernej.skrabec@siol.net, Laurent.pinchart@ideasonboard.com, kernel@pengutronix.de, narmstrong@baylibre.com, airlied@linux.ie, festevam@gmail.com, s.hauer@pengutronix.de, jonas@kwiboo.se, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, kishon@ti.com, a.hajda@samsung.com, vkoul@kernel.org, linux-imx@nxp.com, daniel@ffwll.ch, mchehab@kernel.org, shawnguo@kernel.org, linux-arm-kernel@lists.infradead.org, linux-media@vger.kernel.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Thu, Dec 17, 2020 at 05:59:26PM +0800, Liu Ying wrote: > This patch adds a drm bridge driver for i.MX8qm/qxp display pixel link. > The pixel link forms a standard asynchronous linkage between > pixel sources(display controller or camera module) and pixel > consumers(imaging or displays). It consists of two distinct > functions, a pixel transfer function and a control interface. > > Signed-off-by: Liu Ying > --- > drivers/gpu/drm/bridge/imx/Kconfig | 8 + > drivers/gpu/drm/bridge/imx/Makefile | 1 + > drivers/gpu/drm/bridge/imx/imx8qxp-pixel-link.c | 411 ++++++++++++++++++++++++ > 3 files changed, 420 insertions(+) > create mode 100644 drivers/gpu/drm/bridge/imx/imx8qxp-pixel-link.c > > diff --git a/drivers/gpu/drm/bridge/imx/Kconfig b/drivers/gpu/drm/bridge/imx/Kconfig > index f1c91b6..4d1f027 100644 > --- a/drivers/gpu/drm/bridge/imx/Kconfig > +++ b/drivers/gpu/drm/bridge/imx/Kconfig > @@ -6,3 +6,11 @@ config DRM_IMX8QXP_PIXEL_COMBINER > help > Choose this to enable pixel combiner found in > Freescale i.MX8qm/qxp processors. > + > +config DRM_IMX8QXP_PIXEL_LINK > + tristate "Freescale i.MX8QM/QXP display pixel link" > + depends on OF > + select DRM_KMS_HELPER > + help > + Choose this to enable display pixel link found in > + Freescale i.MX8qm/qxp processors. > diff --git a/drivers/gpu/drm/bridge/imx/Makefile b/drivers/gpu/drm/bridge/imx/Makefile > index 7d7c8d6..c15469f 100644 > --- a/drivers/gpu/drm/bridge/imx/Makefile > +++ b/drivers/gpu/drm/bridge/imx/Makefile > @@ -1 +1,2 @@ > obj-$(CONFIG_DRM_IMX8QXP_PIXEL_COMBINER) += imx8qxp-pixel-combiner.o > +obj-$(CONFIG_DRM_IMX8QXP_PIXEL_LINK) += imx8qxp-pixel-link.o > diff --git a/drivers/gpu/drm/bridge/imx/imx8qxp-pixel-link.c b/drivers/gpu/drm/bridge/imx/imx8qxp-pixel-link.c > new file mode 100644 > index 00000000..1af7467 > --- /dev/null > +++ b/drivers/gpu/drm/bridge/imx/imx8qxp-pixel-link.c > @@ -0,0 +1,411 @@ > +// SPDX-License-Identifier: GPL-2.0+ > + > +/* > + * Copyright 2020 NXP > + */ > + > +#include > +#include > +#include > +#include > +#include > + > +#include > +#include > +#include > + > +#include > + > +#define DRIVER_NAME "imx8qxp-display-pixel-link" > +#define PL_MAX_MST_ADDR 3 > + > +struct imx8qxp_pixel_link { > + struct drm_bridge bridge; > + struct drm_bridge *next_bridge; > + struct device *dev; > + struct imx_sc_ipc *ipc_handle; > + int id; > + int stream_id; > + int dc_id; > + u32 sink_rsc; > + u32 mst_addr; > + u8 mst_addr_ctrl; > + u8 mst_en_ctrl; > + u8 mst_vld_ctrl; > + u8 sync_ctrl; > +}; > + > +#define DEFINE_IMX8QXP_PIXEL_LINK_ENABLE_CTRL(name) \ > +static void \ > +imx8qxp_pixel_link_enable_##name(struct imx8qxp_pixel_link *pl) \ > +{ \ > + int ret = imx_sc_misc_set_control(pl->ipc_handle, pl->sink_rsc, \ > + pl->name##_ctrl, true); \ > + if (ret) \ > + DRM_DEV_ERROR(pl->dev, \ > + "failed to enable DC%d stream%d pixel link " \ > + #name ": %d\n", pl->dc_id, pl->stream_id, ret); \ > +} > + > +DEFINE_IMX8QXP_PIXEL_LINK_ENABLE_CTRL(mst_en) > +DEFINE_IMX8QXP_PIXEL_LINK_ENABLE_CTRL(mst_vld) > +DEFINE_IMX8QXP_PIXEL_LINK_ENABLE_CTRL(sync) > + > +#define DEFINE_IMX8QXP_PIXEL_LINK_DISABLE_CTRL(name) \ > +static int \ > +imx8qxp_pixel_link_disable_##name(struct imx8qxp_pixel_link *pl) \ > +{ \ > + int ret = imx_sc_misc_set_control(pl->ipc_handle, pl->sink_rsc, \ > + pl->name##_ctrl, false); \ > + if (ret) \ > + DRM_DEV_ERROR(pl->dev, \ > + "failed to disable DC%d stream%d pixel link " \ > + #name ": %d\n", pl->dc_id, pl->stream_id, ret); \ > + \ > + return ret; \ > +} > + > +DEFINE_IMX8QXP_PIXEL_LINK_DISABLE_CTRL(mst_en) > +DEFINE_IMX8QXP_PIXEL_LINK_DISABLE_CTRL(mst_vld) > +DEFINE_IMX8QXP_PIXEL_LINK_DISABLE_CTRL(sync) > + > +static void imx8qxp_pixel_link_set_mst_addr(struct imx8qxp_pixel_link *pl) > +{ > + int ret; > + > + ret = imx_sc_misc_set_control(pl->ipc_handle, > + pl->sink_rsc, pl->mst_addr_ctrl, > + pl->mst_addr); > + if (ret) > + DRM_DEV_ERROR(pl->dev, > + "failed to set DC%d stream%d pixel link mst addr(%u): %d\n", > + pl->dc_id, pl->stream_id, pl->mst_addr, ret); > +} > + > +static int imx8qxp_pixel_link_bridge_attach(struct drm_bridge *bridge, > + enum drm_bridge_attach_flags flags) > +{ > + struct imx8qxp_pixel_link *pl = bridge->driver_private; > + > + if (!(flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR)) { > + DRM_DEV_ERROR(pl->dev, > + "do not support creating a drm_connector\n"); > + return -EINVAL; > + } > + > + if (!bridge->encoder) { > + DRM_DEV_ERROR(pl->dev, "missing encoder\n"); > + return -ENODEV; > + } > + > + return drm_bridge_attach(bridge->encoder, > + pl->next_bridge, bridge, > + DRM_BRIDGE_ATTACH_NO_CONNECTOR); > +} > + > +static void > +imx8qxp_pixel_link_bridge_mode_set(struct drm_bridge *bridge, > + const struct drm_display_mode *mode, > + const struct drm_display_mode *adjusted_mode) > +{ > + struct imx8qxp_pixel_link *pl = bridge->driver_private; > + > + imx8qxp_pixel_link_set_mst_addr(pl); > +} > + > +static void imx8qxp_pixel_link_bridge_atomic_enable(struct drm_bridge *bridge, > + struct drm_bridge_state *old_bridge_state) > +{ > + struct imx8qxp_pixel_link *pl = bridge->driver_private; > + > + imx8qxp_pixel_link_enable_mst_en(pl); > + imx8qxp_pixel_link_enable_mst_vld(pl); > + imx8qxp_pixel_link_enable_sync(pl); > +} > + > +static void imx8qxp_pixel_link_bridge_atomic_disable(struct drm_bridge *bridge, > + struct drm_bridge_state *old_bridge_state) > +{ > + struct imx8qxp_pixel_link *pl = bridge->driver_private; > + > + imx8qxp_pixel_link_disable_mst_en(pl); > + imx8qxp_pixel_link_disable_mst_vld(pl); > + imx8qxp_pixel_link_disable_sync(pl); > +} > + > +static const u32 imx8qxp_pixel_link_bus_output_fmts[] = { > + MEDIA_BUS_FMT_RGB888_1X36_CPADLO, > + MEDIA_BUS_FMT_RGB666_1X36_CPADLO, > +}; > + > +static bool imx8qxp_pixel_link_bus_output_fmt_supported(u32 fmt) > +{ > + int i; > + > + for (i = 0; i < ARRAY_SIZE(imx8qxp_pixel_link_bus_output_fmts); i++) { > + if (imx8qxp_pixel_link_bus_output_fmts[i] == fmt) > + return true; > + } > + > + return false; > +} > + > +static u32 * > +imx8qxp_pixel_link_bridge_atomic_get_input_bus_fmts(struct drm_bridge *bridge, > + struct drm_bridge_state *bridge_state, > + struct drm_crtc_state *crtc_state, > + struct drm_connector_state *conn_state, > + u32 output_fmt, > + unsigned int *num_input_fmts) > +{ > + u32 *input_fmts; > + > + if (!imx8qxp_pixel_link_bus_output_fmt_supported(output_fmt)) > + return NULL; > + > + *num_input_fmts = 1; > + > + input_fmts = kmalloc(sizeof(*input_fmts), GFP_KERNEL); > + if (!input_fmts) > + return NULL; > + > + input_fmts[0] = output_fmt; > + > + return input_fmts; > +} > + > +static u32 * > +imx8qxp_pixel_link_bridge_atomic_get_output_bus_fmts(struct drm_bridge *bridge, > + struct drm_bridge_state *bridge_state, > + struct drm_crtc_state *crtc_state, > + struct drm_connector_state *conn_state, > + unsigned int *num_output_fmts) > +{ > + *num_output_fmts = ARRAY_SIZE(imx8qxp_pixel_link_bus_output_fmts); > + return kmemdup(imx8qxp_pixel_link_bus_output_fmts, > + sizeof(imx8qxp_pixel_link_bus_output_fmts), GFP_KERNEL); > +} > + > +static const struct drm_bridge_funcs imx8qxp_pixel_link_bridge_funcs = { > + .atomic_duplicate_state = drm_atomic_helper_bridge_duplicate_state, > + .atomic_destroy_state = drm_atomic_helper_bridge_destroy_state, > + .atomic_reset = drm_atomic_helper_bridge_reset, > + .attach = imx8qxp_pixel_link_bridge_attach, > + .mode_set = imx8qxp_pixel_link_bridge_mode_set, > + .atomic_enable = imx8qxp_pixel_link_bridge_atomic_enable, > + .atomic_disable = imx8qxp_pixel_link_bridge_atomic_disable, > + .atomic_get_input_bus_fmts = > + imx8qxp_pixel_link_bridge_atomic_get_input_bus_fmts, > + .atomic_get_output_bus_fmts = > + imx8qxp_pixel_link_bridge_atomic_get_output_bus_fmts, > +}; > + > +static struct drm_bridge * > +imx8qxp_pixel_link_find_next_bridge(struct imx8qxp_pixel_link *pl) > +{ > + struct device_node *np = pl->dev->of_node; > + struct device_node *port, *ep, *remote; > + struct of_endpoint endpoint; > + struct drm_bridge *next_bridge[2], *err_bridge; > + u32 port_id, vld_port_id; > + int port_cnt = 0, ep_cnt = 0; > + int bridge_sel = 0; /* select the first next bridge by default */ > + int ret; > + > + for (port_id = 1; port_id <= PL_MAX_MST_ADDR + 1; port_id++) { > + port = of_graph_get_port_by_id(np, port_id); > + if (!port) > + continue; > + > + if (of_device_is_available(port)) { > + port_cnt++; > + vld_port_id = port_id; > + } > + > + of_node_put(port); > + } > + > + /* check the number of available output ports */ > + if (port_cnt == 0) { > + DRM_DEV_ERROR(pl->dev, "no available output port\n"); > + return ERR_PTR(-ENODEV); > + } else if (port_cnt > 1) { > + DRM_DEV_ERROR(pl->dev, "invalid available output ports: %d\n", > + port_cnt); > + return ERR_PTR(-ENOTSUPP); > + } > + > + port = of_graph_get_port_by_id(np, vld_port_id); > + if (!port) { > + DRM_DEV_ERROR(pl->dev, "failed to get output port\n"); > + return ERR_PTR(-ENODEV); > + } > + > + for_each_available_child_of_node(port, ep) { for_each_endpoint_of_node > + /* 2 available endpoints at most */ > + if (++ep_cnt > 2) { > + DRM_DEV_ERROR(pl->dev, > + "invalid available endpoints: %d\n", > + ep_cnt); Not really the kernel's job to validate the DT. Just get endpoints 0 and 1 if those are the 2 you want. > + err_bridge = ERR_PTR(-ENOTSUPP); > + of_node_put(ep); > + goto err; > + } > + > + ret = of_graph_parse_endpoint(ep, &endpoint); > + if (ret) { > + DRM_DEV_ERROR(pl->dev, "failed to parse endpoint: %d\n", > + ret); > + err_bridge = ERR_PTR(ret); > + of_node_put(ep); > + goto err; > + } > + > + remote = of_graph_get_remote_port_parent(ep); > + if (!remote || !of_device_is_available(remote)) { > + DRM_DEV_ERROR(pl->dev, > + "endpoint%u has no available remote\n", > + endpoint.id); > + of_node_put(remote); > + of_node_put(ep); > + err_bridge = ERR_PTR(-ENODEV); > + goto err; > + } else if (!of_device_is_available(remote->parent)) { > + DRM_DEV_ERROR(pl->dev, > + "endpoint%u remote parent is not available\n", > + endpoint.id); > + of_node_put(remote); > + of_node_put(ep); > + err_bridge = ERR_PTR(-ENODEV); > + goto err; > + } Rewrite all this in terms of of_graph_get_remote_node(). It takes care of of_device_is_available checking for you. > + > + next_bridge[ep_cnt - 1] = of_drm_find_bridge(remote); > + if (!next_bridge[ep_cnt - 1]) { > + of_node_put(remote); > + of_node_put(ep); > + err_bridge = ERR_PTR(-EPROBE_DEFER); > + goto err; > + } > + > + /* specially select the next bridge with companion PXL2DPI */ > + if (of_find_property(remote, "fsl,companion-pxl2dpi", NULL)) > + bridge_sel = ep_cnt - 1; > + > + of_node_put(remote); > + } > + > + pl->mst_addr = vld_port_id - 1; > + > + of_node_put(port); > + > + return next_bridge[bridge_sel]; > +err: > + of_node_put(port); > + > + return err_bridge; > +} > + > +static int imx8qxp_pixel_link_bridge_probe(struct platform_device *pdev) > +{ > + struct imx8qxp_pixel_link *pl; > + struct device *dev = &pdev->dev; > + struct device_node *np = dev->of_node; > + int ret; > + > + pl = devm_kzalloc(dev, sizeof(*pl), GFP_KERNEL); > + if (!pl) > + return -ENOMEM; > + > + ret = imx_scu_get_handle(&pl->ipc_handle); > + if (ret) { > + if (ret != -EPROBE_DEFER) > + DRM_DEV_ERROR(dev, "failed to get SCU ipc handle: %d\n", > + ret); > + return ret; > + } > + > + pl->id = of_alias_get_id(np, "dc_pl"); What's this needed for? I doubt aliases is the right thing to use here. Rob _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel