From patchwork Fri Apr 6 14:19:35 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Linus Walleij X-Patchwork-Id: 132917 Delivered-To: patch@linaro.org Received: by 10.46.84.29 with SMTP id i29csp844631ljb; Fri, 6 Apr 2018 07:19:47 -0700 (PDT) X-Google-Smtp-Source: AIpwx49sMdC+dcB0+58VxEBg00hJaGGBc488g+b1Fj3kWM60f/FLXYqZFhSaFitLZMHYDb9xvSZO X-Received: by 2002:a17:902:8f8f:: with SMTP id z15-v6mr27010548plo.368.1523024387409; Fri, 06 Apr 2018 07:19:47 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1523024387; cv=none; d=google.com; s=arc-20160816; b=RDd9oF0sWzNCj0sOWG4fUgcHKWm5Qmx7JnLHMwhIXnXYdnouyEwCC6+Le5H6XVN5OB HYJww7Qi7rWsAex0TCS6YYGD5uowAAr0gyS0OmDfpXx7RjWZvrDbouTZhI0qPvHgrtIJ DtBGYtjyjq22ZOG8We50iuQ5UwEQ20lWjdYUU6FFpjhlH20lm04R6aVdNbwHNrDI5HEU iPHIYZc54bL/vygZ8fWUsc+0Mub28tPNQKZiKMM9tKoyLXr0RfMVV74yVVC25IZVgLXd 8bH9Vd/vy/rFV535n57BGBNh3bYJucqyQtYORa3yGaOpmc06ETWLzurLTnFmZIV1ZzH7 ustA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:content-transfer-encoding:cc:list-subscribe :list-help:list-post:list-archive:list-unsubscribe:list-id :precedence:mime-version:references:in-reply-to:message-id:date :subject:to:from:dkim-signature:delivered-to :arc-authentication-results; bh=3H+a0AWwTMdHvQx5kP9o1aI9aMNUZ6KJ00ues46kMDU=; b=oC5rnbFjpEobL9bzcExKVae+J7YkgHfL+2oKSZaHq6/pBXLCbz0XDCHoqrgeqn7KJt aP3aC2Cc2fI86MmYQMQcC6h6MRC0YwLA4SB0u0x3CpGkVY+7ClEBCz9opFbr8BylA+Ee 2Zt4lWT7GfjKIDc7RwQfaec7d88NoiS3i3ek0Ihjx8LDN47nJURMKKhnd7YgAv0yTYt5 r8uWbINDNpuuHMoEBhjD4z3i0Y9ZW2NUHHORmE426gBFTPUoREHXJkjAozuUAiwke1Jt jlnlYd+6ZAJDX8A8t1Hp4lDbg15UqLoQcWrWVT6nqPuJhAQK4vvb1lqFg7hQ9HziuO++ BvXw== ARC-Authentication-Results: i=1; mx.google.com; dkim=neutral (body hash did not verify) header.i=@linaro.org header.s=google header.b=H5dV0PTC; spf=pass (google.com: best guess record for domain of dri-devel-bounces@lists.freedesktop.org designates 131.252.210.177 as permitted sender) smtp.mailfrom=dri-devel-bounces@lists.freedesktop.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from gabe.freedesktop.org (gabe.freedesktop.org. [131.252.210.177]) by mx.google.com with ESMTPS id b89-v6si10902630plb.262.2018.04.06.07.19.47 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Fri, 06 Apr 2018 07:19:47 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of dri-devel-bounces@lists.freedesktop.org designates 131.252.210.177 as permitted sender) client-ip=131.252.210.177; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@linaro.org header.s=google header.b=H5dV0PTC; spf=pass (google.com: best guess record for domain of dri-devel-bounces@lists.freedesktop.org designates 131.252.210.177 as permitted sender) smtp.mailfrom=dri-devel-bounces@lists.freedesktop.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 39F026E97D; Fri, 6 Apr 2018 14:19:46 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from mail-lf0-x243.google.com (mail-lf0-x243.google.com [IPv6:2a00:1450:4010:c07::243]) by gabe.freedesktop.org (Postfix) with ESMTPS id BDD2D6E988 for ; Fri, 6 Apr 2018 14:19:42 +0000 (UTC) Received: by mail-lf0-x243.google.com with SMTP id p142-v6so792860lfd.6 for ; Fri, 06 Apr 2018 07:19:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=0+zmRrO9BO89uqK08WJUwFs7r0N/cCmhTWuQadd9BYI=; b=H5dV0PTC17048nAk/ISKxdiIt+ESxOdCv2jXwO4+gkT3S5yey/5B+scZV0MzIbgMPw ORU7zkEJALQzQFhxx8kJkx+2Df2nqiU4UgHQjrYma8nAFXHuOAH9S55wSpRE3u9B30xT S3gLFazKjFoYrDMV8A7TaDqyLq/onX+teWc00= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=0+zmRrO9BO89uqK08WJUwFs7r0N/cCmhTWuQadd9BYI=; b=tX2bkpqqp0QeVjy4Wi7UiiL5iXDGFcA79uMIlh4qGjoV4HkILjxiMyEdbUMEA0fkmq HfAOnNhm1uBR7iLsTsuEhD0X0DsaKcOMZklPcBPP9DsKdCu+t6+MndYr8NLZCMWeqjDL O2gED1kwAY6Ii4I/3I/dDFAKexc201PyUP8AYNuJNZ9AXyqkh8U/OipFRT/5jiNUnoI8 aF1c62lLts0KnG+99vO2mgy7WHJz3YWx4wzXc5v3RGdJ8FTe4aQElMc1w6jPujhLA1gn l+veKEHYlgnQq0ziM9KiFJ5Etn3aqfvtRVivic8h5gnMl9z/zN37BJkTaDA21F+JRopf dRGA== X-Gm-Message-State: ALQs6tD+KGh3fM8AZ+9PsPqk1Fb6ZRE1T2bb9RHxiG//YQa2AM+5hkS6 Uf/ZckkwCuCS3yCycQTUhXspsA== X-Received: by 2002:a19:6919:: with SMTP id e25-v6mr16195720lfc.52.1523024381061; Fri, 06 Apr 2018 07:19:41 -0700 (PDT) Received: from genomnajs.ideon.se ([85.235.10.227]) by smtp.gmail.com with ESMTPSA id e8-v6sm2098137lfc.88.2018.04.06.07.19.39 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Fri, 06 Apr 2018 07:19:40 -0700 (PDT) From: Linus Walleij To: Daniel Vetter , Jani Nikula , Sean Paul , Eric Anholt , Liviu Dudau Subject: [PATCH 2/2 v2] drm/pl111: Enable device-specific assigned memory Date: Fri, 6 Apr 2018 16:19:35 +0200 Message-Id: <20180406141935.6801-2-linus.walleij@linaro.org> X-Mailer: git-send-email 2.14.3 In-Reply-To: <20180406141935.6801-1-linus.walleij@linaro.org> References: <20180406141935.6801-1-linus.walleij@linaro.org> MIME-Version: 1.0 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: Mali DP Maintainers , linux-arm-kernel@lists.infradead.org, dri-devel@lists.freedesktop.org Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" The Versatile Express has 8 MB of dedicated video RAM (VRAM) on the motherboard, which is what we should be using for the PL111 if available. On this platform, the memory backplane is constructed so that only this memory will work properly with the CLCD on the motherboard, using any other memory region just gives random snow on the display. The CA9 Versatile Express also has a PL111 instance on its core tile. This is OK, it has been tested with the motherboard VRAM and that works just as fine as regular CMA memory. The memory is assigned to the device using the memory-region device tree property and a "shared-dma-pool" reserved memory pool like this: reserved-memory { #address-cells = <1>; #size-cells = <1>; ranges; vram: vram@48000000 { compatible = "shared-dma-pool"; reg = <0x48000000 0x00800000>; no-map; }; }; clcd@1f000 { compatible = "arm,pl111", "arm,primecell"; (...) memory-region = <&vram>; }ยท; Cc: Liviu Dudau Cc: Mali DP Maintainers Signed-off-by: Linus Walleij --- ChangeLog v1->v2: - Make sure to also call of_reserved_mem_device_release() at remove() and errorpath. --- drivers/gpu/drm/pl111/pl111_drv.c | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/pl111/pl111_drv.c b/drivers/gpu/drm/pl111/pl111_drv.c index 4621259d5387..bc57c15d9fe4 100644 --- a/drivers/gpu/drm/pl111/pl111_drv.c +++ b/drivers/gpu/drm/pl111/pl111_drv.c @@ -60,6 +60,7 @@ #include #include #include +#include #include #include @@ -257,6 +258,10 @@ static int pl111_amba_probe(struct amba_device *amba_dev, drm->dev_private = priv; priv->variant = variant; + ret = of_reserved_mem_device_init(dev); + if (!ret) + dev_info(dev, "using device-specific reserved memory\n"); + if (of_property_read_u32(dev->of_node, "max-memory-bandwidth", &priv->memory_bw)) { dev_info(dev, "no max memory bandwidth specified, assume unlimited\n"); @@ -275,7 +280,8 @@ static int pl111_amba_probe(struct amba_device *amba_dev, priv->regs = devm_ioremap_resource(dev, &amba_dev->res); if (IS_ERR(priv->regs)) { dev_err(dev, "%s failed mmio\n", __func__); - return PTR_ERR(priv->regs); + ret = PTR_ERR(priv->regs); + goto mem_rel; } /* This may override some variant settings */ @@ -305,11 +311,15 @@ static int pl111_amba_probe(struct amba_device *amba_dev, dev_unref: drm_dev_unref(drm); +mem_rel: + of_reserved_mem_device_release(dev); + return ret; } static int pl111_amba_remove(struct amba_device *amba_dev) { + struct device *dev = &amba_dev->dev; struct drm_device *drm = amba_get_drvdata(amba_dev); struct pl111_drm_dev_private *priv = drm->dev_private; @@ -319,6 +329,7 @@ static int pl111_amba_remove(struct amba_device *amba_dev) drm_panel_bridge_remove(priv->bridge); drm_mode_config_cleanup(drm); drm_dev_unref(drm); + of_reserved_mem_device_release(dev); return 0; }