2011-03-29 00:22:35

by Andres Salomon

[permalink] [raw]
Subject: [PATCH resend] ARM: mx51_efika: fix build error due to new mfd changes


MFD changes in 4ec1b54c ('mfd: mfd_cell is now implicitly available
to mc13xxx drivers') changed the mc13xxx_platform_data struct layout.
At the time all users were changed, but this driver was introduced in
another tree at the same time. This updates the mc13xxx_platform_data
user, fixing a build error.

Signed-off-by: Andres Salomon <[email protected]>
---
arch/arm/mach-mx5/mx51_efika.c | 6 ++++--
1 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/arch/arm/mach-mx5/mx51_efika.c b/arch/arm/mach-mx5/mx51_efika.c
index 51a67fc..868af8f 100644
--- a/arch/arm/mach-mx5/mx51_efika.c
+++ b/arch/arm/mach-mx5/mx51_efika.c
@@ -572,8 +572,10 @@ static struct mc13xxx_regulator_init_data mx51_efika_regulators[] = {

static struct mc13xxx_platform_data mx51_efika_mc13892_data = {
.flags = MC13XXX_USE_RTC | MC13XXX_USE_REGULATOR,
- .num_regulators = ARRAY_SIZE(mx51_efika_regulators),
- .regulators = mx51_efika_regulators,
+ .regulators = {
+ .num_regulators = ARRAY_SIZE(mx51_efika_regulators),
+ .regulators = mx51_efika_regulators,
+ },
};

static struct spi_board_info mx51_efika_spi_board_info[] __initdata = {
--
1.7.2.5


2011-03-29 00:48:01

by Fabio Estevam

[permalink] [raw]
Subject: Re: [PATCH resend] ARM: mx51_efika: fix build error due to new mfd changes

Andres,

On Mon, Mar 28, 2011 at 9:22 PM, Andres Salomon <[email protected]> wrote:
>
> MFD changes in 4ec1b54c ('mfd: mfd_cell is now implicitly available
> to mc13xxx drivers') changed the mc13xxx_platform_data struct layout.
> At the time all users were changed, but this driver was introduced in
> another tree at the same time.  This updates the mc13xxx_platform_data
> user, fixing a build error.
>
> Signed-off-by: Andres Salomon <[email protected]>
> ---
>  arch/arm/mach-mx5/mx51_efika.c |    6 ++++--
>  1 files changed, 4 insertions(+), 2 deletions(-)

Your patch is in Sascha´s tree now:
http://git.pengutronix.de/?p=imx/linux-2.6.git;a=commitdiff;h=251290a6078cb88382344b3ee535ae8c6254c1b5

Regards,

Fabio Estevam

2011-03-29 00:52:29

by Andres Salomon

[permalink] [raw]
Subject: Re: [PATCH resend] ARM: mx51_efika: fix build error due to new mfd changes

On Mon, 28 Mar 2011 21:47:58 -0300
Fabio Estevam <[email protected]> wrote:

> Andres,
>
> On Mon, Mar 28, 2011 at 9:22 PM, Andres Salomon <[email protected]>
> wrote:
> >
> > MFD changes in 4ec1b54c ('mfd: mfd_cell is now implicitly available
> > to mc13xxx drivers') changed the mc13xxx_platform_data struct
> > layout. At the time all users were changed, but this driver was
> > introduced in another tree at the same time.  This updates the
> > mc13xxx_platform_data user, fixing a build error.
> >
> > Signed-off-by: Andres Salomon <[email protected]>
> > ---
> >  arch/arm/mach-mx5/mx51_efika.c |    6 ++++--
> >  1 files changed, 4 insertions(+), 2 deletions(-)
>
> Your patch is in Sascha´s tree now:
> http://git.pengutronix.de/?p=imx/linux-2.6.git;a=commitdiff;h=251290a6078cb88382344b3ee535ae8c6254c1b5
>
> Regards,
>
> Fabio Estevam

Ah, thanks, I hadn't heard anything about it. Linus, feel free to
ignore my patch.