Hi Greg,
I'm just about to use the generic UIO platform device driver in a
project. I noticed the driver registers itself simply as "uio". We
should choose a more unique name here. The oneliner below goes on top of
the other UIO patches you've got in your queue right now. This is not in
mainline yet.
Thanks,
Hans
-------------------------
The generic UIO platform device driver should be given a unique driver ID and
not just "uio". This is especially important since Magnus Damm announced a
similar driver named uio_pdrv_genirq. We should get the names right before
this hits mainline.
Signed-off-by: Hans J. Koch <[email protected]>
---
drivers/uio/uio_pdrv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Index: linux-2.6.26-rc/drivers/uio/uio_pdrv.c
===================================================================
--- linux-2.6.26-rc.orig/drivers/uio/uio_pdrv.c 2008-07-07 23:57:44.000000000 +0200
+++ linux-2.6.26-rc/drivers/uio/uio_pdrv.c 2008-07-07 23:58:26.000000000 +0200
@@ -12,7 +12,7 @@
#include <linux/uio_driver.h>
#include <linux/stringify.h>
-#define DRIVER_NAME "uio"
+#define DRIVER_NAME "uio_pdrv"
struct uio_platdata {
struct uio_info *uioinfo;
Hi Hans,
Hans J. Koch wrote:
> I'm just about to use the generic UIO platform device driver in a
> project. I noticed the driver registers itself simply as "uio". We
> should choose a more unique name here. The oneliner below goes on top of
> the other UIO patches you've got in your queue right now. This is not in
> mainline yet.
I don't care much, but IMHO "uio" is fine, because the name has a
context (i.e. it's the name of a platform device) and so adding "_pdrv"
doesn't yield a more clear result.
I don't claim to know all or even many platform drivers, the ones I
currently use (and that are in mainline, too) are:
gpio-keys
i2c-gpio
leds-gpio
physmap-flash
serial8250
and for all of these I consider it right not to have a "pdrv" suffix.
> The generic UIO platform device driver should be given a unique driver ID and
> not just "uio". This is especially important since Magnus Damm announced a
> similar driver named uio_pdrv_genirq. We should get the names right before
> this hits mainline.
So using uio_genirq here would be the logical consequence.
But as I said, I don't care much and both options are fine for me.
Best regards
Uwe
--
Uwe Kleine-K?nig, Software Engineer
Digi International GmbH Branch Breisach, K?ferstrasse 8, 79206 Breisach, Germany
Tax: 315/5781/0242 / VAT: DE153662976 / Reg. Amtsgericht Dortmund HRB 13962