[PATCH v2 2/2] pseries: Fix endian issues in cpu hot-removal
Michael Ellerman
mpe at ellerman.id.au
Tue Sep 16 17:00:23 EST 2014
On Mon, 2014-09-15 at 12:16 +0530, Bharata B Rao wrote:
> On Sat, Sep 13, 2014 at 12:41 AM, Thomas Falcon
> <tlfalcon at 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
More information about the Linuxppc-dev
mailing list