* [git:media_stage/master] media: c8sectpfe: remove redundant assignment to pointer tsin
@ 2021-12-07 10:29 Mauro Carvalho Chehab
0 siblings, 0 replies; 3+ messages in thread
From: Mauro Carvalho Chehab @ 2021-12-07 10:29 UTC (permalink / raw)
To: linuxtv-commits; +Cc: kernel-janitors, Patrice Chotard, Colin Ian King
This is an automatic generated email to let you know that the following patch were queued:
Subject: media: c8sectpfe: remove redundant assignment to pointer tsin
Author: Colin Ian King <colin.i.king@gmail.com>
Date: Sun Dec 5 01:37:45 2021 +0100
Pointer tsin is being assigned a value that is never read. The assignment
is redundant and can be removed.
Link: https://lore.kernel.org/linux-media/20211205003745.227491-1-colin.i.king@gmail.com
Cc: kernel-janitors@vger.kernel.org
Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c | 4 ----
1 file changed, 4 deletions(-)
---
diff --git a/drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c b/drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c
index 02dc78bd7fab..e1f520903248 100644
--- a/drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c
+++ b/drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c
@@ -930,12 +930,8 @@ static int configure_channels(struct c8sectpfei *fei)
/* iterate round each tsin and configure memdma descriptor and IB hw */
for_each_child_of_node(np, child) {
-
- tsin = fei->channel_data[index];
-
ret = configure_memdma_and_inputblock(fei,
fei->channel_data[index]);
-
if (ret) {
dev_err(fei->dev,
"configure_memdma_and_inputblock failed\n");
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [git:media_stage/master] media: c8sectpfe: remove redundant assignment to pointer tsin
@ 2021-12-07 10:26 Mauro Carvalho Chehab
0 siblings, 0 replies; 3+ messages in thread
From: Mauro Carvalho Chehab @ 2021-12-07 10:26 UTC (permalink / raw)
To: linuxtv-commits; +Cc: kernel-janitors, Patrice Chotard, Colin Ian King
This is an automatic generated email to let you know that the following patch were queued:
Subject: media: c8sectpfe: remove redundant assignment to pointer tsin
Author: Colin Ian King <colin.i.king@gmail.com>
Date: Sun Dec 5 01:37:45 2021 +0100
Pointer tsin is being assigned a value that is never read. The assignment
is redundant and can be removed.
Link: https://lore.kernel.org/linux-media/20211205003745.227491-1-colin.i.king@gmail.com
Cc: kernel-janitors@vger.kernel.org
Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c | 4 ----
1 file changed, 4 deletions(-)
---
diff --git a/drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c b/drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c
index 02dc78bd7fab..e1f520903248 100644
--- a/drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c
+++ b/drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c
@@ -930,12 +930,8 @@ static int configure_channels(struct c8sectpfei *fei)
/* iterate round each tsin and configure memdma descriptor and IB hw */
for_each_child_of_node(np, child) {
-
- tsin = fei->channel_data[index];
-
ret = configure_memdma_and_inputblock(fei,
fei->channel_data[index]);
-
if (ret) {
dev_err(fei->dev,
"configure_memdma_and_inputblock failed\n");
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [git:media_stage/master] media: c8sectpfe: remove redundant assignment to pointer tsin
@ 2021-12-06 13:46 Mauro Carvalho Chehab
0 siblings, 0 replies; 3+ messages in thread
From: Mauro Carvalho Chehab @ 2021-12-06 13:46 UTC (permalink / raw)
To: linuxtv-commits; +Cc: kernel-janitors, Colin Ian King
This is an automatic generated email to let you know that the following patch were queued:
Subject: media: c8sectpfe: remove redundant assignment to pointer tsin
Author: Colin Ian King <colin.i.king@gmail.com>
Date: Sun Dec 5 01:37:45 2021 +0100
Pointer tsin is being assigned a value that is never read. The assignment
is redundant and can be removed.
Link: https://lore.kernel.org/linux-media/20211205003745.227491-1-colin.i.king@gmail.com
Cc: kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org
Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c | 4 ----
1 file changed, 4 deletions(-)
---
diff --git a/drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c b/drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c
index 02dc78bd7fab..e1f520903248 100644
--- a/drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c
+++ b/drivers/media/platform/sti/c8sectpfe/c8sectpfe-core.c
@@ -930,12 +930,8 @@ static int configure_channels(struct c8sectpfei *fei)
/* iterate round each tsin and configure memdma descriptor and IB hw */
for_each_child_of_node(np, child) {
-
- tsin = fei->channel_data[index];
-
ret = configure_memdma_and_inputblock(fei,
fei->channel_data[index]);
-
if (ret) {
dev_err(fei->dev,
"configure_memdma_and_inputblock failed\n");
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-12-07 10:30 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-07 10:29 [git:media_stage/master] media: c8sectpfe: remove redundant assignment to pointer tsin Mauro Carvalho Chehab
-- strict thread matches above, loose matches on Subject: below --
2021-12-07 10:26 Mauro Carvalho Chehab
2021-12-06 13:46 Mauro Carvalho Chehab
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.