hc
2023-12-11 d2ccde1c8e90d38cee87a1b0309ad2827f3fd30d
kernel/drivers/media/platform/rcar_fdp1.c
....@@ -947,7 +947,7 @@
947947 u32 rndctl;
948948
949949 pstride = q_data->format.plane_fmt[0].bytesperline
950
- << FD1_WPF_PSTRIDE_Y_SHIFT;
950
+ << FD1_WPF_PSTRIDE_Y_SHIFT;
951951
952952 if (q_data->format.num_planes > 1)
953953 pstride |= q_data->format.plane_fmt[1].bytesperline
....@@ -1141,8 +1141,8 @@
11411141 int dstbufs = 1;
11421142
11431143 dprintk(ctx->fdp1, "+ Src: %d : Dst: %d\n",
1144
- v4l2_m2m_num_src_bufs_ready(ctx->fh.m2m_ctx),
1145
- v4l2_m2m_num_dst_bufs_ready(ctx->fh.m2m_ctx));
1144
+ v4l2_m2m_num_src_bufs_ready(ctx->fh.m2m_ctx),
1145
+ v4l2_m2m_num_dst_bufs_ready(ctx->fh.m2m_ctx));
11461146
11471147 /* One output buffer is required for each field */
11481148 if (V4L2_FIELD_HAS_BOTH(src_q_data->format.field))
....@@ -1280,7 +1280,7 @@
12801280
12811281 fdp1_queue_field(ctx, fbuf);
12821282 dprintk(fdp1, "Queued Buffer [%d] last_field:%d\n",
1283
- i, fbuf->last_field);
1283
+ i, fbuf->last_field);
12841284 }
12851285
12861286 /* Queue as many jobs as our data provides for */
....@@ -1339,7 +1339,7 @@
13391339 fdp1_job_free(fdp1, job);
13401340
13411341 dprintk(fdp1, "curr_ctx->num_processed %d curr_ctx->translen %d\n",
1342
- ctx->num_processed, ctx->translen);
1342
+ ctx->num_processed, ctx->translen);
13431343
13441344 if (ctx->num_processed == ctx->translen ||
13451345 ctx->aborting) {
....@@ -1361,10 +1361,10 @@
13611361 static int fdp1_vidioc_querycap(struct file *file, void *priv,
13621362 struct v4l2_capability *cap)
13631363 {
1364
- strlcpy(cap->driver, DRIVER_NAME, sizeof(cap->driver));
1365
- strlcpy(cap->card, DRIVER_NAME, sizeof(cap->card));
1364
+ strscpy(cap->driver, DRIVER_NAME, sizeof(cap->driver));
1365
+ strscpy(cap->card, DRIVER_NAME, sizeof(cap->card));
13661366 snprintf(cap->bus_info, sizeof(cap->bus_info),
1367
- "platform:%s", DRIVER_NAME);
1367
+ "platform:%s", DRIVER_NAME);
13681368 return 0;
13691369 }
13701370
....@@ -1732,8 +1732,8 @@
17321732 static const struct v4l2_ioctl_ops fdp1_ioctl_ops = {
17331733 .vidioc_querycap = fdp1_vidioc_querycap,
17341734
1735
- .vidioc_enum_fmt_vid_cap_mplane = fdp1_enum_fmt_vid_cap,
1736
- .vidioc_enum_fmt_vid_out_mplane = fdp1_enum_fmt_vid_out,
1735
+ .vidioc_enum_fmt_vid_cap = fdp1_enum_fmt_vid_cap,
1736
+ .vidioc_enum_fmt_vid_out = fdp1_enum_fmt_vid_out,
17371737 .vidioc_g_fmt_vid_cap_mplane = fdp1_g_fmt,
17381738 .vidioc_g_fmt_vid_out_mplane = fdp1_g_fmt,
17391739 .vidioc_try_fmt_vid_cap_mplane = fdp1_try_fmt,
....@@ -1995,13 +1995,13 @@
19951995 /* Free smsk_data */
19961996 if (ctx->smsk_cpu) {
19971997 dma_free_coherent(ctx->fdp1->dev, ctx->smsk_size,
1998
- ctx->smsk_cpu, ctx->smsk_addr[0]);
1998
+ ctx->smsk_cpu, ctx->smsk_addr[0]);
19991999 ctx->smsk_addr[0] = ctx->smsk_addr[1] = 0;
20002000 ctx->smsk_cpu = NULL;
20012001 }
20022002
20032003 WARN(!list_empty(&ctx->fields_queue),
2004
- "Buffer queue not empty");
2004
+ "Buffer queue not empty");
20052005 } else {
20062006 /* Empty Capture queues (Jobs) */
20072007 struct fdp1_job *job;
....@@ -2023,10 +2023,10 @@
20232023 fdp1_field_complete(ctx, ctx->previous);
20242024
20252025 WARN(!list_empty(&ctx->fdp1->queued_job_list),
2026
- "Queued Job List not empty");
2026
+ "Queued Job List not empty");
20272027
20282028 WARN(!list_empty(&ctx->fdp1->hw_job_list),
2029
- "HW Job list not empty");
2029
+ "HW Job list not empty");
20302030 }
20312031 }
20322032
....@@ -2112,7 +2112,7 @@
21122112 fdp1_ctrl_deint_menu);
21132113
21142114 ctrl = v4l2_ctrl_new_std(&ctx->hdl, &fdp1_ctrl_ops,
2115
- V4L2_CID_MIN_BUFFERS_FOR_CAPTURE, 1, 2, 1, 1);
2115
+ V4L2_CID_MIN_BUFFERS_FOR_CAPTURE, 1, 2, 1, 1);
21162116 if (ctrl)
21172117 ctrl->flags |= V4L2_CTRL_FLAG_VOLATILE;
21182118
....@@ -2122,6 +2122,7 @@
21222122 if (ctx->hdl.error) {
21232123 ret = ctx->hdl.error;
21242124 v4l2_ctrl_handler_free(&ctx->hdl);
2125
+ kfree(ctx);
21252126 goto done;
21262127 }
21272128
....@@ -2341,16 +2342,16 @@
23412342 vfd->lock = &fdp1->dev_mutex;
23422343 vfd->v4l2_dev = &fdp1->v4l2_dev;
23432344 video_set_drvdata(vfd, fdp1);
2344
- strlcpy(vfd->name, fdp1_videodev.name, sizeof(vfd->name));
2345
+ strscpy(vfd->name, fdp1_videodev.name, sizeof(vfd->name));
23452346
2346
- ret = video_register_device(vfd, VFL_TYPE_GRABBER, 0);
2347
+ ret = video_register_device(vfd, VFL_TYPE_VIDEO, 0);
23472348 if (ret) {
23482349 v4l2_err(&fdp1->v4l2_dev, "Failed to register video device\n");
23492350 goto release_m2m;
23502351 }
23512352
2352
- v4l2_info(&fdp1->v4l2_dev,
2353
- "Device registered as /dev/video%d\n", vfd->num);
2353
+ v4l2_info(&fdp1->v4l2_dev, "Device registered as /dev/video%d\n",
2354
+ vfd->num);
23542355
23552356 /* Power up the cells to read HW */
23562357 pm_runtime_enable(&pdev->dev);
....@@ -2375,7 +2376,7 @@
23752376 break;
23762377 default:
23772378 dev_err(fdp1->dev, "FDP1 Unidentifiable (0x%08x)\n",
2378
- hw_version);
2379
+ hw_version);
23792380 }
23802381
23812382 /* Allow the hw to sleep until an open call puts it to use */