* [PATCH 1/5] [media] s5p-g2d: Remove casting the return value which is a void pointer
@ 2013-09-09 5:52 Jingoo Han
2013-09-09 5:53 ` [PATCH 2/5] [media] m2m-deinterlace: " Jingoo Han
` (3 more replies)
0 siblings, 4 replies; 5+ messages in thread
From: Jingoo Han @ 2013-09-09 5:52 UTC (permalink / raw)
To: 'Mauro Carvalho Chehab'
Cc: linux-media, 'Kamil Debski', 'Kyungmin Park',
'Jingoo Han'
Casting the return value which is a void pointer is redundant.
The conversion from void pointer to any other pointer type is
guaranteed by the C programming language.
Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
drivers/media/platform/s5p-g2d/g2d.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/platform/s5p-g2d/g2d.c b/drivers/media/platform/s5p-g2d/g2d.c
index fd6289d..0b29483 100644
--- a/drivers/media/platform/s5p-g2d/g2d.c
+++ b/drivers/media/platform/s5p-g2d/g2d.c
@@ -840,7 +840,7 @@ put_clk:
static int g2d_remove(struct platform_device *pdev)
{
- struct g2d_dev *dev = (struct g2d_dev *)platform_get_drvdata(pdev);
+ struct g2d_dev *dev = platform_get_drvdata(pdev);
v4l2_info(&dev->v4l2_dev, "Removing " G2D_NAME);
v4l2_m2m_release(dev->m2m_dev);
--
1.7.10.4
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 2/5] [media] m2m-deinterlace: Remove casting the return value which is a void pointer
2013-09-09 5:52 [PATCH 1/5] [media] s5p-g2d: Remove casting the return value which is a void pointer Jingoo Han
@ 2013-09-09 5:53 ` Jingoo Han
2013-09-09 5:54 ` [PATCH 3/5] [media] mem2mem_testdev: " Jingoo Han
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Jingoo Han @ 2013-09-09 5:53 UTC (permalink / raw)
To: 'Mauro Carvalho Chehab'; +Cc: linux-media, 'Jingoo Han'
Casting the return value which is a void pointer is redundant.
The conversion from void pointer to any other pointer type is
guaranteed by the C programming language.
Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
drivers/media/platform/m2m-deinterlace.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/media/platform/m2m-deinterlace.c b/drivers/media/platform/m2m-deinterlace.c
index 540516c..36513e8 100644
--- a/drivers/media/platform/m2m-deinterlace.c
+++ b/drivers/media/platform/m2m-deinterlace.c
@@ -1084,8 +1084,7 @@ free_dev:
static int deinterlace_remove(struct platform_device *pdev)
{
- struct deinterlace_dev *pcdev =
- (struct deinterlace_dev *)platform_get_drvdata(pdev);
+ struct deinterlace_dev *pcdev = platform_get_drvdata(pdev);
v4l2_info(&pcdev->v4l2_dev, "Removing " MEM2MEM_TEST_MODULE_NAME);
v4l2_m2m_release(pcdev->m2m_dev);
--
1.7.10.4
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 3/5] [media] mem2mem_testdev: Remove casting the return value which is a void pointer
2013-09-09 5:52 [PATCH 1/5] [media] s5p-g2d: Remove casting the return value which is a void pointer Jingoo Han
2013-09-09 5:53 ` [PATCH 2/5] [media] m2m-deinterlace: " Jingoo Han
@ 2013-09-09 5:54 ` Jingoo Han
2013-09-09 5:55 ` [PATCH 4/5] [media] ngene: " Jingoo Han
2013-09-09 5:56 ` [PATCH 5/5] [media] ddbridge: " Jingoo Han
3 siblings, 0 replies; 5+ messages in thread
From: Jingoo Han @ 2013-09-09 5:54 UTC (permalink / raw)
To: 'Mauro Carvalho Chehab'; +Cc: linux-media, 'Jingoo Han'
Casting the return value which is a void pointer is redundant.
The conversion from void pointer to any other pointer type is
guaranteed by the C programming language.
Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
drivers/media/platform/mem2mem_testdev.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/media/platform/mem2mem_testdev.c b/drivers/media/platform/mem2mem_testdev.c
index 6a17676..8df5975 100644
--- a/drivers/media/platform/mem2mem_testdev.c
+++ b/drivers/media/platform/mem2mem_testdev.c
@@ -1090,8 +1090,7 @@ unreg_dev:
static int m2mtest_remove(struct platform_device *pdev)
{
- struct m2mtest_dev *dev =
- (struct m2mtest_dev *)platform_get_drvdata(pdev);
+ struct m2mtest_dev *dev = platform_get_drvdata(pdev);
v4l2_info(&dev->v4l2_dev, "Removing " MEM2MEM_TEST_MODULE_NAME);
v4l2_m2m_release(dev->m2m_dev);
--
1.7.10.4
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 4/5] [media] ngene: Remove casting the return value which is a void pointer
2013-09-09 5:52 [PATCH 1/5] [media] s5p-g2d: Remove casting the return value which is a void pointer Jingoo Han
2013-09-09 5:53 ` [PATCH 2/5] [media] m2m-deinterlace: " Jingoo Han
2013-09-09 5:54 ` [PATCH 3/5] [media] mem2mem_testdev: " Jingoo Han
@ 2013-09-09 5:55 ` Jingoo Han
2013-09-09 5:56 ` [PATCH 5/5] [media] ddbridge: " Jingoo Han
3 siblings, 0 replies; 5+ messages in thread
From: Jingoo Han @ 2013-09-09 5:55 UTC (permalink / raw)
To: 'Mauro Carvalho Chehab'; +Cc: linux-media, 'Jingoo Han'
Casting the return value which is a void pointer is redundant.
The conversion from void pointer to any other pointer type is
guaranteed by the C programming language.
Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
drivers/media/pci/ngene/ngene-core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/pci/ngene/ngene-core.c b/drivers/media/pci/ngene/ngene-core.c
index 37ebc42..21d18d4 100644
--- a/drivers/media/pci/ngene/ngene-core.c
+++ b/drivers/media/pci/ngene/ngene-core.c
@@ -1622,7 +1622,7 @@ static void ngene_unlink(struct ngene *dev)
void ngene_shutdown(struct pci_dev *pdev)
{
- struct ngene *dev = (struct ngene *)pci_get_drvdata(pdev);
+ struct ngene *dev = pci_get_drvdata(pdev);
if (!dev || !shutdown_workaround)
return;
--
1.7.10.4
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 5/5] [media] ddbridge: Remove casting the return value which is a void pointer
2013-09-09 5:52 [PATCH 1/5] [media] s5p-g2d: Remove casting the return value which is a void pointer Jingoo Han
` (2 preceding siblings ...)
2013-09-09 5:55 ` [PATCH 4/5] [media] ngene: " Jingoo Han
@ 2013-09-09 5:56 ` Jingoo Han
3 siblings, 0 replies; 5+ messages in thread
From: Jingoo Han @ 2013-09-09 5:56 UTC (permalink / raw)
To: 'Mauro Carvalho Chehab'; +Cc: linux-media, 'Jingoo Han'
Casting the return value which is a void pointer is redundant.
The conversion from void pointer to any other pointer type is
guaranteed by the C programming language.
Signed-off-by: Jingoo Han <jg1.han@samsung.com>
---
drivers/media/pci/ddbridge/ddbridge-core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/pci/ddbridge/ddbridge-core.c b/drivers/media/pci/ddbridge/ddbridge-core.c
index 36e3452..9375f30 100644
--- a/drivers/media/pci/ddbridge/ddbridge-core.c
+++ b/drivers/media/pci/ddbridge/ddbridge-core.c
@@ -1544,7 +1544,7 @@ static void ddb_unmap(struct ddb *dev)
static void ddb_remove(struct pci_dev *pdev)
{
- struct ddb *dev = (struct ddb *) pci_get_drvdata(pdev);
+ struct ddb *dev = pci_get_drvdata(pdev);
ddb_ports_detach(dev);
ddb_i2c_release(dev);
--
1.7.10.4
^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-09-09 5:56 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-09-09 5:52 [PATCH 1/5] [media] s5p-g2d: Remove casting the return value which is a void pointer Jingoo Han
2013-09-09 5:53 ` [PATCH 2/5] [media] m2m-deinterlace: " Jingoo Han
2013-09-09 5:54 ` [PATCH 3/5] [media] mem2mem_testdev: " Jingoo Han
2013-09-09 5:55 ` [PATCH 4/5] [media] ngene: " Jingoo Han
2013-09-09 5:56 ` [PATCH 5/5] [media] ddbridge: " Jingoo Han
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).