Hi Rick and Bin, On Wed, 2018-10-24 at 14:54 +0800, Rick Chen wrote: > > > > Print the address of the u-boot device tree. > > > > > > > > > > This is unnecessary as it is already done by 'fdt' command. > > > > > Hi Bin & Lukas > > At the beginning, I really don't understand the fdt Usage: > Maybe I am not so familiar with fdt. > I can't find any string about fdt_blob. It seem not describe so > clearly > After I trace fdt.c, I finally know to get it as below. > > RISC-V # bdinfo > boot_params = 0x00000400 > ... > baudrate = 38400 bps > fdt_blob = 0x3FF0F340 > RISC-V # > > RISC-V # fdt addr -c > The address of the fdt is 0x3ff0f340 > > Usage: > fdt addr [-c] <addr> [<length>] - Set the [control] fdt location > to <addr> > fdt move <fdt> <newaddr> <length> - Copy the fdt to <addr> and make > it active > fdt resize [<extrasize>] - Resize fdt to size + padding to > 4k addr + some optional <extrasize> if needed > fdt print <path> [<prop>] - Recursive print starting at > <path> > fdt list <path> [<prop>] - Print one level starting at > <path> > fdt get value <var> <path> <prop> - Get <property> and store in > <var> > fdt get name <var> <path> <index> - Get name of node <index> and > store in <var> > fdt get addr <var> <path> <prop> - Get start address of <property> > and store in <var> > fdt get size <var> <path> [<prop>] - Get size of [<property>] or num > nodes and store in <var> > fdt set <path> <prop> [<val>] - Set <property> [to <val>] > fdt mknode <path> <node> - Create a new node after <path> > fdt rm <path> [<prop>] - Delete the node or <property> > fdt header - Display header info > fdt bootcpu <id> - Set boot cpuid > fdt memory <addr> <size> - Add/Update memory node > fdt rsvmem print - Show current mem reserves > fdt rsvmem add <addr> <size> - Add a mem reserve > fdt rsvmem delete <index> - Delete a mem reserves > fdt chosen [<start> <end>] - Add/update the /chosen branch > in the tree > <start>/<end> - initrd > start/end addr > > It seem that other arch (MICROBLAZE, ARM) also print fdt_blob in > bdinfo cmd. > So I think this patch can be accepted. > > Reviewed-by: Rick Chen <r...@andestech.com> > > Rick >
I was not aware that the fdt command can print the location of fdt_blob. I think this patch can be dropped, unless you want me to keep it. Thanks, Lukas > > > > > Signed-off-by: Lukas Auer <lukas.a...@aisec.fraunhofer.de> > > > > --- > > > > > > > > cmd/bdinfo.c | 2 ++ > > > > 1 file changed, 2 insertions(+) > > > > > > > > > > Regards, > > > Bin _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot