diff options
author | Lidza Louina <lidza.louina@gmail.com> | 2013-08-29 19:00:37 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-08-30 14:28:58 -0400 |
commit | c211148fb6355f347ec819a05c3034acd56e75c1 (patch) | |
tree | c0fe6a7abf161d0a20d479e405c6a2342c54b57b | |
parent | f5adda3283fb4a2304a53e56699a9a0aadc3ab69 (diff) |
staging: dgnc: adds TODO
This patchs adds a TODO for the driver.
Signed-off-by: Lidza Louina <lidza.louina@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | drivers/staging/dgnc/TODO | 17 |
1 files changed, 17 insertions, 0 deletions
diff --git a/drivers/staging/dgnc/TODO b/drivers/staging/dgnc/TODO new file mode 100644 index 000000000000..1ff2d1874aa2 --- /dev/null +++ b/drivers/staging/dgnc/TODO | |||
@@ -0,0 +1,17 @@ | |||
1 | * remove kzalloc casts | ||
2 | * checkpatch fixes | ||
3 | * sparse fixes | ||
4 | * fix use of sizeof(). Example replace sizeof(struct board_t) | ||
5 | with sizeof(*brd) and remove sizeof(char) | ||
6 | * change name of board_t to dgnc_board | ||
7 | * split two assignments into the two assignments on two lines; | ||
8 | don't use two equals signs | ||
9 | * remove unecessary comments | ||
10 | * remove unecessary error messages. Example kzalloc() has its | ||
11 | own error message. Adding an extra one is useless. | ||
12 | * use goto statements for error handling when appropriate | ||
13 | * there is a lot of unecessary code in the driver. It was | ||
14 | originally a standalone driver. Remove uneeded code. | ||
15 | |||
16 | Please send patches to Greg Kroah-Hartman <greg@kroah.com> and | ||
17 | Cc: Lidza Louina <lidza.louina@gmail.com> | ||