* [PATCH] flexcop: kill a WARN_ON in xlate_proc_name
@ 2010-09-03 20:44 Kyle McMartin
2010-12-21 21:22 ` Kyle McMartin
0 siblings, 1 reply; 2+ messages in thread
From: Kyle McMartin @ 2010-09-03 20:44 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: linux-kernel
From: Kyle McMartin <kyle@redhat.com>
The flexcop driver is currently trying to register DRIVER_NAME for its
irq name. Use a less braindead name (the module name) to
1) avoid a warning in xlate_proc_name
2) have a nice entry in /proc/irq/$n/
(https://bugzilla.redhat.com/show_bug.cgi?id=575873)
Signed-off-by: Kyle McMartin <kyle@redhat.com>
---
diff --git a/drivers/media/dvb/b2c2/flexcop-pci.c b/drivers/media/dvb/b2c2/flexcop-pci.c
index 227c020..7465308 100644
--- a/drivers/media/dvb/b2c2/flexcop-pci.c
+++ b/drivers/media/dvb/b2c2/flexcop-pci.c
@@ -39,6 +39,7 @@ MODULE_PARM_DESC(debug,
#define DRIVER_VERSION "0.1"
#define DRIVER_NAME "Technisat/B2C2 FlexCop II/IIb/III Digital TV PCI Driver"
+#define FLEXCOP_MODULE_NAME "b2c2-flexcop"
#define DRIVER_AUTHOR "Patrick Boettcher <patrick.boettcher@desy.de>"
struct flexcop_pci {
@@ -299,7 +300,7 @@ static int flexcop_pci_init(struct flexcop_pci *fc_pci)
return ret;
pci_set_master(fc_pci->pdev);
- if ((ret = pci_request_regions(fc_pci->pdev, DRIVER_NAME)) != 0)
+ if ((ret = pci_request_regions(fc_pci->pdev, FLEXCOP_MODULE_NAME)) != 0)
goto err_pci_disable_device;
fc_pci->io_mem = pci_iomap(fc_pci->pdev, 0, 0x800);
@@ -313,7 +314,7 @@ static int flexcop_pci_init(struct flexcop_pci *fc_pci)
pci_set_drvdata(fc_pci->pdev, fc_pci);
spin_lock_init(&fc_pci->irq_lock);
if ((ret = request_irq(fc_pci->pdev->irq, flexcop_pci_isr,
- IRQF_SHARED, DRIVER_NAME, fc_pci)) != 0)
+ IRQF_SHARED, FLEXCOP_MODULE_NAME, fc_pci)) != 0)
goto err_pci_iounmap;
fc_pci->init_state |= FC_PCI_INIT;
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] flexcop: kill a WARN_ON in xlate_proc_name
2010-09-03 20:44 [PATCH] flexcop: kill a WARN_ON in xlate_proc_name Kyle McMartin
@ 2010-12-21 21:22 ` Kyle McMartin
0 siblings, 0 replies; 2+ messages in thread
From: Kyle McMartin @ 2010-12-21 21:22 UTC (permalink / raw)
To: Mauro Carvalho Chehab; +Cc: linux-kernel
On Fri, Sep 03, 2010 at 04:44:46PM -0400, Kyle McMartin wrote:
> From: Kyle McMartin <kyle@redhat.com>
>
> The flexcop driver is currently trying to register DRIVER_NAME for its
> irq name. Use a less braindead name (the module name) to
> 1) avoid a warning in xlate_proc_name
> 2) have a nice entry in /proc/irq/$n/
>
> (https://bugzilla.redhat.com/show_bug.cgi?id=575873)
>
> Signed-off-by: Kyle McMartin <kyle@redhat.com>
ping?
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2010-12-21 21:22 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-09-03 20:44 [PATCH] flexcop: kill a WARN_ON in xlate_proc_name Kyle McMartin
2010-12-21 21:22 ` Kyle McMartin
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.