[U-Boot] fit: Add support for printing more than one FPGA node name
Tom Rini
trini at konsulko.com
Mon Apr 22 17:29:39 UTC 2019
On Wed, Feb 13, 2019 at 08:32:24PM +0800, tien.fong.chee at intel.com wrote:
> From: Tien Fong Chee <tien.fong.chee at intel.com>
>
> This would print out all the FPGA node names setting to fpga property.
>
> Signed-off-by: Tien Fong Chee <tien.fong.chee at intel.com>
> ---
> common/image-fit.c | 14 +++++++++++---
> 1 files changed, 11 insertions(+), 3 deletions(-)
>
> diff --git a/common/image-fit.c b/common/image-fit.c
> index ac901e1..816e17d 100644
> --- a/common/image-fit.c
> +++ b/common/image-fit.c
> @@ -263,6 +263,7 @@ static void fit_conf_print(const void *fit, int noffset, const char *p)
> int ret;
> int fdt_index, loadables_index;
> int ndepth;
> + ulong count;
>
> /* Mandatory properties */
> ret = fit_get_desc(fit, noffset, &desc);
> @@ -299,9 +300,16 @@ static void fit_conf_print(const void *fit, int noffset, const char *p)
> printf("%s\n", uname);
> }
>
> - uname = fdt_getprop(fit, noffset, FIT_FPGA_PROP, NULL);
> - if (uname)
> - printf("%s FPGA: %s\n", p, uname);
> + count = fit_conf_get_prop_node_count(fit, noffset, FIT_FPGA_PROP);
> +
> + for (ndepth = 0; ndepth < count; ndepth++) {
> + int images_noffset = fit_conf_get_prop_node_index(fit, noffset,
> + FIT_FPGA_PROP, ndepth);
> + uname = fit_get_name(fit, images_noffset, NULL);
> +
> + if (uname)
> + printf("%s FPGA: %s\n", p, uname);
> + }
>
> /* Print out all of the specified loadables */
> for (loadables_index = 0;
While I'm fine with the conceptual change here, both
fit_conf_get_prop_node_count / fit_conf_get_prop_node_index are defined
later in the file and while public functions not documented in a header.
Can you please do a patch that does so first, as we otherwise get a
warning? 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/20190422/4a9ae92b/attachment.sig>
More information about the U-Boot
mailing list