Re: [PATCH v2 0/2] Remove unused voltagedomain data for AM33xx

2013-06-17 Thread Santosh Shilimkar
On Monday 17 June 2013 09:16 AM, Rajendra Nayak wrote:
 Hi,
 
 The powerdomain framework currently expects to always have a voltagedomain
 associated with a given powerdomain. We already have AM33xx which
 has no Voltage Controller/Voltage Processor as part of PRCM.
 There are more SoCs' to follow starting with AM437x and DRA7xx
 which do not have VC/VP. All these SoCs', hence do not support any
 form of auto-scaling of voltages, that was possible on earlier OMAPs'
 using the VFSM to trigger an i2c communication to the PMIC automatically
 on hitting a low power state.
 
 Instead of adding dummy voltage domain data files, like was done for the
 AM33xx case, make the powerdomain framework aware of the fact that some
 SoCs' might not really have support for auto-scaling of voltages in hw.
 
 Patches are based of 3.10-rc6 and are boot tested on am335x bone.
 
 Rajendra Nayak (2):
   ARM: OMAP2+: Powerdomain: Remove the need to always have a voltdm
 associated to a pwrdm
   ARM: AM33xx: Remove the unused voltagedomain data
 
  arch/arm/mach-omap2/Makefile  |1 -
  arch/arm/mach-omap2/io.c  |1 -
  arch/arm/mach-omap2/powerdomain.c |5 +++
  arch/arm/mach-omap2/powerdomain.h |2 ++
  arch/arm/mach-omap2/prm33xx.c |7 
  arch/arm/mach-omap2/voltage.h |1 -
  arch/arm/mach-omap2/voltagedomains33xx_data.c |   43 
 -
  7 files changed, 14 insertions(+), 46 deletions(-)
  delete mode 100644 arch/arm/mach-omap2/voltagedomains33xx_data.c
 
Nice !!

FWIW, Acked-by: Santosh Shilimkar santosh.shilim...@ti.com
--
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 0/2] Remove unused voltagedomain data for AM33xx

2013-06-17 Thread Mohammed, Afzal
Hi,

On Mon, Jun 17, 2013 at 18:46:21, Nayak, Rajendra wrote:

 The powerdomain framework currently expects to always have a voltagedomain
 associated with a given powerdomain. We already have AM33xx which
 has no Voltage Controller/Voltage Processor as part of PRCM.
 There are more SoCs' to follow starting with AM437x and DRA7xx
 which do not have VC/VP. All these SoCs', hence do not support any
 form of auto-scaling of voltages, that was possible on earlier OMAPs'
 using the VFSM to trigger an i2c communication to the PMIC automatically
 on hitting a low power state.
 
 Instead of adding dummy voltage domain data files, like was done for the
 AM33xx case, make the powerdomain framework aware of the fact that some
 SoCs' might not really have support for auto-scaling of voltages in hw.
 
 Patches are based of 3.10-rc6 and are boot tested on am335x bone.

Tested-by: Afzal Mohammed af...@ti.com (am335x evm)

Regards
Afzal
--
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 0/2] Remove unused voltagedomain data for AM33xx

2013-06-17 Thread Kevin Hilman
Rajendra Nayak rna...@ti.com writes:

 Hi,

 The powerdomain framework currently expects to always have a voltagedomain
 associated with a given powerdomain. We already have AM33xx which
 has no Voltage Controller/Voltage Processor as part of PRCM.
 There are more SoCs' to follow starting with AM437x and DRA7xx
 which do not have VC/VP. All these SoCs', hence do not support any
 form of auto-scaling of voltages, that was possible on earlier OMAPs'
 using the VFSM to trigger an i2c communication to the PMIC automatically
 on hitting a low power state.

 Instead of adding dummy voltage domain data files, like was done for the
 AM33xx case, make the powerdomain framework aware of the fact that some
 SoCs' might not really have support for auto-scaling of voltages in hw.

 Patches are based of 3.10-rc6 and are boot tested on am335x bone.

 Rajendra Nayak (2):
   ARM: OMAP2+: Powerdomain: Remove the need to always have a voltdm
 associated to a pwrdm
   ARM: AM33xx: Remove the unused voltagedomain data

Thanks, queuing up this series with ack from Santosh and tested-by from
Afzal.  I'll try, but not sure if it can make it for v3.11.

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