* [2.6 patch] MAINTAINERS: remove HGA FRAMEBUFFER DRIVER entry
@ 2008-05-19 22:09 Adrian Bunk
0 siblings, 0 replies; 5+ messages in thread
From: Adrian Bunk @ 2008-05-19 22:09 UTC (permalink / raw)
To: adaplas; +Cc: linux-kernel
- maintainer has not been active for years
- maintainer email bounces
- URL no longer exists
- covered by the FRAMEBUFFER LAYER entry
Signed-off-by: Adrian Bunk <bunk@kernel.org>
---
fc7a740aef47772e5417c71f52aa9726eb5c716d diff --git a/MAINTAINERS b/MAINTAINERS
index 2f74092..af69cab 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1822,13 +1822,6 @@ M: zippel@linux-m68k.org
L: linux-kernel@vger.kernel.org
S: Maintained
-HGA FRAMEBUFFER DRIVER
-P: Ferenc Bakonyi
-M: fero@drama.obuda.kando.hu
-L: linux-nvidia@lists.surfsouth.com
-W: http://drama.obuda.kando.hu/~fero/cgi-bin/hgafb.shtml
-S: Maintained
-
HID CORE LAYER
P: Jiri Kosina
M: jkosina@suse.cz
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [2.6 patch] MAINTAINERS: remove HGA FRAMEBUFFER DRIVER entry
@ 2008-06-01 11:41 devzero
2008-06-02 16:39 ` [PATCH] hgafb: fix module removal Anton Vorontsov
0 siblings, 1 reply; 5+ messages in thread
From: devzero @ 2008-06-01 11:41 UTC (permalink / raw)
To: Adrian Bunk; +Cc: Andrew Morton, linux-kernel
addon:
- the driver has issues - see http://bugzilla.kernel.org/show_bug.cgi?id=9689
- after digging out author's recent email on the net and asking him, he confirmed, that he doesen`t maintain it anymore. he lost his hga hardware in 2001. also see bugzilla entry.
furthermore:
>hgafb: probe of hgafb.0 failed with error -22
>Device 'hgafb.0' does not have a release() function, it is broken and must be fixed.
>WARNING: at drivers/base/core.c:107 device_release()
as that bug hasn't been touched since january and kernel telling that it`s broken, i think it also should depend on BROKEN.
Acked-by: Roland Kletzing <devzero@web.de>
List: linux-kernel
Subject: [2.6 patch] MAINTAINERS: remove HGA FRAMEBUFFER DRIVER entry
From: Adrian Bunk <bunk () kernel ! org>
Date: 2008-05-19 22:09:28
Message-ID: 20080519220928.GW17716 () cs181133002 ! pp ! htv ! fi
[Download message RAW]
- maintainer has not been active for years
- maintainer email bounces
- URL no longer exists
- covered by the FRAMEBUFFER LAYER entry
Signed-off-by: Adrian Bunk <bunk@kernel.org>
---
fc7a740aef47772e5417c71f52aa9726eb5c716d diff --git a/MAINTAINERS b/MAINTAINERS
index 2f74092..af69cab 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1822,13 +1822,6 @@ M: zippel@linux-m68k.org
L: linux-kernel@vger.kernel.org
S: Maintained
-HGA FRAMEBUFFER DRIVER
-P: Ferenc Bakonyi
-M: fero@drama.obuda.kando.hu
-L: linux-nvidia@lists.surfsouth.com
-W: http://drama.obuda.kando.hu/~fero/cgi-bin/hgafb.shtml
-S: Maintained
-
HID CORE LAYER
P: Jiri Kosina
M: jkosina@suse.cz
_______________________________________________________________________
EINE FÜR ALLE: die kostenlose WEB.DE-Plattform für Freunde und Deine
Homepage mit eigenem Namen. Jetzt starten! http://unddu.de/?kid=kid@mf2
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH] hgafb: fix module removal
2008-06-01 11:41 [2.6 patch] MAINTAINERS: remove HGA FRAMEBUFFER DRIVER entry devzero
@ 2008-06-02 16:39 ` Anton Vorontsov
0 siblings, 0 replies; 5+ messages in thread
From: Anton Vorontsov @ 2008-06-02 16:39 UTC (permalink / raw)
To: devzero; +Cc: Adrian Bunk, Andrew Morton, linux-kernel
We should allocate platform device dynamically, otherwise it can't be removed.
Also, back in time it was acceptable to mix devices and platform_devices,
today this might cause all sort of bugs. Fix that too.
This patch was build-tested only.
Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com>
---
On Sun, Jun 01, 2008 at 01:41:30PM +0200, devzero@web.de wrote:
> addon:
>
> - the driver has issues - see http://bugzilla.kernel.org/show_bug.cgi?id=9689
> - after digging out author's recent email on the net and asking him,
> he confirmed, that he doesen`t maintain it anymore. he lost his hga hardware
> in 2001. also see bugzilla entry.
>
> furthermore:
>
> >hgafb: probe of hgafb.0 failed with error -22
> >Device 'hgafb.0' does not have a release() function, it is broken and must be fixed.
> >WARNING: at drivers/base/core.c:107 device_release()
>
> as that bug hasn't been touched since january and kernel telling that it`s
> broken, i think it also should depend on BROKEN.
It should be just trivially fixed. Here is the patch, I can only build
test it though.
drivers/video/hgafb.c | 39 ++++++++++++++++++++-------------------
1 files changed, 20 insertions(+), 19 deletions(-)
diff --git a/drivers/video/hgafb.c b/drivers/video/hgafb.c
index fb9e672..224ffb8 100644
--- a/drivers/video/hgafb.c
+++ b/drivers/video/hgafb.c
@@ -547,7 +547,7 @@ static struct fb_ops hgafb_ops = {
* Initialization
*/
-static int __init hgafb_probe(struct device *device)
+static int __init hgafb_probe(struct platform_device *pdev)
{
struct fb_info *info;
@@ -589,13 +589,13 @@ static int __init hgafb_probe(struct device *device)
printk(KERN_INFO "fb%d: %s frame buffer device\n",
info->node, info->fix.id);
- dev_set_drvdata(device, info);
+ platform_set_drvdata(pdev, info);
return 0;
}
-static int hgafb_remove(struct device *device)
+static int __devexit hgafb_remove(struct platform_device *pdev)
{
- struct fb_info *info = dev_get_drvdata(device);
+ struct fb_info *info = platform_get_drvdata(pdev);
hga_txt_mode();
hga_clear_screen();
@@ -616,16 +616,15 @@ static int hgafb_remove(struct device *device)
return 0;
}
-static struct device_driver hgafb_driver = {
- .name = "hgafb",
- .bus = &platform_bus_type,
+static struct platform_driver hgafb_driver = {
+ .driver = {
+ .name = "hgafb",
+ },
.probe = hgafb_probe,
- .remove = hgafb_remove,
+ .remove = __devexit_p(hgafb_remove),
};
-static struct platform_device hgafb_device = {
- .name = "hgafb",
-};
+static struct platform_device *hgafb_device;
static int __init hgafb_init(void)
{
@@ -634,21 +633,23 @@ static int __init hgafb_init(void)
if (fb_get_options("hgafb", NULL))
return -ENODEV;
- ret = driver_register(&hgafb_driver);
+ ret = platform_driver_register(&hgafb_driver);
+ if (ret)
+ return ret;
- if (!ret) {
- ret = platform_device_register(&hgafb_device);
- if (ret)
- driver_unregister(&hgafb_driver);
+ hgafb_device = platform_device_register_simple("hgafb", -1, NULL, 0);
+ if (!hgafb_device) {
+ platform_driver_unregister(&hgafb_driver);
+ return -ENOMEM;
}
- return ret;
+ return 0;
}
static void __exit hgafb_exit(void)
{
- platform_device_unregister(&hgafb_device);
- driver_unregister(&hgafb_driver);
+ platform_device_unregister(hgafb_device);
+ platform_driver_unregister(&hgafb_driver);
}
/* -------------------------------------------------------------------------
--
1.5.5.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [2.6 patch] MAINTAINERS: remove HGA FRAMEBUFFER DRIVER entry
@ 2008-06-08 19:57 devzero
0 siblings, 0 replies; 5+ messages in thread
From: devzero @ 2008-06-08 19:57 UTC (permalink / raw)
To: linux-kernel; +Cc: akpm, bunk
no need for merging this one ?
List: linux-kernel
Subject: Re: [2.6 patch] MAINTAINERS: remove HGA FRAMEBUFFER DRIVER entry
From: devzero () web ! de
Date: 2008-06-01 11:41:30
Message-ID: 299882843 () web ! de
[Download message RAW]
addon:
- the driver has issues - see http://bugzilla.kernel.org/show_bug.cgi?id=9689
- after digging out author's recent email on the net and asking him, he confirmed, \
that he doesen`t maintain it anymore. he lost his hga hardware in 2001. also see \
bugzilla entry.
furthermore:
> hgafb: probe of hgafb.0 failed with error -22
> Device 'hgafb.0' does not have a release() function, it is broken and must be \
> fixed.
> WARNING: at drivers/base/core.c:107 device_release()
as that bug hasn't been touched since january and kernel telling that it`s broken, i \
think it also should depend on BROKEN.
Acked-by: Roland Kletzing <devzero@web.de>
List: linux-kernel
Subject: [2.6 patch] MAINTAINERS: remove HGA FRAMEBUFFER DRIVER entry
From: Adrian Bunk <bunk () kernel ! org>
Date: 2008-05-19 22:09:28
Message-ID: 20080519220928.GW17716 () cs181133002 ! pp ! htv ! fi
[Download message RAW]
- maintainer has not been active for years
- maintainer email bounces
- URL no longer exists
- covered by the FRAMEBUFFER LAYER entry
Signed-off-by: Adrian Bunk <bunk@kernel.org>
---
fc7a740aef47772e5417c71f52aa9726eb5c716d diff --git a/MAINTAINERS b/MAINTAINERS
index 2f74092..af69cab 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1822,13 +1822,6 @@ M: zippel@linux-m68k.org
L: linux-kernel@vger.kernel.org
S: Maintained
-HGA FRAMEBUFFER DRIVER
-P: Ferenc Bakonyi
-M: fero@drama.obuda.kando.hu
-L: linux-nvidia@lists.surfsouth.com
-W: http://drama.obuda.kando.hu/~fero/cgi-bin/hgafb.shtml
-S: Maintained
-
HID CORE LAYER
P: Jiri Kosina
M: jkosina@suse.cz
_______________________________________________________________________
Jetzt neu! Schützen Sie Ihren PC mit McAfee und WEB.DE. 30 Tage
kostenlos testen. http://www.pc-sicherheit.web.de/startseite/?mc=022220
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [2.6 patch] MAINTAINERS: remove HGA FRAMEBUFFER DRIVER entry
@ 2008-08-06 21:10 Adrian Bunk
0 siblings, 0 replies; 5+ messages in thread
From: Adrian Bunk @ 2008-08-06 21:10 UTC (permalink / raw)
To: adaplas; +Cc: linux-kernel, Andrew Morton
- maintainer has not been active for years
- maintainer email bounces
- URL no longer exists
- covered by the FRAMEBUFFER LAYER entry
Signed-off-by: Adrian Bunk <bunk@kernel.org>
---
This patch has been sent on:
- 20 May 2008
fc7a740aef47772e5417c71f52aa9726eb5c716d diff --git a/MAINTAINERS b/MAINTAINERS
index 2f74092..af69cab 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1822,13 +1822,6 @@ M: zippel@linux-m68k.org
L: linux-kernel@vger.kernel.org
S: Maintained
-HGA FRAMEBUFFER DRIVER
-P: Ferenc Bakonyi
-M: fero@drama.obuda.kando.hu
-L: linux-nvidia@lists.surfsouth.com
-W: http://drama.obuda.kando.hu/~fero/cgi-bin/hgafb.shtml
-S: Maintained
-
HID CORE LAYER
P: Jiri Kosina
M: jkosina@suse.cz
^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2008-08-06 21:19 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-06-01 11:41 [2.6 patch] MAINTAINERS: remove HGA FRAMEBUFFER DRIVER entry devzero
2008-06-02 16:39 ` [PATCH] hgafb: fix module removal Anton Vorontsov
-- strict thread matches above, loose matches on Subject: below --
2008-08-06 21:10 [2.6 patch] MAINTAINERS: remove HGA FRAMEBUFFER DRIVER entry Adrian Bunk
2008-06-08 19:57 devzero
2008-05-19 22:09 Adrian Bunk
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox