diff options
author | Bastian Hecht <hechtb@googlemail.com> | 2012-10-19 12:15:34 +0200 |
---|---|---|
committer | Artem Bityutskiy <artem.bityutskiy@linux.intel.com> | 2012-11-15 15:37:50 +0200 |
commit | e8a9d8f31c592eea89f1b0d3fd425e7a96944e88 (patch) | |
tree | 704a94cb7e7a5ab31e57779c7b719ffc88e1bdda /include/linux/mtd | |
parent | 5de0b52ea8f8f5149502867acff2efb5efaf1fc2 (diff) | |
download | linux-e8a9d8f31c592eea89f1b0d3fd425e7a96944e88.tar.gz linux-e8a9d8f31c592eea89f1b0d3fd425e7a96944e88.tar.bz2 linux-e8a9d8f31c592eea89f1b0d3fd425e7a96944e88.zip |
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/linux/mtd')
-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 { uint8_t done_buff[2048 + 64]; /* max size 2048 + 64 */ int read_bytes; - int index; + unsigned int index; int seqin_column; /* column in SEQIN cmd */ int seqin_page_addr; /* page_addr in SEQIN cmd */ uint32_t seqin_read_cmd; /* read cmd in SEQIN cmd */ |