[PATCH V2] ARM: imx: add cpufreq support for i.mx6sx

[Anson.Huang at freescale.com]

Hi, Shawn
	I am OK with this.

Best Regards.
Anson Huang
 
Freescale Semiconductor Shanghai
Tel:021-28937058


>-----Original Message-----
>From: Shawn Guo [mailto:shawn.guo at freescale.com]
>Sent: Thursday, June 26, 2014 1:55 PM
>To: Fabio Estevam
>Cc: Huang Yongcai-B20788; Sascha Hauer; Russell King; linux-arm-
>kernel at lists.infradead.org
>Subject: Re: [PATCH V2] ARM: imx: add cpufreq support for i.mx6sx
>
>On Wed, Jun 25, 2014 at 10:23:01PM -0300, Fabio Estevam wrote:
>> Hi Anson,
>>
>> On Wed, Jun 25, 2014 at 6:10 AM, Anson Huang <b20788 at freescale.com> wrote:
>> > Add cpufreq support for i.MX6SX, using common i.MX6Q cpufreq driver.
>> >
>> > Signed-off-by: Anson Huang <b20788 at freescale.com>
>> > ---
>> >  arch/arm/mach-imx/mach-imx6sx.c |    3 +++
>> >  1 file changed, 3 insertions(+)
>> >
>> > diff --git a/arch/arm/mach-imx/mach-imx6sx.c
>> > b/arch/arm/mach-imx/mach-imx6sx.c index 74ff7d6..9221e1d 100644
>> > --- a/arch/arm/mach-imx/mach-imx6sx.c
>> > +++ b/arch/arm/mach-imx/mach-imx6sx.c
>> > @@ -42,6 +42,9 @@ static void __init imx6sx_init_irq(void)  static
>> > void __init imx6sx_init_late(void)  {
>> >         imx6q_cpuidle_init();
>> > +
>> > +       if (IS_ENABLED(CONFIG_ARM_IMX6Q_CPUFREQ))
>> > +               platform_device_register_simple("imx6q-cpufreq", -1,
>> > + NULL, 0);
>> >  }
>>
>> With this patch applied I get a flood of these messages:
>>
>> cpu cpu0: failed to scale vddpu up: -22
>> cpufreq: __target_index: Failed to change cpu frequency: -22 cpu cpu0:
>> failed to scale vddpu up: -22
>> cpufreq: __target_index: Failed to change cpu frequency: -22 cpu cpu0:
>> failed to scale vddpu up: -22
>> cpufreq: __target_index: Failed to change cpu frequency: -22
>
>So it's because the driver change for imx6sx hasn't been in place.  I just
>dropped the patch from my branch for now, and will revisit later when driver
>part shows up on my branch.  (Unfortunately, that will be next cycle).
>
>Shawn