Re: [PATCH] Staging: comedi: apci_1564: Fix long CamelCase function names

From: Ian Abbott
Date: Tue Mar 11 2014 - 10:31:38 EST


On 2014-03-09 07:26, Fred Akers wrote:
This patch fixes a few function names that are very long and are
not in the correct naming style

Signed-off-by: Fred Akers <knivey@xxxxxxxxxx>
---
.../comedi/drivers/addi-data/hwdrv_apci1564.c | 62 ++++++++++------------
drivers/staging/comedi/drivers/addi_apci_1564.c | 16 +++---
2 files changed, 36 insertions(+), 42 deletions(-)

diff --git a/drivers/staging/comedi/drivers/addi-data/hwdrv_apci1564.c b/drivers/staging/comedi/drivers/addi-data/hwdrv_apci1564.c
index a31e115..cb426e7 100644
--- a/drivers/staging/comedi/drivers/addi-data/hwdrv_apci1564.c
+++ b/drivers/staging/comedi/drivers/addi-data/hwdrv_apci1564.c
@@ -116,7 +116,7 @@ static unsigned int ui_InterruptData, ui_Type;

/*
+----------------------------------------------------------------------------+
-| Function Name : int i_APCI1564_ConfigDigitalInput |
+| Function Name : int i_apci1564_di_config |

It would be nice to get rid of the silly i_, v_ prefixes as well!

--
-=( Ian Abbott @ MEV Ltd. E-mail: <abbotti@xxxxxxxxx> )=-
-=( Tel: +44 (0)161 477 1898 FAX: +44 (0)161 718 3587 )=-
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/