[PATCH] bdinfo: Show information about fdt blob via bdinfo

[PATCH] bdinfo: Show information about fdt blob via bdinfo

[Heiko Schocher]

PowerPC target supports OF booting, so print the
fdt address and size in bdinfo.

Signed-off-by: Heiko Schocher <hs@denx.de>
---
Travis build:
https://travis-ci.org/hsdenx/u-boot-test/builds/651885430

 cmd/bdinfo.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/cmd/bdinfo.c b/cmd/bdinfo.c
index d6a7175b37..42b0f49b1a 100644
--- a/cmd/bdinfo.c
+++ b/cmd/bdinfo.c
@@ -208,6 +208,9 @@ int do_bdinfo(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 	print_baudrate();
 	print_num("relocaddr", gd->relocaddr);
 	board_detail();
+	print_num("fdt_blob", (ulong)gd->fdt_blob);
+	print_num("new_fdt", (ulong)gd->new_fdt);
+	print_num("fdt_size", (ulong)gd->fdt_size);
 	return 0;
 }
 
-- 
2.24.1

[PATCH] bdinfo: Show information about fdt blob via bdinfo

[Simon Glass]

On Tue, 18 Feb 2020 at 22:19, Heiko Schocher <hs@denx.de> wrote:
>
> PowerPC target supports OF booting, so print the
> fdt address and size in bdinfo.
>
> Signed-off-by: Heiko Schocher <hs@denx.de>
> ---
> Travis build:
> https://travis-ci.org/hsdenx/u-boot-test/builds/651885430
>
>  cmd/bdinfo.c | 3 +++
>  1 file changed, 3 insertions(+)

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

[PATCH] bdinfo: Show information about fdt blob via bdinfo

[Heiko Schocher]

Hello Tom,

Am 19.02.2020 um 06:19 schrieb Heiko Schocher:
> PowerPC target supports OF booting, so print the
> fdt address and size in bdinfo.
> 
> Signed-off-by: Heiko Schocher <hs@denx.de>
> ---
> Travis build:
> https://travis-ci.org/hsdenx/u-boot-test/builds/651885430
> 
>   cmd/bdinfo.c | 3 +++
>   1 file changed, 3 insertions(+)

Any issues with this patch, or do you plan to apply it after release?

Thanks!

bye,
Heiko
-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52   Fax: +49-8142-66989-80   Email: hs at denx.de

[PATCH] bdinfo: Show information about fdt blob via bdinfo

[Heiko Schocher]

Hello Tom,

Am 09.04.2020 um 07:14 schrieb Heiko Schocher:
> Hello Tom,
> 
> Am 19.02.2020 um 06:19 schrieb Heiko Schocher:
>> PowerPC target supports OF booting, so print the
>> fdt address and size in bdinfo.
>>
>> Signed-off-by: Heiko Schocher <hs@denx.de>
>> ---
>> Travis build:
>> https://travis-ci.org/hsdenx/u-boot-test/builds/651885430
>>
>> ? cmd/bdinfo.c | 3 +++
>> ? 1 file changed, 3 insertions(+)
> 
> Any issues with this patch, or do you plan to apply it after release?
> 
> Thanks!

Any issues?

bye,
Heiko
-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52   Fax: +49-8142-66989-80   Email: hs at denx.de

[PATCH] bdinfo: Show information about fdt blob via bdinfo

[Bin Meng]

Hi Heiko,

On Mon, May 18, 2020 at 12:28 PM Heiko Schocher <hs@denx.de> wrote:
>
> Hello Tom,
>
> Am 09.04.2020 um 07:14 schrieb Heiko Schocher:
> > Hello Tom,
> >
> > Am 19.02.2020 um 06:19 schrieb Heiko Schocher:
> >> PowerPC target supports OF booting, so print the
> >> fdt address and size in bdinfo.
> >>
> >> Signed-off-by: Heiko Schocher <hs@denx.de>
> >> ---
> >> Travis build:
> >> https://travis-ci.org/hsdenx/u-boot-test/builds/651885430
> >>
> >>   cmd/bdinfo.c | 3 +++
> >>   1 file changed, 3 insertions(+)
> >
> > Any issues with this patch, or do you plan to apply it after release?
> >
> > Thanks!
>
> Any issues?

I think it's better to have this patch rebased on top of Simon's
bdinfo clean-up series.
http://patchwork.ozlabs.org/project/uboot/list/?series=176007

Regards,
Bin

[PATCH] bdinfo: Show information about fdt blob via bdinfo

[Tom Rini]

On Mon, May 18, 2020 at 01:42:32PM +0800, Bin Meng wrote:
> Hi Heiko,
> 
> On Mon, May 18, 2020 at 12:28 PM Heiko Schocher <hs@denx.de> wrote:
> >
> > Hello Tom,
> >
> > Am 09.04.2020 um 07:14 schrieb Heiko Schocher:
> > > Hello Tom,
> > >
> > > Am 19.02.2020 um 06:19 schrieb Heiko Schocher:
> > >> PowerPC target supports OF booting, so print the
> > >> fdt address and size in bdinfo.
> > >>
> > >> Signed-off-by: Heiko Schocher <hs@denx.de>
> > >> ---
> > >> Travis build:
> > >> https://travis-ci.org/hsdenx/u-boot-test/builds/651885430
> > >>
> > >>   cmd/bdinfo.c | 3 +++
> > >>   1 file changed, 3 insertions(+)
> > >
> > > Any issues with this patch, or do you plan to apply it after release?
> > >
> > > Thanks!
> >
> > Any issues?
> 
> I think it's better to have this patch rebased on top of Simon's
> bdinfo clean-up series.
> http://patchwork.ozlabs.org/project/uboot/list/?series=176007

Yes, if Simon's series doesn't already address this problem, it should.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200518/3d74ff9e/attachment.sig>

[PATCH] bdinfo: Show information about fdt blob via bdinfo

[Heiko Schocher]

Hello Tom, Bin,

Am 18.05.2020 um 17:17 schrieb Tom Rini:
> On Mon, May 18, 2020 at 01:42:32PM +0800, Bin Meng wrote:
>> Hi Heiko,
>>
>> On Mon, May 18, 2020 at 12:28 PM Heiko Schocher <hs@denx.de> wrote:
>>>
>>> Hello Tom,
>>>
>>> Am 09.04.2020 um 07:14 schrieb Heiko Schocher:
>>>> Hello Tom,
>>>>
>>>> Am 19.02.2020 um 06:19 schrieb Heiko Schocher:
>>>>> PowerPC target supports OF booting, so print the
>>>>> fdt address and size in bdinfo.
>>>>>
>>>>> Signed-off-by: Heiko Schocher <hs@denx.de>
>>>>> ---
>>>>> Travis build:
>>>>> https://travis-ci.org/hsdenx/u-boot-test/builds/651885430
>>>>>
>>>>>    cmd/bdinfo.c | 3 +++
>>>>>    1 file changed, 3 insertions(+)
>>>>
>>>> Any issues with this patch, or do you plan to apply it after release?
>>>>
>>>> Thanks!
>>>
>>> Any issues?
>>
>> I think it's better to have this patch rebased on top of Simon's
>> bdinfo clean-up series.
>> http://patchwork.ozlabs.org/project/uboot/list/?series=176007
> 
> Yes, if Simon's series doesn't already address this problem, it should.

Yep, I send a new patch (if necessary) after Simons patches are merged.

bye,
Heiko
-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52   Fax: +49-8142-66989-80   Email: hs at denx.de