The function unflatten_dt_node() is called recursively to unflatten
device nodes and properties in the FDT blob. It looks complicated
and hard to be understood.

This splits the function into 3 functions: populate_properties(),
populate_node() and unflatten_dt_node(). populate_properties(),
which is called by populate_node(), creates properties for the
indicated device node. The later one creates the device nodes
from FDT blob. populate_node() gets the offset in FDT blob for
next device nodes and then calls populate_node(). No logical
changes introduced.
Signed-off-by: NGavin Shan <gwshan@linux.vnet.ibm.com>
Acked-by: NRob Herring <robh@kernel.org>
Signed-off-by: NRob Herring <robh@kernel.org>

When CONFIG_OF is disabled, we have to include linux/errno.h before
including of_graph.h, or get build errors like in the newly added
sun4i drm driver:

In file included from ../drivers/gpu/drm/sun4i/sun4i_drv.c:14:0:
include/linux/of_graph.h: In function 'of_graph_parse_endpoint':
include/linux/of_graph.h:58:10: error: 'ENOSYS' undeclared (first use in this function)

A better solution is to ensure that the header can be included
by itself, so let's include linux/errno.h here to fix the error
we just got, and any similar future error.
Signed-off-by: NArnd Bergmann <arnd@arndb.de>
Fixes: 9026e0d1 ("drm: Add Allwinner A10 Display Engine support")
Signed-off-by: NRob Herring <robh@kernel.org>

This function increments refcount.  This is worth noting.
Signed-off-by: NMasahiro Yamada <yamada.masahiro@socionext.com>
Reviewed-by: NFrank Rowand <frank.rowand@am.sony.com>
Signed-off-by: NRob Herring <robh@kernel.org>

Signed-off-by: NEric Engestrom <eric@engestrom.ch>
Signed-off-by: NRob Herring <robh@kernel.org>

Signed-off-by: NEric Engestrom <eric@engestrom.ch>
Signed-off-by: NRob Herring <robh@kernel.org>

Signed-off-by: NEric Engestrom <eric@engestrom.ch>
Signed-off-by: NRob Herring <robh@kernel.org>

Signed-off-by: NEric Engestrom <eric@engestrom.ch>
Acked-by: NViresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: NRob Herring <robh@kernel.org>

Signed-off-by: NEric Engestrom <eric@engestrom.ch>
Signed-off-by: NRob Herring <robh@kernel.org>

Signed-off-by: NEric Engestrom <eric@engestrom.ch>
Signed-off-by: NRob Herring <robh@kernel.org>

Signed-off-by: NEric Engestrom <eric@engestrom.ch>
Signed-off-by: NRob Herring <robh@kernel.org>

Signed-off-by: NEric Engestrom <eric@engestrom.ch>
Signed-off-by: NRob Herring <robh@kernel.org>

Signed-off-by: NEric Engestrom <eric@engestrom.ch>
Signed-off-by: NRob Herring <robh@kernel.org>

Signed-off-by: NEric Engestrom <eric@engestrom.ch>
Signed-off-by: NRob Herring <robh@kernel.org>

Signed-off-by: NEric Engestrom <eric@engestrom.ch>
Signed-off-by: NRob Herring <robh@kernel.org>

Signed-off-by: NEric Engestrom <eric@engestrom.ch>
Signed-off-by: NRob Herring <robh@kernel.org>

Signed-off-by: NEric Engestrom <eric@engestrom.ch>
Signed-off-by: NRob Herring <robh@kernel.org>

Signed-off-by: NEric Engestrom <eric@engestrom.ch>
Reviewed-by: NHeiko Stuebner <heiko@sntech.de>
[robh: s/describe/described/]
Signed-off-by: NRob Herring <robh@kernel.org>

Signed-off-by: NEric Engestrom <eric@engestrom.ch>
Signed-off-by: NRob Herring <robh@kernel.org>

All other UART DT binding documentation is under
Documentation/devicetree/bindings/serial/.
Signed-off-by: NGeert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: NRob Herring <robh@kernel.org>

change 100mA -> 100uA
Signed-off-by: NH. Nikolaus Schaller <hns@goldelico.com>
Signed-off-by: NRob Herring <robh@kernel.org>

Signed-off-by: NGeert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: NRob Herring <robh@kernel.org>

Signed-off-by: NGeert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: NRob Herring <robh@kernel.org>

Signed-off-by: NGeert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: NRob Herring <robh@kernel.org>

Remove the 0x in the unit address because it shouldn't be there.

Cc: Joao Pinto <jpinto@synopsys.com>
Signed-off-by: NStephen Boyd <stephen.boyd@linaro.org>
Signed-off-by: NRob Herring <robh@kernel.org>

Device tree binding for NVIDIA Tegra have traditionally carried the
"nvidia," vendor prefix in the filename. A couple of odd ones don't, so
fix them up for consistency.

Also rename existing bindings to reflect the first compatible value that
they document. This wasn't done consistently either.
Acked-by: NStephen Warren <swarren@nvidia.com>
Signed-off-by: NThierry Reding <treding@nvidia.com>
Signed-off-by: NRob Herring <robh@kernel.org>

When Tegra124 support was first merged the unit-addresses of all devices
were listed with a "0," prefix to encode the reg property's second cell.
It turns out that this notation is not correct, and the "," separator is
only used to separate fields in the unit address (such as the device and
function number in PCI devices), not individual cells for addresses
with more than one cell.
Acked-by: NStephen Warren <swarren@nvidia.com>
Signed-off-by: NThierry Reding <treding@nvidia.com>
Signed-off-by: NRob Herring <robh@kernel.org>

This patch adds Inforce Computing to vendor prefix list.
This vendor makes boards like IFC6410, IFC6540 based on Qualcomm SOCs.
Signed-off-by: NSrinivas Kandagatla <srinivas.kandagatla@linaro.org>
Signed-off-by: NRob Herring <robh@kernel.org>

This patch adds Arrow Electronics to vendor perfix list, as this vendor
makes some of the Qualcomm SOC based 96boards like DB600c and DB410c.
Signed-off-by: NSrinivas Kandagatla <srinivas.kandagatla@linaro.org>
Signed-off-by: NRob Herring <robh@kernel.org>

Signed-off-by: NSergio Prado <sergio.prado@e-labworks.com>
Signed-off-by: NRob Herring <robh@kernel.org>

Adding support for the tps659038 pmic so it doesn't generate a warning
when running the patch check script to
Documentation/devicetree/bindings/regulator/palmas-pmic.txt

Adding a note that the tps659037 device is a OTP spin of the
tps659038 pmic and device compatible.
Signed-off-by: NSchuyler Patton <spatton@ti.com>
Signed-off-by: NKeerthy <j-keerthy@ti.com>
Acked-by: NRob Herring <robh@kernel.org>
Signed-off-by: NRob Herring <robh@kernel.org>

Commit eb3fcf00 ("dt-bindings: consolidate interrupt controller
bindings") moved the binding documentation for the ARM GIC from
arm/gic.txt to interrupt-controller/arm,gic.txt. However, there are
still some binding documents referring to the old path. Update these
binding documents to use the correct location.

Fixes: eb3fcf00 ("dt-bindings: consolidate interrupt controller bindings")
Signed-off-by: NJon Hunter <jonathanh@nvidia.com>
Acked-by: NLinus Walleij <linus.walleij@linaro.org>
Acked-by: NMatthias Brugger <matthias.bgg@gmail.com>
Signed-off-by: NRob Herring <robh@kernel.org>

Drop #address-cells and #size-cells, which are not required by the
gpio-keys binding documentation, as button sub-nodes are not devices.

Rename sub-nodes to avoid new dtc unit address warnings when copied.

While at it, adopt the dashes convention for the node name.
Reported-by: NJulien Chauveau <chauveau.julien@gmail.com>
Cc: Julien Chauveau <chauveau.julien@gmail.com>
Cc: Javier Martinez Canillas <javier@dowhile0.org>
Cc: Geert Uytterhoeven <geert@linux-m68k.org>
Signed-off-by: NAndreas Färber <afaerber@suse.de>
Reviewed-by: NJavier Martinez Canillas <javier@osg.samsung.com>
Reviewed-by: NJulien Chauveau <chauveau.julien@gmail.com>
Signed-off-by: NRob Herring <robh@kernel.org>

Remove the usage of of_parse_phandle_with_args() and replace
it by the phandle-iterator implementation so that we can
parse out all of the potentially present 128 stream-ids.
Signed-off-by: NJoerg Roedel <jroedel@suse.de>
Acked-by: NWill Deacon <will.deacon@arm.com>
Signed-off-by: NRob Herring <robh@kernel.org>

This helper function can be used to copy the arguments of a
phandle to an array.
Signed-off-by: NJoerg Roedel <jroedel@suse.de>
Signed-off-by: NRob Herring <robh@kernel.org>

With this macro any user can easily iterate over a list of
phandles. The patch also converts __of_parse_phandle_with_args()
to make use of the macro.

The of_count_phandle_with_args() function is not converted,
because the macro hides the return value of of_phandle_iterator_init(),
which is needed in there.
Signed-off-by: NJoerg Roedel <jroedel@suse.de>
Signed-off-by: NRob Herring <robh@kernel.org>

The index = -1 case in __of_parse_phandle_with_args() is
used to just return the number of phandles. That special
case needs extra handling, so move it to the place where it
is needed: of_count_phandle_with_args().

This allows to further simplify __of_parse_phandle_with_args()
later on.
Signed-off-by: NJoerg Roedel <jroedel@suse.de>
Signed-off-by: NRob Herring <robh@kernel.org>

Move the code to walk over the phandles out of the loop in
__of_parse_phandle_with_args() to a separate function that
just works with the iterator handle: of_phandle_iterator_next().
Signed-off-by: NJoerg Roedel <jroedel@suse.de>
Signed-off-by: NRob Herring <robh@kernel.org>

This struct carrys all necessary information to iterate over
a list of phandles and extract the arguments. Add an
init-function for the iterator and make use of it in
__of_parse_phandle_with_args().
Signed-off-by: NJoerg Roedel <jroedel@suse.de>
Signed-off-by: NRob Herring <robh@kernel.org>

On arm CONFIG_CMDLINE_EXTEND does not append build-in
cmdline in kernel to U-boot parameters. Fix it here.
Theoretically this patch should repair kdump work where
it adds elfcorehdr= and memmap additional parameters
to second kernel.
Signed-off-by: NMax Uvarov <muvarov@gmail.com>
Signed-off-by: NRob Herring <robh@kernel.org>

The newly added dtc warning to check DT unit-address without reg
property and vice-versa generates lots of warnings. Turn off the check
unless building with W=1 or W=2.
Signed-off-by: NRob Herring <robh@kernel.org>
Cc: Michal Marek <mmarek@suse.com>
Cc: linux-kbuild@vger.kernel.org