* [U-Boot] [PATCH] fit: Add support for printing more than one FPGA node name
@ 2019-02-13 12:32 tien.fong.chee at intel.com
2019-04-22 17:29 ` [U-Boot] " Tom Rini
2019-05-19 20:44 ` [U-Boot] [PATCH] " Tom Rini
0 siblings, 2 replies; 6+ messages in thread
From: tien.fong.chee at intel.com @ 2019-02-13 12:32 UTC (permalink / raw)
To: u-boot
From: Tien Fong Chee <tien.fong.chee@intel.com>
This would print out all the FPGA node names setting to fpga property.
Signed-off-by: Tien Fong Chee <tien.fong.chee@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;
--
1.7.7.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [U-Boot] fit: Add support for printing more than one FPGA node name
2019-02-13 12:32 [U-Boot] [PATCH] fit: Add support for printing more than one FPGA node name tien.fong.chee at intel.com
@ 2019-04-22 17:29 ` Tom Rini
2019-04-26 6:12 ` Chee, Tien Fong
2019-05-19 20:44 ` [U-Boot] [PATCH] " Tom Rini
1 sibling, 1 reply; 6+ messages in thread
From: Tom Rini @ 2019-04-22 17:29 UTC (permalink / raw)
To: u-boot
On Wed, Feb 13, 2019 at 08:32:24PM +0800, tien.fong.chee at intel.com wrote:
> From: Tien Fong Chee <tien.fong.chee@intel.com>
>
> This would print out all the FPGA node names setting to fpga property.
>
> Signed-off-by: Tien Fong Chee <tien.fong.chee@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>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] fit: Add support for printing more than one FPGA node name
2019-04-22 17:29 ` [U-Boot] " Tom Rini
@ 2019-04-26 6:12 ` Chee, Tien Fong
2019-04-27 14:43 ` Tom Rini
0 siblings, 1 reply; 6+ messages in thread
From: Chee, Tien Fong @ 2019-04-26 6:12 UTC (permalink / raw)
To: u-boot
On Mon, 2019-04-22 at 13:29 -0400, Tom Rini wrote:
> On Wed, Feb 13, 2019 at 08:32:24PM +0800, tien.fong.chee at intel.com
> wrote:
>
> >
> > From: Tien Fong Chee <tien.fong.chee@intel.com>
> >
> > This would print out all the FPGA node names setting to fpga
> > property.
> >
> > Signed-off-by: Tien Fong Chee <tien.fong.chee@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!
The header file in this patch https://patchwork.ozlabs.org/patch/105828
9/ , unfortunately the review taking a bit longer than what i expected.
I would update you once the whole series patches are accepted.
Thanks a lot.
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] fit: Add support for printing more than one FPGA node name
2019-04-26 6:12 ` Chee, Tien Fong
@ 2019-04-27 14:43 ` Tom Rini
2019-05-15 4:20 ` Chee, Tien Fong
0 siblings, 1 reply; 6+ messages in thread
From: Tom Rini @ 2019-04-27 14:43 UTC (permalink / raw)
To: u-boot
On Fri, Apr 26, 2019 at 06:12:40AM +0000, Chee, Tien Fong wrote:
> On Mon, 2019-04-22 at 13:29 -0400, Tom Rini wrote:
> > On Wed, Feb 13, 2019 at 08:32:24PM +0800, tien.fong.chee at intel.com
> > wrote:
> >
> > >
> > > From: Tien Fong Chee <tien.fong.chee@intel.com>
> > >
> > > This would print out all the FPGA node names setting to fpga
> > > property.
> > >
> > > Signed-off-by: Tien Fong Chee <tien.fong.chee@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!
> The header file in this patch https://patchwork.ozlabs.org/patch/1058289/,
> unfortunately the review taking a bit longer than what i expected.
>
> I would update you once the whole series patches are accepted.
Ah, OK, I'll hold off until the other series is complete.
--
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/20190427/c5c09647/attachment.sig>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] fit: Add support for printing more than one FPGA node name
2019-04-27 14:43 ` Tom Rini
@ 2019-05-15 4:20 ` Chee, Tien Fong
0 siblings, 0 replies; 6+ messages in thread
From: Chee, Tien Fong @ 2019-05-15 4:20 UTC (permalink / raw)
To: u-boot
On Sat, 2019-04-27 at 10:43 -0400, Tom Rini wrote:
> On Fri, Apr 26, 2019 at 06:12:40AM +0000, Chee, Tien Fong wrote:
> >
> > On Mon, 2019-04-22 at 13:29 -0400, Tom Rini wrote:
> > >
> > > On Wed, Feb 13, 2019 at 08:32:24PM +0800, tien.fong.chee at intel.co
> > > m
> > > wrote:
> > >
> > > >
> > > >
> > > > From: Tien Fong Chee <tien.fong.chee@intel.com>
> > > >
> > > > This would print out all the FPGA node names setting to fpga
> > > > property.
> > > >
> > > > Signed-off-by: Tien Fong Chee <tien.fong.chee@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!
> > The header file in this patch https://patchwork.ozlabs.org/patch/10
> > 58289/,
> > unfortunately the review taking a bit longer than what i expected.
> >
> > I would update you once the whole series patches are accepted.
> Ah, OK, I'll hold off until the other series is complete.
The series is already in mainline master branch.
Thanks.
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH] fit: Add support for printing more than one FPGA node name
2019-02-13 12:32 [U-Boot] [PATCH] fit: Add support for printing more than one FPGA node name tien.fong.chee at intel.com
2019-04-22 17:29 ` [U-Boot] " Tom Rini
@ 2019-05-19 20:44 ` Tom Rini
1 sibling, 0 replies; 6+ messages in thread
From: Tom Rini @ 2019-05-19 20:44 UTC (permalink / raw)
To: u-boot
On Wed, Feb 13, 2019 at 08:32:24PM +0800, tien.fong.chee at intel.com wrote:
> From: Tien Fong Chee <tien.fong.chee@intel.com>
>
> This would print out all the FPGA node names setting to fpga property.
>
> Signed-off-by: Tien Fong Chee <tien.fong.chee@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;
There's a problem here. When I use buildman to make multiple boards
(say 'arm', but something smaller should trigger it too), all of my
builds get stuck on mkimage doing something. A build of a single board
is fine.
--
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/20190519/84c90e31/attachment.sig>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-05-19 20:44 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-02-13 12:32 [U-Boot] [PATCH] fit: Add support for printing more than one FPGA node name tien.fong.chee at intel.com
2019-04-22 17:29 ` [U-Boot] " Tom Rini
2019-04-26 6:12 ` Chee, Tien Fong
2019-04-27 14:43 ` Tom Rini
2019-05-15 4:20 ` Chee, Tien Fong
2019-05-19 20:44 ` [U-Boot] [PATCH] " Tom Rini
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.