Hi David,

On Fri, Sep 23, 2016 at 03:31:20PM +1000, David Gibson wrote:

On Thu, Sep 22, 2016 at 09:39:22AM +0300, Maxime Ripard wrote:

quoted

Hi David,

On Mon, Aug 22, 2016 at 09:01:56AM -0400, David Gibson wrote:

quoted

On Mon, Aug 22, 2016 at 12:07:21PM +0200, Maxime Ripard wrote:

quoted

Hi David,

On Mon, Aug 01, 2016 at 01:36:42PM +1000, David Gibson wrote:

quoted

On Fri, Jul 29, 2016 at 11:55:48AM +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.

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

Applied, though I made a couple of tiny tweaks, see below.

Awesome, thanks!

Do you still have comments on the last 3 patches?

Sorry, I got sidetracked by other things.

The only remaining thing I'd like some changes in is the error
reporting.

The biggset problem is that in some places you are returning
BADSTRUCTURE for badly formatter fixup or other overlay properties.
BADSTRUCTURE should only be used when there is a problem in the
encoding of the tree structure itself, not a problem in the encoding
within a specific property or properties.

I don't think your code should ever generate BADSTRUCTURE directly
(but it could return it when it gets that value returned from a lower
level function).

What I was hoping to have a closer look at was to see if I'd prefer
just a single 'BADOVERLAY' error for any problem in te overlay
encoding, or if I'd prefer sveral: BADFIXUP, BADTARGET, or something
like that.  I'll try to get a closer look, but it probably won't
happen this week (I'm at KVM Forum).

Did you have time to think about this?

Finally, yes.  Just sent the detailed reply.

I'll have a look, thanks!

Expect a new version some time next week.

Maxime

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