RE: [PATCH 2/2] OMAP3: RX-51: define vdds_csib regulator supply

2011-05-02 Thread kalle.jokiniemi
Hi,

  -Original Message-
  From: ext Tony Lindgren [mailto:t...@atomide.com]
  Sent: 29. huhtikuuta 2011 12:14
  To: Jokiniemi Kalle (Nokia-SD/Tampere)
  Cc: laurent.pinch...@ideasonboard.com; mche...@infradead.org; linux-
  o...@vger.kernel.org; linux-me...@vger.kernel.org
  Subject: Re: [PATCH 2/2] OMAP3: RX-51: define vdds_csib regulator supply
  
  * Kalle Jokiniemi kalle.jokini...@nokia.com [110429 00:09]:
   The RX-51 uses the CSIb IO complex for camera operation. The
   board file is missing definition for the regulator supplying
   the CSIb complex, so this is added for better power
   management.
  
   Signed-off-by: Kalle Jokiniemi kalle.jokini...@nokia.com
  
  This looks safe to merge along with the other patch
  
  Acked-by: Tony Lindgren t...@atomide.com

Thanks Tony, adding Mauro's correct email...

- Kalle




  
   ---
arch/arm/mach-omap2/board-rx51-peripherals.c |9 +
1 files changed, 9 insertions(+), 0 deletions(-)
  
   diff --git a/arch/arm/mach-omap2/board-rx51-peripherals.c
  b/arch/arm/mach-omap2/board-rx51-peripherals.c
   index bbcb677..1324ba3 100644
   --- a/arch/arm/mach-omap2/board-rx51-peripherals.c
   +++ b/arch/arm/mach-omap2/board-rx51-peripherals.c
   @@ -337,6 +337,13 @@ static struct omap2_hsmmc_info mmc[] __initdata =
  {
static struct regulator_consumer_supply rx51_vmmc1_supply =
  REGULATOR_SUPPLY(vmmc, omap_hsmmc.0);
  
   +static struct regulator_consumer_supply rx51_vaux2_supplies[] = {
   +  REGULATOR_SUPPLY(vdds_csib, omap3isp),
   +  {
   +  .supply = vaux2,
   +  },
   +};
   +
static struct regulator_consumer_supply rx51_vaux3_supply =
  REGULATOR_SUPPLY(vmmc, omap_hsmmc.1);
  
   @@ -400,6 +407,8 @@ static struct regulator_init_data rx51_vaux2 = {
  .valid_ops_mask = REGULATOR_CHANGE_MODE
  | REGULATOR_CHANGE_STATUS,
  },
   +  .num_consumer_supplies  = ARRAY_SIZE(rx51_vaux2_supplies),
   +  .consumer_supplies  = rx51_vaux2_supplies,
};
  
/* VAUX3 - adds more power to VIO_18 rail */
   --
   1.7.1
  
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH 2/2] OMAP3: RX-51: define vdds_csib regulator supply

2011-04-29 Thread Kalle Jokiniemi
The RX-51 uses the CSIb IO complex for camera operation. The
board file is missing definition for the regulator supplying
the CSIb complex, so this is added for better power
management.

Signed-off-by: Kalle Jokiniemi kalle.jokini...@nokia.com
---
 arch/arm/mach-omap2/board-rx51-peripherals.c |9 +
 1 files changed, 9 insertions(+), 0 deletions(-)

diff --git a/arch/arm/mach-omap2/board-rx51-peripherals.c 
b/arch/arm/mach-omap2/board-rx51-peripherals.c
index bbcb677..1324ba3 100644
--- a/arch/arm/mach-omap2/board-rx51-peripherals.c
+++ b/arch/arm/mach-omap2/board-rx51-peripherals.c
@@ -337,6 +337,13 @@ static struct omap2_hsmmc_info mmc[] __initdata = {
 static struct regulator_consumer_supply rx51_vmmc1_supply =
REGULATOR_SUPPLY(vmmc, omap_hsmmc.0);
 
+static struct regulator_consumer_supply rx51_vaux2_supplies[] = {
+   REGULATOR_SUPPLY(vdds_csib, omap3isp),
+   {
+   .supply = vaux2,
+   },
+};
+
 static struct regulator_consumer_supply rx51_vaux3_supply =
REGULATOR_SUPPLY(vmmc, omap_hsmmc.1);
 
@@ -400,6 +407,8 @@ static struct regulator_init_data rx51_vaux2 = {
.valid_ops_mask = REGULATOR_CHANGE_MODE
| REGULATOR_CHANGE_STATUS,
},
+   .num_consumer_supplies  = ARRAY_SIZE(rx51_vaux2_supplies),
+   .consumer_supplies  = rx51_vaux2_supplies,
 };
 
 /* VAUX3 - adds more power to VIO_18 rail */
-- 
1.7.1

--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 2/2] OMAP3: RX-51: define vdds_csib regulator supply

2011-04-29 Thread Tony Lindgren
* Kalle Jokiniemi kalle.jokini...@nokia.com [110429 00:09]:
 The RX-51 uses the CSIb IO complex for camera operation. The
 board file is missing definition for the regulator supplying
 the CSIb complex, so this is added for better power
 management.
 
 Signed-off-by: Kalle Jokiniemi kalle.jokini...@nokia.com

This looks safe to merge along with the other patch:

Acked-by: Tony Lindgren t...@atomide.com

 ---
  arch/arm/mach-omap2/board-rx51-peripherals.c |9 +
  1 files changed, 9 insertions(+), 0 deletions(-)
 
 diff --git a/arch/arm/mach-omap2/board-rx51-peripherals.c 
 b/arch/arm/mach-omap2/board-rx51-peripherals.c
 index bbcb677..1324ba3 100644
 --- a/arch/arm/mach-omap2/board-rx51-peripherals.c
 +++ b/arch/arm/mach-omap2/board-rx51-peripherals.c
 @@ -337,6 +337,13 @@ static struct omap2_hsmmc_info mmc[] __initdata = {
  static struct regulator_consumer_supply rx51_vmmc1_supply =
   REGULATOR_SUPPLY(vmmc, omap_hsmmc.0);
  
 +static struct regulator_consumer_supply rx51_vaux2_supplies[] = {
 + REGULATOR_SUPPLY(vdds_csib, omap3isp),
 + {
 + .supply = vaux2,
 + },
 +};
 +
  static struct regulator_consumer_supply rx51_vaux3_supply =
   REGULATOR_SUPPLY(vmmc, omap_hsmmc.1);
  
 @@ -400,6 +407,8 @@ static struct regulator_init_data rx51_vaux2 = {
   .valid_ops_mask = REGULATOR_CHANGE_MODE
   | REGULATOR_CHANGE_STATUS,
   },
 + .num_consumer_supplies  = ARRAY_SIZE(rx51_vaux2_supplies),
 + .consumer_supplies  = rx51_vaux2_supplies,
  };
  
  /* VAUX3 - adds more power to VIO_18 rail */
 -- 
 1.7.1
 
--
To unsubscribe from this list: send the line unsubscribe linux-omap in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html