Thread (2 messages) flat view 2 messages, 2 authors, 2016-07-21

Re: [PATCH v2 5/6] libfdt: Add fdt_setprop_inplace_namelen_partial

From: Maxime Ripard <hidden>
Date: 2016-07-18 13:45:58

Hi David,

On Tue, Jul 12, 2016 at 09:45:20PM +1000, David Gibson wrote:
On Mon, Jul 11, 2016 at 09:56:22PM +0200, Maxime Ripard wrote:
quoted
Add a function to modify inplace only a portion of a property..

This is especially useful when the property is an array of values, and you
want to update one of them without changing the DT size.

Signed-off-by: Maxime Ripard <maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
Acked-by: Simon Glass <redacted>
Reviewed-by: David Gibson <redacted>

Now it just needs some testcases..
Since fdt_setprop_inplace relies on it now, it already has some test
coverage.

However, I tried to add some testcases for this to the test_tree1, but
it was generating a lot of errors because it seems that not all tests
are run from test_tree1.dts and generated by some other mean I
couldn't identify. Could you point me to the right direction?

Thanks,
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help