Re: [PATCH] of: Fix issue where code would fall through to error case.

2016-11-29 Thread Frank Rowand
On 11/29/16 07:06, Rob Herring wrote: > On Mon, Nov 28, 2016 at 9:30 AM, Frank Rowand wrote: >> On 11/26/16 13:39, Frank Rowand wrote: >>> On 11/23/16 13:58, Rob Herring wrote: On Thu, Nov 17, 2016 at 6:10 PM, Moritz Fischer wrote: > On Thu, Nov 17, 2016 at 4:02 PM, Frank Rowand >

Re: [PATCH] of: Fix issue where code would fall through to error case.

2016-11-29 Thread Moritz Fischer
Hi Rob, On Tue, Nov 29, 2016 at 09:06:08AM -0600, Rob Herring wrote: > On Mon, Nov 28, 2016 at 9:30 AM, Frank Rowand wrote: > > On 11/26/16 13:39, Frank Rowand wrote: > >> On 11/23/16 13:58, Rob Herring wrote: > >>> On Thu, Nov 17, 2016 at 6:10 PM, Moritz Fischer > >>> wrote: > On Thu, Nov

Re: [PATCH] of: Fix issue where code would fall through to error case.

2016-11-29 Thread Rob Herring
On Mon, Nov 28, 2016 at 9:30 AM, Frank Rowand wrote: > On 11/26/16 13:39, Frank Rowand wrote: >> On 11/23/16 13:58, Rob Herring wrote: >>> On Thu, Nov 17, 2016 at 6:10 PM, Moritz Fischer >>> wrote: On Thu, Nov 17, 2016 at 4:02 PM, Frank Rowand wrote: > On 11/17/16 15:40, Frank Row

Re: [PATCH] of: Fix issue where code would fall through to error case.

2016-11-28 Thread Frank Rowand
On 11/26/16 13:39, Frank Rowand wrote: > On 11/23/16 13:58, Rob Herring wrote: >> On Thu, Nov 17, 2016 at 6:10 PM, Moritz Fischer >> wrote: >>> On Thu, Nov 17, 2016 at 4:02 PM, Frank Rowand >>> wrote: On 11/17/16 15:40, Frank Rowand wrote: > On 11/17/16 15:25, Moritz Fischer wrote:

Re: [PATCH] of: Fix issue where code would fall through to error case.

2016-11-26 Thread Frank Rowand
On 11/23/16 13:58, Rob Herring wrote: > On Thu, Nov 17, 2016 at 6:10 PM, Moritz Fischer > wrote: >> On Thu, Nov 17, 2016 at 4:02 PM, Frank Rowand wrote: >>> On 11/17/16 15:40, Frank Rowand wrote: On 11/17/16 15:25, Moritz Fischer wrote: > No longer fall through into the error case that p

Re: [PATCH] of: Fix issue where code would fall through to error case.

2016-11-23 Thread Rob Herring
On Thu, Nov 17, 2016 at 6:10 PM, Moritz Fischer wrote: > On Thu, Nov 17, 2016 at 4:02 PM, Frank Rowand wrote: >> On 11/17/16 15:40, Frank Rowand wrote: >>> On 11/17/16 15:25, Moritz Fischer wrote: No longer fall through into the error case that prints out an error if no error (err = 0)

Re: [PATCH] of: Fix issue where code would fall through to error case.

2016-11-17 Thread Moritz Fischer
On Thu, Nov 17, 2016 at 4:02 PM, Frank Rowand wrote: > On 11/17/16 15:40, Frank Rowand wrote: >> On 11/17/16 15:25, Moritz Fischer wrote: >>> No longer fall through into the error case that prints out >>> an error if no error (err = 0) occurred. >>> >>> Fixes d9181b20a83(of: Add back an error mess

Re: [PATCH] of: Fix issue where code would fall through to error case.

2016-11-17 Thread Frank Rowand
On 11/17/16 15:40, Frank Rowand wrote: > On 11/17/16 15:25, Moritz Fischer wrote: >> No longer fall through into the error case that prints out >> an error if no error (err = 0) occurred. >> >> Fixes d9181b20a83(of: Add back an error message, restructured) >> Signed-off-by: Moritz Fischer >> --- >

Re: [PATCH] of: Fix issue where code would fall through to error case.

2016-11-17 Thread Frank Rowand
On 11/17/16 15:25, Moritz Fischer wrote: > No longer fall through into the error case that prints out > an error if no error (err = 0) occurred. > > Fixes d9181b20a83(of: Add back an error message, restructured) > Signed-off-by: Moritz Fischer > --- > drivers/of/resolver.c | 6 +- > 1 file c