diff options
author | Tyrel Datwyler <tyreld@linux.vnet.ibm.com> | 2013-08-14 22:23:45 -0700 |
---|---|---|
committer | Benjamin Herrenschmidt <benh@kernel.crashing.org> | 2013-08-27 14:45:10 +1000 |
commit | d8e533b45fd98aa72957d9580d97aa0ab45265ad (patch) | |
tree | 5893062a920f299f6f8ed0d1323a81667bfd53f1 /arch/powerpc/platforms | |
parent | b3f6a45925232d29519007f1ba6687e1e790fb5d (diff) | |
download | op-kernel-dev-d8e533b45fd98aa72957d9580d97aa0ab45265ad.zip op-kernel-dev-d8e533b45fd98aa72957d9580d97aa0ab45265ad.tar.gz |
powerpc/pseries: Fix creation of loop in device node property list
The update_dt_prop helper function fails to set the IN/OUT parameter prop to
NULL after a complete property has been parsed from the work area returned by
the ibm,update-properties rtas function. This results in the property list of
the device node being updated is corrupted and becomes a loop since the same
property structure is used repeatedly.
Signed-off-by: Tyrel Datwyler <tyreld@linux.vnet.ibm.com>
Acked-by: Nathan Fontenot <nfont@linux.vnet.ibm.com>
Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Diffstat (limited to 'arch/powerpc/platforms')
-rw-r--r-- | arch/powerpc/platforms/pseries/mobility.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/powerpc/platforms/pseries/mobility.c b/arch/powerpc/platforms/pseries/mobility.c index 3d01eee..f28abee 100644 --- a/arch/powerpc/platforms/pseries/mobility.c +++ b/arch/powerpc/platforms/pseries/mobility.c @@ -119,7 +119,7 @@ static int update_dt_property(struct device_node *dn, struct property **prop, if (!more) { of_update_property(dn, new_prop); - new_prop = NULL; + *prop = NULL; } return 0; |