* [PATCH] staging: media: tegra-vde: Remove variable
@ 2019-05-30 20:44 Nishka Dasgupta
2019-05-30 20:46 ` Greg KH
0 siblings, 1 reply; 2+ messages in thread
From: Nishka Dasgupta @ 2019-05-30 20:44 UTC (permalink / raw)
To: digetx, mchehab, gregkh, thierry.reding, jonathan, linux-media,
linux-tegra, devel, linux-kernel
Cc: Nishka Dasgupta
Remove unnecessary variable iram_tables and use its value directly.
Issue found using Coccinelle.
Signed-off-by: Nishka Dasgupta <nishkadg.linux@gmail.com>
---
drivers/staging/media/tegra-vde/tegra-vde.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/media/tegra-vde/tegra-vde.c b/drivers/staging/media/tegra-vde/tegra-vde.c
index a5020dbf6eef..3205973218e4 100644
--- a/drivers/staging/media/tegra-vde/tegra-vde.c
+++ b/drivers/staging/media/tegra-vde/tegra-vde.c
@@ -273,12 +273,10 @@ static void tegra_vde_setup_iram_entry(struct tegra_vde *vde,
unsigned int row,
u32 value1, u32 value2)
{
- u32 *iram_tables = vde->iram;
-
trace_vde_setup_iram_entry(table, row, value1, value2);
- iram_tables[0x20 * table + row * 2] = value1;
- iram_tables[0x20 * table + row * 2 + 1] = value2;
+ vde->iram[0x20 * table + row * 2] = value1;
+ vde->iram[0x20 * table + row * 2 + 1] = value2;
}
static void tegra_vde_setup_iram_tables(struct tegra_vde *vde,
--
2.19.1
^ permalink raw reply related [flat|nested] 2+ messages in thread* Re: [PATCH] staging: media: tegra-vde: Remove variable
2019-05-30 20:44 [PATCH] staging: media: tegra-vde: Remove variable Nishka Dasgupta
@ 2019-05-30 20:46 ` Greg KH
0 siblings, 0 replies; 2+ messages in thread
From: Greg KH @ 2019-05-30 20:46 UTC (permalink / raw)
To: Nishka Dasgupta
Cc: digetx, mchehab, thierry.reding, jonathan, linux-media,
linux-tegra, devel, linux-kernel
On Fri, May 31, 2019 at 02:14:39AM +0530, Nishka Dasgupta wrote:
> Remove unnecessary variable iram_tables and use its value directly.
> Issue found using Coccinelle.
>
> Signed-off-by: Nishka Dasgupta <nishkadg.linux@gmail.com>
> ---
> drivers/staging/media/tegra-vde/tegra-vde.c | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/staging/media/tegra-vde/tegra-vde.c b/drivers/staging/media/tegra-vde/tegra-vde.c
> index a5020dbf6eef..3205973218e4 100644
> --- a/drivers/staging/media/tegra-vde/tegra-vde.c
> +++ b/drivers/staging/media/tegra-vde/tegra-vde.c
> @@ -273,12 +273,10 @@ static void tegra_vde_setup_iram_entry(struct tegra_vde *vde,
> unsigned int row,
> u32 value1, u32 value2)
> {
> - u32 *iram_tables = vde->iram;
> -
> trace_vde_setup_iram_entry(table, row, value1, value2);
>
> - iram_tables[0x20 * table + row * 2] = value1;
> - iram_tables[0x20 * table + row * 2 + 1] = value2;
> + vde->iram[0x20 * table + row * 2] = value1;
> + vde->iram[0x20 * table + row * 2 + 1] = value2;
Ick, no, why? why why why?
You keep sending these patches, please take a bit of time to think about
what you are doing and see if the change actually makes sense.
Again, here, no, it does not.
greg k-h
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-05-30 20:46 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-05-30 20:44 [PATCH] staging: media: tegra-vde: Remove variable Nishka Dasgupta
2019-05-30 20:46 ` Greg KH
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox