2023-08-11 02:59:52

by Daniel Golle

[permalink] [raw]
Subject: [PATCH v4 1/8] dt-bindings: mtd: add basic bindings for UBI

Add basic bindings for UBI devices and volumes.

Signed-off-by: Daniel Golle <[email protected]>
---
.../bindings/mtd/partitions/linux,ubi.yaml | 65 +++++++++++++++++++
.../bindings/mtd/partitions/ubi-volume.yaml | 36 ++++++++++
2 files changed, 101 insertions(+)
create mode 100644 Documentation/devicetree/bindings/mtd/partitions/linux,ubi.yaml
create mode 100644 Documentation/devicetree/bindings/mtd/partitions/ubi-volume.yaml

diff --git a/Documentation/devicetree/bindings/mtd/partitions/linux,ubi.yaml b/Documentation/devicetree/bindings/mtd/partitions/linux,ubi.yaml
new file mode 100644
index 0000000000000..a06c1666d5184
--- /dev/null
+++ b/Documentation/devicetree/bindings/mtd/partitions/linux,ubi.yaml
@@ -0,0 +1,65 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/mtd/partitions/linux,ubi.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Unsorted Block Images
+
+description: |
+ UBI ("Unsorted Block Images") is a volume management system for raw
+ flash devices which manages multiple logical volumes on a single
+ physical flash device and spreads the I/O load (i.e, wear-leveling)
+ across whole flash chip.
+
+maintainers:
+ - Daniel Golle <[email protected]>
+
+allOf:
+ - $ref: partition.yaml#
+
+properties:
+ compatible:
+ const: linux,ubi
+
+ volumes:
+ type: object
+ description: UBI Volumes
+
+ patternProperties:
+ "^ubi-volume-.*$":
+ $ref: /schemas/mtd/partitions/ubi-volume.yaml
+
+ unevaluatedProperties: false
+
+required:
+ - compatible
+
+unevaluatedProperties: false
+
+examples:
+ - |
+ partitions {
+ compatible = "fixed-partitions";
+ #address-cells = <1>;
+ #size-cells = <1>;
+
+ partition@0 {
+ reg = <0x0 0x100000>;
+ label = "bootloader";
+ read-only;
+ };
+
+ partition@100000 {
+ reg = <0x100000 0x1ff00000>;
+ label = "ubi";
+ compatible = "linux,ubi";
+
+ volumes {
+ ubi-volume-caldata {
+ volume-id = <2>;
+ volume-name = "rf";
+ };
+ };
+ };
+ };
diff --git a/Documentation/devicetree/bindings/mtd/partitions/ubi-volume.yaml b/Documentation/devicetree/bindings/mtd/partitions/ubi-volume.yaml
new file mode 100644
index 0000000000000..e14c85bbeeb91
--- /dev/null
+++ b/Documentation/devicetree/bindings/mtd/partitions/ubi-volume.yaml
@@ -0,0 +1,36 @@
+# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/mtd/partitions/ubi-volume.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: UBI volume
+
+description: |
+ This binding describes a single UBI volume. Volumes can be matches either
+ by their ID or their name, or both.
+
+maintainers:
+ - Daniel Golle <[email protected]>
+
+properties:
+ compatible: true
+
+ volume-id:
+ $ref: /schemas/types.yaml#/definitions/uint32
+ description:
+ Match UBI volume ID
+
+ volume-name:
+ $ref: /schemas/types.yaml#/definitions/string
+ description:
+ Match UBI volume name
+
+anyOf:
+ - required:
+ - volume-id
+
+ - required:
+ - volume-name
+
+unevaluatedProperties: false
--
2.41.0


2023-09-11 22:23:02

by Daniel Golle

[permalink] [raw]
Subject: Re: [PATCH v4 1/8] dt-bindings: mtd: add basic bindings for UBI

On Mon, Aug 21, 2023 at 12:31:22PM -0500, Rob Herring wrote:
> On Fri, 11 Aug 2023 02:36:37 +0100, Daniel Golle wrote:
> > Add basic bindings for UBI devices and volumes.
> >
> Reviewed-by: Rob Herring <[email protected]>
>

Patch 3 - 7 have previously been discussed on the mailing list, patch
1 and 2 have been reviewed by Rob, patch 8 ("mtd: ubi: provide NVMEM
layer over UBI volumes") still needs review.

Is there anything I can do from my end to have this series moving
forward?

Patchwork: https://patchwork.ozlabs.org/project/linux-mtd/list/?series=368347


Thank you for your advise!


Daniel

2023-09-13 18:31:10

by Tudor Ambarus

[permalink] [raw]
Subject: Re: [PATCH v4 1/8] dt-bindings: mtd: add basic bindings for UBI

Hi!

On 9/11/23 20:01, Daniel Golle wrote:
> Is there anything I can do from my end to have this series moving forward?

You could review other UBI submissions to reduce the review load on
Richard. I guess the queue of UBI pending patches is at:
https://patchwork.ozlabs.org/project/linux-mtd/list/?series=&submitter=&state=&q=ubi&archive=&delegate=

Cheers,
ta