Skip to content

Commit

Permalink
mtd: sh_flctl: Use memcpy() instead of using a loop
Browse files Browse the repository at this point in the history
Elements have been copied "manually" in a loop. Better use memcpy().

Signed-off-by: Bastian Hecht <hechtb@gmail.com>
Reviewed-by: Simon Horman <horms@verge.net.au>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@linux.intel.com>
Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
  • Loading branch information
Bastian Hecht authored and David Woodhouse committed Sep 29, 2012
1 parent 2361f73 commit d76236f
Showing 1 changed file with 7 additions and 6 deletions.
13 changes: 7 additions & 6 deletions drivers/mtd/nand/sh_flctl.c
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@
#include <linux/platform_device.h>
#include <linux/pm_runtime.h>
#include <linux/slab.h>
#include <linux/string.h>

#include <linux/mtd/mtd.h>
#include <linux/mtd/nand.h>
Expand Down Expand Up @@ -746,10 +747,9 @@ static void flctl_select_chip(struct mtd_info *mtd, int chipnr)
static void flctl_write_buf(struct mtd_info *mtd, const uint8_t *buf, int len)
{
struct sh_flctl *flctl = mtd_to_flctl(mtd);
int i, index = flctl->index;
int index = flctl->index;

for (i = 0; i < len; i++)
flctl->done_buff[index + i] = buf[i];
memcpy(&flctl->done_buff[index], buf, len);
flctl->index += len;
}

Expand Down Expand Up @@ -778,10 +778,11 @@ static uint16_t flctl_read_word(struct mtd_info *mtd)

static void flctl_read_buf(struct mtd_info *mtd, uint8_t *buf, int len)
{
int i;
struct sh_flctl *flctl = mtd_to_flctl(mtd);
int index = flctl->index;

for (i = 0; i < len; i++)
buf[i] = flctl_read_byte(mtd);
memcpy(buf, &flctl->done_buff[index], len);
flctl->index += len;
}

static int flctl_verify_buf(struct mtd_info *mtd, const u_char *buf, int len)
Expand Down

0 comments on commit d76236f

Please sign in to comment.