mtd: nand: mxs_nand: add minimal ECC support

[u-boot] / drivers / dfu / dfu_ram.c

diff --git a/drivers/dfu/dfu_ram.c b/drivers/dfu/dfu_ram.c
index e094a946f6be708dfc31a23fdaf48bc7d13f7b9b..cc98668e7a474081d098ccd9d530d958898c90a0 100644 (file)
--- a/drivers/dfu/dfu_ram.c
+++ b/drivers/dfu/dfu_ram.c
@@ -1,3 +1,4 @@
+// SPDX-License-Identifier: GPL-2.0+
 /*
  * (C) Copyright 2013
  * Afzal Mohammed <afzal.mohd.ma@gmail.com>
@@ -5,8 +6,6 @@
  * Reference: dfu_mmc.c
  * Copyright (C) 2012 Samsung Electronics
  * author: Lukasz Majewski <l.majewski@samsung.com>
- *
- * SPDX-License-Identifier:    GPL-2.0+
  */
 
 #include <common.h>
@@ -18,12 +17,12 @@ static int dfu_transfer_medium_ram(enum dfu_op op, struct dfu_entity *dfu,
                                   u64 offset, void *buf, long *len)
 {
        if (dfu->layout != DFU_RAM_ADDR) {
-               error("unsupported layout: %s\n", dfu_get_layout(dfu->layout));
+               pr_err("unsupported layout: %s\n", dfu_get_layout(dfu->layout));
                return  -EINVAL;
        }
 
        if (offset > dfu->data.ram.size) {
-               error("request exceeds allowed area\n");
+               pr_err("request exceeds allowed area\n");
                return -EINVAL;
        }
 
@@ -41,9 +40,11 @@ static int dfu_write_medium_ram(struct dfu_entity *dfu, u64 offset,
        return dfu_transfer_medium_ram(DFU_OP_WRITE, dfu, offset, buf, len);
 }
 
-long dfu_get_medium_size_ram(struct dfu_entity *dfu)
+int dfu_get_medium_size_ram(struct dfu_entity *dfu, u64 *size)
 {
-       return dfu->data.ram.size;
+       *size = dfu->data.ram.size;
+
+       return 0;
 }
 
 static int dfu_read_medium_ram(struct dfu_entity *dfu, u64 offset,
@@ -54,19 +55,26 @@ static int dfu_read_medium_ram(struct dfu_entity *dfu, u64 offset,
 
 int dfu_fill_entity_ram(struct dfu_entity *dfu, char *devstr, char *s)
 {
-       char *st;
+       const char *argv[3];
+       const char **parg = argv;
+
+       for (; parg < argv + sizeof(argv) / sizeof(*argv); ++parg) {
+               *parg = strsep(&s, " ");
+               if (*parg == NULL) {
+                       pr_err("Invalid number of arguments.\n");
+                       return -ENODEV;
+               }
+       }
 
        dfu->dev_type = DFU_DEV_RAM;
-       st = strsep(&s, " ");
-       if (strcmp(st, "ram")) {
-               error("unsupported device: %s\n", st);
+       if (strcmp(argv[0], "ram")) {
+               pr_err("unsupported device: %s\n", argv[0]);
                return -ENODEV;
        }
 
        dfu->layout = DFU_RAM_ADDR;
-       dfu->data.ram.start = (void *)simple_strtoul(s, &s, 16);
-       s++;
-       dfu->data.ram.size = simple_strtoul(s, &s, 16);
+       dfu->data.ram.start = (void *)simple_strtoul(argv[1], NULL, 16);
+       dfu->data.ram.size = simple_strtoul(argv[2], NULL, 16);
 
        dfu->write_medium = dfu_write_medium_ram;
        dfu->get_medium_size = dfu_get_medium_size_ram;