<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 23, 2016 at 8:24 AM, Simon Glass <span dir="ltr"><<a href="mailto:sjg@chromium.org" target="_blank">sjg@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">[resend from correct address]<br>
<div class="HOEnZb"><div class="h5"><br>
On 23 November 2016 at 09:13, Simon Glass <<a href="mailto:sjg@google.com">sjg@google.com</a>> wrote:<br>
> Hi Maxim,<br>
><br>
> On 22 November 2016 at 16:56,  <<a href="mailto:maxims@google.com">maxims@google.com</a>> wrote:<br>
>> From: Maxim Sloyko <<a href="mailto:maxims@google.com">maxims@google.com</a>><br>
>><br>
><br>
> For the subject, 'Add' rather than 'Added' (we use present tense)<br></div></div></blockquote><div><br></div><div>Done.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">
><br>
>> This is needed by I2C driver.<br>
>><br>
>> Signed-off-by: Maxim Sloyko <<a href="mailto:maxims@google.com">maxims@google.com</a>><br>
>> ---<br>
>>  arch/arm/include/asm/arch-<wbr>aspeed/ast_scu.h |  1 +<br>
>>  arch/arm/mach-aspeed/ast-scu.c             | 11 +++++++++++<br>
>>  2 files changed, 12 insertions(+)<br>
>><br>
>> diff --git a/arch/arm/include/asm/arch-<wbr>aspeed/ast_scu.h b/arch/arm/include/asm/arch-<wbr>aspeed/ast_scu.h<br>
>> index d248416..eb5aaa2 100644<br>
>> --- a/arch/arm/include/asm/arch-<wbr>aspeed/ast_scu.h<br>
>> +++ b/arch/arm/include/asm/arch-<wbr>aspeed/ast_scu.h<br>
>> @@ -38,6 +38,7 @@ extern void ast_scu_get_who_init_dram(<wbr>void);<br>
>>  extern u32 ast_get_clk_source(void);<br>
>>  extern u32 ast_get_h_pll_clk(void);<br>
>>  extern u32 ast_get_ahbclk(void);<br>
>> +extern u32 ast_get_apbclk(void);<br>
><br>
> Please add a comment as to what this does and what it returns.<br></div></div></blockquote><div><br></div><div>Done.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">
><br>
>><br>
>>  extern u32 ast_scu_get_vga_memsize(void);<br>
>><br>
>> diff --git a/arch/arm/mach-aspeed/ast-<wbr>scu.c b/arch/arm/mach-aspeed/ast-<wbr>scu.c<br>
>> index 280c421..e00dbe2 100644<br>
>> --- a/arch/arm/mach-aspeed/ast-<wbr>scu.c<br>
>> +++ b/arch/arm/mach-aspeed/ast-<wbr>scu.c<br>
>> @@ -318,6 +318,17 @@ u32 ast_get_ahbclk(void)<br>
>><br>
>>  #endif /* AST_SOC_G5 */<br>
>><br>
>> +u32 ast_get_apbclk(void)<br>
>> +{<br>
>> +       u32 h_pll = ast_get_h_pll_clk();<br>
><br>
> Can this be ulong, or is there a reason it has to be exactly 32 bits?<br>
><br>
> blank line here (between declarations and code)<br>
><br>
>> +       /* The formula for converting the bit pattern to divisor is<br>
><br>
> /*<br>
>  * The formula...<br>
>  * ...<br>
>  */<br></div></div></blockquote><div><br></div><div>Done.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">
><br>
>> +        * (4 + 4 * DIV), according to datasheet<br>
>> +        */<br>
>> +       u32 apb_div = 4 + 4 * SCU_GET_PCLK_DIV(ast_scu_read(<wbr>AST_SCU_CLK_SEL));<br>
>> +       return h_pll / apb_div;<br>
>> +}<br>
>> +<br>
>> +<br>
>>  void ast_scu_show_system_info(void)<br>
>>  {<br>
>><br>
>> --<br>
>> 2.8.0.rc3.226.g39d4020<br>
>><br>
><br>
> Regards,<br>
> Simon<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div><b>M</b>axim <b>S</b>loyko</div></div>
</div></div>