linux-toradex.git/drivers/of/unittest.c, branch v4.9.95

of: unittest: use of_platform_default_populate() to populate default bus

2016-06-23T20:00:59+00:00

Use helper of_platform_default_populate() in linux/of_platform
when possible, instead of calling of_platform_populate() with
the default match table.

Cc: Rob Herring 
Cc: Frank Rowand 
Cc: Grant Likely 
Signed-off-by: Kefeng Wang 
Signed-off-by: Rob Herring

Merge tag 'devicetree-for-4.7' of git://git.kernel.org/pub/scm/linux/kernel/git/robh/linux

2016-05-20T21:51:34+00:00

Pull devicetree updates from Rob Herring:

- Rewrite of the unflattening code to avoid recursion and lessen the
stack usage.

- Rewrite of the phandle args parsing code to get rid of the fixed args
size. This is needed for IOMMU code.

- Sync to latest dtc which adds more dts style checking. These
warnings are enabled with "W=1" compiles.

- Tegra documentation updates related to the above warnings.

- A bunch of spelling and other doc fixes.

- Various vendor prefix additions.

* tag 'devicetree-for-4.7' of git://git.kernel.org/pub/scm/linux/kernel/git/robh/linux: (52 commits)
devicetree: Add Creative Technology vendor id
gpio: dt-bindings: add ibm,ppc4xx-gpio binding
of/unittest: Remove unnecessary module.h header inclusion
drivers/of: Fix build warning in populate_node()
drivers/of: Fix depth when unflattening devicetree
of: dynamic: changeset prop-update revert fix
drivers/of: Export of_detach_node()
drivers/of: Return allocated memory from of_fdt_unflatten_tree()
drivers/of: Specify parent node in of_fdt_unflatten_tree()
drivers/of: Rename unflatten_dt_node()
drivers/of: Avoid recursively calling unflatten_dt_node()
drivers/of: Split unflatten_dt_node()
of: include errno.h in of_graph.h
of: document refcount incrementation of of_get_cpu_node()
Documentation: dt: soc: fix spelling mistakes
Documentation: dt: power: fix spelling mistake
Documentation: dt: pinctrl: fix spelling mistake
Documentation: dt: opp: fix spelling mistake
Documentation: dt: net: fix spelling mistakes
Documentation: dt: mtd: fix spelling mistake
...

of/unittest: Remove unnecessary module.h header inclusion

2016-05-19T14:16:36+00:00

The OF_UNITTEST Kconfig symbol is bool so this unittest can only be
built-in and not build as a module. Also, nothing defined in this
header file used so is not necessary to include it.

Signed-off-by: Javier Martinez Canillas 
Signed-off-by: Rob Herring

drivers/of: Specify parent node in of_fdt_unflatten_tree()

2016-05-16T12:22:35+00:00

This adds one more argument to of_fdt_unflatten_tree() to specify
the parent node of the FDT blob that is going to be unflattened.
In the result, the function can be used to unflatten FDT blob that
represents device sub-tree in PowerNV PCI hotplug driver.

Cc: Jyri Sarha 
Signed-off-by: Gavin Shan 
Acked-by: Rob Herring 
Acked-by: Jyri Sarha 
Signed-off-by: Rob Herring

of/unittest: convert to use an explicit i2c mux core

2016-04-22T13:00:18+00:00

Allocate an explicit i2c mux core to handle parent and child adapters
etc. Update the select op to be in terms of the i2c mux core instead
of the child adapter.

Acked-by: Rob Herring 
Signed-off-by: Peter Rosin 
Signed-off-by: Wolfram Sang

of/unittest: fix infinite loop in of_unittest_destroy_tracked_overlays()

2016-03-03T22:51:31+00:00

of_overlay_destroy() can return `-ENODEV' error code once it
failed to find the requested overlay in `ov_idr'. However,
of_unittest_destroy_tracked_overlays() does not handle this
error code correctly and continues to call of_overlay_destroy()
on the 'missing' overlay over and over again. This results in
a printk flood

[..]
[   33.497583] of_overlay_destroy: Could not find overlay #6
[   33.497583] of_overlay_destroy: Could not find overlay #6
[   33.497584] ### dt-test ### of_unittest_destroy_tracked_overlays: overlay destroy failed for #6
[   33.497584] ### dt-test ### of_unittest_destroy_tracked_overlays: overlay destroy failed for #6
[   33.497586] of_overlay_destroy: Could not find overlay #6
[   33.497586] of_overlay_destroy: Could not find overlay #6
[   33.497587] ### dt-test ### of_unittest_destroy_tracked_overlays: overlay destroy failed for #6
[   33.497587] ### dt-test ### of_unittest_destroy_tracked_overlays: overlay destroy failed for #6
[..]

which is not really good due to printk design, and can lead to soft
lockups, hard lockups, etc. (depending on the context console_unlock()
is being called from). The problem has bee observed in real life
and reported by Ying Huang.

This patch does not address the root cause of missing overlay in
`ov_idr', it fixes the endless loop only.

Signed-off-by: Sergey Senozhatsky 
Reported-by: kernel test robot 
Link: lkml.kernel.org/r/87fuwk1c0o.fsf@yhuang-dev.intel.com
Signed-off-by: Rob Herring

drivers/of: Export OF changeset functions

2016-01-13T22:10:37+00:00

The PowerNV PCI hotplug driver is going to use the OF changeset
to manage the changed device sub-tree. This exports those OF
changeset functions for that.

Signed-off-by: Gavin Shan 
Acked-by: Wolfram Sang 
Tested-by: Wolfram Sang 
Signed-off-by: Rob Herring

of/unittest: Show broken behaviour in the platform bus

2016-01-05T15:20:40+00:00

Add a single resource to the test bus device to exercise the platform
bus code a little more. This isn't strictly a devicetree test, but it is
a corner case that the devicetree runs into. Until we've got platform
device unittests, it can live here. It doesn't need to be an explicit
text because the kernel will oops when it is wrong.

Cc: Pantelis Antoniou 
Cc: Rob Herring 
Cc: Greg Kroah-Hartman 
Cc: Ricardo Ribalda Delgado 
Signed-off-by: Grant Likely 
[wsa: added the comment provided by Grant, rebased, and tested]
Signed-off-by: Wolfram Sang 
Signed-off-by: Rob Herring

of/unittest: add missing of_node_put

2015-10-22T14:26:28+00:00

for_each_child_of_node performs an of_node_get on each iteration, so
a break out of the loop requires an of_node_put.

A simplified version of the semantic patch that fixes this problem is as
follows (http://coccinelle.lip6.fr):

// 
@@
expression root,e;
local idexpression child;
@@

 for_each_child_of_node(root, child) {
   ... when != of_node_put(child)
       when != e = child
(
   return child;
|
+  of_node_put(child);
?  return ...;
)
   ...
 }
// 

Combine the puts into code at the end of the function, for conciseness.

Signed-off-by: Julia Lawall 
Signed-off-by: Rob Herring

of: Drop owner assignment from platform and i2c driver

2015-07-27T13:24:39+00:00

platform_driver and i2c_driver do not need to set an owner because core
will set it.

Signed-off-by: Krzysztof Kozlowski 
Signed-off-by: Rob Herring