From patchwork Wed Jun 13 14:07:11 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Maxime Ripard X-Patchwork-Id: 10462341 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 E10B660329 for ; Wed, 13 Jun 2018 14:18:28 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 344FB28833 for ; Wed, 13 Jun 2018 14:18:27 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 2909B28872; Wed, 13 Jun 2018 14:18:27 +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=-2.9 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,MAILING_LIST_MULTI autolearn=unavailable version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 50E2528833 for ; Wed, 13 Jun 2018 14:18:26 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:MIME-Version:Cc:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:References: In-Reply-To:Message-Id:Date:Subject:To:From:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Owner; bh=y7/9zLl4jdM0N0LQQVCvv3R2Z2/jPPm6C5qTuUoquKs=; b=iMXGADVVbcsxafE4/A5hfKIndp dpQqo8wZkQHrsvv/dwBwHs1lzMga7d1o3gQK0yLdu+J5frHVILy8v3g1fnQR4kr019SpfVwrSg/DQ X5fPyZvifjPFAS5SKuZZ1phKeyPzS80cgEPnBjIJo1Wfp3Nzu3xCH/CSAggDHVhEwQfsoD7MuZOd0 EZx64Xq6gHo/dCHNdQI0NRSCdEgF1u8wI+X+qvDHea6t4oQoOgq0rHaet+WhTC6PRl5BsUsYrMaZy WpmeSZfWsOROmTMOPneYLVdG+w8oQRUeVeemJaTP2e13NxZvzNYC90Zz0Uqij8n9oaSDCy808hCaU LJ8fZF4w==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1fT6bK-0000TG-Dj; Wed, 13 Jun 2018 14:18:18 +0000 Received: from mail.bootlin.com ([62.4.15.54]) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1fT6R6-0007dC-Kt for linux-arm-kernel@lists.infradead.org; Wed, 13 Jun 2018 14:07:59 +0000 Received: by mail.bootlin.com (Postfix, from userid 110) id E6CD220A39; Wed, 13 Jun 2018 16:07:18 +0200 (CEST) Received: from localhost (AAubervilliers-681-1-37-30.w90-88.abo.wanadoo.fr [90.88.156.30]) by mail.bootlin.com (Postfix) with ESMTPSA id B5AFD20728; Wed, 13 Jun 2018 16:07:18 +0200 (CEST) From: Maxime Ripard To: hans.verkuil@cisco.com, acourbot@chromium.org, sakari.ailus@linux.intel.com, Laurent Pinchart Subject: [PATCH 6/9] media: cedrus: Add ops structure Date: Wed, 13 Jun 2018 16:07:11 +0200 Message-Id: <20180613140714.1686-7-maxime.ripard@bootlin.com> X-Mailer: git-send-email 2.17.0 In-Reply-To: <20180613140714.1686-1-maxime.ripard@bootlin.com> References: <20180613140714.1686-1-maxime.ripard@bootlin.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20180613_070744_997934_D2D82D88 X-CRM114-Status: GOOD ( 18.18 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Maxime Ripard , jenskuske@gmail.com, linux-sunxi@googlegroups.com, linux-kernel@vger.kernel.org, tfiga@chromium.org, Paul Kocialkowski , Chen-Yu Tsai , posciak@chromium.org, Thomas Petazzoni , nicolas.dufresne@collabora.com, linux-arm-kernel@lists.infradead.org, linux-media@vger.kernel.org MIME-Version: 1.0 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP In order to increase the number of codecs supported, we need to decouple the MPEG2 only code that was there up until now and turn it into something a bit more generic. Do that by introducing an intermediate ops structure that would need to be filled by each supported codec. Start by implementing in that structure the setup and trigger hooks that are currently the only functions being implemented by codecs support. To do so, we need to store the current codec in use, which we do at start_streaming time. Signed-off-by: Maxime Ripard Acked-by: Paul Kocialkowski --- .../platform/sunxi/cedrus/sunxi_cedrus.c | 2 ++ .../sunxi/cedrus/sunxi_cedrus_common.h | 11 +++++++ .../platform/sunxi/cedrus/sunxi_cedrus_dec.c | 10 +++--- .../sunxi/cedrus/sunxi_cedrus_mpeg2.c | 11 +++++-- .../sunxi/cedrus/sunxi_cedrus_mpeg2.h | 33 ------------------- .../sunxi/cedrus/sunxi_cedrus_video.c | 17 +++++++++- 6 files changed, 42 insertions(+), 42 deletions(-) delete mode 100644 drivers/media/platform/sunxi/cedrus/sunxi_cedrus_mpeg2.h diff --git a/drivers/media/platform/sunxi/cedrus/sunxi_cedrus.c b/drivers/media/platform/sunxi/cedrus/sunxi_cedrus.c index ccd41d9a3e41..bc80480f5dfd 100644 --- a/drivers/media/platform/sunxi/cedrus/sunxi_cedrus.c +++ b/drivers/media/platform/sunxi/cedrus/sunxi_cedrus.c @@ -244,6 +244,8 @@ static int sunxi_cedrus_probe(struct platform_device *pdev) if (ret) return ret; + dev->dec_ops[SUNXI_CEDRUS_CODEC_MPEG2] = &sunxi_cedrus_dec_ops_mpeg2; + ret = v4l2_device_register(&pdev->dev, &dev->v4l2_dev); if (ret) goto unreg_media; diff --git a/drivers/media/platform/sunxi/cedrus/sunxi_cedrus_common.h b/drivers/media/platform/sunxi/cedrus/sunxi_cedrus_common.h index a5f83c452006..c2e2c92d103b 100644 --- a/drivers/media/platform/sunxi/cedrus/sunxi_cedrus_common.h +++ b/drivers/media/platform/sunxi/cedrus/sunxi_cedrus_common.h @@ -75,6 +75,7 @@ struct sunxi_cedrus_ctx { struct v4l2_pix_format_mplane src_fmt; struct sunxi_cedrus_fmt *vpu_dst_fmt; struct v4l2_pix_format_mplane dst_fmt; + enum sunxi_cedrus_codec current_codec; struct v4l2_ctrl_handler hdl; struct v4l2_ctrl *ctrls[SUNXI_CEDRUS_CTRL_MAX]; @@ -107,6 +108,14 @@ struct sunxi_cedrus_buffer *vb2_to_cedrus_buffer(const struct vb2_buffer *p) return vb2_v4l2_to_cedrus_buffer(to_vb2_v4l2_buffer(p)); } +struct sunxi_cedrus_dec_ops { + void (*setup)(struct sunxi_cedrus_ctx *ctx, + struct sunxi_cedrus_run *run); + void (*trigger)(struct sunxi_cedrus_ctx *ctx); +}; + +extern struct sunxi_cedrus_dec_ops sunxi_cedrus_dec_ops_mpeg2; + struct sunxi_cedrus_dev { struct v4l2_device v4l2_dev; struct video_device vfd; @@ -130,6 +139,8 @@ struct sunxi_cedrus_dev { struct reset_control *rstc; struct regmap *syscon; + + struct sunxi_cedrus_dec_ops *dec_ops[SUNXI_CEDRUS_CODEC_LAST]; }; static inline void sunxi_cedrus_write(struct sunxi_cedrus_dev *dev, diff --git a/drivers/media/platform/sunxi/cedrus/sunxi_cedrus_dec.c b/drivers/media/platform/sunxi/cedrus/sunxi_cedrus_dec.c index f274408ab5a7..5e552fa05274 100644 --- a/drivers/media/platform/sunxi/cedrus/sunxi_cedrus_dec.c +++ b/drivers/media/platform/sunxi/cedrus/sunxi_cedrus_dec.c @@ -28,7 +28,6 @@ #include #include "sunxi_cedrus_common.h" -#include "sunxi_cedrus_mpeg2.h" #include "sunxi_cedrus_dec.h" #include "sunxi_cedrus_hw.h" @@ -77,6 +76,7 @@ void sunxi_cedrus_device_work(struct work_struct *work) void sunxi_cedrus_device_run(void *priv) { struct sunxi_cedrus_ctx *ctx = priv; + struct sunxi_cedrus_dev *dev = ctx->dev; struct sunxi_cedrus_run run = { 0 }; struct media_request *src_req, *dst_req; unsigned long flags; @@ -120,8 +120,6 @@ void sunxi_cedrus_device_run(void *priv) case V4L2_PIX_FMT_MPEG2_FRAME: CHECK_CONTROL(ctx, SUNXI_CEDRUS_CTRL_DEC_MPEG2_FRAME_HDR); run.mpeg2.hdr = get_ctrl_ptr(ctx, SUNXI_CEDRUS_CTRL_DEC_MPEG2_FRAME_HDR); - sunxi_cedrus_mpeg2_setup(ctx, &run); - break; default: @@ -129,6 +127,9 @@ void sunxi_cedrus_device_run(void *priv) } #undef CHECK_CONTROL + if (!ctx->job_abort) + dev->dec_ops[ctx->current_codec]->setup(ctx, &run); + unlock_complete: spin_unlock_irqrestore(&ctx->dev->irq_lock, flags); @@ -143,8 +144,7 @@ void sunxi_cedrus_device_run(void *priv) spin_lock_irqsave(&ctx->dev->irq_lock, flags); if (!ctx->job_abort) { - if (ctx->vpu_src_fmt->fourcc == V4L2_PIX_FMT_MPEG2_FRAME) - sunxi_cedrus_mpeg2_trigger(ctx); + dev->dec_ops[ctx->current_codec]->trigger(ctx); } else { v4l2_m2m_buf_done(run.src, VB2_BUF_STATE_ERROR); v4l2_m2m_buf_done(run.dst, VB2_BUF_STATE_ERROR); diff --git a/drivers/media/platform/sunxi/cedrus/sunxi_cedrus_mpeg2.c b/drivers/media/platform/sunxi/cedrus/sunxi_cedrus_mpeg2.c index d1d7a3cfce0d..e25075bb5779 100644 --- a/drivers/media/platform/sunxi/cedrus/sunxi_cedrus_mpeg2.c +++ b/drivers/media/platform/sunxi/cedrus/sunxi_cedrus_mpeg2.c @@ -52,8 +52,8 @@ static const u8 mpeg_default_non_intra_quant[64] = { #define m_niq(i) ((i << 8) | mpeg_default_non_intra_quant[i]) -void sunxi_cedrus_mpeg2_setup(struct sunxi_cedrus_ctx *ctx, - struct sunxi_cedrus_run *run) +static void sunxi_cedrus_mpeg2_setup(struct sunxi_cedrus_ctx *ctx, + struct sunxi_cedrus_run *run) { struct sunxi_cedrus_dev *dev = ctx->dev; const struct v4l2_ctrl_mpeg2_frame_hdr *frame_hdr = run->mpeg2.hdr; @@ -148,9 +148,14 @@ void sunxi_cedrus_mpeg2_setup(struct sunxi_cedrus_ctx *ctx, sunxi_cedrus_write(dev, src_buf_addr + VBV_SIZE - 1, VE_MPEG_VLD_END); } -void sunxi_cedrus_mpeg2_trigger(struct sunxi_cedrus_ctx *ctx) +static void sunxi_cedrus_mpeg2_trigger(struct sunxi_cedrus_ctx *ctx) { struct sunxi_cedrus_dev *dev = ctx->dev; sunxi_cedrus_write(dev, VE_TRIG_MPEG2, VE_MPEG_TRIGGER); } + +struct sunxi_cedrus_dec_ops sunxi_cedrus_dec_ops_mpeg2 = { + .setup = sunxi_cedrus_mpeg2_setup, + .trigger = sunxi_cedrus_mpeg2_trigger, +}; diff --git a/drivers/media/platform/sunxi/cedrus/sunxi_cedrus_mpeg2.h b/drivers/media/platform/sunxi/cedrus/sunxi_cedrus_mpeg2.h deleted file mode 100644 index 4c380becfa1a..000000000000 --- a/drivers/media/platform/sunxi/cedrus/sunxi_cedrus_mpeg2.h +++ /dev/null @@ -1,33 +0,0 @@ -/* - * Sunxi-Cedrus VPU driver - * - * Copyright (C) 2018 Paul Kocialkowski - * Copyright (C) 2016 Florent Revest - * - * Based on the vim2m driver, that is: - * - * Copyright (c) 2009-2010 Samsung Electronics Co., Ltd. - * Pawel Osciak, - * Marek Szyprowski, - * - * This software is licensed under the terms of the GNU General Public - * License version 2, as published by the Free Software Foundation, and - * may be copied, distributed, and modified under those terms. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - */ - -#ifndef _SUNXI_CEDRUS_MPEG2_H_ -#define _SUNXI_CEDRUS_MPEG2_H_ - -struct sunxi_cedrus_ctx; -struct sunxi_cedrus_run; - -void sunxi_cedrus_mpeg2_setup(struct sunxi_cedrus_ctx *ctx, - struct sunxi_cedrus_run *run); -void sunxi_cedrus_mpeg2_trigger(struct sunxi_cedrus_ctx *ctx); - -#endif diff --git a/drivers/media/platform/sunxi/cedrus/sunxi_cedrus_video.c b/drivers/media/platform/sunxi/cedrus/sunxi_cedrus_video.c index 089abfe6bfeb..fb7b081a5bb7 100644 --- a/drivers/media/platform/sunxi/cedrus/sunxi_cedrus_video.c +++ b/drivers/media/platform/sunxi/cedrus/sunxi_cedrus_video.c @@ -28,7 +28,6 @@ #include #include "sunxi_cedrus_common.h" -#include "sunxi_cedrus_mpeg2.h" #include "sunxi_cedrus_dec.h" #include "sunxi_cedrus_hw.h" @@ -414,6 +413,21 @@ static int sunxi_cedrus_buf_prepare(struct vb2_buffer *vb) return 0; } +static int sunxi_cedrus_start_streaming(struct vb2_queue *q, unsigned int count) +{ + struct sunxi_cedrus_ctx *ctx = vb2_get_drv_priv(q); + + switch (ctx->vpu_src_fmt->fourcc) { + case V4L2_PIX_FMT_MPEG2_FRAME: + ctx->current_codec = SUNXI_CEDRUS_CODEC_MPEG2; + break; + default: + return -EINVAL; + } + + return 0; +} + static void sunxi_cedrus_stop_streaming(struct vb2_queue *q) { struct sunxi_cedrus_ctx *ctx = vb2_get_drv_priv(q); @@ -462,6 +476,7 @@ static struct vb2_ops sunxi_cedrus_qops = { .buf_cleanup = sunxi_cedrus_buf_cleanup, .buf_queue = sunxi_cedrus_buf_queue, .buf_request_complete = sunxi_cedrus_buf_request_complete, + .start_streaming = sunxi_cedrus_start_streaming, .stop_streaming = sunxi_cedrus_stop_streaming, .wait_prepare = vb2_ops_wait_prepare, .wait_finish = vb2_ops_wait_finish,