Re: [PATCH v2 0/6] Introduce UBI block device

2024-05-24 Thread Dario Binacchi
Hi Alexey,

On Thu, May 23, 2024 at 12:59 PM Alexey Romanov
 wrote:
>
> On Thu, May 23, 2024 at 12:51:33PM +0200, Michael Nazzareno Trimarchi wrote:
> > Hi
> >
> > We queue them this afternoon
>
> Thank you.
>
> I also wanted to remind you that there is still a similiar patchest with
> mtdblock support 
> (https://lore.kernel.org/all/20240506135607.aprjqn7uyng2jgp5@cab-wsm-0029881/)
> that should be applied to bracnh before this.
>
> >
> > Michael
> >
> > On Thu, May 23, 2024 at 12:49 PM Alexey Romanov
> >  wrote:
> > >
> > >
> > > Hi Michael,
> > >
> > > On Mon, May 06, 2024 at 03:59:52PM +0200, Michael Nazzareno Trimarchi 
> > > wrote:
> > > > Hi Alexey
> > > >
> > > > Sorry will will put on CI
> > >
> > > Any news?
> > >
> > > >
> > > > Michael
> > > >
> > > > On Mon, May 6, 2024 at 3:58 PM Alexey Romanov
> > > >  wrote:
> > > > >
> > > > > Hello! Ping.
> > > > >
> > > > > On Mon, Mar 25, 2024 at 05:41:42PM +0300, Alexey Romanov wrote:
> > > > > > Hello!
> > > > > >
> > > > > > This series adds support for the UBI block device, which
> > > > > > allows to read/write data block by block. For example,
> > > > > > it can now be used for BCB or Android AB command:
> > > > > >
> > > > > >   $ bcb load ubi 0 part_name
> > > > > >
> > > > > > Tested only on SPI NAND, so bind is made only for
> > > > > > SPI NAND drivers. Can be used with mtdblock device [1].
> > > > > >
> > > > > > ---
> > > > > >
> > > > > > Changes V1 -> V2 [2]:
> > > > > >
> > > > > >  - Rebased over mtdblock v2 patchset [3].
> > > > > >  - Compile UBI partitions support only if CONFIG_BLK option
> > > > > >is enabled.
> > > > > >
> > > > > > - Links:
> > > > > >
> > > > > >   [1] 
> > > > > > https://lore.kernel.org/all/20240227100441.1811047-1-avroma...@salutedevices.com/
> > > > > >   [2] 
> > > > > > https://lore.kernel.org/all/20240306134906.1179285-1-avroma...@salutedevices.com/
> > > > > >   [3] 
> > > > > > https://lore.kernel.org/all/20240307130726.1582487-1-avroma...@salutedevices.com/
> > > > > >
> > > > > > Alexey Romanov (6):
> > > > > >   ubi: allow to read from volume with offset
> > > > > >   ubi: allow to write to volume with offset
> > > > > >   drivers: introduce UBI block abstraction
> > > > > >   disk: don't try search for partition type if already set
> > > > > >   disk: support UBI partitions
> > > > > >   spinand: bind UBI block
> > > > > >
> > > > > >  cmd/ubi.c   |  77 +++--
> > > > > >  disk/part.c |   7 ++
> > > > > >  drivers/block/blk-uclass.c  |   1 +
> > > > > >  drivers/mtd/nand/spi/core.c |   8 ++-
> > > > > >  drivers/mtd/ubi/Makefile|   1 +
> > > > > >  drivers/mtd/ubi/block.c | 130 
> > > > > > 
> > > > > >  drivers/mtd/ubi/part.c  |  99 +++
> > > > > >  env/ubi.c   |  16 ++---
> > > > > >  include/part.h  |   2 +
> > > > > >  include/ubi_uboot.h |   8 ++-
> > > > > >  10 files changed, 332 insertions(+), 17 deletions(-)
> > > > > >  create mode 100644 drivers/mtd/ubi/block.c
> > > > > >  create mode 100644 drivers/mtd/ubi/part.c
> > > > > >
> > > > > > --
> > > > > > 2.34.1
> > > > > >
> > > > >
> > > > > --
> > > > > Thank you,
> > > > > Alexey
> > > >
> > > >
> > > >
> > > > --
> > > > Michael Nazzareno Trimarchi
> > > > Co-Founder & Chief Executive Officer
> > > > M. +39 347 913 2170
> > > > mich...@amarulasolutions.com
> > > > __
> > > >
> > > > Amarula Solutions BV
> > > > Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
> > > > T. +31 (0)85 111 9172
> > > > i...@amarulasolutions.com
> > > > www.amarulasolutions.com
> > >
> > > --
> > > Thank you,
> > > Alexey
> >
> >
> >
> > --
> > Michael Nazzareno Trimarchi
> > Co-Founder & Chief Executive Officer
> > M. +39 347 913 2170
> > mich...@amarulasolutions.com
> > __
> >
> > Amarula Solutions BV
> > Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
> > T. +31 (0)85 111 9172
> > i...@amarulasolutions.com
> > www.amarulasolutions.com
>
> --
> Thank you,
> Alexey

I applied your series

https://patchwork.ozlabs.org/user/todo/uboot/?series=401667
https://patchwork.ozlabs.org/user/todo/uboot/?series=400302

but the CI fails. The first applied series already has problems:

Building current source for 1 boards (1 thread, 64 jobs per thread)
53 x86: + coreboot
54+In file included from include/linux/printk.h:4,
55+ from include/linux/kernel.h:5,
56+ from include/linux/bitops.h:22,
57+ from include/uuid.h:13,
58+ from include/part.h:11,
59+ from drivers/mtd/mtdblock.c:55:
60+drivers/mtd/mtdblock.c: In function 'mtd_blk_read':
61+drivers/mtd/mtdblock.c:100:32: error: format '%lx' expects argument
of type 'long unsigned int', but argument 7 has type 'lbaint_t' {aka
'long long unsigned int'} [-Werror=format=]
62+ 100 | pr_err("mtdblock: failed to read block 0x%lx\n", cur);
63+ | ^~~~
64+include/log.h:157:21: note: in def

Re: [PATCH v2 0/6] Introduce UBI block device

2024-05-23 Thread Alexey Romanov
On Thu, May 23, 2024 at 12:51:33PM +0200, Michael Nazzareno Trimarchi wrote:
> Hi
> 
> We queue them this afternoon

Thank you.

I also wanted to remind you that there is still a similiar patchest with
mtdblock support 
(https://lore.kernel.org/all/20240506135607.aprjqn7uyng2jgp5@cab-wsm-0029881/)
that should be applied to bracnh before this.

> 
> Michael
> 
> On Thu, May 23, 2024 at 12:49 PM Alexey Romanov
>  wrote:
> >
> >
> > Hi Michael,
> >
> > On Mon, May 06, 2024 at 03:59:52PM +0200, Michael Nazzareno Trimarchi wrote:
> > > Hi Alexey
> > >
> > > Sorry will will put on CI
> >
> > Any news?
> >
> > >
> > > Michael
> > >
> > > On Mon, May 6, 2024 at 3:58 PM Alexey Romanov
> > >  wrote:
> > > >
> > > > Hello! Ping.
> > > >
> > > > On Mon, Mar 25, 2024 at 05:41:42PM +0300, Alexey Romanov wrote:
> > > > > Hello!
> > > > >
> > > > > This series adds support for the UBI block device, which
> > > > > allows to read/write data block by block. For example,
> > > > > it can now be used for BCB or Android AB command:
> > > > >
> > > > >   $ bcb load ubi 0 part_name
> > > > >
> > > > > Tested only on SPI NAND, so bind is made only for
> > > > > SPI NAND drivers. Can be used with mtdblock device [1].
> > > > >
> > > > > ---
> > > > >
> > > > > Changes V1 -> V2 [2]:
> > > > >
> > > > >  - Rebased over mtdblock v2 patchset [3].
> > > > >  - Compile UBI partitions support only if CONFIG_BLK option
> > > > >is enabled.
> > > > >
> > > > > - Links:
> > > > >
> > > > >   [1] 
> > > > > https://lore.kernel.org/all/20240227100441.1811047-1-avroma...@salutedevices.com/
> > > > >   [2] 
> > > > > https://lore.kernel.org/all/20240306134906.1179285-1-avroma...@salutedevices.com/
> > > > >   [3] 
> > > > > https://lore.kernel.org/all/20240307130726.1582487-1-avroma...@salutedevices.com/
> > > > >
> > > > > Alexey Romanov (6):
> > > > >   ubi: allow to read from volume with offset
> > > > >   ubi: allow to write to volume with offset
> > > > >   drivers: introduce UBI block abstraction
> > > > >   disk: don't try search for partition type if already set
> > > > >   disk: support UBI partitions
> > > > >   spinand: bind UBI block
> > > > >
> > > > >  cmd/ubi.c   |  77 +++--
> > > > >  disk/part.c |   7 ++
> > > > >  drivers/block/blk-uclass.c  |   1 +
> > > > >  drivers/mtd/nand/spi/core.c |   8 ++-
> > > > >  drivers/mtd/ubi/Makefile|   1 +
> > > > >  drivers/mtd/ubi/block.c | 130 
> > > > > 
> > > > >  drivers/mtd/ubi/part.c  |  99 +++
> > > > >  env/ubi.c   |  16 ++---
> > > > >  include/part.h  |   2 +
> > > > >  include/ubi_uboot.h |   8 ++-
> > > > >  10 files changed, 332 insertions(+), 17 deletions(-)
> > > > >  create mode 100644 drivers/mtd/ubi/block.c
> > > > >  create mode 100644 drivers/mtd/ubi/part.c
> > > > >
> > > > > --
> > > > > 2.34.1
> > > > >
> > > >
> > > > --
> > > > Thank you,
> > > > Alexey
> > >
> > >
> > >
> > > --
> > > Michael Nazzareno Trimarchi
> > > Co-Founder & Chief Executive Officer
> > > M. +39 347 913 2170
> > > mich...@amarulasolutions.com
> > > __
> > >
> > > Amarula Solutions BV
> > > Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
> > > T. +31 (0)85 111 9172
> > > i...@amarulasolutions.com
> > > www.amarulasolutions.com
> >
> > --
> > Thank you,
> > Alexey
> 
> 
> 
> -- 
> Michael Nazzareno Trimarchi
> Co-Founder & Chief Executive Officer
> M. +39 347 913 2170
> mich...@amarulasolutions.com
> __
> 
> Amarula Solutions BV
> Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
> T. +31 (0)85 111 9172
> i...@amarulasolutions.com
> www.amarulasolutions.com

-- 
Thank you,
Alexey

Re: [PATCH v2 0/6] Introduce UBI block device

2024-05-23 Thread Alexey Romanov

Hi Michael,

On Mon, May 06, 2024 at 03:59:52PM +0200, Michael Nazzareno Trimarchi wrote:
> Hi Alexey
> 
> Sorry will will put on CI

Any news?

> 
> Michael
> 
> On Mon, May 6, 2024 at 3:58 PM Alexey Romanov
>  wrote:
> >
> > Hello! Ping.
> >
> > On Mon, Mar 25, 2024 at 05:41:42PM +0300, Alexey Romanov wrote:
> > > Hello!
> > >
> > > This series adds support for the UBI block device, which
> > > allows to read/write data block by block. For example,
> > > it can now be used for BCB or Android AB command:
> > >
> > >   $ bcb load ubi 0 part_name
> > >
> > > Tested only on SPI NAND, so bind is made only for
> > > SPI NAND drivers. Can be used with mtdblock device [1].
> > >
> > > ---
> > >
> > > Changes V1 -> V2 [2]:
> > >
> > >  - Rebased over mtdblock v2 patchset [3].
> > >  - Compile UBI partitions support only if CONFIG_BLK option
> > >is enabled.
> > >
> > > - Links:
> > >
> > >   [1] 
> > > https://lore.kernel.org/all/20240227100441.1811047-1-avroma...@salutedevices.com/
> > >   [2] 
> > > https://lore.kernel.org/all/20240306134906.1179285-1-avroma...@salutedevices.com/
> > >   [3] 
> > > https://lore.kernel.org/all/20240307130726.1582487-1-avroma...@salutedevices.com/
> > >
> > > Alexey Romanov (6):
> > >   ubi: allow to read from volume with offset
> > >   ubi: allow to write to volume with offset
> > >   drivers: introduce UBI block abstraction
> > >   disk: don't try search for partition type if already set
> > >   disk: support UBI partitions
> > >   spinand: bind UBI block
> > >
> > >  cmd/ubi.c   |  77 +++--
> > >  disk/part.c |   7 ++
> > >  drivers/block/blk-uclass.c  |   1 +
> > >  drivers/mtd/nand/spi/core.c |   8 ++-
> > >  drivers/mtd/ubi/Makefile|   1 +
> > >  drivers/mtd/ubi/block.c | 130 
> > >  drivers/mtd/ubi/part.c  |  99 +++
> > >  env/ubi.c   |  16 ++---
> > >  include/part.h  |   2 +
> > >  include/ubi_uboot.h |   8 ++-
> > >  10 files changed, 332 insertions(+), 17 deletions(-)
> > >  create mode 100644 drivers/mtd/ubi/block.c
> > >  create mode 100644 drivers/mtd/ubi/part.c
> > >
> > > --
> > > 2.34.1
> > >
> >
> > --
> > Thank you,
> > Alexey
> 
> 
> 
> -- 
> Michael Nazzareno Trimarchi
> Co-Founder & Chief Executive Officer
> M. +39 347 913 2170
> mich...@amarulasolutions.com
> __
> 
> Amarula Solutions BV
> Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
> T. +31 (0)85 111 9172
> i...@amarulasolutions.com
> www.amarulasolutions.com

-- 
Thank you,
Alexey

Re: [PATCH v2 0/6] Introduce UBI block device

2024-05-23 Thread Michael Nazzareno Trimarchi
Hi

We queue them this afternoon

Michael

On Thu, May 23, 2024 at 12:49 PM Alexey Romanov
 wrote:
>
>
> Hi Michael,
>
> On Mon, May 06, 2024 at 03:59:52PM +0200, Michael Nazzareno Trimarchi wrote:
> > Hi Alexey
> >
> > Sorry will will put on CI
>
> Any news?
>
> >
> > Michael
> >
> > On Mon, May 6, 2024 at 3:58 PM Alexey Romanov
> >  wrote:
> > >
> > > Hello! Ping.
> > >
> > > On Mon, Mar 25, 2024 at 05:41:42PM +0300, Alexey Romanov wrote:
> > > > Hello!
> > > >
> > > > This series adds support for the UBI block device, which
> > > > allows to read/write data block by block. For example,
> > > > it can now be used for BCB or Android AB command:
> > > >
> > > >   $ bcb load ubi 0 part_name
> > > >
> > > > Tested only on SPI NAND, so bind is made only for
> > > > SPI NAND drivers. Can be used with mtdblock device [1].
> > > >
> > > > ---
> > > >
> > > > Changes V1 -> V2 [2]:
> > > >
> > > >  - Rebased over mtdblock v2 patchset [3].
> > > >  - Compile UBI partitions support only if CONFIG_BLK option
> > > >is enabled.
> > > >
> > > > - Links:
> > > >
> > > >   [1] 
> > > > https://lore.kernel.org/all/20240227100441.1811047-1-avroma...@salutedevices.com/
> > > >   [2] 
> > > > https://lore.kernel.org/all/20240306134906.1179285-1-avroma...@salutedevices.com/
> > > >   [3] 
> > > > https://lore.kernel.org/all/20240307130726.1582487-1-avroma...@salutedevices.com/
> > > >
> > > > Alexey Romanov (6):
> > > >   ubi: allow to read from volume with offset
> > > >   ubi: allow to write to volume with offset
> > > >   drivers: introduce UBI block abstraction
> > > >   disk: don't try search for partition type if already set
> > > >   disk: support UBI partitions
> > > >   spinand: bind UBI block
> > > >
> > > >  cmd/ubi.c   |  77 +++--
> > > >  disk/part.c |   7 ++
> > > >  drivers/block/blk-uclass.c  |   1 +
> > > >  drivers/mtd/nand/spi/core.c |   8 ++-
> > > >  drivers/mtd/ubi/Makefile|   1 +
> > > >  drivers/mtd/ubi/block.c | 130 
> > > >  drivers/mtd/ubi/part.c  |  99 +++
> > > >  env/ubi.c   |  16 ++---
> > > >  include/part.h  |   2 +
> > > >  include/ubi_uboot.h |   8 ++-
> > > >  10 files changed, 332 insertions(+), 17 deletions(-)
> > > >  create mode 100644 drivers/mtd/ubi/block.c
> > > >  create mode 100644 drivers/mtd/ubi/part.c
> > > >
> > > > --
> > > > 2.34.1
> > > >
> > >
> > > --
> > > Thank you,
> > > Alexey
> >
> >
> >
> > --
> > Michael Nazzareno Trimarchi
> > Co-Founder & Chief Executive Officer
> > M. +39 347 913 2170
> > mich...@amarulasolutions.com
> > __
> >
> > Amarula Solutions BV
> > Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
> > T. +31 (0)85 111 9172
> > i...@amarulasolutions.com
> > www.amarulasolutions.com
>
> --
> Thank you,
> Alexey



-- 
Michael Nazzareno Trimarchi
Co-Founder & Chief Executive Officer
M. +39 347 913 2170
mich...@amarulasolutions.com
__

Amarula Solutions BV
Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
T. +31 (0)85 111 9172
i...@amarulasolutions.com
www.amarulasolutions.com


Re: [PATCH v2 0/6] Introduce UBI block device

2024-05-06 Thread Alexey Romanov
Hello! Ping.

On Mon, Mar 25, 2024 at 05:41:42PM +0300, Alexey Romanov wrote:
> Hello!
> 
> This series adds support for the UBI block device, which
> allows to read/write data block by block. For example,
> it can now be used for BCB or Android AB command:
> 
>   $ bcb load ubi 0 part_name
> 
> Tested only on SPI NAND, so bind is made only for
> SPI NAND drivers. Can be used with mtdblock device [1].
> 
> --- 
> 
> Changes V1 -> V2 [2]:
> 
>  - Rebased over mtdblock v2 patchset [3].
>  - Compile UBI partitions support only if CONFIG_BLK option
>is enabled.
> 
> - Links:
> 
>   [1] 
> https://lore.kernel.org/all/20240227100441.1811047-1-avroma...@salutedevices.com/
>   [2] 
> https://lore.kernel.org/all/20240306134906.1179285-1-avroma...@salutedevices.com/
>   [3] 
> https://lore.kernel.org/all/20240307130726.1582487-1-avroma...@salutedevices.com/
> 
> Alexey Romanov (6):
>   ubi: allow to read from volume with offset
>   ubi: allow to write to volume with offset
>   drivers: introduce UBI block abstraction
>   disk: don't try search for partition type if already set
>   disk: support UBI partitions
>   spinand: bind UBI block
> 
>  cmd/ubi.c   |  77 +++--
>  disk/part.c |   7 ++
>  drivers/block/blk-uclass.c  |   1 +
>  drivers/mtd/nand/spi/core.c |   8 ++-
>  drivers/mtd/ubi/Makefile|   1 +
>  drivers/mtd/ubi/block.c | 130 
>  drivers/mtd/ubi/part.c  |  99 +++
>  env/ubi.c   |  16 ++---
>  include/part.h  |   2 +
>  include/ubi_uboot.h |   8 ++-
>  10 files changed, 332 insertions(+), 17 deletions(-)
>  create mode 100644 drivers/mtd/ubi/block.c
>  create mode 100644 drivers/mtd/ubi/part.c
> 
> -- 
> 2.34.1
> 

-- 
Thank you,
Alexey

Re: [PATCH v2 0/6] Introduce UBI block device

2024-05-06 Thread Heiko Schocher

Hello Alexey,

On 06.05.24 15:58, Alexey Romanov wrote:

Hello! Ping.


okay from my side... I think I acked them already

@Dario: IIRC the patchset is delegated to you...

If it is okay, I can apply it in u-boot-ubi and send a pull request to
Tom, but please, send a Acked-by or Reviewed-by and delegate it back to me!

Thanks!

bye,
Heiko


On Mon, Mar 25, 2024 at 05:41:42PM +0300, Alexey Romanov wrote:

Hello!

This series adds support for the UBI block device, which
allows to read/write data block by block. For example,
it can now be used for BCB or Android AB command:

   $ bcb load ubi 0 part_name

Tested only on SPI NAND, so bind is made only for
SPI NAND drivers. Can be used with mtdblock device [1].

---

Changes V1 -> V2 [2]:

  - Rebased over mtdblock v2 patchset [3].
  - Compile UBI partitions support only if CONFIG_BLK option
is enabled.

- Links:

   [1] 
https://lore.kernel.org/all/20240227100441.1811047-1-avroma...@salutedevices.com/
   [2] 
https://lore.kernel.org/all/20240306134906.1179285-1-avroma...@salutedevices.com/
   [3] 
https://lore.kernel.org/all/20240307130726.1582487-1-avroma...@salutedevices.com/

Alexey Romanov (6):
   ubi: allow to read from volume with offset
   ubi: allow to write to volume with offset
   drivers: introduce UBI block abstraction
   disk: don't try search for partition type if already set
   disk: support UBI partitions
   spinand: bind UBI block

  cmd/ubi.c   |  77 +++--
  disk/part.c |   7 ++
  drivers/block/blk-uclass.c  |   1 +
  drivers/mtd/nand/spi/core.c |   8 ++-
  drivers/mtd/ubi/Makefile|   1 +
  drivers/mtd/ubi/block.c | 130 
  drivers/mtd/ubi/part.c  |  99 +++
  env/ubi.c   |  16 ++---
  include/part.h  |   2 +
  include/ubi_uboot.h |   8 ++-
  10 files changed, 332 insertions(+), 17 deletions(-)
  create mode 100644 drivers/mtd/ubi/block.c
  create mode 100644 drivers/mtd/ubi/part.c

--
2.34.1





--
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


Re: [PATCH v2 0/6] Introduce UBI block device

2024-05-06 Thread Michael Nazzareno Trimarchi
Hi Alexey

Sorry will will put on CI

Michael

On Mon, May 6, 2024 at 3:58 PM Alexey Romanov
 wrote:
>
> Hello! Ping.
>
> On Mon, Mar 25, 2024 at 05:41:42PM +0300, Alexey Romanov wrote:
> > Hello!
> >
> > This series adds support for the UBI block device, which
> > allows to read/write data block by block. For example,
> > it can now be used for BCB or Android AB command:
> >
> >   $ bcb load ubi 0 part_name
> >
> > Tested only on SPI NAND, so bind is made only for
> > SPI NAND drivers. Can be used with mtdblock device [1].
> >
> > ---
> >
> > Changes V1 -> V2 [2]:
> >
> >  - Rebased over mtdblock v2 patchset [3].
> >  - Compile UBI partitions support only if CONFIG_BLK option
> >is enabled.
> >
> > - Links:
> >
> >   [1] 
> > https://lore.kernel.org/all/20240227100441.1811047-1-avroma...@salutedevices.com/
> >   [2] 
> > https://lore.kernel.org/all/20240306134906.1179285-1-avroma...@salutedevices.com/
> >   [3] 
> > https://lore.kernel.org/all/20240307130726.1582487-1-avroma...@salutedevices.com/
> >
> > Alexey Romanov (6):
> >   ubi: allow to read from volume with offset
> >   ubi: allow to write to volume with offset
> >   drivers: introduce UBI block abstraction
> >   disk: don't try search for partition type if already set
> >   disk: support UBI partitions
> >   spinand: bind UBI block
> >
> >  cmd/ubi.c   |  77 +++--
> >  disk/part.c |   7 ++
> >  drivers/block/blk-uclass.c  |   1 +
> >  drivers/mtd/nand/spi/core.c |   8 ++-
> >  drivers/mtd/ubi/Makefile|   1 +
> >  drivers/mtd/ubi/block.c | 130 
> >  drivers/mtd/ubi/part.c  |  99 +++
> >  env/ubi.c   |  16 ++---
> >  include/part.h  |   2 +
> >  include/ubi_uboot.h |   8 ++-
> >  10 files changed, 332 insertions(+), 17 deletions(-)
> >  create mode 100644 drivers/mtd/ubi/block.c
> >  create mode 100644 drivers/mtd/ubi/part.c
> >
> > --
> > 2.34.1
> >
>
> --
> Thank you,
> Alexey



-- 
Michael Nazzareno Trimarchi
Co-Founder & Chief Executive Officer
M. +39 347 913 2170
mich...@amarulasolutions.com
__

Amarula Solutions BV
Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
T. +31 (0)85 111 9172
i...@amarulasolutions.com
www.amarulasolutions.com


Re: [PATCH v2 0/6] Introduce UBI block device

2024-04-03 Thread Alexey Romanov
Hi guys! Ping.

On Tue, Mar 26, 2024 at 05:21:29AM +0100, Heiko Schocher wrote:
> Hello Alexey,
> 
> On 25.03.24 15:41, Alexey Romanov wrote:
> > Hello!
> > 
> > This series adds support for the UBI block device, which
> > allows to read/write data block by block. For example,
> > it can now be used for BCB or Android AB command:
> > 
> >$ bcb load ubi 0 part_name
> > 
> > Tested only on SPI NAND, so bind is made only for
> > SPI NAND drivers. Can be used with mtdblock device [1].
> > 
> > ---
> > 
> > Changes V1 -> V2 [2]:
> > 
> >   - Rebased over mtdblock v2 patchset [3].
> >   - Compile UBI partitions support only if CONFIG_BLK option
> > is enabled.
> 
> Thanks!
> 
> > - Links:
> > 
> >[1] 
> > https://lore.kernel.org/all/20240227100441.1811047-1-avroma...@salutedevices.com/
> >[2] 
> > https://lore.kernel.org/all/20240306134906.1179285-1-avroma...@salutedevices.com/
> >[3] 
> > https://lore.kernel.org/all/20240307130726.1582487-1-avroma...@salutedevices.com/
> > 
> > Alexey Romanov (6):
> >ubi: allow to read from volume with offset
> >ubi: allow to write to volume with offset
> >drivers: introduce UBI block abstraction
> >disk: don't try search for partition type if already set
> >disk: support UBI partitions
> >spinand: bind UBI block
> > 
> >   cmd/ubi.c   |  77 +++--
> >   disk/part.c |   7 ++
> >   drivers/block/blk-uclass.c  |   1 +
> >   drivers/mtd/nand/spi/core.c |   8 ++-
> >   drivers/mtd/ubi/Makefile|   1 +
> >   drivers/mtd/ubi/block.c | 130 
> >   drivers/mtd/ubi/part.c  |  99 +++
> >   env/ubi.c   |  16 ++---
> >   include/part.h  |   2 +
> >   include/ubi_uboot.h |   8 ++-
> >   10 files changed, 332 insertions(+), 17 deletions(-)
> >   create mode 100644 drivers/mtd/ubi/block.c
> >   create mode 100644 drivers/mtd/ubi/part.c
> 
> Looks good to me now.
> 
> @Dario: I saw that series is delegetad to you in patchwork.
>  From my side it is fine. If you think I should  pick it up, please
>  send a Reviewed/Acked-by for the SPI NAND parts, and notify me,
>  thanks!
> 
> 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

-- 
Thank you,
Alexey

Re: [PATCH v2 0/6] Introduce UBI block device

2024-03-25 Thread Heiko Schocher

Hello Alexey,

On 25.03.24 15:41, Alexey Romanov wrote:

Hello!

This series adds support for the UBI block device, which
allows to read/write data block by block. For example,
it can now be used for BCB or Android AB command:

   $ bcb load ubi 0 part_name

Tested only on SPI NAND, so bind is made only for
SPI NAND drivers. Can be used with mtdblock device [1].

---

Changes V1 -> V2 [2]:

  - Rebased over mtdblock v2 patchset [3].
  - Compile UBI partitions support only if CONFIG_BLK option
is enabled.


Thanks!


- Links:

   [1] 
https://lore.kernel.org/all/20240227100441.1811047-1-avroma...@salutedevices.com/
   [2] 
https://lore.kernel.org/all/20240306134906.1179285-1-avroma...@salutedevices.com/
   [3] 
https://lore.kernel.org/all/20240307130726.1582487-1-avroma...@salutedevices.com/

Alexey Romanov (6):
   ubi: allow to read from volume with offset
   ubi: allow to write to volume with offset
   drivers: introduce UBI block abstraction
   disk: don't try search for partition type if already set
   disk: support UBI partitions
   spinand: bind UBI block

  cmd/ubi.c   |  77 +++--
  disk/part.c |   7 ++
  drivers/block/blk-uclass.c  |   1 +
  drivers/mtd/nand/spi/core.c |   8 ++-
  drivers/mtd/ubi/Makefile|   1 +
  drivers/mtd/ubi/block.c | 130 
  drivers/mtd/ubi/part.c  |  99 +++
  env/ubi.c   |  16 ++---
  include/part.h  |   2 +
  include/ubi_uboot.h |   8 ++-
  10 files changed, 332 insertions(+), 17 deletions(-)
  create mode 100644 drivers/mtd/ubi/block.c
  create mode 100644 drivers/mtd/ubi/part.c


Looks good to me now.

@Dario: I saw that series is delegetad to you in patchwork.
 From my side it is fine. If you think I should  pick it up, please
 send a Reviewed/Acked-by for the SPI NAND parts, and notify me,
 thanks!

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 v2 0/6] Introduce UBI block device

2024-03-25 Thread Alexey Romanov
Hello!

This series adds support for the UBI block device, which
allows to read/write data block by block. For example,
it can now be used for BCB or Android AB command:

  $ bcb load ubi 0 part_name

Tested only on SPI NAND, so bind is made only for
SPI NAND drivers. Can be used with mtdblock device [1].

--- 

Changes V1 -> V2 [2]:

 - Rebased over mtdblock v2 patchset [3].
 - Compile UBI partitions support only if CONFIG_BLK option
   is enabled.

- Links:

  [1] 
https://lore.kernel.org/all/20240227100441.1811047-1-avroma...@salutedevices.com/
  [2] 
https://lore.kernel.org/all/20240306134906.1179285-1-avroma...@salutedevices.com/
  [3] 
https://lore.kernel.org/all/20240307130726.1582487-1-avroma...@salutedevices.com/

Alexey Romanov (6):
  ubi: allow to read from volume with offset
  ubi: allow to write to volume with offset
  drivers: introduce UBI block abstraction
  disk: don't try search for partition type if already set
  disk: support UBI partitions
  spinand: bind UBI block

 cmd/ubi.c   |  77 +++--
 disk/part.c |   7 ++
 drivers/block/blk-uclass.c  |   1 +
 drivers/mtd/nand/spi/core.c |   8 ++-
 drivers/mtd/ubi/Makefile|   1 +
 drivers/mtd/ubi/block.c | 130 
 drivers/mtd/ubi/part.c  |  99 +++
 env/ubi.c   |  16 ++---
 include/part.h  |   2 +
 include/ubi_uboot.h |   8 ++-
 10 files changed, 332 insertions(+), 17 deletions(-)
 create mode 100644 drivers/mtd/ubi/block.c
 create mode 100644 drivers/mtd/ubi/part.c

-- 
2.34.1