diff options
author | Bastian Hecht <hechtb@googlemail.com> | 2012-10-19 06:15:34 -0400 |
---|---|---|
committer | Artem Bityutskiy <artem.bityutskiy@linux.intel.com> | 2012-11-15 08:37:50 -0500 |
commit | e8a9d8f31c592eea89f1b0d3fd425e7a96944e88 (patch) | |
tree | 704a94cb7e7a5ab31e57779c7b719ffc88e1bdda /include | |
parent | 5de0b52ea8f8f5149502867acff2efb5efaf1fc2 (diff) |
mtd: sh_flctl: Minor cleanups
Some small fixes to avoid sparse and smatch complain. Other cosmetic fixes
as well.
- Change of the type of the member index in struct sh_flctl from signed
to unsigned. We use index by addressing array members, so unsigned is more
concise here. Adapt functions relying on sh_flctl::index.
- Remove a blurring cast in write_fiforeg().
- Apply consistent naming scheme when refering to the data buffer.
- Shorten some unnecessarily verbose functions.
- Remove spaces at start of lines.
Signed-off-by: Bastian Hecht <hechtb@gmail.com>
Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Diffstat (limited to 'include')
-rw-r--r-- | include/linux/mtd/sh_flctl.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/mtd/sh_flctl.h b/include/linux/mtd/sh_flctl.h index 01e4b15b280e..481557688d05 100644 --- a/include/linux/mtd/sh_flctl.h +++ b/include/linux/mtd/sh_flctl.h | |||
@@ -147,7 +147,7 @@ struct sh_flctl { | |||
147 | 147 | ||
148 | uint8_t done_buff[2048 + 64]; /* max size 2048 + 64 */ | 148 | uint8_t done_buff[2048 + 64]; /* max size 2048 + 64 */ |
149 | int read_bytes; | 149 | int read_bytes; |
150 | int index; | 150 | unsigned int index; |
151 | int seqin_column; /* column in SEQIN cmd */ | 151 | int seqin_column; /* column in SEQIN cmd */ |
152 | int seqin_page_addr; /* page_addr in SEQIN cmd */ | 152 | int seqin_page_addr; /* page_addr in SEQIN cmd */ |
153 | uint32_t seqin_read_cmd; /* read cmd in SEQIN cmd */ | 153 | uint32_t seqin_read_cmd; /* read cmd in SEQIN cmd */ |