Skip to content

Commit 6168cbb

Browse files
6by96by9
committed
media: hevc_dec: Add in downstream single planar SAND variant
Upstream will take the multi-planar SAND format, but add back in the downstream single planar variant for backwards compatibility Signed-off-by: Dave Stevenson <[email protected]>
1 parent 2a167a9 commit 6168cbb

File tree

2 files changed

+153
-46
lines changed

2 files changed

+153
-46
lines changed

‎drivers/media/platform/raspberrypi/hevc_dec/hevc_d_h265.c‎

Lines changed: 71 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -1748,12 +1748,26 @@ static void hevc_d_h265_setup(struct hevc_d_ctx *ctx, struct hevc_d_run *run)
17481748
de->bit_copy_gptr = ctx->bitbufs + ctx->p1idx;
17491749
de->bit_copy_len = 0;
17501750

1751-
de->luma_stride = ctx->dst_fmt.height * 128;
1752-
de->frame_luma_addr =
1753-
vb2_dma_contig_plane_dma_addr(&run->dst->vb2_buf, 0);
1754-
de->chroma_stride = de->luma_stride / 2;
1755-
de->frame_chroma_addr =
1756-
vb2_dma_contig_plane_dma_addr(&run->dst->vb2_buf, 1);
1751+
switch (ctx->dst_fmt.pixelformat) {
1752+
case V4L2_PIX_FMT_NV12MT_COL128:
1753+
case V4L2_PIX_FMT_NV12MT_10_COL128:
1754+
de->luma_stride = ctx->dst_fmt.height * 128;
1755+
de->frame_luma_addr =
1756+
vb2_dma_contig_plane_dma_addr(&run->dst->vb2_buf, 0);
1757+
de->chroma_stride = de->luma_stride / 2;
1758+
de->frame_chroma_addr =
1759+
vb2_dma_contig_plane_dma_addr(&run->dst->vb2_buf, 1);
1760+
break;
1761+
case V4L2_PIX_FMT_NV12_COL128:
1762+
case V4L2_PIX_FMT_NV12_10_COL128:
1763+
de->luma_stride = ctx->dst_fmt.plane_fmt[0].bytesperline * 128;
1764+
de->frame_luma_addr =
1765+
vb2_dma_contig_plane_dma_addr(&run->dst->vb2_buf, 0);
1766+
de->chroma_stride = de->luma_stride;
1767+
de->frame_chroma_addr = de->frame_luma_addr +
1768+
(ctx->dst_fmt.height * 128);
1769+
break;
1770+
}
17571771
de->frame_aux = NULL;
17581772

17591773
if (s->sps.bit_depth_luma_minus8 !=
@@ -1766,15 +1780,19 @@ static void hevc_d_h265_setup(struct hevc_d_ctx *ctx, struct hevc_d_run *run)
17661780
}
17671781
if (s->sps.bit_depth_luma_minus8 == 0) {
17681782
if (ctx->dst_fmt.pixelformat !=
1769-
V4L2_PIX_FMT_NV12MT_COL128) {
1783+
V4L2_PIX_FMT_NV12MT_COL128 &&
1784+
ctx->dst_fmt.pixelformat !=
1785+
V4L2_PIX_FMT_NV12_COL128) {
17701786
v4l2_err(&dev->v4l2_dev,
17711787
"Pixel format %#x != NV12MT_COL128 for 8-bit output",
17721788
ctx->dst_fmt.pixelformat);
17731789
goto fail;
17741790
}
17751791
} else if (s->sps.bit_depth_luma_minus8 == 2) {
17761792
if (ctx->dst_fmt.pixelformat !=
1777-
V4L2_PIX_FMT_NV12MT_10_COL128) {
1793+
V4L2_PIX_FMT_NV12MT_10_COL128 &&
1794+
ctx->dst_fmt.pixelformat !=
1795+
V4L2_PIX_FMT_NV12_10_COL128) {
17781796
v4l2_err(&dev->v4l2_dev,
17791797
"Pixel format %#x != NV12MT_10_COL128 for 10-bit output",
17801798
ctx->dst_fmt.pixelformat);
@@ -1786,20 +1804,40 @@ static void hevc_d_h265_setup(struct hevc_d_ctx *ctx, struct hevc_d_run *run)
17861804
s->sps.bit_depth_luma_minus8 + 8);
17871805
goto fail;
17881806
}
1789-
if (run->dst->vb2_buf.num_planes != 2) {
1790-
v4l2_warn(&dev->v4l2_dev, "Capture planes (%d) != 2\n",
1791-
run->dst->vb2_buf.num_planes);
1792-
goto fail;
1793-
}
1794-
if (run->dst->planes[0].length < ctx->dst_fmt.plane_fmt[0].sizeimage ||
1795-
run->dst->planes[1].length < ctx->dst_fmt.plane_fmt[1].sizeimage) {
1796-
v4l2_warn(&dev->v4l2_dev,
1797-
"Capture planes length (%d/%d) < sizeimage (%d/%d)\n",
1798-
run->dst->planes[0].length,
1799-
run->dst->planes[1].length,
1800-
ctx->dst_fmt.plane_fmt[0].sizeimage,
1801-
ctx->dst_fmt.plane_fmt[1].sizeimage);
1802-
goto fail;
1807+
switch (ctx->dst_fmt.pixelformat) {
1808+
case V4L2_PIX_FMT_NV12MT_COL128:
1809+
case V4L2_PIX_FMT_NV12MT_10_COL128:
1810+
if (run->dst->vb2_buf.num_planes != 2) {
1811+
v4l2_warn(&dev->v4l2_dev, "Capture planes (%d) != 2\n",
1812+
run->dst->vb2_buf.num_planes);
1813+
goto fail;
1814+
}
1815+
if (run->dst->planes[0].length < ctx->dst_fmt.plane_fmt[0].sizeimage ||
1816+
run->dst->planes[1].length < ctx->dst_fmt.plane_fmt[1].sizeimage) {
1817+
v4l2_warn(&dev->v4l2_dev,
1818+
"Capture planes length (%d/%d) < sizeimage (%d/%d)\n",
1819+
run->dst->planes[0].length,
1820+
run->dst->planes[1].length,
1821+
ctx->dst_fmt.plane_fmt[0].sizeimage,
1822+
ctx->dst_fmt.plane_fmt[1].sizeimage);
1823+
goto fail;
1824+
}
1825+
break;
1826+
case V4L2_PIX_FMT_NV12_COL128:
1827+
case V4L2_PIX_FMT_NV12_10_COL128:
1828+
if (run->dst->vb2_buf.num_planes != 1) {
1829+
v4l2_warn(&dev->v4l2_dev, "Capture planes (%d) != 1\n",
1830+
run->dst->vb2_buf.num_planes);
1831+
goto fail;
1832+
}
1833+
if (run->dst->planes[0].length < ctx->dst_fmt.plane_fmt[0].sizeimage) {
1834+
v4l2_warn(&dev->v4l2_dev,
1835+
"Capture planes length (%d) < sizeimage (%d)\n",
1836+
run->dst->planes[0].length,
1837+
ctx->dst_fmt.plane_fmt[0].sizeimage);
1838+
goto fail;
1839+
}
1840+
break;
18031841
}
18041842

18051843
/*
@@ -1993,8 +2031,13 @@ static void hevc_d_h265_setup(struct hevc_d_ctx *ctx, struct hevc_d_run *run)
19932031

19942032
de->ref_addrs[i][0] =
19952033
vb2_dma_contig_plane_dma_addr(buf, 0);
1996-
de->ref_addrs[i][1] =
1997-
vb2_dma_contig_plane_dma_addr(buf, 1);
2034+
if (ctx->dst_fmt.pixelformat == V4L2_PIX_FMT_NV12MT_COL128 ||
2035+
ctx->dst_fmt.pixelformat == V4L2_PIX_FMT_NV12MT_10_COL128)
2036+
de->ref_addrs[i][1] =
2037+
vb2_dma_contig_plane_dma_addr(buf, 1);
2038+
else
2039+
de->ref_addrs[i][1] = de->ref_addrs[i][0] +
2040+
(ctx->dst_fmt.height * 128);
19982041
}
19992042

20002043
/* Move DPB from temp */
@@ -2629,9 +2672,11 @@ static int try_ctrl_sps(struct v4l2_ctrl *ctrl)
26292672
return 0;
26302673

26312674
if ((sps->bit_depth_luma_minus8 == 0 &&
2632-
ctx->dst_fmt.pixelformat != V4L2_PIX_FMT_NV12MT_COL128) ||
2675+
ctx->dst_fmt.pixelformat != V4L2_PIX_FMT_NV12MT_COL128 &&
2676+
ctx->dst_fmt.pixelformat != V4L2_PIX_FMT_NV12_COL128) ||
26332677
(sps->bit_depth_luma_minus8 == 2 &&
2634-
ctx->dst_fmt.pixelformat != V4L2_PIX_FMT_NV12MT_10_COL128)) {
2678+
ctx->dst_fmt.pixelformat != V4L2_PIX_FMT_NV12MT_10_COL128 &&
2679+
ctx->dst_fmt.pixelformat != V4L2_PIX_FMT_NV12_10_COL128)) {
26352680
v4l2_warn(&dev->v4l2_dev,
26362681
"SPS luma depth %d does not match capture format\n",
26372682
sps->bit_depth_luma_minus8 + 8);

‎drivers/media/platform/raspberrypi/hevc_dec/hevc_d_video.c‎

Lines changed: 82 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -150,17 +150,57 @@ static void hevc_d_prepare_dst_format(struct v4l2_pix_format_mplane *pix_fmt)
150150
bytesperline = width * 4 / 3;
151151
sizeimage = bytesperline * height;
152152
break;
153+
154+
case V4L2_PIX_FMT_NV12_COL128:
155+
/* Width rounds up to columns */
156+
width = ALIGN(width, 128);
157+
height = ALIGN(height, 8);
158+
159+
/* column height
160+
* Accept suggested shape if at least min & < 2 * min
161+
*/
162+
bytesperline = constrain2x(bytesperline, height * 3 / 2);
163+
sizeimage = bytesperline * width;
164+
break;
165+
166+
case V4L2_PIX_FMT_NV12_10_COL128:
167+
/* width in pixels (3 pels = 4 bytes) rounded to 128 byte
168+
* columns
169+
*/
170+
width = ALIGN(((width + 2) / 3), 32) * 3;
171+
height = ALIGN(height, 8);
172+
173+
/* column height
174+
* Accept suggested shape if at least min & < 2 * min
175+
*/
176+
bytesperline = constrain2x(bytesperline, height * 3 / 2);
177+
sizeimage = bytesperline * width * 4 / 3;
178+
break;
179+
153180
}
154181

155182
pix_fmt->width = width;
156183
pix_fmt->height = height;
157184

158185
pix_fmt->field = V4L2_FIELD_NONE;
159-
pix_fmt->plane_fmt[0].bytesperline = bytesperline;
160-
pix_fmt->plane_fmt[0].sizeimage = sizeimage;
161-
pix_fmt->plane_fmt[1].bytesperline = bytesperline;
162-
pix_fmt->plane_fmt[1].sizeimage = sizeimage / 2;
163-
pix_fmt->num_planes = 2;
186+
187+
switch (pix_fmt->pixelformat) {
188+
default:
189+
case V4L2_PIX_FMT_NV12MT_COL128:
190+
case V4L2_PIX_FMT_NV12MT_10_COL128:
191+
pix_fmt->plane_fmt[0].bytesperline = bytesperline;
192+
pix_fmt->plane_fmt[0].sizeimage = sizeimage;
193+
pix_fmt->plane_fmt[1].bytesperline = bytesperline;
194+
pix_fmt->plane_fmt[1].sizeimage = sizeimage / 2;
195+
pix_fmt->num_planes = 2;
196+
break;
197+
case V4L2_PIX_FMT_NV12_COL128:
198+
case V4L2_PIX_FMT_NV12_10_COL128:
199+
pix_fmt->plane_fmt[0].bytesperline = bytesperline;
200+
pix_fmt->plane_fmt[0].sizeimage = sizeimage;
201+
pix_fmt->num_planes = 1;
202+
break;
203+
}
164204
}
165205

166206
static int hevc_d_querycap(struct file *file, void *priv,
@@ -245,19 +285,30 @@ static int hevc_d_hevc_validate_sps(const struct v4l2_ctrl_hevc_sps * const sps)
245285
static u32 pixelformat_from_sps(const struct v4l2_ctrl_hevc_sps * const sps,
246286
const int index)
247287
{
288+
static const u32 all_formats[] = {
289+
V4L2_PIX_FMT_NV12MT_COL128,
290+
V4L2_PIX_FMT_NV12MT_10_COL128,
291+
V4L2_PIX_FMT_NV12_COL128,
292+
V4L2_PIX_FMT_NV12_10_COL128,
293+
};
248294
u32 pf = 0;
249295

250296
if (!is_sps_set(sps) || !hevc_d_hevc_validate_sps(sps)) {
251297
/* Treat this as an error? For now return both */
252-
if (index == 0)
253-
pf = V4L2_PIX_FMT_NV12MT_COL128;
254-
else if (index == 1)
255-
pf = V4L2_PIX_FMT_NV12MT_10_COL128;
256-
} else if (index == 0) {
257-
if (sps->bit_depth_luma_minus8 == 0)
258-
pf = V4L2_PIX_FMT_NV12MT_COL128;
259-
else if (sps->bit_depth_luma_minus8 == 2)
260-
pf = V4L2_PIX_FMT_NV12MT_10_COL128;
298+
if (index < ARRAY_SIZE(all_formats))
299+
pf = all_formats[index];
300+
} else {
301+
if (index == 0) {
302+
if (sps->bit_depth_luma_minus8 == 0)
303+
pf = V4L2_PIX_FMT_NV12MT_COL128;
304+
else if (sps->bit_depth_luma_minus8 == 2)
305+
pf = V4L2_PIX_FMT_NV12MT_10_COL128;
306+
} else if (index == 1) {
307+
if (sps->bit_depth_luma_minus8 == 0)
308+
pf = V4L2_PIX_FMT_NV12_COL128;
309+
else if (sps->bit_depth_luma_minus8 == 2)
310+
pf = V4L2_PIX_FMT_NV12_10_COL128;
311+
}
261312
}
262313

263314
return pf;
@@ -466,17 +517,28 @@ static int hevc_d_queue_setup(struct vb2_queue *vq, unsigned int *nbufs,
466517
pix_fmt = get_dst_fmt(ctx);
467518

468519
if (*nplanes) {
469-
if (*nplanes < 2 ||
470-
sizes[0] < pix_fmt->plane_fmt[0].sizeimage ||
471-
sizes[1] < pix_fmt->plane_fmt[1].sizeimage)
472-
return -EINVAL;
520+
if (pix_fmt->pixelformat == V4L2_PIX_FMT_NV12MT_COL128 ||
521+
pix_fmt->pixelformat == V4L2_PIX_FMT_NV12MT_10_COL128) {
522+
if (*nplanes < 2 ||
523+
sizes[0] < pix_fmt->plane_fmt[0].sizeimage ||
524+
sizes[1] < pix_fmt->plane_fmt[1].sizeimage)
525+
return -EINVAL;
526+
} else {
527+
if (sizes[0] < pix_fmt->plane_fmt[0].sizeimage)
528+
return -EINVAL;
529+
}
473530
} else {
474531
sizes[0] = pix_fmt->plane_fmt[0].sizeimage;
475532
if (V4L2_TYPE_IS_OUTPUT(vq->type)) {
476533
*nplanes = 1;
477534
} else {
478-
sizes[1] = pix_fmt->plane_fmt[1].sizeimage;
479-
*nplanes = 2;
535+
if (pix_fmt->pixelformat == V4L2_PIX_FMT_NV12MT_COL128 ||
536+
pix_fmt->pixelformat == V4L2_PIX_FMT_NV12MT_10_COL128) {
537+
sizes[1] = pix_fmt->plane_fmt[1].sizeimage;
538+
*nplanes = 2;
539+
} else {
540+
*nplanes = 1;
541+
}
480542
}
481543
}
482544

0 commit comments

Comments
 (0)