From patchwork Mon Mar 13 20:42:23 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sergei Shtylyov X-Patchwork-Id: 9622075 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 9E01060244 for ; Mon, 13 Mar 2017 20:42:49 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 8D71328490 for ; Mon, 13 Mar 2017 20:42:49 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 81EF6284D9; Mon, 13 Mar 2017 20:42:49 +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=-6.9 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,RCVD_IN_DNSWL_HI autolearn=unavailable version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id DBCB528490 for ; Mon, 13 Mar 2017 20:42:48 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752928AbdCMUmr (ORCPT ); Mon, 13 Mar 2017 16:42:47 -0400 Received: from mail-lf0-f44.google.com ([209.85.215.44]:33166 "EHLO mail-lf0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752235AbdCMUmp (ORCPT ); Mon, 13 Mar 2017 16:42:45 -0400 Received: by mail-lf0-f44.google.com with SMTP id a6so68746594lfa.0 for ; Mon, 13 Mar 2017 13:42:44 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cogentembedded-com.20150623.gappssmtp.com; s=20150623; h=from:message-id:user-agent:date:to:cc:subject:mime-version :content-disposition; bh=Yz1TnrCwo9Iyvt9UIXvBojy1hQeZ4iZfXBXJl2OlGIk=; b=wD+LuiVwK+b654ln/avAHsoicyZiU0au/tRQgP1f9SEtW2yw6a5k3ghhgaEu6G8zPv yMfo9dksijCAy1sktAKg/ADn+G/ki5eQU2X/TNnR6nMZLuf7UsKFPOZeIMmw7Vl6Q7Dk tJc60P5W79nV0WwC1DvuQ8Y956mpv2xzjzHezl1wOdBbcFhFzloZASrGBr2ckTT/bcI3 64wp377lGo1eHBc2Om9Rhq/SUxc6k3GDsqDnL7JGk8Vi8KsZa5fwRcdBDzGbV+/nnPu1 eUUI7rujwAMd+ttA432Lkn7G9GXWuf+kajy71683BI51ImG95Kj6YzQ9TcvmAj5kjXCr ZFmA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:message-id:user-agent:date:to:cc:subject :mime-version:content-disposition; bh=Yz1TnrCwo9Iyvt9UIXvBojy1hQeZ4iZfXBXJl2OlGIk=; b=Bkl3o+z2pNZiZaYIMA3YGUSBX/HuAGHmcANh7VO20Olkd9wPWjieAS+b/3HHKiJaB6 M7++XshqE/czcOnRKIt5XLPMwfBy7kI3UgbU8827/NCnq7NYwebkWHVM3xXygEAQ+GvN 2/FhKR5FT9xye5MQBjuCHoMigrLLAhVG9cnVJ+mHZEBzY0OUgH6+TE0b86kzaJq17cuV mXletnIPmJi7IDHel2W3/8XmEVz6j4PJ69kCSIOlO5XnVK9tCUo8fDfza3bvtm8D7z/W EYYRMz4SCz77FNOWia3CbvUvbDHZ04w07GEbX+Qwnvny2s+sktdxhYIjGXVlWFgM9RqA JByQ== X-Gm-Message-State: AMke39mml/k6ctVLRyw7o8eliLRV/vSJpqN1p0Gj++hgN0p7SgFxR17InTUayXxsfOm/IA== X-Received: by 10.25.38.83 with SMTP id m80mr8989581lfm.166.1489437763370; Mon, 13 Mar 2017 13:42:43 -0700 (PDT) Received: from wasted.cogentembedded.com ([31.173.84.203]) by smtp.gmail.com with ESMTPSA id h23sm3895223lji.34.2017.03.13.13.42.40 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 13 Mar 2017 13:42:42 -0700 (PDT) From: Sergei Shtylyov X-Google-Original-From: "Sergei Shtylyov" Received: by wasted.cogentembedded.com (sSMTP sendmail emulation); Mon, 13 Mar 2017 23:42:39 +0300 Message-Id: <20170313204239.637629122@cogentembedded.com> User-Agent: quilt/0.64 Date: Mon, 13 Mar 2017 23:42:23 +0300 To: Rob Herring , Mark Rutland , Mauro Carvalho Chehab , devicetree@vger.kernel.org, linux-media@vger.kernel.org Cc: linux-renesas-soc@vger.kernel.org, Sergei Shtylyov Subject: [PATCH] media: platform: rcar_imr: add IMR-LSX3 support MIME-Version: 1.0 Content-Disposition: inline; filename=media-platform-rcar_imr-add-IMR-LSX3-support.patch Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Add support for the image renderer light SRAM extended 3 (IMR-LSX3) found only in the R-Car V2H (R8A7792) SoC. It differs from IMR-LX4 in that it supports only planar video formats but can use the video capture data for the textures. Signed-off-by: Sergei Shtylyov --- This patch is against the 'media_tree.git' repo's 'master' branch plus the latest version of the Renesas IMR driver... Documentation/devicetree/bindings/media/rcar_imr.txt | 11 +- drivers/media/platform/rcar_imr.c | 93 ++++++++++++++++--- 2 files changed, 90 insertions(+), 14 deletions(-) Index: media_tree/Documentation/devicetree/bindings/media/rcar_imr.txt =================================================================== --- media_tree.orig/Documentation/devicetree/bindings/media/rcar_imr.txt +++ media_tree/Documentation/devicetree/bindings/media/rcar_imr.txt @@ -8,9 +8,14 @@ and drawing with respect to any shape th Required properties: -- compatible: "renesas,-imr-lx4", "renesas,imr-lx4" as a fallback for - the image renderer light extended 4 (IMR-LX4) found in the R-Car gen3 SoCs, - where the examples with are: +- compatible: + "renesas,-imr-lsx3", "renesas,imr-lsx3" as a fallback for the image + renderer light SRAM extended 3 (IMR-LSX3) found in the R-Car gen2 SoCs, where + the examples with are: + - "renesas,r8a7792-imr-lsx3" for R-Car V2H; + "renesas,-imr-lx4", "renesas,imr-lx4" as a fallback for the image + renderer light extended 4 (IMR-LX4) found in the R-Car gen3 SoCs, where the + examples with are: - "renesas,r8a7795-imr-lx4" for R-Car H3, - "renesas,r8a7796-imr-lx4" for R-Car M3-W. - reg: offset and length of the register block; Index: media_tree/drivers/media/platform/rcar_imr.c =================================================================== --- media_tree.orig/drivers/media/platform/rcar_imr.c +++ media_tree/drivers/media/platform/rcar_imr.c @@ -1,5 +1,5 @@ /* - * rcar_imr.c -- R-Car IMR-LX4 Driver + * rcar_imr.c -- R-Car IMR-LSX3/LX4 Driver * * Copyright (C) 2015-2017 Cogent Embedded, Inc. * @@ -14,7 +14,7 @@ #include #include #include -#include +#include #include #include #include @@ -81,8 +81,21 @@ struct imr_format_info { u32 flags; }; +enum imr_gen { + IMR_LSX3, + IMR_LX4, +}; + +/* IMR type specific data */ +struct imr_type { + enum imr_gen gen; + const struct imr_format_info *formats; + unsigned int num_formats; +}; + /* per-device data */ struct imr_device { + const struct imr_type *type; struct device *dev; struct clk *clock; void __iomem *mmio; @@ -180,6 +193,7 @@ struct imr_ctx { #define IMR_IMR_IEM BIT(1) #define IMR_IMR_INM BIT(2) +#define IMR_CMRCR_TXTM BIT(0) /* IMR-LSX3 only */ #define IMR_CMRCR_LUCE BIT(1) #define IMR_CMRCR_CLCE BIT(2) #define IMR_CMRCR_DUV_SHIFT 3 @@ -282,6 +296,34 @@ static u32 __imr_flags_common(u32 iflags return iflags & oflags & IMR_F_PLANES_MASK; } +static const struct imr_format_info imr_lsx3_formats[] = { + { + .name = "YUV 4:2:2 semiplanar (NV16)", + .fourcc = V4L2_PIX_FMT_NV16, + .flags = IMR_F_Y8 | IMR_F_UV8 | IMR_F_PLANAR, + }, + { + .name = "Greyscale 8-bit", + .fourcc = V4L2_PIX_FMT_GREY, + .flags = IMR_F_Y8 | IMR_F_PLANAR, + }, + { + .name = "Greyscale 10-bit", + .fourcc = V4L2_PIX_FMT_Y10, + .flags = IMR_F_Y8 | IMR_F_Y10 | IMR_F_PLANAR, + }, + { + .name = "Greyscale 12-bit", + .fourcc = V4L2_PIX_FMT_Y12, + .flags = IMR_F_Y8 | IMR_F_Y10 | IMR_F_Y12 | IMR_F_PLANAR, + }, + { + .name = "Chrominance UV 8-bit", + .fourcc = V4L2_PIX_FMT_UV8, + .flags = IMR_F_UV8 | IMR_F_PLANAR, + }, +}; + static const struct imr_format_info imr_lx4_formats[] = { { .name = "YUV 4:2:2 semiplanar (NV16)", @@ -335,6 +377,18 @@ static const struct imr_format_info imr_ }, }; +static const struct imr_type imr_lsx3 = { + .gen = IMR_LSX3, + .formats = imr_lsx3_formats, + .num_formats = ARRAY_SIZE(imr_lsx3_formats), +}; + +static const struct imr_type imr_lx4 = { + .gen = IMR_LX4, + .formats = imr_lx4_formats, + .num_formats = ARRAY_SIZE(imr_lx4_formats), +}; + /* mesh configuration constructor */ static struct imr_cfg *imr_cfg_create(struct imr_ctx *ctx, u32 dl_size, u32 dl_start) @@ -767,7 +821,8 @@ static void imr_dl_program_setup(struct "setup %ux%u -> %ux%u mapping (type=%x)\n", w, h, W, H, type); /* set triangle mode register from user-supplied descriptor */ - *dl++ = IMR_OP_WTS(IMR_TRIMCR, 0x004F); + *dl++ = IMR_OP_WTS(IMR_TRIMCR, + ctx->imr->type->gen == IMR_LX4 ? 0x004F : 0x007F); /* set automatic source/destination coordinates generation flags */ *dl++ = IMR_OP_WTS(IMR_TRIMSR, __imr_auto_sg_dg_tcm(type) | @@ -823,7 +878,7 @@ static void imr_dl_program_setup(struct *dl++ = IMR_OP_WTS(IMR_SSTR, w << (iflags & IMR_F_UV10 ? 1 : 0)); } - } else { + } else if (ctx->imr->type->gen == IMR_LX4) { u16 src_fmt = (iflags & IMR_F_UV_SWAP ? IMR_CMRCR2_UVFORM : 0) | (iflags & IMR_F_YUV_SWAP ? IMR_CMRCR2_YUV422FORM : 0); @@ -864,6 +919,9 @@ static void imr_dl_program_setup(struct *dl++ = IMR_OP_WTS(IMR_DSTR, W << (cflags & IMR_F_Y10 ? 2 : 1)); } + } else { + /* this shouldn't happen! */ + BUG(); } /* @@ -1114,6 +1172,7 @@ out: /* test if a format is supported */ static int __imr_try_fmt(struct imr_ctx *ctx, struct v4l2_format *f) { + const struct imr_type *type = ctx->imr->type; struct v4l2_pix_format *pix = &f->fmt.pix; u32 fourcc = pix->pixelformat; int i; @@ -1122,8 +1181,8 @@ static int __imr_try_fmt(struct imr_ctx * both output and capture interface have the same set of * supported formats */ - for (i = 0; i < ARRAY_SIZE(imr_lx4_formats); i++) { - if (fourcc == imr_lx4_formats[i].fourcc) { + for (i = 0; i < type->num_formats; i++) { + if (fourcc == type->formats[i].fourcc) { /* fix up format specification as needed */ pix->field = V4L2_FIELD_NONE; @@ -1174,9 +1233,12 @@ static int imr_querycap(struct file *fil /* enumerate supported formats */ static int imr_enum_fmt(struct file *file, void *priv, struct v4l2_fmtdesc *f) { + struct imr_ctx *ctx = fh_to_ctx(priv); + const struct imr_type *type = ctx->imr->type; + /* no distinction between output/capture formats */ - if (f->index < ARRAY_SIZE(imr_lx4_formats)) { - const struct imr_format_info *fmt = &imr_lx4_formats[f->index]; + if (f->index < type->num_formats) { + const struct imr_format_info *fmt = &type->formats[f->index]; strlcpy(f->description, fmt->name, sizeof(f->description)); f->pixelformat = fmt->fourcc; @@ -1246,7 +1308,7 @@ static int imr_s_fmt(struct file *file, /* processing is locked? TBD */ q_data->fmt = f->fmt.pix; - q_data->flags = imr_lx4_formats[i].flags; + q_data->flags = ctx->imr->type->formats[i].flags; /* set default crop factors */ if (!V4L2_TYPE_IS_OUTPUT(f->type)) { @@ -1622,6 +1684,8 @@ static void imr_device_run(void *priv) wmb(); /* start rendering operation */ + if (imr->type->gen != IMR_LX4) + iowrite32(IMR_CMRCR_TXTM, imr->mmio + IMR_CMRCSR); iowrite32(IMR_CR_RS, imr->mmio + IMR_CR); /* timestamp input buffer */ @@ -1776,16 +1840,22 @@ handled: static int imr_probe(struct platform_device *pdev) { + const struct imr_type *type; struct imr_device *imr; struct resource *res; int ret; + type = of_device_get_match_data(&pdev->dev); + if (!type) + return -ENODEV; + imr = devm_kzalloc(&pdev->dev, sizeof(*imr), GFP_KERNEL); if (!imr) return -ENOMEM; mutex_init(&imr->mutex); spin_lock_init(&imr->lock); + imr->type = type; imr->dev = &pdev->dev; /* memory-mapped registers */ @@ -1919,7 +1989,8 @@ static const struct dev_pm_ops imr_pm_op /* device table */ static const struct of_device_id imr_of_match[] = { - { .compatible = "renesas,imr-lx4" }, + { .compatible = "renesas,imr-lsx3", .data = &imr_lsx3, }, + { .compatible = "renesas,imr-lx4", .data = &imr_lx4, }, { }, }; @@ -1939,5 +2010,5 @@ module_platform_driver(imr_platform_driv MODULE_ALIAS("imr"); MODULE_AUTHOR("Cogent Embedded Inc. "); -MODULE_DESCRIPTION("Renesas IMR-LX4 Driver"); +MODULE_DESCRIPTION("Renesas IMR-LSX3/LX4 Driver"); MODULE_LICENSE("GPL");