Re: [PATCH] libfdt/tests: Add fdt_path_offset_namelen() test

2015-04-07 Thread Peter Hurley
On 04/07/2015 12:49 AM, David Gibson wrote: On Sun, Apr 05, 2015 at 12:05:56AM -0400, Peter Hurley wrote: Add unit test for fdt_path_offset_namelen(). Verify partial path- descending retrieves the same node offset as fdt_subnode_offset(). Verify parsing correctness with multiple path

Re: [PATCH] libfdt/tests: Add fdt_path_offset_namelen() test

2015-04-07 Thread Peter Hurley
On 04/07/2015 07:12 AM, Peter Hurley wrote: On 04/07/2015 12:49 AM, David Gibson wrote: I did spot a case where the existing code is arguably incorrect: using fdt_path_offset_namelen(fdt, /somenode\0foo, 13) will return the same as fdt_path_offset(fdt, /somenode), although I think ideally

Re: [PATCH] libfdt: Add fdt_path_offset_namelen()

2015-04-04 Thread Peter Hurley
On 03/13/2015 01:43 AM, David Gibson wrote: On Tue, Mar 10, 2015 at 09:47:44AM -0400, Peter Hurley wrote: Hi David, On 03/09/2015 08:17 PM, David Gibson wrote: On Fri, Mar 06, 2015 at 10:12:38AM -0500, Peter Hurley wrote: Properties may contain path names which are not NUL-terminated

[PATCH] libfdt/tests: Add fdt_path_offset_namelen() test

2015-04-04 Thread Peter Hurley
Add unit test for fdt_path_offset_namelen(). Verify partial path- descending retrieves the same node offset as fdt_subnode_offset(). Verify parsing correctness with multiple path separators, both mid-path and trailing. Signed-off-by: Peter Hurley pe...@hurleysoftware.com --- tests/Makefile.tests