2020-04-17 08:58:38

by Jason Yan

Subject: [PATCH] video: fbdev: aty128fb: remove unused 'sdr_64'

Fix the following gcc warning:

drivers/video/fbdev/aty/aty128fb.c:337:36: warning: ‘sdr_64’ defined but
not used [-Wunused-const-variable=]
static const struct aty128_meminfo sdr_64 = {
^~~~~~

Reported-by: Hulk Robot <[email protected]>
Signed-off-by: Jason Yan <[email protected]>
---
drivers/video/fbdev/aty/aty128fb.c | 14 --------------
1 file changed, 14 deletions(-)

diff --git a/drivers/video/fbdev/aty/aty128fb.c b/drivers/video/fbdev/aty/aty128fb.c
index d7e41c8dd533..d05d4195acad 100644
--- a/drivers/video/fbdev/aty/aty128fb.c
+++ b/drivers/video/fbdev/aty/aty128fb.c
@@ -334,20 +334,6 @@ static const struct aty128_meminfo sdr_128 = {
.name = "128-bit SDR SGRAM (1:1)",
};

-static const struct aty128_meminfo sdr_64 = {
- .ML = 4,
- .MB = 8,
- .Trcd = 3,
- .Trp = 3,
- .Twr = 1,
- .CL = 3,
- .Tr2w = 1,
- .LoopLatency = 17,
- .DspOn = 46,
- .Rloop = 17,
- .name = "64-bit SDR SGRAM (1:1)",
-};
-
static const struct aty128_meminfo sdr_sgram = {
.ML = 4,
.MB = 4,
--
2.21.1

2020-04-17 14:12:19

by Bartlomiej Zolnierkiewicz

Subject: Re: [PATCH] video: fbdev: aty128fb: remove unused 'sdr_64'

On 4/17/20 11:23 AM, Jason Yan wrote:
> Fix the following gcc warning:
>
> drivers/video/fbdev/aty/aty128fb.c:337:36: warning: ‘sdr_64’ defined but
> not used [-Wunused-const-variable=]
> static const struct aty128_meminfo sdr_64 = {
> ^~~~~~
>
> Reported-by: Hulk Robot <[email protected]>
> Signed-off-by: Jason Yan <[email protected]>

Patch queued for v5.8, thanks.

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics

> ---
> drivers/video/fbdev/aty/aty128fb.c | 14 --------------
> 1 file changed, 14 deletions(-)
>
> diff --git a/drivers/video/fbdev/aty/aty128fb.c b/drivers/video/fbdev/aty/aty128fb.c
> index d7e41c8dd533..d05d4195acad 100644
> --- a/drivers/video/fbdev/aty/aty128fb.c
> +++ b/drivers/video/fbdev/aty/aty128fb.c
> @@ -334,20 +334,6 @@ static const struct aty128_meminfo sdr_128 = {
> .name = "128-bit SDR SGRAM (1:1)",
> };
>
> -static const struct aty128_meminfo sdr_64 = {
> - .ML = 4,
> - .MB = 8,
> - .Trcd = 3,
> - .Trp = 3,
> - .Twr = 1,
> - .CL = 3,
> - .Tr2w = 1,
> - .LoopLatency = 17,
> - .DspOn = 46,
> - .Rloop = 17,
> - .name = "64-bit SDR SGRAM (1:1)",
> -};
> -
> static const struct aty128_meminfo sdr_sgram = {
> .ML = 4,
> .MB = 4,
>