Re: [U-Boot] [PATCH] libfdt: introduce function fdt_get_max_phandle
On May 26, 2010, at 9:38 PM, Jerry Van Baren wrote: Wolfgang Denk wrote: Dear Timur Tabi, In message 1274308618-2974-1-git-send-email-ti...@freescale.com you wrote: Introduce function fdt_get_max_phandle(), which returns the largest value of all phandles in a device tree. This is useful for allocating a new phandle property, since all phandles must be unique. Signed-off-by: Timur Tabi ti...@freescale.com --- include/libfdt.h| 20 lib/libfdt/fdt_ro.c | 17 + 2 files changed, 37 insertions(+), 0 deletions(-) Please submit as part of a patch / patch series that acrtually uses this feature. As is, it's just dead code that has no users. I'm happy with this in principle, and will ack formally when the community is OK with the code change. It looks like a positive improvement to me, reducing complexity and addressing Timur/Freescale's needs. Anyway, I envision acking the patch and having FSL include it in whatever MPC8xxx patchset that actually needs the changes. Thanks and best regards, gvb We should NOT add code to libfdt if we can avoid it. We should be doing this in fdt_support.c. libfdt changes should first go to dtc. - k ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] libfdt: introduce function fdt_get_max_phandle
Dear Timur Tabi, In message 1274308618-2974-1-git-send-email-ti...@freescale.com you wrote: Introduce function fdt_get_max_phandle(), which returns the largest value of all phandles in a device tree. This is useful for allocating a new phandle property, since all phandles must be unique. Signed-off-by: Timur Tabi ti...@freescale.com --- include/libfdt.h| 20 lib/libfdt/fdt_ro.c | 17 + 2 files changed, 37 insertions(+), 0 deletions(-) Please submit as part of a patch / patch series that acrtually uses this feature. As is, it's just dead code that has no users. +int fdt_get_max_phandle(const void *fdt) +{ + int offset; + uint32_t temp, phandle = 0; + + FDT_CHECK_HEADER(fdt); + + for (offset = fdt_next_node(fdt, -1, NULL); offset = 0; + offset = fdt_next_node(fdt, offset, NULL)) { + temp = fdt_get_phandle(fdt, offset); + if (temp phandle) + phandle = temp; + } + + return phandle; +} As fdt_get_phandle() returns a uint32_t, fdt_get_max_phandle() should return the same type, not int. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de Star Trek Lives! ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] libfdt: introduce function fdt_get_max_phandle
Wolfgang Denk wrote: Dear Timur Tabi, In message 1274308618-2974-1-git-send-email-ti...@freescale.com you wrote: Introduce function fdt_get_max_phandle(), which returns the largest value of all phandles in a device tree. This is useful for allocating a new phandle property, since all phandles must be unique. Signed-off-by: Timur Tabi ti...@freescale.com --- include/libfdt.h| 20 lib/libfdt/fdt_ro.c | 17 + 2 files changed, 37 insertions(+), 0 deletions(-) Please submit as part of a patch / patch series that acrtually uses this feature. As is, it's just dead code that has no users. I'm happy with this in principle, and will ack formally when the community is OK with the code change. It looks like a positive improvement to me, reducing complexity and addressing Timur/Freescale's needs. Anyway, I envision acking the patch and having FSL include it in whatever MPC8xxx patchset that actually needs the changes. Thanks and best regards, gvb [snip improvement suggestion] ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot