Re: [PATCH v2] ARM: OMAP4: twl-common: Support for additional devices on i2c1 bus

2012-08-07 Thread Tony Lindgren 
* Peter Ujfalusi peter.ujfal...@ti.com [120723 01:52]:
 Hi Tony,
 
 On 07/13/2012 10:38 AM, Peter Ujfalusi wrote:
  On OMAP4 the i2c1 bus is dedicated for the PMIC and audio related devices.
  Manufacturers can opt to use different codec than twl6040 and also can add
  audio related IC to the bus (external amplifier for example on SDP4430).
  
  Make it possible to add differnet set of additional devices to i2c1 bus on
  OMAP4 boards.
 
 Would it be possible to queue this patch for 3.6?

Sorry things were getting already too complex for v3.6 merge window..
So v3.7 it will be. Looks like this need to be updated against v3.6-rc1
to apply.

Regards,

Tony
--
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 v2] ARM: OMAP4: twl-common: Support for additional devices on i2c1 bus

2012-08-07 Thread Peter Ujfalusi 
Hi Tony,

On 08/07/2012 10:20 AM, Tony Lindgren wrote:
 Sorry things were getting already too complex for v3.6 merge window..

OK.

 So v3.7 it will be. Looks like this need to be updated against v3.6-rc1
 to apply.

I'll resend the updated patch on top of 3.6-rc1

Regards,
Péter
--
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 v2] ARM: OMAP4: twl-common: Support for additional devices on i2c1 bus

2012-07-23 Thread Peter Ujfalusi 
Hi Tony,

On 07/13/2012 10:38 AM, Peter Ujfalusi wrote:
 On OMAP4 the i2c1 bus is dedicated for the PMIC and audio related devices.
 Manufacturers can opt to use different codec than twl6040 and also can add
 audio related IC to the bus (external amplifier for example on SDP4430).
 
 Make it possible to add differnet set of additional devices to i2c1 bus on
 OMAP4 boards.

Would it be possible to queue this patch for 3.6?

Thank you,

Péter

 Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com
 ---
 
 Changes since v1:
 Generated against l-o/master
 
 Regards,
 Peter
 
  arch/arm/mach-omap2/board-4430sdp.c|   12 +-
  arch/arm/mach-omap2/board-omap4panda.c |   12 +-
  arch/arm/mach-omap2/twl-common.c   |   35 +--
  arch/arm/mach-omap2/twl-common.h   |3 +-
  4 files changed, 32 insertions(+), 30 deletions(-)
 
 diff --git a/arch/arm/mach-omap2/board-4430sdp.c 
 b/arch/arm/mach-omap2/board-4430sdp.c
 index ad8a7d9..26466f2 100644
 --- a/arch/arm/mach-omap2/board-4430sdp.c
 +++ b/arch/arm/mach-omap2/board-4430sdp.c
 @@ -547,6 +547,14 @@ static struct twl6040_platform_data twl6040_data = {
   .irq_base   = TWL6040_CODEC_IRQ_BASE,
  };
  
 +static struct i2c_board_info __initdata sdp4430_i2c_1_boardinfo[] = {
 + {
 + I2C_BOARD_INFO(twl6040, 0x4b),
 + .irq = OMAP44XX_IRQ_SYS_2N,
 + .platform_data = twl6040_data,
 + },
 +};
 +
  static struct twl4030_platform_data sdp4430_twldata = {
   /* Regulators */
   .vusim  = sdp4430_vusim,
 @@ -580,8 +588,8 @@ static int __init omap4_i2c_init(void)
   TWL_COMMON_REGULATOR_CLK32KG |
   TWL_COMMON_REGULATOR_V1V8 |
   TWL_COMMON_REGULATOR_V2V1);
 - omap4_pmic_init(twl6030, sdp4430_twldata,
 - twl6040_data, OMAP44XX_IRQ_SYS_2N);
 + omap4_pmic_init(twl6030, sdp4430_twldata, sdp4430_i2c_1_boardinfo,
 + ARRAY_SIZE(sdp4430_i2c_1_boardinfo));
   omap_register_i2c_bus(2, 400, NULL, 0);
   omap_register_i2c_bus(3, 400, sdp4430_i2c_3_boardinfo,
   ARRAY_SIZE(sdp4430_i2c_3_boardinfo));
 diff --git a/arch/arm/mach-omap2/board-omap4panda.c 
 b/arch/arm/mach-omap2/board-omap4panda.c
 index b627cdc..5becba2 100644
 --- a/arch/arm/mach-omap2/board-omap4panda.c
 +++ b/arch/arm/mach-omap2/board-omap4panda.c
 @@ -266,6 +266,14 @@ static struct twl6040_platform_data twl6040_data = {
   .irq_base   = TWL6040_CODEC_IRQ_BASE,
  };
  
 +static struct i2c_board_info __initdata panda_i2c_1_boardinfo[] = {
 + {
 + I2C_BOARD_INFO(twl6040, 0x4b),
 + .irq = OMAP44XX_IRQ_SYS_2N,
 + .platform_data = twl6040_data,
 + },
 +};
 +
  /* Panda board uses the common PMIC configuration */
  static struct twl4030_platform_data omap4_panda_twldata;
  
 @@ -293,8 +301,8 @@ static int __init omap4_panda_i2c_init(void)
   TWL_COMMON_REGULATOR_CLK32KG |
   TWL_COMMON_REGULATOR_V1V8 |
   TWL_COMMON_REGULATOR_V2V1);
 - omap4_pmic_init(twl6030, omap4_panda_twldata,
 - twl6040_data, OMAP44XX_IRQ_SYS_2N);
 + omap4_pmic_init(twl6030, omap4_panda_twldata, panda_i2c_1_boardinfo,
 + ARRAY_SIZE(panda_i2c_1_boardinfo));
   omap_register_i2c_bus(2, 400, NULL, 0);
   /*
* Bus 3 is attached to the DVI port where devices like the pico DLP
 diff --git a/arch/arm/mach-omap2/twl-common.c 
 b/arch/arm/mach-omap2/twl-common.c
 index 3882f3c..d52be85 100644
 --- a/arch/arm/mach-omap2/twl-common.c
 +++ b/arch/arm/mach-omap2/twl-common.c
 @@ -39,16 +39,6 @@ static struct i2c_board_info __initdata 
 pmic_i2c_board_info = {
   .flags  = I2C_CLIENT_WAKE,
  };
  
 -static struct i2c_board_info __initdata omap4_i2c1_board_info[] = {
 - {
 - .addr   = 0x48,
 - .flags  = I2C_CLIENT_WAKE,
 - },
 - {
 - I2C_BOARD_INFO(twl6040, 0x4b),
 - },
 -};
 -
  #if defined(CONFIG_ARCH_OMAP3) || defined(CONFIG_ARCH_OMAP4)
  static int twl_set_voltage(void *data, int target_uV)
  {
 @@ -77,30 +67,25 @@ void __init omap_pmic_init(int bus, u32 clkrate,
  
  void __init omap4_pmic_init(const char *pmic_type,
   struct twl4030_platform_data *pmic_data,
 - struct twl6040_platform_data *twl6040_data, int twl6040_irq)
 + struct i2c_board_info *devices, int nr_devices)
  {
   /* PMIC part*/
   omap_mux_init_signal(sys_nirq1, OMAP_PIN_INPUT_PULLUP | 
 OMAP_PIN_OFF_WAKEUPENABLE);
 - strncpy(omap4_i2c1_board_info[0].type, pmic_type,
 - sizeof(omap4_i2c1_board_info[0].type));
 - omap4_i2c1_board_info[0].irq = OMAP44XX_IRQ_SYS_1N;
 - omap4_i2c1_board_info[0].platform_data = pmic_data;
 -
 - /* TWL6040 audio IC part */
 - omap4_i2c1_board_info[1].irq = twl6040_irq;
 -

Re: [PATCH v2] ARM: OMAP4: twl-common: Support for additional devices on i2c1 bus

2012-07-13 Thread Menon, Nishanth 
On Fri, Jul 13, 2012 at 3:38 AM, Peter Ujfalusi peter.ujfal...@ti.com wrote:
 On OMAP4 the i2c1 bus is dedicated for the PMIC and audio related devices.
 Manufacturers can opt to use different codec than twl6040 and also can add
 audio related IC to the bus (external amplifier for example on SDP4430).

 Make it possible to add differnet set of additional devices to i2c1 bus on
 OMAP4 boards.

 Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com
 ---

 Changes since v1:
 Generated against l-o/master

 Regards,
 Peter

  arch/arm/mach-omap2/board-4430sdp.c|   12 +-
  arch/arm/mach-omap2/board-omap4panda.c |   12 +-
  arch/arm/mach-omap2/twl-common.c   |   35 +--
  arch/arm/mach-omap2/twl-common.h   |3 +-
  4 files changed, 32 insertions(+), 30 deletions(-)

 diff --git a/arch/arm/mach-omap2/board-4430sdp.c 
 b/arch/arm/mach-omap2/board-4430sdp.c
 index ad8a7d9..26466f2 100644
 --- a/arch/arm/mach-omap2/board-4430sdp.c
 +++ b/arch/arm/mach-omap2/board-4430sdp.c
 @@ -547,6 +547,14 @@ static struct twl6040_platform_data twl6040_data = {
 .irq_base   = TWL6040_CODEC_IRQ_BASE,
  };

 +static struct i2c_board_info __initdata sdp4430_i2c_1_boardinfo[] = {
 +   {
 +   I2C_BOARD_INFO(twl6040, 0x4b),
 +   .irq = OMAP44XX_IRQ_SYS_2N,
 +   .platform_data = twl6040_data,
 +   },
 +};
 +
  static struct twl4030_platform_data sdp4430_twldata = {
 /* Regulators */
 .vusim  = sdp4430_vusim,
 @@ -580,8 +588,8 @@ static int __init omap4_i2c_init(void)
 TWL_COMMON_REGULATOR_CLK32KG |
 TWL_COMMON_REGULATOR_V1V8 |
 TWL_COMMON_REGULATOR_V2V1);
 -   omap4_pmic_init(twl6030, sdp4430_twldata,
 -   twl6040_data, OMAP44XX_IRQ_SYS_2N);
 +   omap4_pmic_init(twl6030, sdp4430_twldata, sdp4430_i2c_1_boardinfo,
 +   ARRAY_SIZE(sdp4430_i2c_1_boardinfo));

We still need a way to switch to I2C highspeed mode
omap4_pmic_init still does register in 400KHz mode, while setting a bit in 6040
should let us talk 3.3MHz on the bus.

Regards,
Nishanth Menon
--
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 v2] ARM: OMAP4: twl-common: Support for additional devices on i2c1 bus

2012-07-13 Thread Peter Ujfalusi 
On 07/13/2012 12:14 PM, Menon, Nishanth wrote:
 We still need a way to switch to I2C highspeed mode
 omap4_pmic_init still does register in 400KHz mode, while setting a bit in 
 6040
 should let us talk 3.3MHz on the bus.

We need to have a way to do this runtime I think. The twl6040 comes up with
100kbps as a default speed. So far we had no issue with 400kbps bus, but if we
increase it we might not be able to communicate with twl6040...

-- 
Péter


--
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