Skip to content

Commit e6026eb

Browse files
Tom Rixmiquelraynal
authored andcommitted
mtd: lpddr_cmds: remove unused words variable
clang with W=1 reports drivers/mtd/lpddr/lpddr_cmds.c:409:31: error: variable 'words' set but not used [-Werror,-Wunused-but-set-variable] int ret, wbufsize, word_gap, words; ^ This variable is not used so remove it. Signed-off-by: Tom Rix <trix@redhat.com> Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com> Link: https://lore.kernel.org/linux-mtd/20230328000620.1778033-1-trix@redhat.com
1 parent 1f79a61 commit e6026eb

1 file changed

Lines changed: 2 additions & 5 deletions

File tree

drivers/mtd/lpddr/lpddr_cmds.c

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -406,7 +406,7 @@ static int do_write_buffer(struct map_info *map, struct flchip *chip,
406406
{
407407
struct lpddr_private *lpddr = map->fldrv_priv;
408408
map_word datum;
409-
int ret, wbufsize, word_gap, words;
409+
int ret, wbufsize, word_gap;
410410
const struct kvec *vec;
411411
unsigned long vec_seek;
412412
unsigned long prog_buf_ofs;
@@ -421,10 +421,7 @@ static int do_write_buffer(struct map_info *map, struct flchip *chip,
421421
}
422422
/* Figure out the number of words to write */
423423
word_gap = (-adr & (map_bankwidth(map)-1));
424-
words = (len - word_gap + map_bankwidth(map) - 1) / map_bankwidth(map);
425-
if (!word_gap) {
426-
words--;
427-
} else {
424+
if (word_gap) {
428425
word_gap = map_bankwidth(map) - word_gap;
429426
adr -= word_gap;
430427
datum = map_word_ff(map);

0 commit comments

Comments
 (0)