Re: [PATCH v2 2/2] of: Add unit tests for applying overlays

2017-04-25 Thread Frank Rowand
On 04/25/17 09:44, Rob Herring wrote: > On Mon, Apr 24, 2017 at 6:05 PM, wrote: >> From: Frank Rowand >> >> Existing overlay unit tests examine individual pieces of the overlay >> code. The new tests target the entire process of applying an

Re: [PATCH v2 2/2] of: Add unit tests for applying overlays

2017-04-25 Thread Frank Rowand
On 04/25/17 09:44, Rob Herring wrote: > On Mon, Apr 24, 2017 at 6:05 PM, wrote: >> From: Frank Rowand >> >> Existing overlay unit tests examine individual pieces of the overlay >> code. The new tests target the entire process of applying an overlay. > > Just a few nits. > >> Signed-off-by:

Re: [PATCH v2 2/2] of: Add unit tests for applying overlays

2017-04-25 Thread Rob Herring
On Mon, Apr 24, 2017 at 6:05 PM, wrote: > From: Frank Rowand > > Existing overlay unit tests examine individual pieces of the overlay > code. The new tests target the entire process of applying an overlay. Just a few nits. > Signed-off-by: Frank

Re: [PATCH v2 2/2] of: Add unit tests for applying overlays

2017-04-25 Thread Rob Herring
On Mon, Apr 24, 2017 at 6:05 PM, wrote: > From: Frank Rowand > > Existing overlay unit tests examine individual pieces of the overlay > code. The new tests target the entire process of applying an overlay. Just a few nits. > Signed-off-by: Frank Rowand [...] > diff --git

[PATCH v2 2/2] of: Add unit tests for applying overlays

2017-04-24 Thread frowand . list
From: Frank Rowand Existing overlay unit tests examine individual pieces of the overlay code. The new tests target the entire process of applying an overlay. Signed-off-by: Frank Rowand --- There are checkpatch warnings. I have reviewed them and

[PATCH v2 2/2] of: Add unit tests for applying overlays

2017-04-24 Thread frowand . list
From: Frank Rowand Existing overlay unit tests examine individual pieces of the overlay code. The new tests target the entire process of applying an overlay. Signed-off-by: Frank Rowand --- There are checkpatch warnings. I have reviewed them and feel they can be ignored. drivers/of/fdt.c