diff options
author | Jeff Mahoney <jeffm@suse.com> | 2016-05-06 14:02:54 -0400 |
---|---|---|
committer | Jens Axboe <axboe@fb.com> | 2016-05-06 14:51:10 -0400 |
commit | 57aac2f1be4a0711b7f24f7d367e8672ebaa3844 (patch) | |
tree | 43c7aef6fafb634bd86eabb345dd0227aa8e989a | |
parent | 87c32077819c695cbc5ab00226a28010cd5806c3 (diff) |
lightnvm: fix "warning: ‘ret’ may be used uninitialized"
This fixes the following warnings:
drivers/lightnvm/sysblk.c:125:9: warning: ‘ret’ may be used
uninitialized in this function
drivers/lightnvm/sysblk.c:275:15: warning: ‘ret’ may be used
uninitialized in this function
In both cases, ret is only set from within a loop that may not be entered.
Signed-off-by: Jeff Mahoney <jeffm@suse.com>
Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: Matias Bjørling <m@bjorling.me>
Signed-off-by: Jens Axboe <axboe@fb.com>
-rw-r--r-- | drivers/lightnvm/sysblk.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/lightnvm/sysblk.c b/drivers/lightnvm/sysblk.c index 321de1f154c5..b1e1404e6951 100644 --- a/drivers/lightnvm/sysblk.c +++ b/drivers/lightnvm/sysblk.c | |||
@@ -122,7 +122,7 @@ static int nvm_get_all_sysblks(struct nvm_dev *dev, struct sysblk_scan *s, | |||
122 | struct ppa_addr *ppas, nvm_bb_update_fn *fn) | 122 | struct ppa_addr *ppas, nvm_bb_update_fn *fn) |
123 | { | 123 | { |
124 | struct ppa_addr dppa; | 124 | struct ppa_addr dppa; |
125 | int i, ret; | 125 | int i, ret = 0; |
126 | 126 | ||
127 | s->nr_ppas = 0; | 127 | s->nr_ppas = 0; |
128 | 128 | ||
@@ -272,7 +272,7 @@ static int nvm_write_and_verify(struct nvm_dev *dev, struct nvm_sb_info *info, | |||
272 | { | 272 | { |
273 | struct nvm_system_block nvmsb; | 273 | struct nvm_system_block nvmsb; |
274 | void *buf; | 274 | void *buf; |
275 | int i, sect, ret, bufsz; | 275 | int i, sect, ret = 0, bufsz; |
276 | struct ppa_addr *ppas; | 276 | struct ppa_addr *ppas; |
277 | 277 | ||
278 | nvm_cpu_to_sysblk(&nvmsb, info); | 278 | nvm_cpu_to_sysblk(&nvmsb, info); |