[U-Boot] [PATCH v4 5/6] doc: dtbinding: Add file system firmware loader binding document

[U-Boot] [PATCH v4 5/6] doc: dtbinding: Add file system firmware loader binding document

[tien.fong.chee at intel.com]

From: Tien Fong Chee <tien.fong.chee@intel.com>

Add a document to describe file system firmware loader binding
information.

Signed-off-by: Tien Fong Chee <tien.fong.chee@intel.com>
---
 doc/device-tree-bindings/chosen.txt         | 21 +++++++++++++
 doc/device-tree-bindings/misc/fs_loader.txt | 48 +++++++++++++++++++++++++++++
 2 files changed, 69 insertions(+)
 create mode 100644 doc/device-tree-bindings/misc/fs_loader.txt

diff --git a/doc/device-tree-bindings/chosen.txt b/doc/device-tree-bindings/chosen.txt
index c96b8f7..dbce047 100644
--- a/doc/device-tree-bindings/chosen.txt
+++ b/doc/device-tree-bindings/chosen.txt
@@ -73,3 +73,24 @@ Example
 		u-boot,spl-boot-order = "same-as-spl", &sdmmc, "/sdhci at fe330000";
 	};
 };
+
+firmware-loader property
+------------------------
+Multiple file system firmware loader nodes could be defined in device trees for
+multiple storage type and their default partition, then a property
+"firmware-loader" can be used to pass default firmware loader
+node(default storage type) to the firmware loader driver.
+
+Example
+-------
+/ {
+	chosen {
+		firmware-loader = &fs_loader0;
+	};
+
+	fs_loader0: fs-loader at 0 {
+		u-boot,dm-pre-reloc;
+		compatible = "u-boot,fs-loader";
+		phandlepart = <&mmc 1>;
+	};
+};
diff --git a/doc/device-tree-bindings/misc/fs_loader.txt b/doc/device-tree-bindings/misc/fs_loader.txt
new file mode 100644
index 0000000..884fbf4
--- /dev/null
+++ b/doc/device-tree-bindings/misc/fs_loader.txt
@@ -0,0 +1,48 @@
+* File system firmware loader
+
+Required properties:
+--------------------
+
+- compatible: should contain "u-boot,fs-loader"
+- phandlepart: which block storage device and partition the image loading from,
+	       this property is required for mmc, usb and sata. This is unsigned
+	       32-bit array. For example phandlepart=<&mmc_0 1>, meaning use
+	       that MMC0 node pointer, partition 1.
+- mdtpart: which partition of ubi the image loading from, this property is
+	   required for ubi and mounting.
+- ubivol: which volume of ubi the image loading from, this property is required
+	  for ubi and mounting.
+
+Example of storage device and partition search set for mmc, usb, sata and
+ubi in device tree source as shown in below:
+
+	Example of storage type and device partition search set for mmc, usb,
+	sata and ubi as shown in below:
+	Example for mmc:
+	fs_loader0: fs-loader at 0 {
+		u-boot,dm-pre-reloc;
+		compatible = "u-boot,fs-loader";
+		phandlepart = <&mmc_0 1>;
+	};
+
+	Example for usb:
+	fs_loader1: fs-loader at 1 {
+		u-boot,dm-pre-reloc;
+		compatible = "u-boot,fs-loader";
+		phandlepart = <&usb0 1>;
+	};
+
+	Example for sata:
+	fs_loader2: fs-loader at 2 {
+		u-boot,dm-pre-reloc;
+		compatible = "u-boot,fs-loader";
+		phandlepart = <&sata0 1>;
+	};
+
+	Example for ubi:
+	fs_loader3: fs-loader at 3 {
+		u-boot,dm-pre-reloc;
+		compatible = "u-boot,fs-loader";
+		mtdpart = "UBI",
+		ubivol = "ubi0";
+	};
-- 
2.2.0

[U-Boot] [PATCH v4 5/6] doc: dtbinding: Add file system firmware loader binding document

[Simon Glass]

On 6 July 2018 at 02:27,  <tien.fong.chee@intel.com> wrote:
> From: Tien Fong Chee <tien.fong.chee@intel.com>
>
> Add a document to describe file system firmware loader binding
> information.
>
> Signed-off-by: Tien Fong Chee <tien.fong.chee@intel.com>
> ---
>  doc/device-tree-bindings/chosen.txt         | 21 +++++++++++++
>  doc/device-tree-bindings/misc/fs_loader.txt | 48 +++++++++++++++++++++++++++++
>  2 files changed, 69 insertions(+)
>  create mode 100644 doc/device-tree-bindings/misc/fs_loader.txt

Reviewed-by: Simon Glass <sjg@chromium.org>

Apart from the naming of phandlepart, as I mentioned in the previous patch

[U-Boot] [PATCH v4 5/6] doc: dtbinding: Add file system firmware loader binding document

[Chee, Tien Fong]

On Wed, 2018-07-11 at 08:02 -0600, Simon Glass wrote:
> On 6 July 2018 at 02:27,  <tien.fong.chee@intel.com> wrote:
> > 
> > From: Tien Fong Chee <tien.fong.chee@intel.com>
> > 
> > Add a document to describe file system firmware loader binding
> > information.
> > 
> > Signed-off-by: Tien Fong Chee <tien.fong.chee@intel.com>
> > ---
> >  doc/device-tree-bindings/chosen.txt         | 21 +++++++++++++
> >  doc/device-tree-bindings/misc/fs_loader.txt | 48
> > +++++++++++++++++++++++++++++
> >  2 files changed, 69 insertions(+)
> >  create mode 100644 doc/device-tree-bindings/misc/fs_loader.txt
> Reviewed-by: Simon Glass <sjg@chromium.org>
> 
> Apart from the naming of phandlepart, as I mentioned in the previous
> patch
Noted.

[U-Boot] [PATCH v4 5/6] doc: dtbinding: Add file system firmware loader binding document

[Chee, Tien Fong]

On Wed, 2018-07-11 at 08:02 -0600, Simon Glass wrote:
> On 6 July 2018 at 02:27,  <tien.fong.chee@intel.com> wrote:
> > 
> > From: Tien Fong Chee <tien.fong.chee@intel.com>
> > 
> > Add a document to describe file system firmware loader binding
> > information.
> > 
> > Signed-off-by: Tien Fong Chee <tien.fong.chee@intel.com>
> > ---
> >  doc/device-tree-bindings/chosen.txt         | 21 +++++++++++++
> >  doc/device-tree-bindings/misc/fs_loader.txt | 48
> > +++++++++++++++++++++++++++++
> >  2 files changed, 69 insertions(+)
> >  create mode 100644 doc/device-tree-bindings/misc/fs_loader.txt
> Reviewed-by: Simon Glass <sjg@chromium.org>
> 
> Apart from the naming of phandlepart, as I mentioned in the previous
> patch
Okay.

[U-Boot] [U-Boot, v4, 5/6] doc: dtbinding: Add file system firmware loader binding document

[Tom Rini]

On Fri, Jul 06, 2018 at 04:27:35PM +0800, tien.fong.chee at intel.com wrote:

> From: Tien Fong Chee <tien.fong.chee@intel.com>
> 
> Add a document to describe file system firmware loader binding
> information.
> 
> Signed-off-by: Tien Fong Chee <tien.fong.chee@intel.com>
> Reviewed-by: Simon Glass <sjg@chromium.org>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180929/e9d5d576/attachment.sig>