* [PATCH] video: fbdev: radeon: Remove dead code
@ 2020-02-18 16:15 ` Souptick Joarder
2020-03-02 15:42 ` Bartlomiej Zolnierkiewicz
0 siblings, 1 reply; 2+ messages in thread
From: Souptick Joarder @ 2020-02-18 16:15 UTC (permalink / raw)
To: benh, b.zolnierkie; +Cc: linux-fbdev, linux-kernel, dri-devel, Souptick Joarder
This is dead code since 3.15 and can be removed if not
going to be useful further.
Signed-off-by: Souptick Joarder <jrdr.linux@gmail.com>
---
drivers/video/fbdev/aty/radeon_base.c | 16 ----------------
1 file changed, 16 deletions(-)
diff --git a/drivers/video/fbdev/aty/radeon_base.c b/drivers/video/fbdev/aty/radeon_base.c
index 3af00e3..ccf888e 100644
--- a/drivers/video/fbdev/aty/radeon_base.c
+++ b/drivers/video/fbdev/aty/radeon_base.c
@@ -849,12 +849,6 @@ static int radeonfb_check_var (struct fb_var_screeninfo *var, struct fb_info *in
case 9 ... 16:
v.bits_per_pixel = 16;
break;
- case 17 ... 24:
-#if 0 /* Doesn't seem to work */
- v.bits_per_pixel = 24;
- break;
-#endif
- return -EINVAL;
case 25 ... 32:
v.bits_per_pixel = 32;
break;
@@ -2548,16 +2542,6 @@ static void radeonfb_pci_unregister(struct pci_dev *pdev)
if (rinfo->mon2_EDID)
sysfs_remove_bin_file(&rinfo->pdev->dev.kobj, &edid2_attr);
-#if 0
- /* restore original state
- *
- * Doesn't quite work yet, I suspect if we come from a legacy
- * VGA mode (or worse, text mode), we need to do some VGA black
- * magic here that I know nothing about. --BenH
- */
- radeon_write_mode (rinfo, &rinfo->init_state, 1);
- #endif
-
del_timer_sync(&rinfo->lvds_timer);
arch_phys_wc_del(rinfo->wc_cookie);
unregister_framebuffer(info);
--
1.9.1
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] video: fbdev: radeon: Remove dead code
2020-02-18 16:15 ` [PATCH] video: fbdev: radeon: Remove dead code Souptick Joarder
@ 2020-03-02 15:42 ` Bartlomiej Zolnierkiewicz
0 siblings, 0 replies; 2+ messages in thread
From: Bartlomiej Zolnierkiewicz @ 2020-03-02 15:42 UTC (permalink / raw)
To: Souptick Joarder; +Cc: linux-fbdev, linux-kernel, dri-devel
On 2/18/20 5:15 PM, Souptick Joarder wrote:
> This is dead code since 3.15 and can be removed if not
> going to be useful further.
>
> Signed-off-by: Souptick Joarder <jrdr.linux@gmail.com>
Patch queued for v5.7, thanks.
Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics
> ---
> drivers/video/fbdev/aty/radeon_base.c | 16 ----------------
> 1 file changed, 16 deletions(-)
>
> diff --git a/drivers/video/fbdev/aty/radeon_base.c b/drivers/video/fbdev/aty/radeon_base.c
> index 3af00e3..ccf888e 100644
> --- a/drivers/video/fbdev/aty/radeon_base.c
> +++ b/drivers/video/fbdev/aty/radeon_base.c
> @@ -849,12 +849,6 @@ static int radeonfb_check_var (struct fb_var_screeninfo *var, struct fb_info *in
> case 9 ... 16:
> v.bits_per_pixel = 16;
> break;
> - case 17 ... 24:
> -#if 0 /* Doesn't seem to work */
> - v.bits_per_pixel = 24;
> - break;
> -#endif
> - return -EINVAL;
> case 25 ... 32:
> v.bits_per_pixel = 32;
> break;
> @@ -2548,16 +2542,6 @@ static void radeonfb_pci_unregister(struct pci_dev *pdev)
> if (rinfo->mon2_EDID)
> sysfs_remove_bin_file(&rinfo->pdev->dev.kobj, &edid2_attr);
>
> -#if 0
> - /* restore original state
> - *
> - * Doesn't quite work yet, I suspect if we come from a legacy
> - * VGA mode (or worse, text mode), we need to do some VGA black
> - * magic here that I know nothing about. --BenH
> - */
> - radeon_write_mode (rinfo, &rinfo->init_state, 1);
> - #endif
> -
> del_timer_sync(&rinfo->lvds_timer);
> arch_phys_wc_del(rinfo->wc_cookie);
> unregister_framebuffer(info);
>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-03-02 15:43 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <CGME20200218160846eucas1p238fa498fc5fc35f4865f0920c5f7c76f@eucas1p2.samsung.com>
2020-02-18 16:15 ` [PATCH] video: fbdev: radeon: Remove dead code Souptick Joarder
2020-03-02 15:42 ` Bartlomiej Zolnierkiewicz
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).