Re: [PATCH 10/35] mfd: ab8500-core: Rework MFD sub-deviceinitialisation structures

From: Lee Jones
Date: Wed Feb 20 2013 - 03:10:10 EST


On Tue, 19 Feb 2013, Arnd Bergmann wrote:

> On Friday 15 February 2013, Lee Jones wrote:
> > + {
> > + .name = "ab8500-charger",
> > + .of_compatible = "stericsson,ab8500-charger",
> > + .num_resources = ARRAY_SIZE(ab8500_charger_resources),
> > + .resources = ab8500_charger_resources,
> > +#ifndef CONFIG_OF
> > + .platform_data = &ab8500_bm_data,
> > + .pdata_size = sizeof(ab8500_bm_data),
> > +#endif
> > + },
>
> Is this #ifdef correct? I would have expected "#ifdef CONFIG_ATAGS"
> here.

No, it's not correct. See: 23a04f9f40f2b32ee593b768483105b1c776814d

I will fixup when the branched is rebased post-merge.

--
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org â Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/