From patchwork Mon Apr 23 07:20:28 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Linus Walleij X-Patchwork-Id: 133967 Delivered-To: patch@linaro.org Received: by 10.46.151.6 with SMTP id r6csp674927lji; Mon, 23 Apr 2018 00:20:36 -0700 (PDT) X-Google-Smtp-Source: AIpwx48R+X0plPEMr5gRuDKotHSLIErLlJpuWm4Va4bYqrqmctK5RT4PBTnnlsDIPgkCJpwkHQFj X-Received: by 2002:a17:902:a70c:: with SMTP id w12-v6mr19490995plq.74.1524468036157; Mon, 23 Apr 2018 00:20:36 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1524468036; cv=none; d=google.com; s=arc-20160816; b=SKykwQgHTtZWaCkOAfwXzQV/TW13UJvU33DmzaV456nAf8do7YUxnER2dz3e+eANbz Uo4e7J+Wnvl0riym8lRqqPuNiiaNozpn69l048RhoDB5PyITRqbwGY8uKgBlEGqRLanJ GY4recjT8po7xHNDctF2I1AsrV7YjSo6xbkZ2qcxdkNaQ03s442FfGbWTKuS/EQMlQDx FFcEcXJBY2ZV64BW+070BfICgxZDjt5ZKVYWvNos63Te+0mXRAAawjJnKL7ldfMjVjAH ACbDA/ftxgNiWbjUVGs+8hT+/s0SIwln9M1yHdYTShbeQ8dAh01KYiWrWsh3jO7Uvkin bd5A== 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:message-id:date:subject:to:from :dkim-signature:delivered-to:arc-authentication-results; bh=Ob165D8AVO4VRxs3TWacsBcGdbFpuy4Zp5T60JoXFIE=; b=S+DxaSBncZv1zTB1e40YSO3eHV5od2tbU2+ar8EKWNRKcWqbvuS6V96H6vv7okVmct xMpPqUzh2P4jw8gkqOx1p1uYzK7XjILjfDcUnv61spxu7C/jqi9SuZAyHXLzP4oUcMWJ JukBnOmDdo89PEm3j1VY3VE2rtNQJCyH82ENZpC5cNIYIKiPwljrPqMwjZpzatybzEe2 IxPMupOF0XUMOJknwCmWjX7gtL4RUVr0/khfwqthsRiGX6/ONr6tLNtcAkvqDtWPcMRr DL77bNL7ELV8PgCf0uri+5GcuW619uCxGF2PGJwFevLNvUS7MRg15kHDJdDnuts23TNq mphg== ARC-Authentication-Results: i=1; mx.google.com; dkim=neutral (body hash did not verify) header.i=@linaro.org header.s=google header.b=R7SGXmPa; 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 33-v6si11816850plb.19.2018.04.23.00.20.35 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Mon, 23 Apr 2018 00:20:36 -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=R7SGXmPa; 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 C8E926E161; Mon, 23 Apr 2018 07:20:34 +0000 (UTC) X-Original-To: dri-devel@lists.freedesktop.org Delivered-To: dri-devel@lists.freedesktop.org Received: from mail-lf0-x244.google.com (mail-lf0-x244.google.com [IPv6:2a00:1450:4010:c07::244]) by gabe.freedesktop.org (Postfix) with ESMTPS id D48FC6E161 for ; Mon, 23 Apr 2018 07:20:33 +0000 (UTC) Received: by mail-lf0-x244.google.com with SMTP id y15-v6so5243100lfj.0 for ; Mon, 23 Apr 2018 00:20:33 -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:mime-version :content-transfer-encoding; bh=8BGe+s4gpI0NYQQpQhcCeD3dl6y2DtRAw8WjWZvdG54=; b=R7SGXmPaK767KrXxpP+D+UkWIGVUDEhWCgoN99Mwf/s6b64zfyBPwIPBBhdCzjDFjr 5sdEmltnZW94LAJFep69/nO/jw4w410nOTxg6CTwpj4KpL3ABuuFRKqWKjz3QXIVVKbY DKNPvepjRo/j2yyVuCo+NkEsL8tRazCQ9/N/M= 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:mime-version :content-transfer-encoding; bh=8BGe+s4gpI0NYQQpQhcCeD3dl6y2DtRAw8WjWZvdG54=; b=GqQ9IOKOdxzvrAiXBQhs093WgQBvp3JehqsDMxEbJ0Y5c2hatDRquPzuibuuAWY6Ye wAbXOsqYmHBZDomXy0nLrphnMU4qcLeTsDdZf5ezfEQMCJSkJ/oy9OKzytzP+61m64Lb BF70G6SZ66x5fGEJna/FesPe3Bx77ifHMWpUZnJBL/7dBlvrHm03pI8OHtkkgKBqOY6t WN6pDGoFvuEPylTJTCaOWMweSOxT+IWqWXbe19lYmtsUccwcI0QpTtMy138bl+AwpzY1 X3gvhrWSvesneLPXsqc9/E7J8vayE4cG7lt7jx4+URVJTJj1r6eAbAb8dlaTIkJQhEZH LhBw== X-Gm-Message-State: ALQs6tB3E3vpToRmYNZ9VfP+knW7rRQetKPHKzily8RG7LuyUIsDVUpP DWJDA3uWk6ejlprlZAcOld3QyQ== X-Received: by 2002:a19:17e9:: with SMTP id 102-v6mr8452391lfx.131.1524468032157; Mon, 23 Apr 2018 00:20:32 -0700 (PDT) Received: from genomnajs.ideon.se ([85.235.10.227]) by smtp.gmail.com with ESMTPSA id l24sm875933ljg.58.2018.04.23.00.20.30 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Mon, 23 Apr 2018 00:20:30 -0700 (PDT) From: Linus Walleij To: Daniel Vetter , Jani Nikula , Sean Paul , Eric Anholt , Liviu Dudau Subject: [PATCH 2/2 v4] drm/pl111: Enable device-specific assigned memory Date: Mon, 23 Apr 2018 09:20:28 +0200 Message-Id: <20180423072028.6739-1-linus.walleij@linaro.org> X-Mailer: git-send-email 2.14.3 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: Robin Murphy , 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 area just gives random snow on the display. The CA9 Versatile Express also has a PL111 instance on its core tile that can address all memory, and this does not have the restriction. 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 Cc: Robin Murphy Reviewed-by: Eric Anholt Signed-off-by: Linus Walleij --- ChangeLog v3->v4: - Collect Eric's review-tag. ChangeLog v2->v3: - Fix error path so we uref the memory properly. - Augment the GEM buffer import to return an error pointer if we try to import a buffer when using device-assigned memory: we can only scan out the special memory and the GEM buffers are not copied but referenced by pointer. ChangeLog v1->v2: - Make sure to also call of_reserved_mem_device_release() at remove() and errorpath. --- drivers/gpu/drm/pl111/pl111_drm.h | 1 + drivers/gpu/drm/pl111/pl111_drv.c | 33 +++++++++++++++++++++++++++++++-- 2 files changed, 32 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/pl111/pl111_drm.h b/drivers/gpu/drm/pl111/pl111_drm.h index 8639b2d4ddf7..ce4501d0ab48 100644 --- a/drivers/gpu/drm/pl111/pl111_drm.h +++ b/drivers/gpu/drm/pl111/pl111_drm.h @@ -79,6 +79,7 @@ struct pl111_drm_dev_private { const struct pl111_variant_data *variant; void (*variant_display_enable) (struct drm_device *drm, u32 format); void (*variant_display_disable) (struct drm_device *drm); + bool use_device_memory; }; int pl111_display_init(struct drm_device *dev); diff --git a/drivers/gpu/drm/pl111/pl111_drv.c b/drivers/gpu/drm/pl111/pl111_drv.c index 4621259d5387..5fcf21837746 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 @@ -207,6 +208,23 @@ static int pl111_modeset_init(struct drm_device *dev) return ret; } +struct drm_gem_object *pl111_gem_import_sg_table(struct drm_device *dev, + struct dma_buf_attachment *attach, + struct sg_table *sgt) +{ + struct pl111_drm_dev_private *priv = dev->dev_private; + + /* + * When using device-specific reserved memory we can't import + * DMA buffers: those are passed by reference in any global + * memory and we can only handle a specific range of memory. + */ + if (priv->use_device_memory) + return ERR_PTR(-EINVAL); + + return drm_gem_cma_prime_import_sg_table(dev, attach, sgt); +} + DEFINE_DRM_GEM_CMA_FOPS(drm_fops); static struct drm_driver pl111_drm_driver = { @@ -227,7 +245,7 @@ static struct drm_driver pl111_drm_driver = { .prime_handle_to_fd = drm_gem_prime_handle_to_fd, .prime_fd_to_handle = drm_gem_prime_fd_to_handle, .gem_prime_import = drm_gem_prime_import, - .gem_prime_import_sg_table = drm_gem_cma_prime_import_sg_table, + .gem_prime_import_sg_table = pl111_gem_import_sg_table, .gem_prime_export = drm_gem_prime_export, .gem_prime_get_sg_table = drm_gem_cma_prime_get_sg_table, @@ -257,6 +275,12 @@ 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"); + priv->use_device_memory = true; + } + 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 +299,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 dev_unref; } /* This may override some variant settings */ @@ -305,11 +330,14 @@ static int pl111_amba_probe(struct amba_device *amba_dev, dev_unref: drm_dev_unref(drm); + 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 +347,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; }