Re: [PATCH v1 3/6] drivers: introduce UBI block abstraction

2024-03-22 Thread Heiko Schocher

Hello Alexey,

On 06.03.24 14:49, Alexey Romanov wrote:

UBI block is an virtual device, that runs on top
of the MTD layer. The blocks are UBI volumes.
Intended to be used in combination with other MTD
drivers.

Despite the fact that it, like mtdblock abstraction,
it used with UCLASS_MTD, they can be used together
on the system without conflicting. For example,
using bcb command:

   # Trying to load bcb via mtdblock:
   $ bcb load mtd 0 mtd_partition_name

   # Trying to load bcb via UBI block:
   $ bcb load ubi 1 ubi_volume_name

User always must attach UBI layer (for example, using
ubi_part()) before using UBI block device.

Signed-off-by: Alexey Romanov 
---
  drivers/block/blk-uclass.c |   1 +
  drivers/mtd/ubi/Makefile   |   1 +
  drivers/mtd/ubi/block.c| 130 +
  include/ubi_uboot.h|   4 ++
  4 files changed, 136 insertions(+)
  create mode 100644 drivers/mtd/ubi/block.c


Reviewed-by: Heiko Schocher 
Acked-by: Heiko Schocher 

bye,
Heiko
--
DENX Software Engineering GmbH,  Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52   Fax: +49-8142-66989-80   Email: h...@denx.de


[PATCH v1 3/6] drivers: introduce UBI block abstraction

2024-03-06 Thread Alexey Romanov
UBI block is an virtual device, that runs on top
of the MTD layer. The blocks are UBI volumes.
Intended to be used in combination with other MTD
drivers.

Despite the fact that it, like mtdblock abstraction,
it used with UCLASS_MTD, they can be used together
on the system without conflicting. For example,
using bcb command:

  # Trying to load bcb via mtdblock:
  $ bcb load mtd 0 mtd_partition_name

  # Trying to load bcb via UBI block:
  $ bcb load ubi 1 ubi_volume_name

User always must attach UBI layer (for example, using
ubi_part()) before using UBI block device.

Signed-off-by: Alexey Romanov 
---
 drivers/block/blk-uclass.c |   1 +
 drivers/mtd/ubi/Makefile   |   1 +
 drivers/mtd/ubi/block.c| 130 +
 include/ubi_uboot.h|   4 ++
 4 files changed, 136 insertions(+)
 create mode 100644 drivers/mtd/ubi/block.c

diff --git a/drivers/block/blk-uclass.c b/drivers/block/blk-uclass.c
index ab0a9105c9..8a457e9f70 100644
--- a/drivers/block/blk-uclass.c
+++ b/drivers/block/blk-uclass.c
@@ -38,6 +38,7 @@ static struct {
{ UCLASS_BLKMAP, "blkmap" },
{ UCLASS_RKMTD, "rkmtd" },
{ UCLASS_MTD, "mtd" },
+   { UCLASS_MTD, "ubi" },
 };
 
 static enum uclass_id uclass_name_to_iftype(const char *uclass_idname)
diff --git a/drivers/mtd/ubi/Makefile b/drivers/mtd/ubi/Makefile
index 30d00fbdfe..67b1a05348 100644
--- a/drivers/mtd/ubi/Makefile
+++ b/drivers/mtd/ubi/Makefile
@@ -7,3 +7,4 @@ obj-y += attach.o build.o vtbl.o vmt.o upd.o kapi.o eba.o io.o 
wl.o crc32.o
 obj-$(CONFIG_MTD_UBI_FASTMAP) += fastmap.o
 obj-y += misc.o
 obj-y += debug.o
+obj-$(CONFIG_BLK) += block.o
diff --git a/drivers/mtd/ubi/block.c b/drivers/mtd/ubi/block.c
new file mode 100644
index 00..95d5ef5752
--- /dev/null
+++ b/drivers/mtd/ubi/block.c
@@ -0,0 +1,130 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * (C) Copyright 2024 SaluteDevices, Inc.
+ *
+ * Author: Alexey Romanov 
+ */
+
+#include 
+#include 
+#include 
+#include 
+#include 
+
+int ubi_bind(struct udevice *dev)
+{
+   struct blk_desc *bdesc;
+   struct udevice *bdev;
+   int ret;
+
+   ret = blk_create_devicef(dev, "ubi_blk", "blk", UCLASS_MTD,
+dev_seq(dev), 512, 0, );
+   if (ret) {
+   pr_err("Cannot create block device");
+   return ret;
+   }
+
+   bdesc = dev_get_uclass_plat(bdev);
+
+   bdesc->bdev = bdev;
+   bdesc->part_type = PART_TYPE_UBI;
+
+   return 0;
+}
+
+static struct ubi_device *get_ubi_device(void)
+{
+   return ubi_devices[0];
+}
+
+static char *get_volume_name(int vol_id)
+{
+   struct ubi_device *ubi = get_ubi_device();
+   int i;
+
+   for (i = 0; i < (ubi->vtbl_slots + 1); i++) {
+   struct ubi_volume *volume = ubi->volumes[i];
+
+   if (!volume)
+   continue;
+
+   if (volume->vol_id >= UBI_INTERNAL_VOL_START)
+   continue;
+
+   if (volume->vol_id == vol_id)
+   return volume->name;
+   }
+
+   return NULL;
+}
+
+static ulong ubi_bread(struct udevice *dev, lbaint_t start, lbaint_t blkcnt,
+  void *dst)
+{
+   struct blk_desc *block_dev = dev_get_uclass_plat(dev);
+   char *volume_name = get_volume_name(block_dev->hwpart);
+   unsigned int size = blkcnt * block_dev->blksz;
+   loff_t offset = start * block_dev->blksz;
+   int ret;
+
+   if (!volume_name) {
+   pr_err("%s: failed to find volume name for blk=%ld", __func__, 
start);
+   return -EINVAL;
+   }
+
+   ret = ubi_volume_read(volume_name, dst, offset, size);
+   if (ret) {
+   pr_err("%s: failed to read from %s UBI volume\n", __func__, 
volume_name);
+   return ret;
+   }
+
+   return blkcnt;
+}
+
+static ulong ubi_bwrite(struct udevice *dev, lbaint_t start, lbaint_t blkcnt,
+   const void *src)
+{
+   struct blk_desc *block_dev = dev_get_uclass_plat(dev);
+   char *volume_name = get_volume_name(block_dev->hwpart);
+   unsigned int size = blkcnt * block_dev->blksz;
+   loff_t offset = start * block_dev->blksz;
+   int ret;
+
+   if (!volume_name) {
+   pr_err("%s: failed to find volume for blk=%ld", __func__, 
start);
+   return -EINVAL;
+   }
+
+   ret = ubi_volume_write(volume_name, (void *)src, offset, size);
+   if (ret) {
+   pr_err("%s: failed to write from %s UBI volume\n", __func__, 
volume_name);
+   return ret;
+   }
+
+   return blkcnt;
+}
+
+static int ubi_blk_probe(struct udevice *dev)
+{
+   int ret;
+
+   ret = device_probe(dev);
+   if (ret) {
+   pr_err("Probing %s failed (err=%d)\n", dev->name, ret);
+   return ret;
+   }
+
+   return 0;
+}
+
+static const struct blk_ops ubi_blk_ops = {
+   .read = ubi_bread,
+