* [U-Boot] [PATCH] SPL: Add FIT data-position property support
@ 2017-11-28 3:19 Peng Fan
2017-11-28 5:00 ` Simon Glass
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Peng Fan @ 2017-11-28 3:19 UTC (permalink / raw)
To: u-boot
For external data, FIT has a optional property "data-position" which
can set the external data to a fixed offset to FIT beginning.
Add the support for this property in SPL FIT.
Signed-off-by: Ye Li <ye.li@nxp.com>
Signed-off-by: Peng Fan <peng.fan@nxp.com>
Cc: Simon Glass <sjg@chromium.org>
Cc: Pantelis Antoniou <pantelis.antoniou@konsulko.com>
Cc: "Andrew F. Davis" <afd@ti.com>
Cc: Igor Grinberg <grinberg@compulab.co.il>
Cc: "tomas.melin at vaisala.com" <tomas.melin@vaisala.com>
Cc: Kever Yang <kever.yang@rock-chips.com>
Cc: Andre Przywara <andre.przywara@arm.com>
Cc: York Sun <york.sun@nxp.com>
Cc: Lokesh Vutla <lokeshvutla@ti.com>
Cc: "Cooper Jr., Franklin" <fcooper@ti.com>
Cc: George McCollister <george.mccollister@gmail.com>
Cc: Tuomas Tynkkynen <tuomas.tynkkynen@iki.fi>
Cc: Jean-Jacques Hiblot <jjhiblot@ti.com>
Cc: Rick Altherr <raltherr@google.com>
Cc: Tom Rini <trini@konsulko.com>
---
common/image-fit.c | 25 +++++++++++++++++++++++++
common/spl/spl_fit.c | 11 +++++++++--
include/image.h | 3 +++
3 files changed, 37 insertions(+), 2 deletions(-)
diff --git a/common/image-fit.c b/common/image-fit.c
index 7f17fd1410..b785d8a36e 100644
--- a/common/image-fit.c
+++ b/common/image-fit.c
@@ -806,6 +806,31 @@ int fit_image_get_data_offset(const void *fit, int noffset, int *data_offset)
return 0;
}
+/**
+ * Get 'data-position' property from a given image node.
+ *
+ * @fit: pointer to the FIT image header
+ * @noffset: component image node offset
+ * @data_position: holds the data-position property
+ *
+ * returns:
+ * 0, on success
+ * -ENOENT if the property could not be found
+ */
+int fit_image_get_data_position(const void *fit, int noffset,
+ int *data_position)
+{
+ const fdt32_t *val;
+
+ val = fdt_getprop(fit, noffset, FIT_DATA_POSITION_PROP, NULL);
+ if (!val)
+ return -ENOENT;
+
+ *data_position = fdt32_to_cpu(*val);
+
+ return 0;
+}
+
/**
* Get 'data-size' property from a given image node.
*
diff --git a/common/spl/spl_fit.c b/common/spl/spl_fit.c
index 32d9ee5901..c915b4b149 100644
--- a/common/spl/spl_fit.c
+++ b/common/spl/spl_fit.c
@@ -143,6 +143,7 @@ static int spl_load_fit_image(struct spl_load_info *info, ulong sector,
int align_len = ARCH_DMA_MINALIGN - 1;
uint8_t image_comp = -1, type = -1;
const void *data;
+ bool external_data = false;
if (IS_ENABLED(CONFIG_SPL_OS_BOOT) && IS_ENABLED(CONFIG_SPL_GZIP)) {
if (fit_image_get_comp(fit, node, &image_comp))
@@ -159,9 +160,15 @@ static int spl_load_fit_image(struct spl_load_info *info, ulong sector,
if (fit_image_get_load(fit, node, &load_addr))
load_addr = image_info->load_addr;
- if (!fit_image_get_data_offset(fit, node, &offset)) {
- /* External data */
+ if (!fit_image_get_data_position(fit, node, &offset)) {
+ external_data = true;
+ } else if (!fit_image_get_data_offset(fit, node, &offset)) {
offset += base_offset;
+ external_data = true;
+ }
+
+ if (external_data) {
+ /* External data */
if (fit_image_get_data_size(fit, node, &len))
return -ENOENT;
diff --git a/include/image.h b/include/image.h
index 127cfc5148..15e171d037 100644
--- a/include/image.h
+++ b/include/image.h
@@ -886,6 +886,7 @@ int bootz_setup(ulong image, ulong *start, ulong *end);
/* image node */
#define FIT_DATA_PROP "data"
+#define FIT_DATA_POSITION_PROP "data-position"
#define FIT_DATA_OFFSET_PROP "data-offset"
#define FIT_DATA_SIZE_PROP "data-size"
#define FIT_TIMESTAMP_PROP "timestamp"
@@ -967,6 +968,8 @@ int fit_image_get_entry(const void *fit, int noffset, ulong *entry);
int fit_image_get_data(const void *fit, int noffset,
const void **data, size_t *size);
int fit_image_get_data_offset(const void *fit, int noffset, int *data_offset);
+int fit_image_get_data_position(const void *fit, int noffset,
+ int *data_position);
int fit_image_get_data_size(const void *fit, int noffset, int *data_size);
int fit_image_hash_get_algo(const void *fit, int noffset, char **algo);
--
2.14.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] SPL: Add FIT data-position property support
2017-11-28 3:19 [U-Boot] [PATCH] SPL: Add FIT data-position property support Peng Fan
@ 2017-11-28 5:00 ` Simon Glass
2017-11-28 17:14 ` York Sun
2017-11-29 5:23 ` Tomas Melin
2 siblings, 0 replies; 6+ messages in thread
From: Simon Glass @ 2017-11-28 5:00 UTC (permalink / raw)
To: u-boot
On 27 November 2017 at 20:19, Peng Fan <peng.fan@nxp.com> wrote:
> For external data, FIT has a optional property "data-position" which
> can set the external data to a fixed offset to FIT beginning.
> Add the support for this property in SPL FIT.
>
> Signed-off-by: Ye Li <ye.li@nxp.com>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> Cc: Simon Glass <sjg@chromium.org>
> Cc: Pantelis Antoniou <pantelis.antoniou@konsulko.com>
> Cc: "Andrew F. Davis" <afd@ti.com>
> Cc: Igor Grinberg <grinberg@compulab.co.il>
> Cc: "tomas.melin at vaisala.com" <tomas.melin@vaisala.com>
> Cc: Kever Yang <kever.yang@rock-chips.com>
> Cc: Andre Przywara <andre.przywara@arm.com>
> Cc: York Sun <york.sun@nxp.com>
> Cc: Lokesh Vutla <lokeshvutla@ti.com>
> Cc: "Cooper Jr., Franklin" <fcooper@ti.com>
> Cc: George McCollister <george.mccollister@gmail.com>
> Cc: Tuomas Tynkkynen <tuomas.tynkkynen@iki.fi>
> Cc: Jean-Jacques Hiblot <jjhiblot@ti.com>
> Cc: Rick Altherr <raltherr@google.com>
> Cc: Tom Rini <trini@konsulko.com>
> ---
> common/image-fit.c | 25 +++++++++++++++++++++++++
> common/spl/spl_fit.c | 11 +++++++++--
> include/image.h | 3 +++
> 3 files changed, 37 insertions(+), 2 deletions(-)
Reviewed-by: Simon Glass <sjg@chromium.org>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] SPL: Add FIT data-position property support
2017-11-28 3:19 [U-Boot] [PATCH] SPL: Add FIT data-position property support Peng Fan
2017-11-28 5:00 ` Simon Glass
@ 2017-11-28 17:14 ` York Sun
2017-12-05 5:02 ` Peng Fan
2017-11-29 5:23 ` Tomas Melin
2 siblings, 1 reply; 6+ messages in thread
From: York Sun @ 2017-11-28 17:14 UTC (permalink / raw)
To: u-boot
On 11/27/2017 07:20 PM, Peng Fan wrote:
> For external data, FIT has a optional property "data-position" which
> can set the external data to a fixed offset to FIT beginning.
> Add the support for this property in SPL FIT.
>
> Signed-off-by: Ye Li <ye.li@nxp.com>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> Cc: Simon Glass <sjg@chromium.org>
> Cc: Pantelis Antoniou <pantelis.antoniou@konsulko.com>
> Cc: "Andrew F. Davis" <afd@ti.com>
> Cc: Igor Grinberg <grinberg@compulab.co.il>
> Cc: "tomas.melin at vaisala.com" <tomas.melin@vaisala.com>
> Cc: Kever Yang <kever.yang@rock-chips.com>
> Cc: Andre Przywara <andre.przywara@arm.com>
> Cc: York Sun <york.sun@nxp.com>
> Cc: Lokesh Vutla <lokeshvutla@ti.com>
> Cc: "Cooper Jr., Franklin" <fcooper@ti.com>
> Cc: George McCollister <george.mccollister@gmail.com>
> Cc: Tuomas Tynkkynen <tuomas.tynkkynen@iki.fi>
> Cc: Jean-Jacques Hiblot <jjhiblot@ti.com>
> Cc: Rick Altherr <raltherr@google.com>
> Cc: Tom Rini <trini@konsulko.com>
> ---
> common/image-fit.c | 25 +++++++++++++++++++++++++
> common/spl/spl_fit.c | 11 +++++++++--
> include/image.h | 3 +++
> 3 files changed, 37 insertions(+), 2 deletions(-)
Looks good. Would it be appropriate to update
doc/uImage.FIT/source_file_format.txt to include this property?
York
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] SPL: Add FIT data-position property support
2017-11-28 3:19 [U-Boot] [PATCH] SPL: Add FIT data-position property support Peng Fan
2017-11-28 5:00 ` Simon Glass
2017-11-28 17:14 ` York Sun
@ 2017-11-29 5:23 ` Tomas Melin
2 siblings, 0 replies; 6+ messages in thread
From: Tomas Melin @ 2017-11-29 5:23 UTC (permalink / raw)
To: u-boot
On 11/28/2017 05:19 AM, Peng Fan wrote:
> For external data, FIT has a optional property "data-position" which
> can set the external data to a fixed offset to FIT beginning.
> Add the support for this property in SPL FIT.
>
> Signed-off-by: Ye Li <ye.li@nxp.com>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> Cc: Simon Glass <sjg@chromium.org>
> Cc: Pantelis Antoniou <pantelis.antoniou@konsulko.com>
> Cc: "Andrew F. Davis" <afd@ti.com>
> Cc: Igor Grinberg <grinberg@compulab.co.il>
> Cc: "tomas.melin at vaisala.com" <tomas.melin@vaisala.com>
> Cc: Kever Yang <kever.yang@rock-chips.com>
> Cc: Andre Przywara <andre.przywara@arm.com>
> Cc: York Sun <york.sun@nxp.com>
> Cc: Lokesh Vutla <lokeshvutla@ti.com>
> Cc: "Cooper Jr., Franklin" <fcooper@ti.com>
> Cc: George McCollister <george.mccollister@gmail.com>
> Cc: Tuomas Tynkkynen <tuomas.tynkkynen@iki.fi>
> Cc: Jean-Jacques Hiblot <jjhiblot@ti.com>
> Cc: Rick Altherr <raltherr@google.com>
> Cc: Tom Rini <trini@konsulko.com>
> ---
> common/image-fit.c | 25 +++++++++++++++++++++++++
> common/spl/spl_fit.c | 11 +++++++++--
> include/image.h | 3 +++
> 3 files changed, 37 insertions(+), 2 deletions(-)
>
Reviewed-by: Tomas Melin <tomas.melin@vaisala.com>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] SPL: Add FIT data-position property support
2017-11-28 17:14 ` York Sun
@ 2017-12-05 5:02 ` Peng Fan
2017-12-05 5:06 ` York Sun
0 siblings, 1 reply; 6+ messages in thread
From: Peng Fan @ 2017-12-05 5:02 UTC (permalink / raw)
To: u-boot
On Tue, Nov 28, 2017 at 05:14:58PM +0000, York Sun wrote:
>On 11/27/2017 07:20 PM, Peng Fan wrote:
>> For external data, FIT has a optional property "data-position" which
>> can set the external data to a fixed offset to FIT beginning.
>> Add the support for this property in SPL FIT.
>>
>> Signed-off-by: Ye Li <ye.li@nxp.com>
>> Signed-off-by: Peng Fan <peng.fan@nxp.com>
>> Cc: Simon Glass <sjg@chromium.org>
>> Cc: Pantelis Antoniou <pantelis.antoniou@konsulko.com>
>> Cc: "Andrew F. Davis" <afd@ti.com>
>> Cc: Igor Grinberg <grinberg@compulab.co.il>
>> Cc: "tomas.melin at vaisala.com" <tomas.melin@vaisala.com>
>> Cc: Kever Yang <kever.yang@rock-chips.com>
>> Cc: Andre Przywara <andre.przywara@arm.com>
>> Cc: York Sun <york.sun@nxp.com>
>> Cc: Lokesh Vutla <lokeshvutla@ti.com>
>> Cc: "Cooper Jr., Franklin" <fcooper@ti.com>
>> Cc: George McCollister <george.mccollister@gmail.com>
>> Cc: Tuomas Tynkkynen <tuomas.tynkkynen@iki.fi>
>> Cc: Jean-Jacques Hiblot <jjhiblot@ti.com>
>> Cc: Rick Altherr <raltherr@google.com>
>> Cc: Tom Rini <trini@konsulko.com>
>> ---
>> common/image-fit.c | 25 +++++++++++++++++++++++++
>> common/spl/spl_fit.c | 11 +++++++++--
>> include/image.h | 3 +++
>> 3 files changed, 37 insertions(+), 2 deletions(-)
>
>
>Looks good. Would it be appropriate to update
>doc/uImage.FIT/source_file_format.txt to include this property?
All,
Is the following explaination ok?
diff --git a/doc/uImage.FIT/source_file_format.txt b/doc/uImage.FIT/source_file_format.txt
index 6f727a1e8a..0db23db6c7 100644
--- a/doc/uImage.FIT/source_file_format.txt
+++ b/doc/uImage.FIT/source_file_format.txt
@@ -288,7 +288,8 @@ In this case the 'data' property is omitted. Instead you can use:
The 'data-offset' property can be substituted with 'data-position', which
defines an absolute position or address as the offset. This is helpful when
-booting U-Boot proper before performing relocation.
+booting U-Boot proper before performing relocation. Pass `-p [offset]` to
+mkimage to enable `data-position`.
If it is ok, I'll add it into V2.
Thanks,
Peng.
>
>York
>
>_______________________________________________
>U-Boot mailing list
>U-Boot at lists.denx.de
>https://lists.denx.de/listinfo/u-boot
--
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] SPL: Add FIT data-position property support
2017-12-05 5:02 ` Peng Fan
@ 2017-12-05 5:06 ` York Sun
0 siblings, 0 replies; 6+ messages in thread
From: York Sun @ 2017-12-05 5:06 UTC (permalink / raw)
To: u-boot
> On Dec 4, 2017, at 21:02, Peng Fan <van.freenix@gmail.com> wrote:
>
>> On Tue, Nov 28, 2017 at 05:14:58PM +0000, York Sun wrote:
>>> On 11/27/2017 07:20 PM, Peng Fan wrote:
>>> For external data, FIT has a optional property "data-position" which
>>> can set the external data to a fixed offset to FIT beginning.
>>> Add the support for this property in SPL FIT.
>>>
>>> Signed-off-by: Ye Li <ye.li@nxp.com>
>>> Signed-off-by: Peng Fan <peng.fan@nxp.com>
>>> Cc: Simon Glass <sjg@chromium.org>
>>> Cc: Pantelis Antoniou <pantelis.antoniou@konsulko.com>
>>> Cc: "Andrew F. Davis" <afd@ti.com>
>>> Cc: Igor Grinberg <grinberg@compulab.co.il>
>>> Cc: "tomas.melin at vaisala.com" <tomas.melin@vaisala.com>
>>> Cc: Kever Yang <kever.yang@rock-chips.com>
>>> Cc: Andre Przywara <andre.przywara@arm.com>
>>> Cc: York Sun <york.sun@nxp.com>
>>> Cc: Lokesh Vutla <lokeshvutla@ti.com>
>>> Cc: "Cooper Jr., Franklin" <fcooper@ti.com>
>>> Cc: George McCollister <george.mccollister@gmail.com>
>>> Cc: Tuomas Tynkkynen <tuomas.tynkkynen@iki.fi>
>>> Cc: Jean-Jacques Hiblot <jjhiblot@ti.com>
>>> Cc: Rick Altherr <raltherr@google.com>
>>> Cc: Tom Rini <trini@konsulko.com>
>>> ---
>>> common/image-fit.c | 25 +++++++++++++++++++++++++
>>> common/spl/spl_fit.c | 11 +++++++++--
>>> include/image.h | 3 +++
>>> 3 files changed, 37 insertions(+), 2 deletions(-)
>>
>>
>> Looks good. Would it be appropriate to update
>> doc/uImage.FIT/source_file_format.txt to include this property?
>
> All,
>
> Is the following explaination ok?
>
> diff --git a/doc/uImage.FIT/source_file_format.txt b/doc/uImage.FIT/source_file_format.txt
> index 6f727a1e8a..0db23db6c7 100644
> --- a/doc/uImage.FIT/source_file_format.txt
> +++ b/doc/uImage.FIT/source_file_format.txt
> @@ -288,7 +288,8 @@ In this case the 'data' property is omitted. Instead you can use:
>
> The 'data-offset' property can be substituted with 'data-position', which
> defines an absolute position or address as the offset. This is helpful when
> -booting U-Boot proper before performing relocation.
> +booting U-Boot proper before performing relocation. Pass `-p [offset]` to
> +mkimage to enable `data-position`.
>
> If it is ok, I'll add it into V2.
>
It's something. I am OK.
York
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-12-05 5:06 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-28 3:19 [U-Boot] [PATCH] SPL: Add FIT data-position property support Peng Fan
2017-11-28 5:00 ` Simon Glass
2017-11-28 17:14 ` York Sun
2017-12-05 5:02 ` Peng Fan
2017-12-05 5:06 ` York Sun
2017-11-29 5:23 ` Tomas Melin
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.