From patchwork Sat Jun 24 23:05:21 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Rob Clark X-Patchwork-Id: 780418 X-Patchwork-Delegate: trini@ti.com Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.denx.de (dione.denx.de [81.169.180.215]) by ozlabs.org (Postfix) with ESMTP id 3ww9t71zRfz9s3s for ; Sun, 25 Jun 2017 09:06:23 +1000 (AEST) Authentication-Results: ozlabs.org; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.b="JIKkGfCx"; dkim-atps=neutral Received: by lists.denx.de (Postfix, from userid 105) id ECCE1C21D5C; Sat, 24 Jun 2017 23:06:10 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on lists.denx.de X-Spam-Level: X-Spam-Status: No, score=-0.0 required=5.0 tests=FREEMAIL_FROM, RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL, T_DKIM_INVALID autolearn=unavailable autolearn_force=no version=3.4.0 Received: from lists.denx.de (localhost [IPv6:::1]) by lists.denx.de (Postfix) with ESMTP id BDE38C21D85; Sat, 24 Jun 2017 23:05:45 +0000 (UTC) Received: by lists.denx.de (Postfix, from userid 105) id 37833C21D5C; Sat, 24 Jun 2017 23:05:40 +0000 (UTC) Received: from mail-qk0-f193.google.com (mail-qk0-f193.google.com [209.85.220.193]) by lists.denx.de (Postfix) with ESMTPS id 62184C21D7C for ; Sat, 24 Jun 2017 23:05:36 +0000 (UTC) Received: by mail-qk0-f193.google.com with SMTP id 16so10609372qkg.2 for ; Sat, 24 Jun 2017 16:05:36 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=4zMHTtg09bALGubAfmTWikHKaElEc3sf40uR2aJQVvw=; b=JIKkGfCx1A75R24868x+yjyp4Q/j4o8RKq7fUE//RQJkedp6W4q76ml3GGyvPU5xtR k3nmdEOfVRzsl6gxA/CD7Dr9SbpynQUtj2I49cWwV9UEvYvn8gspvIZt/yigJH0tW3xj UZwub/5yJJVBaWWIcftjR5SiuYIr9Zr6mjnhqfLzRvsftOIsRhDlR0JCU42YYbl20WYg hBMnfBDz7NIXLw3hvUwDZ7zZ3yRtftOhTiiHdTRGDhfSOvfC6azSjNdf9p/0YDJK774k zL6DFKqMH5v5hskYYrncHYSq/IRNRSSJHdh46pKOy7mxa1wiaLrIglyd3La/NFoR7+xE mu1w== 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; bh=4zMHTtg09bALGubAfmTWikHKaElEc3sf40uR2aJQVvw=; b=JFoLqOCVHRq/oJYIaPxVmq+GCQKpn/mHzuN9/iuc4ZxuM5s19zQN2OVruz0/2YhdHb ZwatlHMXBoP2igCzjWtVWovswjNo5fdCuL9h0BaBrZ3FJPD00xSzFOMMMb4+/XCnolTJ pKwtPFrXczJ3jdl5MukLxNm1RcFlYLY0/RCYXZTX20U9bjJ9B3/OOdqD8O9Q/jUtdh6D 4iqcOpvRLJ5Uc2pDFm8m5/dLoTA1jv+AHnz5tHoHsx28XvW+kSutp3oZu/inmf1cuTMA wOcoo4+7Ts/ZEM1deskFC2LTF2qQMlSI8OAetWxMqlFot0DeE6O+26t2dYNGP2rmDLwQ X/iQ== X-Gm-Message-State: AKS2vOzEMHBk7gKG7Z6uwgfF5ShX9YbgWXiRp7cBCNJhwBO15Mm6//SB BElJclsM4OiLeU3J1Gc= X-Received: by 10.55.214.221 with SMTP id p90mr16083746qkl.228.1498345535104; Sat, 24 Jun 2017 16:05:35 -0700 (PDT) Received: from localhost ([2601:184:4780:aac0:25f8:dd96:a084:785a]) by smtp.gmail.com with ESMTPSA id b28sm6666238qte.23.2017.06.24.16.05.33 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Sat, 24 Jun 2017 16:05:34 -0700 (PDT) From: Rob Clark To: U-Boot Mailing List Date: Sat, 24 Jun 2017 19:05:21 -0400 Message-Id: <20170624230523.2327-4-robdclark@gmail.com> X-Mailer: git-send-email 2.9.4 In-Reply-To: <20170624230523.2327-1-robdclark@gmail.com> References: <20170624230523.2327-1-robdclark@gmail.com> Subject: [U-Boot] [PATCH 3/5] video: simplefb X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.18 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" Not really qcom specific, but for now qcom/lk is the one firmware that is (afaiu) setting up the appropriate dt node for pre-configured display. Uses the generic simple-framebuffer DT bindings so this should be useful on other platforms. Signed-off-by: Rob Clark --- drivers/video/Kconfig | 10 +++++++ drivers/video/Makefile | 2 +- drivers/video/simplefb.c | 68 ++++++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 79 insertions(+), 1 deletion(-) create mode 100644 drivers/video/simplefb.c diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig index 61dfed8..8eb0359 100644 --- a/drivers/video/Kconfig +++ b/drivers/video/Kconfig @@ -628,4 +628,14 @@ config VIDEO_DW_HDMI rather requires a SoC-specific glue driver to call it), it can not be enabled from the configuration menu. +config VIDEO_SIMPLE + bool "Simple display driver for preconfigured display" + help + Enables a simple generic display driver which utilizes the + simple-framebuffer devicetree bindings. + + This driver assumes that the display hardware has been initialized + before u-boot starts, and u-boot will simply render to the pre- + allocated frame buffer surface. + endmenu diff --git a/drivers/video/Makefile b/drivers/video/Makefile index ac5371f..52f50f6 100644 --- a/drivers/video/Makefile +++ b/drivers/video/Makefile @@ -57,7 +57,7 @@ obj-$(CONFIG_FORMIKE) += formike.o obj-$(CONFIG_LG4573) += lg4573.o obj-$(CONFIG_AM335X_LCD) += am335x-fb.o obj-$(CONFIG_VIDEO_DW_HDMI) += dw_hdmi.o - +obj-$(CONFIG_VIDEO_SIMPLE) += simplefb.o obj-${CONFIG_VIDEO_TEGRA124} += tegra124/ obj-${CONFIG_EXYNOS_FB} += exynos/ obj-${CONFIG_VIDEO_ROCKCHIP} += rockchip/ diff --git a/drivers/video/simplefb.c b/drivers/video/simplefb.c new file mode 100644 index 0000000..7c3e8e5 --- /dev/null +++ b/drivers/video/simplefb.c @@ -0,0 +1,68 @@ +/* + * (C) Copyright 2017 Rob Clark + * + * SPDX-License-Identifier: GPL-2.0+ + */ + +#include +#include +#include +#include +#include + +static int simple_video_probe(struct udevice *dev) +{ + struct video_uc_platdata *plat = dev_get_uclass_platdata(dev); + struct video_priv *uc_priv = dev_get_uclass_priv(dev); + const void *blob = gd->fdt_blob; + const int node = dev_of_offset(dev); + const char *format; + fdt_addr_t base, size; + + if (fdtdec_decode_region(blob, node, "reg", &base, &size)) { + debug("%s: Failed to decode memory region\n", __func__); + return -EINVAL; + } + + debug("%s: base=%llx, size=%llu\n", __func__, base, size); + + // TODO is there some way to reserve the framebuffer + // region so it isn't clobbered? + plat->base = base; + plat->size = size; + + video_set_flush_dcache(dev, true); + + debug("%s: Query resolution...\n", __func__); + + uc_priv->xsize = fdtdec_get_uint(blob, node, "width", 0); + uc_priv->ysize = fdtdec_get_uint(blob, node, "height", 0); + uc_priv->rot = 0; + + format = fdt_getprop(blob, node, "format", NULL); + debug("%s: %dx%d@%s\n", __func__, uc_priv->xsize, uc_priv->ysize, format); + + if (strcmp(format, "r5g6b5") == 0) { + uc_priv->bpix = VIDEO_BPP16; + } else if (strcmp(format, "a8b8g8r8") == 0) { + uc_priv->bpix = VIDEO_BPP32; + } else { + printf("%s: invalid format: %s\n", __func__, format); + return -EINVAL; + } + + return 0; +} + +static const struct udevice_id simple_video_ids[] = { + { .compatible = "simple-framebuffer" }, + { } +}; + +U_BOOT_DRIVER(simple_video) = { + .name = "simple_video", + .id = UCLASS_VIDEO, + .of_match = simple_video_ids, + .probe = simple_video_probe, + .flags = DM_FLAG_PRE_RELOC, +};