On Mon, 2014-09-15 at 12:16 +0530, Bharata B Rao wrote: > On Sat, Sep 13, 2014 at 12:41 AM, Thomas Falcon > <tlfal...@linux.vnet.ibm.com> wrote: > > > > diff --git a/arch/powerpc/platforms/pseries/dlpar.c > > b/arch/powerpc/platforms/pseries/dlpar.c > > index 9e9f30b2..343dfdf 100644 > > --- a/arch/powerpc/platforms/pseries/dlpar.c > > +++ b/arch/powerpc/platforms/pseries/dlpar.c > > @@ -496,15 +498,15 @@ out: > > static ssize_t dlpar_cpu_release(const char *buf, size_t count) > > { > > struct device_node *dn; > > - const u32 *drc_index; > > + const u32 drc_index; > > int rc; > > > > dn = of_find_node_by_path(buf); > > if (!dn) > > return -EINVAL; > > > > - drc_index = of_get_property(dn, "ibm,my-drc-index", NULL); > > - if (!drc_index) { > > + rc = of_property_read_u32(dn, "ibm,my-drc-index", &drc_index); > > Use of const for drc_index causes compilation problems.
Yes that's clearly wrong. Please fix and retest. cheers _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev