linux-toradex.git/drivers/of/resolver.c, branch v4.10

of: Fix issue where code would fall through to error case.

2016-12-09T16:33:03+00:00

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 
Reviewed-by: Frank Rowand 
Signed-off-by: Rob Herring

of: Remove unused variable overlay_symbols

2016-11-15T15:17:18+00:00

Remove unused pointer to node "__symbols__".

Signed-off-by: Frank Rowand 
Signed-off-by: Rob Herring

of: Move setting of pointer to beside test for non-null

2016-11-15T15:17:17+00:00

Value of pointer was calculated in an earlier block than
where it was used.  Move it down into the block where it
is used, immediately before where is is checked to be valid.

Signed-off-by: Frank Rowand 
Signed-off-by: Rob Herring

of: Add back an error message, restructured

2016-11-15T15:17:17+00:00

Add a single pr_err() to cover a range of errors that were reported
by several pr_err() that were removed earlier in this series.

Signed-off-by: Frank Rowand 
Signed-off-by: Rob Herring

of: Update comments to reflect changes and increase clarity

2016-11-15T15:17:16+00:00

Update comments to better explain what functions are doing.

Signed-off-by: Frank Rowand 
Signed-off-by: Rob Herring

of: Remove redundant size check

2016-11-15T15:17:16+00:00

Remove a redundant check of buffer size.

Signed-off-by: Frank Rowand 
Signed-off-by: Rob Herring

of: Update structure of code to be clearer, also remove BUG_ON()

2016-11-15T15:17:15+00:00

Remove BUG_ON(), which is frowned upon and not needed here.
Restructure to remove some excessive complexity.

Signed-off-by: Frank Rowand 
Signed-off-by: Rob Herring

of: Rename variables to better reflect purpose or follow convention

2016-11-15T15:17:15+00:00

Rename variables to better reflect what their purpose is.  As a side
effect, this reduces the need for some of the comments previously
removed in this series.

Signed-off-by: Frank Rowand 
Signed-off-by: Rob Herring

of: Remove prefix "__of_" from local function names

2016-11-15T15:17:14+00:00

Remove "__of_" prefix from local function names.  The pattern of
a leading "__" is used in drivers/of/ to signify a function that
must be called with a lock held.  These functions do not fit
that pattern.

Signed-off-by: Frank Rowand 
Signed-off-by: Rob Herring

of: Rename functions to more accurately reflect what they do

2016-11-15T15:17:14+00:00

Some function names are misleading or do not provide a good
sense of what they do.  Rename the functions to ne more
informative.

Signed-off-by: Frank Rowand 
Signed-off-by: Rob Herring