From: Adrian Ratiu <adrian.ratiu@collabora.com>
To: Ezequiel Garcia <ezequiel@collabora.com>,
Philipp Zabel <p.zabel@pengutronix.de>
Cc: Mark Brown <broonie@kernel.org>,
Mauro Carvalho Chehab <mchehab@kernel.org>,
Fruehberger Peter <Peter.Fruehberger@de.bosch.com>,
kuhanh.murugasen.krishnan@intel.com,
Daniel Vetter <daniel@ffwll.ch>,
kernel@collabora.com, linux-media@vger.kernel.org,
linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org
Subject: [PATCH 10/18] media: hantro: convert G1 h264 decoder to regmap fields
Date: Mon, 12 Oct 2020 23:59:49 +0300 [thread overview]
Message-ID: <20201012205957.889185-11-adrian.ratiu@collabora.com> (raw)
In-Reply-To: <20201012205957.889185-1-adrian.ratiu@collabora.com>
Populate the regmap field API for G1 h264 decoding and convert the
G1 h264 decoder source to use the new API. This is done because we
will add support for the newer VC8000D core which will configure
the regmap API fields differently to match its own hwreg layout.
Signed-off-by: Adrian Ratiu <adrian.ratiu@collabora.com>
---
.../staging/media/hantro/hantro_g1_h264_dec.c | 71 ++++++++++-------
drivers/staging/media/hantro/hantro_regmap.c | 79 ++++++++++++++++++-
drivers/staging/media/hantro/hantro_regmap.h | 26 +++++-
3 files changed, 145 insertions(+), 31 deletions(-)
diff --git a/drivers/staging/media/hantro/hantro_g1_h264_dec.c b/drivers/staging/media/hantro/hantro_g1_h264_dec.c
index 845bef73d218..8592dfabbc5e 100644
--- a/drivers/staging/media/hantro/hantro_g1_h264_dec.c
+++ b/drivers/staging/media/hantro/hantro_g1_h264_dec.c
@@ -18,6 +18,9 @@
#include "hantro_g1_regs.h"
#include "hantro_hw.h"
#include "hantro_v4l2.h"
+#include "hantro_regmap.h"
+
+extern struct regmap_config hantro_regmap_dec;
static void set_params(struct hantro_ctx *ctx)
{
@@ -27,10 +30,15 @@ static void set_params(struct hantro_ctx *ctx)
const struct v4l2_ctrl_h264_pps *pps = ctrls->pps;
struct vb2_v4l2_buffer *src_buf = hantro_get_src_buf(ctx);
struct hantro_dev *vpu = ctx->dev;
+ struct hantro_regmap_fields_dec *fields = vpu->reg_fields_dec;
+ u32 width = MB_WIDTH(ctx->src_fmt.width);
+ u32 height = MB_HEIGHT(ctx->src_fmt.height);
u32 reg;
+ regmap_field_write(fields->dec_axi_wr_id, 0x0);
+
/* Decoder control register 0. */
- reg = G1_REG_DEC_CTRL0_DEC_AXI_WR_ID(0x0);
+ reg = 0;
if (sps->flags & V4L2_H264_SPS_FLAG_MB_ADAPTIVE_FRAME_FIELD)
reg |= G1_REG_DEC_CTRL0_SEQ_MBAFF_E;
if (sps->profile_idc > 66) {
@@ -50,10 +58,11 @@ static void set_params(struct hantro_ctx *ctx)
vdpu_write_relaxed(vpu, reg, G1_REG_DEC_CTRL0);
/* Decoder control register 1. */
- reg = G1_REG_DEC_CTRL1_PIC_MB_WIDTH(MB_WIDTH(ctx->src_fmt.width)) |
- G1_REG_DEC_CTRL1_PIC_MB_HEIGHT_P(MB_HEIGHT(ctx->src_fmt.height)) |
- G1_REG_DEC_CTRL1_REF_FRAMES(sps->max_num_ref_frames);
- vdpu_write_relaxed(vpu, reg, G1_REG_DEC_CTRL1);
+ regmap_field_write(fields->dec_pic_width, width);
+ regmap_field_write(fields->dec_pic_height, height);
+
+ regmap_field_write(fields->dec_num_ref_frames,
+ sps->max_num_ref_frames);
/* Decoder control register 2. */
reg = G1_REG_DEC_CTRL2_CH_QP_OFFSET(pps->chroma_qp_index_offset) |
@@ -66,10 +75,11 @@ static void set_params(struct hantro_ctx *ctx)
vdpu_write_relaxed(vpu, reg, G1_REG_DEC_CTRL2);
/* Decoder control register 3. */
- reg = G1_REG_DEC_CTRL3_START_CODE_E |
- G1_REG_DEC_CTRL3_INIT_QP(pps->pic_init_qp_minus26 + 26) |
- G1_REG_DEC_CTRL3_STREAM_LEN(vb2_get_plane_payload(&src_buf->vb2_buf, 0));
- vdpu_write_relaxed(vpu, reg, G1_REG_DEC_CTRL3);
+ regmap_field_write(fields->dec_start_code_e, 1);
+ regmap_field_write(fields->dec_init_qp,
+ pps->pic_init_qp_minus26 + 26);
+ regmap_field_write(fields->dec_stream_len,
+ vb2_get_plane_payload(&src_buf->vb2_buf, 0));
/* Decoder control register 4. */
reg = G1_REG_DEC_CTRL4_FRAMENUM_LEN(sps->log2_max_frame_num_minus4 + 4) |
@@ -121,8 +131,7 @@ static void set_params(struct hantro_ctx *ctx)
vdpu_write_relaxed(vpu, 0, G1_REG_REF_BUF_CTRL);
/* Reference picture buffer control register 2. */
- vdpu_write_relaxed(vpu, G1_REG_REF_BUF_CTRL2_APF_THRESHOLD(8),
- G1_REG_REF_BUF_CTRL2);
+ regmap_field_write(fields->dec_apf_threshold, 8);
}
static void set_ref(struct hantro_ctx *ctx)
@@ -221,7 +230,6 @@ static void set_ref(struct hantro_ctx *ctx)
/* Set up addresses of DPB buffers. */
for (i = 0; i < HANTRO_H264_DPB_SIZE; i++) {
dma_addr_t dma_addr = hantro_h264_get_ref_buf(ctx, i);
-
vdpu_write_relaxed(vpu, dma_addr, G1_REG_ADDR_REF(i));
}
}
@@ -231,6 +239,7 @@ static void set_buffers(struct hantro_ctx *ctx)
const struct hantro_h264_dec_ctrls *ctrls = &ctx->h264_dec.ctrls;
struct vb2_v4l2_buffer *src_buf, *dst_buf;
struct hantro_dev *vpu = ctx->dev;
+ struct hantro_regmap_fields_dec *fields = vpu->reg_fields_dec;
dma_addr_t src_dma, dst_dma;
size_t offset = 0;
@@ -239,14 +248,14 @@ static void set_buffers(struct hantro_ctx *ctx)
/* Source (stream) buffer. */
src_dma = vb2_dma_contig_plane_dma_addr(&src_buf->vb2_buf, 0);
- vdpu_write_relaxed(vpu, src_dma, G1_REG_ADDR_STR);
+ regmap_field_write(fields->dec_addr_str, src_dma);
/* Destination (decoded frame) buffer. */
dst_dma = hantro_get_dec_buf_addr(ctx, &dst_buf->vb2_buf);
/* Adjust dma addr to start at second line for bottom field */
if (ctrls->decode->flags & V4L2_H264_DECODE_PARAM_FLAG_BOTTOM_FIELD)
offset = ALIGN(ctx->src_fmt.width, MB_DIM);
- vdpu_write_relaxed(vpu, dst_dma + offset, G1_REG_ADDR_DST);
+ regmap_field_write(fields->dec_addr_dst, dst_dma + offset);
/* Higher profiles require DMV buffer appended to reference frames. */
if (ctrls->sps->profile_idc > 66 && ctrls->decode->nal_ref_idc) {
@@ -266,16 +275,18 @@ static void set_buffers(struct hantro_ctx *ctx)
if (ctrls->decode->flags & V4L2_H264_DECODE_PARAM_FLAG_BOTTOM_FIELD)
offset += 32 * MB_WIDTH(ctx->src_fmt.width) *
MB_HEIGHT(ctx->src_fmt.height);
- vdpu_write_relaxed(vpu, dst_dma + offset, G1_REG_ADDR_DIR_MV);
+ regmap_field_write(fields->dec_addr_dir_mv, dst_dma + offset);
}
/* Auxiliary buffer prepared in hantro_g1_h264_dec_prepare_table(). */
- vdpu_write_relaxed(vpu, ctx->h264_dec.priv.dma, G1_REG_ADDR_QTABLE);
+ regmap_field_write(fields->dec_addr_qtable, ctx->h264_dec.priv.dma);
}
void hantro_g1_h264_dec_run(struct hantro_ctx *ctx)
{
struct hantro_dev *vpu = ctx->dev;
+ struct hantro_regmap_fields_dec *fields = vpu->reg_fields_dec;
+ int reg;
/* Prepare the H264 decoder context. */
if (hantro_h264_dec_prepare_run(ctx))
@@ -288,17 +299,23 @@ void hantro_g1_h264_dec_run(struct hantro_ctx *ctx)
hantro_end_prepare_run(ctx);
+ switch (vpu->core_hw_dec_rev) {
+ case HANTRO_G1_REV:
+ reg = G1_REG_CONFIG_DEC_TIMEOUT_E |
+ G1_REG_CONFIG_DEC_OUT_ENDIAN |
+ G1_REG_CONFIG_DEC_STRENDIAN_E |
+ G1_REG_CONFIG_DEC_OUTSWAP32_E |
+ G1_REG_CONFIG_DEC_INSWAP32_E |
+ G1_REG_CONFIG_DEC_STRSWAP32_E;
+ vdpu_write_relaxed(vpu, reg, G1_REG_CONFIG);
+ break;
+ /* TODO: add VC8000 support */
+ }
+
+ regmap_field_write(fields->dec_clk_gate_e, 1);
+ regmap_field_write(fields->dec_max_burst, 16);
+ regmap_field_write(fields->dec_axi_rd_id, 16);
+
/* Start decoding! */
- vdpu_write_relaxed(vpu,
- G1_REG_CONFIG_DEC_AXI_RD_ID(0xffu) |
- G1_REG_CONFIG_DEC_TIMEOUT_E |
- G1_REG_CONFIG_DEC_OUT_ENDIAN |
- G1_REG_CONFIG_DEC_STRENDIAN_E |
- G1_REG_CONFIG_DEC_MAX_BURST(16) |
- G1_REG_CONFIG_DEC_OUTSWAP32_E |
- G1_REG_CONFIG_DEC_INSWAP32_E |
- G1_REG_CONFIG_DEC_STRSWAP32_E |
- G1_REG_CONFIG_DEC_CLK_GATE_E,
- G1_REG_CONFIG);
vdpu_write(vpu, G1_REG_INTERRUPT_DEC_E, G1_REG_INTERRUPT);
}
diff --git a/drivers/staging/media/hantro/hantro_regmap.c b/drivers/staging/media/hantro/hantro_regmap.c
index 2fc409cbd797..fbc39abedc7d 100644
--- a/drivers/staging/media/hantro/hantro_regmap.c
+++ b/drivers/staging/media/hantro/hantro_regmap.c
@@ -33,12 +33,54 @@ struct regmap_config hantro_regmap_enc = {
.name = "hantro_regmap_enc",
};
+struct hantro_field_dec {
+ struct reg_field cfg_dec_axi_rd_id;
+ struct reg_field cfg_dec_axi_wr_id;
+ struct reg_field cfg_dec_rlc_mode_e;
+ struct reg_field cfg_dec_mode;
+ struct reg_field cfg_dec_max_burst;
+ struct reg_field cfg_dec_apf_threshold;
+ struct reg_field cfg_dec_stream_len;
+ struct reg_field cfg_dec_init_qp;
+ struct reg_field cfg_dec_start_code_e;
+ struct reg_field cfg_dec_pic_width;
+ struct reg_field cfg_dec_pic_height;
+ struct reg_field cfg_dec_num_ref_frames;
+ struct reg_field cfg_dec_scaling_list_e;
+ struct reg_field cfg_dec_addr_str;
+ struct reg_field cfg_dec_addr_dst;
+ struct reg_field cfg_dec_ilace_mode;
+ struct reg_field cfg_dec_addr_qtable;
+ struct reg_field cfg_dec_addr_dir_mv;
+ struct reg_field cfg_dec_tiled_mode_lsb;
+ struct reg_field cfg_dec_clk_gate_e;
+};
+
struct hantro_field_enc {
/* TODO: populate encoder fields */
};
-struct hantro_field_dec {
- /* TODO: populate decoder fields */
+static const struct hantro_field_dec g1_field = {
+ .cfg_dec_tiled_mode_lsb = REG_FIELD(SWREG(2), 7, 7),
+ .cfg_dec_clk_gate_e = REG_FIELD(SWREG(2), 10, 10),
+ .cfg_dec_axi_rd_id = REG_FIELD(SWREG(2), 24, 31),
+ .cfg_dec_axi_wr_id = REG_FIELD(SWREG(3), 0, 7),
+ .cfg_dec_rlc_mode_e = REG_FIELD(SWREG(3), 27, 27),
+ .cfg_dec_mode = REG_FIELD(SWREG(3), 28, 31),
+ .cfg_dec_max_burst = REG_FIELD(SWREG(2), 0, 4),
+ .cfg_dec_apf_threshold = REG_FIELD(SWREG(55), 0, 13),
+ .cfg_dec_stream_len = REG_FIELD(SWREG(6), 0, 23),
+ .cfg_dec_init_qp = REG_FIELD(SWREG(6), 25, 30),
+ .cfg_dec_start_code_e = REG_FIELD(SWREG(6), 31, 31),
+ .cfg_dec_pic_width = REG_FIELD(SWREG(4), 23, 31),
+ .cfg_dec_pic_height = REG_FIELD(SWREG(4), 11, 18),
+ .cfg_dec_num_ref_frames = REG_FIELD(SWREG(4), 0, 4),
+ .cfg_dec_scaling_list_e = REG_FIELD(SWREG(5), 24, 24),
+ .cfg_dec_addr_str = REG_FIELD(SWREG(12), 0, 31),
+ .cfg_dec_addr_dst = REG_FIELD(SWREG(13), 0, 31),
+ .cfg_dec_ilace_mode = REG_FIELD(SWREG(13), 1, 1),
+ .cfg_dec_addr_qtable = REG_FIELD(SWREG(40), 0, 31),
+ .cfg_dec_addr_dir_mv = REG_FIELD(SWREG(41), 0, 31),
};
#define INIT_FIELD_CFG(f, codec, conf) ({ \
@@ -61,7 +103,27 @@ static int hantro_regmap_fields_init_dec(struct hantro_dev *vpu,
if (!vpu->reg_fields_dec)
return -ENOMEM;
- /* TODO: add decoder fields */
+ /* Decoder */
+ INIT_DEC_FIELD(dec_axi_wr_id);
+ INIT_DEC_FIELD(dec_axi_rd_id);
+ INIT_DEC_FIELD(dec_rlc_mode_e);
+ INIT_DEC_FIELD(dec_mode);
+ INIT_DEC_FIELD(dec_max_burst);
+ INIT_DEC_FIELD(dec_apf_threshold);
+ INIT_DEC_FIELD(dec_stream_len);
+ INIT_DEC_FIELD(dec_init_qp);
+ INIT_DEC_FIELD(dec_start_code_e);
+ INIT_DEC_FIELD(dec_pic_width);
+ INIT_DEC_FIELD(dec_pic_height);
+ INIT_DEC_FIELD(dec_num_ref_frames);
+ INIT_DEC_FIELD(dec_scaling_list_e);
+ INIT_DEC_FIELD(dec_addr_str);
+ INIT_DEC_FIELD(dec_addr_dst);
+ INIT_DEC_FIELD(dec_ilace_mode);
+ INIT_DEC_FIELD(dec_addr_qtable);
+ INIT_DEC_FIELD(dec_addr_dir_mv);
+ INIT_DEC_FIELD(dec_tiled_mode_lsb);
+ INIT_DEC_FIELD(dec_clk_gate_e);
return 0;
}
@@ -133,6 +195,17 @@ int hantro_regmap_init_dec(struct hantro_dev *vpu)
clk_bulk_disable(vpu->variant->num_clocks, vpu->clocks);
+ switch (vpu->core_hw_dec_rev) {
+ case HANTRO_G1_REV:
+ hantro_regmap_dec.max_register = 0x1D8;
+ field = &g1_field;
+ break;
+ default:
+ dev_err(vpu->dev, "Decoder revision 0x%x not supported by driver.\n",
+ vpu->core_hw_dec_rev);
+ return -ENODEV;
+ }
+
vpu->regs_dec = devm_regmap_init_mmio(vpu->dev, dec_base,
&hantro_regmap_dec);
if (IS_ERR(vpu->regs_dec)) {
diff --git a/drivers/staging/media/hantro/hantro_regmap.h b/drivers/staging/media/hantro/hantro_regmap.h
index 52668a8bafb9..e94fdc055784 100644
--- a/drivers/staging/media/hantro/hantro_regmap.h
+++ b/drivers/staging/media/hantro/hantro_regmap.h
@@ -9,8 +9,32 @@
#ifndef HANTRO_REGMAP_H_
#define HANTRO_REGMAP_H_
+#define HANTRO_G1_REV 0x6731
+
+#define SWREG(nr) ((nr) << 2)
+
struct hantro_regmap_fields_dec {
- /* TODO: populate decoder fields */
+ /* Decoder */
+ struct regmap_field *dec_axi_rd_id;
+ struct regmap_field *dec_axi_wr_id;
+ struct regmap_field *dec_max_burst;
+ struct regmap_field *dec_rlc_mode_e;
+ struct regmap_field *dec_mode;
+ struct regmap_field *dec_apf_threshold;
+ struct regmap_field *dec_stream_len;
+ struct regmap_field *dec_init_qp;
+ struct regmap_field *dec_start_code_e;
+ struct regmap_field *dec_pic_width;
+ struct regmap_field *dec_pic_height;
+ struct regmap_field *dec_num_ref_frames;
+ struct regmap_field *dec_scaling_list_e;
+ struct regmap_field *dec_addr_str;
+ struct regmap_field *dec_addr_dst;
+ struct regmap_field *dec_ilace_mode;
+ struct regmap_field *dec_addr_qtable;
+ struct regmap_field *dec_addr_dir_mv;
+ struct regmap_field *dec_tiled_mode_lsb;
+ struct regmap_field *dec_clk_gate_e;
};
struct hantro_regmap_fields_enc {
--
2.28.0
next prev parent reply other threads:[~2020-10-12 20:59 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-10-12 20:59 [PATCH 00/18] Add Hantro regmap and VC8000 h264 decode support Adrian Ratiu
2020-10-12 20:59 ` [PATCH 01/18] media: hantro: document all int reg bits up to vc8000 Adrian Ratiu
2020-10-12 20:59 ` [PATCH 02/18] media: hantro: make consistent use of decimal register notation Adrian Ratiu
2020-10-12 20:59 ` [PATCH 03/18] media: hantro: make G1_REG_SOFT_RESET Rockchip specific Adrian Ratiu
2020-10-12 20:59 ` [PATCH 04/18] media: hantro: add reset controller support Adrian Ratiu
2020-10-13 8:11 ` Philipp Zabel
2020-10-12 20:59 ` [PATCH 05/18] media: hantro: prepare clocks before variant inits are run Adrian Ratiu
2020-10-12 20:59 ` [PATCH 06/18] media: hantro: imx8mq: simplify ctrlblk reset logic Adrian Ratiu
2020-10-12 20:59 ` [PATCH 07/18] regmap: mmio: add config option to allow relaxed MMIO accesses Adrian Ratiu
2020-10-13 10:26 ` Mark Brown
2020-10-14 11:51 ` Adrian Ratiu
2020-10-14 12:12 ` Mark Brown
2020-10-14 13:00 ` Adrian Ratiu
2020-10-12 20:59 ` [PATCH 08/18] media: hantro: add initial MMIO regmap infrastructure Adrian Ratiu
2020-10-12 20:59 ` [PATCH 09/18] media: hantro: default regmap to relaxed MMIO Adrian Ratiu
2020-10-12 20:59 ` Adrian Ratiu [this message]
2020-10-12 20:59 ` [PATCH 11/18] media: hantro: convert G1 postproc to regmap Adrian Ratiu
2020-10-12 20:59 ` [PATCH 12/18] media: hantro: add VC8000D h264 decoding Adrian Ratiu
2020-10-12 20:59 ` [PATCH 13/18] media: hantro: add VC8000D postproc support Adrian Ratiu
2020-10-12 20:59 ` [PATCH 14/18] media: hantro: make PP enablement logic a bit smarter Adrian Ratiu
2020-10-12 20:59 ` [PATCH 15/18] media: hantro: add user-selectable, platform-selectable H264 High10 Adrian Ratiu
2020-10-12 20:59 ` [PATCH 16/18] media: hantro: rename h264_dec as it's not G1 specific anymore Adrian Ratiu
2020-10-12 20:59 ` [PATCH 17/18] media: hantro: add dump registers debug option before decode start Adrian Ratiu
2020-10-12 20:59 ` [PATCH 18/18] media: hantro: document encoder reg fields Adrian Ratiu
2020-10-12 23:39 ` [PATCH 00/18] Add Hantro regmap and VC8000 h264 decode support Jonas Karlman
2020-10-13 6:48 ` Adrian Ratiu
2020-10-29 12:38 ` Ezequiel Garcia
2020-10-29 16:21 ` Jonas Karlman
2020-11-03 15:27 ` Ezequiel Garcia
2020-10-29 13:07 ` Ezequiel Garcia
2020-10-29 14:15 ` Robin Murphy
2020-10-29 14:48 ` Mark Brown
2020-10-29 16:27 ` Ezequiel Garcia
2020-10-29 17:59 ` Mark Brown
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20201012205957.889185-11-adrian.ratiu@collabora.com \
--to=adrian.ratiu@collabora.com \
--cc=Peter.Fruehberger@de.bosch.com \
--cc=broonie@kernel.org \
--cc=daniel@ffwll.ch \
--cc=ezequiel@collabora.com \
--cc=kernel@collabora.com \
--cc=kuhanh.murugasen.krishnan@intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=mchehab@kernel.org \
--cc=p.zabel@pengutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).