Skip to content

Commit 085b26d

Browse files
slongerbeammchehab
authored andcommitted
media: imx: Rename functions that add IPU-internal subdevs
For the functions that add and remove the internal IPU subdevice descriptors, rename them to make clear they are the subdevs internal to the IPU. Also rename the platform data structure for the internal IPU subdevices. No functional changes. Signed-off-by: Steve Longerbeam <[email protected]> Acked-by: Philipp Zabel <[email protected]> Cc: [email protected] Signed-off-by: Hans Verkuil <[email protected]> Signed-off-by: Mauro Carvalho Chehab <[email protected]>
1 parent 107927f commit 085b26d

File tree

5 files changed

+15
-15
lines changed

5 files changed

+15
-15
lines changed

drivers/staging/media/imx/imx-ic-common.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ static struct imx_ic_ops *ic_ops[IC_NUM_OPS] = {
2626

2727
static int imx_ic_probe(struct platform_device *pdev)
2828
{
29-
struct imx_media_internal_sd_platformdata *pdata;
29+
struct imx_media_ipu_internal_sd_pdata *pdata;
3030
struct imx_ic_priv *priv;
3131
int ret;
3232

drivers/staging/media/imx/imx-media-dev.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -477,10 +477,10 @@ static int imx_media_probe(struct platform_device *pdev)
477477
goto cleanup;
478478
}
479479

480-
ret = imx_media_add_internal_subdevs(imxmd);
480+
ret = imx_media_add_ipu_internal_subdevs(imxmd);
481481
if (ret) {
482482
v4l2_err(&imxmd->v4l2_dev,
483-
"add_internal_subdevs failed with %d\n", ret);
483+
"add_ipu_internal_subdevs failed with %d\n", ret);
484484
goto cleanup;
485485
}
486486

@@ -491,7 +491,7 @@ static int imx_media_probe(struct platform_device *pdev)
491491
return 0;
492492

493493
del_int:
494-
imx_media_remove_internal_subdevs(imxmd);
494+
imx_media_remove_ipu_internal_subdevs(imxmd);
495495
cleanup:
496496
v4l2_async_notifier_cleanup(&imxmd->notifier);
497497
v4l2_device_unregister(&imxmd->v4l2_dev);
@@ -508,7 +508,7 @@ static int imx_media_remove(struct platform_device *pdev)
508508
v4l2_info(&imxmd->v4l2_dev, "Removing imx-media\n");
509509

510510
v4l2_async_notifier_unregister(&imxmd->notifier);
511-
imx_media_remove_internal_subdevs(imxmd);
511+
imx_media_remove_ipu_internal_subdevs(imxmd);
512512
v4l2_async_notifier_cleanup(&imxmd->notifier);
513513
media_device_unregister(&imxmd->md);
514514
v4l2_device_unregister(&imxmd->v4l2_dev);

drivers/staging/media/imx/imx-media-internal-sd.c

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
/*
22
* Media driver for Freescale i.MX5/6 SOC
33
*
4-
* Adds the internal subdevices and the media links between them.
4+
* Adds the IPU internal subdevices and the media links between them.
55
*
66
* Copyright (c) 2016 Mentor Graphics Inc.
77
*
@@ -192,7 +192,7 @@ static struct v4l2_subdev *find_sink(struct imx_media_dev *imxmd,
192192

193193
/*
194194
* retrieve IPU id from subdev name, note: can't get this from
195-
* struct imx_media_internal_sd_platformdata because if src is
195+
* struct imx_media_ipu_internal_sd_pdata because if src is
196196
* a CSI, it has different struct ipu_client_platformdata which
197197
* does not contain IPU id.
198198
*/
@@ -270,7 +270,7 @@ static int add_internal_subdev(struct imx_media_dev *imxmd,
270270
const struct internal_subdev *isd,
271271
int ipu_id)
272272
{
273-
struct imx_media_internal_sd_platformdata pdata;
273+
struct imx_media_ipu_internal_sd_pdata pdata;
274274
struct platform_device_info pdevinfo = {};
275275
struct platform_device *pdev;
276276

@@ -328,7 +328,7 @@ static int add_ipu_internal_subdevs(struct imx_media_dev *imxmd, int ipu_id)
328328
return 0;
329329
}
330330

331-
int imx_media_add_internal_subdevs(struct imx_media_dev *imxmd)
331+
int imx_media_add_ipu_internal_subdevs(struct imx_media_dev *imxmd)
332332
{
333333
int ret;
334334

@@ -343,11 +343,11 @@ int imx_media_add_internal_subdevs(struct imx_media_dev *imxmd)
343343
return 0;
344344

345345
remove:
346-
imx_media_remove_internal_subdevs(imxmd);
346+
imx_media_remove_ipu_internal_subdevs(imxmd);
347347
return ret;
348348
}
349349

350-
void imx_media_remove_internal_subdevs(struct imx_media_dev *imxmd)
350+
void imx_media_remove_ipu_internal_subdevs(struct imx_media_dev *imxmd)
351351
{
352352
struct imx_media_async_subdev *imxasd;
353353
struct v4l2_async_subdev *asd;

drivers/staging/media/imx/imx-media-vdic.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -940,7 +940,7 @@ static const struct v4l2_subdev_internal_ops vdic_internal_ops = {
940940

941941
static int imx_vdic_probe(struct platform_device *pdev)
942942
{
943-
struct imx_media_internal_sd_platformdata *pdata;
943+
struct imx_media_ipu_internal_sd_pdata *pdata;
944944
struct vdic_priv *priv;
945945
int ret;
946946

drivers/staging/media/imx/imx-media.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,7 @@ struct imx_media_pad_vdev {
115115
struct list_head list;
116116
};
117117

118-
struct imx_media_internal_sd_platformdata {
118+
struct imx_media_ipu_internal_sd_pdata {
119119
char sd_name[V4L2_SUBDEV_NAME_SIZE];
120120
u32 grp_id;
121121
int ipu_id;
@@ -252,10 +252,10 @@ struct imx_media_fim *imx_media_fim_init(struct v4l2_subdev *sd);
252252
void imx_media_fim_free(struct imx_media_fim *fim);
253253

254254
/* imx-media-internal-sd.c */
255-
int imx_media_add_internal_subdevs(struct imx_media_dev *imxmd);
255+
int imx_media_add_ipu_internal_subdevs(struct imx_media_dev *imxmd);
256256
int imx_media_create_ipu_internal_links(struct imx_media_dev *imxmd,
257257
struct v4l2_subdev *sd);
258-
void imx_media_remove_internal_subdevs(struct imx_media_dev *imxmd);
258+
void imx_media_remove_ipu_internal_subdevs(struct imx_media_dev *imxmd);
259259

260260
/* imx-media-of.c */
261261
int imx_media_add_of_subdevs(struct imx_media_dev *dev,

0 commit comments

Comments
 (0)