<div class="gmail_quote"><div>Hi Sascha,</div><div><br></div><div>Thanks for your thorough review. I have two feedbacks to your commends. Sorry for delayed response, cause I had a hard time due to my computer crash and data loss.</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="im">
&gt; diff --git a/arch/arm/mach-mx5/cpu.c b/arch/arm/mach-mx5/cpu.c<br>
&gt; index 2d37785..83add9c 100644<br>
&gt; --- a/arch/arm/mach-mx5/cpu.c<br>
&gt; +++ b/arch/arm/mach-mx5/cpu.c<br>
&gt; @@ -22,6 +22,8 @@ static int cpu_silicon_rev = -1;<br>
&gt;<br>
&gt;  #define SI_REV 0x48<br>
&gt;<br>
&gt; +struct cpu_wp *(*get_cpu_wp)(int *wp);<br>
&gt; +<br>
<br>
</div>This is not needed.<br>
<div><div></div><div class="h5"><br></div></div></blockquote><div>This is needed, otherwise it does not pass compile. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div><div class="h5"><br></div></div><div class="im">
&gt; +             return ret;<br>
&gt; +     }<br>
&gt; +<br>
&gt; +     cpufreq_frequency_table_get_attr(imx_freq_table, policy-&gt;cpu);<br>
&gt; +     return 0;<br>
&gt; +}<br>
&gt; +<br>
&gt; +static int mxc_cpufreq_exit(struct cpufreq_policy *policy)<br>
&gt; +{<br>
&gt; +     cpufreq_frequency_table_put_attr(policy-&gt;cpu);<br>
&gt; +<br>
&gt; +     /* Reset CPU to 665MHz */<br>
&gt; +     set_cpu_freq(arm_normal_clk);<br>
<br>
</div>arm_normal_clk is initialized to cpu_freq_khz_max * 1000 and never touched<br>
again. It would be clearer here to remove arm_normal_clk and write<br>
cpu_freq_khz_max * 1000 directly here. Then you can also remove the<br>
comment which is wrong for most i.MXs, even for the i.MX51.<br>
<br></blockquote><div><br></div><div>Actually, this is arm_normal_clk is touched later in mxc_cpufreq_exit() fuction.</div><div><br></div><div>Yong</div></div>