| Message ID | 1326733853-13956-2-git-send-email-paul.liu@linaro.org |
|---|---|
| State | Changes Requested |
| Headers | show |
On Tue, Jan 17, 2012 at 01:10:53AM +0800, Ying-Chun Liu (PaulLiu) wrote: > +#define DA9052_LDO1_VOLT_UPPER 1800 > +#define DA9052_LDO1_VOLT_LOWER 600 > +#define DA9052_LDO1_VOLT_STEP 50 This is almost certainly wrong - you should rarely if ever need the voltage step in a consumer or machine driver. > +#define DA9052_LDO(max, min, rname, suspend_mv) \ > +{\ > + .constraints = {\ > + .name = (rname), \ > + .max_uV = (max) * 1000,\ > + .min_uV = (min) * 1000,\ > + .valid_ops_mask = REGULATOR_CHANGE_VOLTAGE\ > + |REGULATOR_CHANGE_STATUS | REGULATOR_CHANGE_MODE,\ > + .valid_modes_mask = REGULATOR_MODE_NORMAL,\ This looks *very* odd - apart from anything else you're setting REGULATOR_CHANGE_MODE but only permitting one mode. You're also allowing things to be changed but not providing any consumers which means nothing can ever change any of them. It looks awfully like you've just copied the supported feature set of the PMIC rather than configured things for your board. > +static struct regulator_init_data da9052_regulators_init[] = { > + /* BUCKS */ > + DA9052_LDO(DA9052_BUCK_CORE_PRO_VOLT_UPPER, > + DA9052_BUCK_CORE_PRO_VOLT_LOWER, "DA9052_BUCK_CORE", 850), If you're going to specify names they should be names which are meaningful for your board - usually the name used to identify the relevant rail or rails in the schematic. > +#define MX53_LOCO_DA9052_IRQ (6*32 + 11) /* GPIO7_11 */ > + > +static int __init loco_da9052_init(struct da9052 *da9052) > +{ > + /* Configuring for DA9052 interrupt servce */ > + /* s3c_gpio_setpull(DA9052_IRQ_PIN, S3C_GPIO_PULL_UP); */ > + I apprecate that my code can be inspirational at times but you probably don't want to cut'n'paste this bit. :) > + /* Set interrupt as LOW LEVEL interrupt source */ > + irq_set_irq_type(gpio_to_irq(MX53_LOCO_DA9052_IRQ), > + IRQF_TRIGGER_LOW); Why is this needed? The PMIC driver should do the right thing when it requests the interrupt... > @@ -273,6 +397,10 @@ static struct i2c_board_info mx53loco_i2c_devices[] = { > { > I2C_BOARD_INFO("mma8450", 0x1C), > }, > + { > + I2C_BOARD_INFO("da9053-aa", 0x90 >> 1), > + .platform_data = &da9052_plat, > + }, You're mixing different ways of numbering I2C devices here.
"Ying-Chun Liu (PaulLiu)" <paul.liu@linaro.org> writes: Hi, > From: "Ying-Chun Liu (PaulLiu)" <paul.liu@linaro.org> > > Add DA9052 PMIC support for Freescale QuickStart Loco board. > The model of PMIC on QuickStart Loco board is "da9053-aa". > > Signed-off-by: Ying-Chun Liu (PaulLiu) <paul.liu@linaro.org> > Cc: Amit Kucheria <amit.kucheria@canonical.com> > Cc: Sascha Hauer <kernel@pengutronix.de> > --- > arch/arm/mach-mx5/board-mx53_loco.c | 128 +++++++++++++++++++++++++++++++++ > arch/arm/plat-mxc/include/mach/irqs.h | 10 +++- > 2 files changed, 137 insertions(+), 1 deletions(-) > > diff --git a/arch/arm/mach-mx5/board-mx53_loco.c b/arch/arm/mach-mx5/board-mx53_loco.c > index fd8b524..61dd8c9 100644 > --- a/arch/arm/mach-mx5/board-mx53_loco.c > +++ b/arch/arm/mach-mx5/board-mx53_loco.c > @@ -23,10 +23,21 @@ > #include <linux/delay.h> > #include <linux/gpio.h> > #include <linux/i2c.h> > +#include <linux/module.h> > +#include <linux/platform_device.h> > +#include <linux/irq.h> > +#include <linux/interrupt.h> > +#include <linux/err.h> > +#include <linux/regulator/machine.h> > +#include <linux/regulator/fixed.h> > +#include <linux/mfd/da9052/da9052.h> > +#include <linux/mfd/da9052/pdata.h> > > #include <mach/common.h> > #include <mach/hardware.h> > #include <mach/iomux-mx53.h> > +#include <mach/irqs.h> > +#include <mach/gpio.h> > > #include <asm/mach-types.h> > #include <asm/mach/arch.h> > @@ -45,6 +56,32 @@ > #define LOCO_SD1_CD IMX_GPIO_NR(3, 13) > #define LOCO_ACCEL_EN IMX_GPIO_NR(6, 14) > > +#define DA9052_LDO1_VOLT_UPPER 1800 > +#define DA9052_LDO1_VOLT_LOWER 600 > +#define DA9052_LDO1_VOLT_STEP 50 > +#define DA9052_LDO2_VOLT_UPPER 1800 > +#define DA9052_LDO2_VOLT_LOWER 600 > +#define DA9052_LDO2_VOLT_STEP 25 > +#define DA9052_LDO34_VOLT_UPPER 3300 > +#define DA9052_LDO34_VOLT_LOWER 1725 > +#define DA9052_LDO34_VOLT_STEP 25 > +#define DA9052_LDO567810_VOLT_UPPER 3600 > +#define DA9052_LDO567810_VOLT_LOWER 1200 > +#define DA9052_LDO567810_VOLT_STEP 50 > +#define DA9052_LDO9_VOLT_STEP 50 > +#define DA9052_LDO9_VOLT_LOWER 1250 > +#define DA9052_LDO9_VOLT_UPPER 3650 > +/* Buck Config Validation Macros */ > +#define DA9052_BUCK_CORE_PRO_VOLT_UPPER 2075 > +#define DA9052_BUCK_CORE_PRO_VOLT_LOWER 500 > +#define DA9052_BUCK_CORE_PRO_STEP 25 > +#define DA9052_BUCK_MEM_VOLT_UPPER 2500 > +#define DA9052_BUCK_MEM_VOLT_LOWER 925 > +#define DA9052_BUCK_MEM_STEP 25 > +#define DA9052_BUCK_PERI_VOLT_UPPER 2500 > +#define DA9052_BUCK_PERI_VOLT_LOWER 925 > +#define DA9052_BUCK_PERI_STEP 25 > + The _STEP #defines looks unused. What about removing them ? > static iomux_v3_cfg_t mx53_loco_pads[] = { > /* FEC */ > MX53_PAD_FEC_MDC__FEC_MDC, > @@ -227,6 +264,93 @@ static const struct esdhc_platform_data mx53_loco_sd3_data __initconst = { > .wp_type = ESDHC_WP_GPIO, > }; > > +#define DA9052_LDO(max, min, rname, suspend_mv) \ > +{\ > + .constraints = {\ > + .name = (rname), \ > + .max_uV = (max) * 1000,\ > + .min_uV = (min) * 1000,\ > + .valid_ops_mask = REGULATOR_CHANGE_VOLTAGE\ > + |REGULATOR_CHANGE_STATUS | REGULATOR_CHANGE_MODE,\ > + .valid_modes_mask = REGULATOR_MODE_NORMAL,\ > + .state_mem = { \ > + .uV = suspend_mv * 1000, \ > + .mode = REGULATOR_MODE_NORMAL, \ > + .enabled = (0 == suspend_mv) ? 0 : 1, \ > + .disabled = 0, \ > + }, \ > + },\ > +} > + > +/* currently the suspend_mv here takes no effects for DA9053 > +preset-voltage have to be done in the latest stage during > +suspend*/ > +static struct regulator_init_data da9052_regulators_init[] = { > + /* BUCKS */ > + DA9052_LDO(DA9052_BUCK_CORE_PRO_VOLT_UPPER, > + DA9052_BUCK_CORE_PRO_VOLT_LOWER, "DA9052_BUCK_CORE", > 850), You're using some #define for min/max. Why not for suspend_mv ? Also, if #defines are similar enough, I guess you can go further with something like (untested) : #define DA9052_LDO(prefix, rname) \ {\ .constraints = {\ .name = (rname), \ .max_uV = (prefix ## _VOLT_UPPER) * 1000,\ .min_uV = (prefix ## _VOLT_LOWER) * 1000,\ .valid_ops_mask = REGULATOR_CHANGE_VOLTAGE\ |REGULATOR_CHANGE_STATUS | REGULATOR_CHANGE_MODE,\ .valid_modes_mask = REGULATOR_MODE_NORMAL,\ .state_mem = { \ .uV = (prefix ## _VOLT_SUSP) * 1000, \ .mode = REGULATOR_MODE_NORMAL, \ .enabled = (0 == (prefix ## _VOLT_SUSP)) ? 0 : 1, \ .disabled = 0, \ }, \ },\ } and then: DA9052(DA9052_BUCK_CORE_PRO, "DA9052_BUCK_CORE"), > + DA9052_LDO(DA9052_BUCK_CORE_PRO_VOLT_UPPER, > + DA9052_BUCK_CORE_PRO_VOLT_LOWER, "DA9052_BUCK_PRO", 950), > + DA9052_LDO(DA9052_BUCK_MEM_VOLT_UPPER, > + DA9052_BUCK_MEM_VOLT_LOWER, "DA9052_BUCK_MEM", 1500), > + DA9052_LDO(DA9052_BUCK_PERI_VOLT_UPPER, > + DA9052_BUCK_PERI_VOLT_LOWER, "DA9052_BUCK_PERI", 2500), > + DA9052_LDO(DA9052_LDO1_VOLT_UPPER, > + DA9052_LDO1_VOLT_LOWER, "DA9052_LDO1", 1300), > + DA9052_LDO(DA9052_LDO2_VOLT_UPPER, > + DA9052_LDO2_VOLT_LOWER, "DA9052_LDO2", 1300), > + DA9052_LDO(DA9052_LDO34_VOLT_UPPER, > + DA9052_LDO34_VOLT_LOWER, "DA9052_LDO3", 3300), > + DA9052_LDO(DA9052_LDO34_VOLT_UPPER, > + DA9052_LDO34_VOLT_LOWER, "DA9052_LDO4", 2775), > + DA9052_LDO(DA9052_LDO567810_VOLT_UPPER, > + DA9052_LDO567810_VOLT_LOWER, "DA9052_LDO5", 1300), > + DA9052_LDO(DA9052_LDO567810_VOLT_UPPER, > + DA9052_LDO567810_VOLT_LOWER, "DA9052_LDO6", 1200), > + DA9052_LDO(DA9052_LDO567810_VOLT_UPPER, > + DA9052_LDO567810_VOLT_LOWER, "DA9052_LDO7", 2750), > + DA9052_LDO(DA9052_LDO567810_VOLT_UPPER, > + DA9052_LDO567810_VOLT_LOWER, "DA9052_LDO8", 1800), > + DA9052_LDO(DA9052_LDO9_VOLT_UPPER, > + DA9052_LDO9_VOLT_LOWER, "DA9052_LDO9", 2500), > + DA9052_LDO(DA9052_LDO567810_VOLT_UPPER, > + DA9052_LDO567810_VOLT_LOWER, "DA9052_LDO10", 1200), > +}; > + > +#define MX53_LOCO_DA9052_IRQ (6*32 + 11) /* GPIO7_11 */ you're aware that there's a IMX_GPIO_NR() macro for defining gpio, right ? Moreover, why not putting it with other #defines for gpio in the top of the file ? > + > +static int __init loco_da9052_init(struct da9052 *da9052) > +{ > + /* Configuring for DA9052 interrupt servce */ > + /* s3c_gpio_setpull(DA9052_IRQ_PIN, S3C_GPIO_PULL_UP); */ > + > + /* Set interrupt as LOW LEVEL interrupt source */ > + irq_set_irq_type(gpio_to_irq(MX53_LOCO_DA9052_IRQ), > + IRQF_TRIGGER_LOW); > + return 0; > +} > + > +static struct da9052_pdata __initdata da9052_plat = { > + .init = loco_da9052_init, > + .irq_base = MXC_PMIC_IRQ_START, > + .regulators = { > + &da9052_regulators_init[0], > + &da9052_regulators_init[1], > + &da9052_regulators_init[2], > + &da9052_regulators_init[3], > + &da9052_regulators_init[4], > + &da9052_regulators_init[5], > + &da9052_regulators_init[6], > + &da9052_regulators_init[7], > + &da9052_regulators_init[8], > + &da9052_regulators_init[9], > + &da9052_regulators_init[10], > + &da9052_regulators_init[11], > + &da9052_regulators_init[12], > + &da9052_regulators_init[13], > + }, > +}; > + > static inline void mx53_loco_fec_reset(void) > { > int ret; > @@ -273,6 +397,10 @@ static struct i2c_board_info mx53loco_i2c_devices[] = { > { > I2C_BOARD_INFO("mma8450", 0x1C), > }, > + { > + I2C_BOARD_INFO("da9053-aa", 0x90 >> 1), > + .platform_data = &da9052_plat, > + }, > }; > > static void __init mx53_loco_board_init(void) > diff --git a/arch/arm/plat-mxc/include/mach/irqs.h b/arch/arm/plat-mxc/include/mach/irqs.h > index fd9efb0..9fb56eb 100644 > --- a/arch/arm/plat-mxc/include/mach/irqs.h > +++ b/arch/arm/plat-mxc/include/mach/irqs.h > @@ -53,7 +53,15 @@ > #endif > /* REVISIT: Add IPU irqs on IMX51 */ > > -#define NR_IRQS (MXC_IPU_IRQ_START + MX3_IPU_IRQS) > +#define MXC_PMIC_IRQ_START (MXC_IPU_IRQ_START + MX3_IPU_IRQS) > + > +#ifdef CONFIG_MACH_MX53_LOCO > +#define MXC_PMIC_IRQS 32 > +#else > +#define MXC_PMIC_IRQS 0 > +#endif So, each board using a pmic needing some irqs will need to add a #ifdef/#define combo ? Can it be made more generic ? How will it work with a kernel compiled for several machines, say loco and an other using a pmic using more than 32 irqs ? > + > +#define NR_IRQS (MXC_PMIC_IRQ_START + MXC_PMIC_IRQS) > > extern int imx_irq_set_priority(unsigned char irq, unsigned char prio); Regards, Arnaud
On Tue, Jan 17, 2012 at 01:10:53AM +0800, Ying-Chun Liu (PaulLiu) wrote: > diff --git a/arch/arm/mach-mx5/board-mx53_loco.c b/arch/arm/mach-mx5/board-mx53_loco.c > index fd8b524..61dd8c9 100644 > --- a/arch/arm/mach-mx5/board-mx53_loco.c > +++ b/arch/arm/mach-mx5/board-mx53_loco.c > @@ -23,10 +23,21 @@ > #include <linux/delay.h> > #include <linux/gpio.h> You have this ^^^^^^^ include, which includes asm/gpio.h, and in turn mach/gpio.h, but... > #include <linux/i2c.h> > +#include <linux/module.h> > +#include <linux/platform_device.h> > +#include <linux/irq.h> > +#include <linux/interrupt.h> > +#include <linux/err.h> > +#include <linux/regulator/machine.h> > +#include <linux/regulator/fixed.h> > +#include <linux/mfd/da9052/da9052.h> > +#include <linux/mfd/da9052/pdata.h> > > #include <mach/common.h> > #include <mach/hardware.h> > #include <mach/iomux-mx53.h> > +#include <mach/irqs.h> > +#include <mach/gpio.h> you include mach/gpio.h here? I don't see why you need mach/irqs.h either. That brings into this another question: you're adding 9 new linux/ includes above. How many of those do you _actually_ need? Eg, I don't see you adding anything from linux/err.h to this file, so why do you need this header? Please don't add include files unnecessarily.
Thanks all. I'm preparing another new upload to fix all the problems. (2012年01月17日 05:08), Arnaud Patard (Rtp) wrote: > "Ying-Chun Liu (PaulLiu)" <paul.liu@linaro.org> writes: > > Hi, > >> From: "Ying-Chun Liu (PaulLiu)" <paul.liu@linaro.org> >> + >> +#define MX53_LOCO_DA9052_IRQ (6*32 + 11) /* GPIO7_11 */ > > you're aware that there's a IMX_GPIO_NR() macro for defining gpio, > right ? Moreover, why not putting it with other #defines for gpio in the > top of the file ? Thanks. My fault. I'll change to use this macro. > >> diff --git a/arch/arm/plat-mxc/include/mach/irqs.h b/arch/arm/plat-mxc/include/mach/irqs.h >> index fd9efb0..9fb56eb 100644 >> --- a/arch/arm/plat-mxc/include/mach/irqs.h >> +++ b/arch/arm/plat-mxc/include/mach/irqs.h >> @@ -53,7 +53,15 @@ >> #endif >> /* REVISIT: Add IPU irqs on IMX51 */ >> >> -#define NR_IRQS (MXC_IPU_IRQ_START + MX3_IPU_IRQS) >> +#define MXC_PMIC_IRQ_START (MXC_IPU_IRQ_START + MX3_IPU_IRQS) >> + >> +#ifdef CONFIG_MACH_MX53_LOCO >> +#define MXC_PMIC_IRQS 32 >> +#else >> +#define MXC_PMIC_IRQS 0 >> +#endif > > So, each board using a pmic needing some irqs will need to add a > #ifdef/#define combo ? Can it be made more generic ? How will it work > with a kernel compiled for several machines, say loco and an other using > a pmic using more than 32 irqs ? > Sorry. Any possible suggestions? I'm thinking about using CONFIG_SPARSE_IRQ and then we can use .nr_irqs in MACHINE_START block so I can acquire more irqs there. But I don't know if this is another intrusive way. Many Thanks, Paul
diff --git a/arch/arm/mach-mx5/board-mx53_loco.c b/arch/arm/mach-mx5/board-mx53_loco.c index fd8b524..61dd8c9 100644 --- a/arch/arm/mach-mx5/board-mx53_loco.c +++ b/arch/arm/mach-mx5/board-mx53_loco.c @@ -23,10 +23,21 @@ #include <linux/delay.h> #include <linux/gpio.h> #include <linux/i2c.h> +#include <linux/module.h> +#include <linux/platform_device.h> +#include <linux/irq.h> +#include <linux/interrupt.h> +#include <linux/err.h> +#include <linux/regulator/machine.h> +#include <linux/regulator/fixed.h> +#include <linux/mfd/da9052/da9052.h> +#include <linux/mfd/da9052/pdata.h> #include <mach/common.h> #include <mach/hardware.h> #include <mach/iomux-mx53.h> +#include <mach/irqs.h> +#include <mach/gpio.h> #include <asm/mach-types.h> #include <asm/mach/arch.h> @@ -45,6 +56,32 @@ #define LOCO_SD1_CD IMX_GPIO_NR(3, 13) #define LOCO_ACCEL_EN IMX_GPIO_NR(6, 14) +#define DA9052_LDO1_VOLT_UPPER 1800 +#define DA9052_LDO1_VOLT_LOWER 600 +#define DA9052_LDO1_VOLT_STEP 50 +#define DA9052_LDO2_VOLT_UPPER 1800 +#define DA9052_LDO2_VOLT_LOWER 600 +#define DA9052_LDO2_VOLT_STEP 25 +#define DA9052_LDO34_VOLT_UPPER 3300 +#define DA9052_LDO34_VOLT_LOWER 1725 +#define DA9052_LDO34_VOLT_STEP 25 +#define DA9052_LDO567810_VOLT_UPPER 3600 +#define DA9052_LDO567810_VOLT_LOWER 1200 +#define DA9052_LDO567810_VOLT_STEP 50 +#define DA9052_LDO9_VOLT_STEP 50 +#define DA9052_LDO9_VOLT_LOWER 1250 +#define DA9052_LDO9_VOLT_UPPER 3650 +/* Buck Config Validation Macros */ +#define DA9052_BUCK_CORE_PRO_VOLT_UPPER 2075 +#define DA9052_BUCK_CORE_PRO_VOLT_LOWER 500 +#define DA9052_BUCK_CORE_PRO_STEP 25 +#define DA9052_BUCK_MEM_VOLT_UPPER 2500 +#define DA9052_BUCK_MEM_VOLT_LOWER 925 +#define DA9052_BUCK_MEM_STEP 25 +#define DA9052_BUCK_PERI_VOLT_UPPER 2500 +#define DA9052_BUCK_PERI_VOLT_LOWER 925 +#define DA9052_BUCK_PERI_STEP 25 + static iomux_v3_cfg_t mx53_loco_pads[] = { /* FEC */ MX53_PAD_FEC_MDC__FEC_MDC, @@ -227,6 +264,93 @@ static const struct esdhc_platform_data mx53_loco_sd3_data __initconst = { .wp_type = ESDHC_WP_GPIO, }; +#define DA9052_LDO(max, min, rname, suspend_mv) \ +{\ + .constraints = {\ + .name = (rname), \ + .max_uV = (max) * 1000,\ + .min_uV = (min) * 1000,\ + .valid_ops_mask = REGULATOR_CHANGE_VOLTAGE\ + |REGULATOR_CHANGE_STATUS | REGULATOR_CHANGE_MODE,\ + .valid_modes_mask = REGULATOR_MODE_NORMAL,\ + .state_mem = { \ + .uV = suspend_mv * 1000, \ + .mode = REGULATOR_MODE_NORMAL, \ + .enabled = (0 == suspend_mv) ? 0 : 1, \ + .disabled = 0, \ + }, \ + },\ +} + +/* currently the suspend_mv here takes no effects for DA9053 +preset-voltage have to be done in the latest stage during +suspend*/ +static struct regulator_init_data da9052_regulators_init[] = { + /* BUCKS */ + DA9052_LDO(DA9052_BUCK_CORE_PRO_VOLT_UPPER, + DA9052_BUCK_CORE_PRO_VOLT_LOWER, "DA9052_BUCK_CORE", 850), + DA9052_LDO(DA9052_BUCK_CORE_PRO_VOLT_UPPER, + DA9052_BUCK_CORE_PRO_VOLT_LOWER, "DA9052_BUCK_PRO", 950), + DA9052_LDO(DA9052_BUCK_MEM_VOLT_UPPER, + DA9052_BUCK_MEM_VOLT_LOWER, "DA9052_BUCK_MEM", 1500), + DA9052_LDO(DA9052_BUCK_PERI_VOLT_UPPER, + DA9052_BUCK_PERI_VOLT_LOWER, "DA9052_BUCK_PERI", 2500), + DA9052_LDO(DA9052_LDO1_VOLT_UPPER, + DA9052_LDO1_VOLT_LOWER, "DA9052_LDO1", 1300), + DA9052_LDO(DA9052_LDO2_VOLT_UPPER, + DA9052_LDO2_VOLT_LOWER, "DA9052_LDO2", 1300), + DA9052_LDO(DA9052_LDO34_VOLT_UPPER, + DA9052_LDO34_VOLT_LOWER, "DA9052_LDO3", 3300), + DA9052_LDO(DA9052_LDO34_VOLT_UPPER, + DA9052_LDO34_VOLT_LOWER, "DA9052_LDO4", 2775), + DA9052_LDO(DA9052_LDO567810_VOLT_UPPER, + DA9052_LDO567810_VOLT_LOWER, "DA9052_LDO5", 1300), + DA9052_LDO(DA9052_LDO567810_VOLT_UPPER, + DA9052_LDO567810_VOLT_LOWER, "DA9052_LDO6", 1200), + DA9052_LDO(DA9052_LDO567810_VOLT_UPPER, + DA9052_LDO567810_VOLT_LOWER, "DA9052_LDO7", 2750), + DA9052_LDO(DA9052_LDO567810_VOLT_UPPER, + DA9052_LDO567810_VOLT_LOWER, "DA9052_LDO8", 1800), + DA9052_LDO(DA9052_LDO9_VOLT_UPPER, + DA9052_LDO9_VOLT_LOWER, "DA9052_LDO9", 2500), + DA9052_LDO(DA9052_LDO567810_VOLT_UPPER, + DA9052_LDO567810_VOLT_LOWER, "DA9052_LDO10", 1200), +}; + +#define MX53_LOCO_DA9052_IRQ (6*32 + 11) /* GPIO7_11 */ + +static int __init loco_da9052_init(struct da9052 *da9052) +{ + /* Configuring for DA9052 interrupt servce */ + /* s3c_gpio_setpull(DA9052_IRQ_PIN, S3C_GPIO_PULL_UP); */ + + /* Set interrupt as LOW LEVEL interrupt source */ + irq_set_irq_type(gpio_to_irq(MX53_LOCO_DA9052_IRQ), + IRQF_TRIGGER_LOW); + return 0; +} + +static struct da9052_pdata __initdata da9052_plat = { + .init = loco_da9052_init, + .irq_base = MXC_PMIC_IRQ_START, + .regulators = { + &da9052_regulators_init[0], + &da9052_regulators_init[1], + &da9052_regulators_init[2], + &da9052_regulators_init[3], + &da9052_regulators_init[4], + &da9052_regulators_init[5], + &da9052_regulators_init[6], + &da9052_regulators_init[7], + &da9052_regulators_init[8], + &da9052_regulators_init[9], + &da9052_regulators_init[10], + &da9052_regulators_init[11], + &da9052_regulators_init[12], + &da9052_regulators_init[13], + }, +}; + static inline void mx53_loco_fec_reset(void) { int ret; @@ -273,6 +397,10 @@ static struct i2c_board_info mx53loco_i2c_devices[] = { { I2C_BOARD_INFO("mma8450", 0x1C), }, + { + I2C_BOARD_INFO("da9053-aa", 0x90 >> 1), + .platform_data = &da9052_plat, + }, }; static void __init mx53_loco_board_init(void) diff --git a/arch/arm/plat-mxc/include/mach/irqs.h b/arch/arm/plat-mxc/include/mach/irqs.h index fd9efb0..9fb56eb 100644 --- a/arch/arm/plat-mxc/include/mach/irqs.h +++ b/arch/arm/plat-mxc/include/mach/irqs.h @@ -53,7 +53,15 @@ #endif /* REVISIT: Add IPU irqs on IMX51 */ -#define NR_IRQS (MXC_IPU_IRQ_START + MX3_IPU_IRQS) +#define MXC_PMIC_IRQ_START (MXC_IPU_IRQ_START + MX3_IPU_IRQS) + +#ifdef CONFIG_MACH_MX53_LOCO +#define MXC_PMIC_IRQS 32 +#else +#define MXC_PMIC_IRQS 0 +#endif + +#define NR_IRQS (MXC_PMIC_IRQ_START + MXC_PMIC_IRQS) extern int imx_irq_set_priority(unsigned char irq, unsigned char prio);