> On 13 Sep 2017, at 22:41, Philipp Tomsich 
> <philipp.toms...@theobroma-systems.com> wrote:
> 
> 
> 
> On Wed, 13 Sep 2017, David Wu wrote:
> 
>> The clk_saradc is dividing from the 24M, clk_saradc=24MHz/(saradc_div_con+1).
>> Saradc integer divider control register is 8-bits width.
>> 
>> Signed-off-by: David Wu <david...@rock-chips.com 
>> <mailto:david...@rock-chips.com>>
> 
> Reviewed-by: Philipp Tomsich <philipp.toms...@theobroma-systems.com 
> <mailto:philipp.toms...@theobroma-systems.com>>
> 
> See below for comments.
> 
>> ---
>> arch/arm/include/asm/arch-rockchip/cru_rk3368.h |  5 ++++
>> drivers/clk/rockchip/clk_rk3368.c               | 32 
>> +++++++++++++++++++++++++
>> 2 files changed, 37 insertions(+)
>> 
>> diff --git a/arch/arm/include/asm/arch-rockchip/cru_rk3368.h 
>> b/arch/arm/include/asm/arch-rockchip/cru_rk3368.h
>> index 2b1197f..31f7685 100644
>> --- a/arch/arm/include/asm/arch-rockchip/cru_rk3368.h
>> +++ b/arch/arm/include/asm/arch-rockchip/cru_rk3368.h
>> @@ -89,6 +89,11 @@ enum {
>>      MCU_CLK_DIV_SHIFT               = 0,
>>      MCU_CLK_DIV_MASK                = GENMASK(4, 0),
>> 
>> +    /* CLKSEL_CON25 */
>> +    CLK_SARADC_DIV_CON_SHIFT        = 8,
>> +    CLK_SARADC_DIV_CON_MASK         = 0xff << CLK_SARADC_DIV_CON_SHIFT,
> 
> Please use GENMASK.
> 
>> +    CLK_SARADC_DIV_CON_WIDTH        = 8,
>> +
>>      /* CLKSEL43_CON */
>>      GMAC_MUX_SEL_EXTCLK             = BIT(8),
>> 
>> diff --git a/drivers/clk/rockchip/clk_rk3368.c 
>> b/drivers/clk/rockchip/clk_rk3368.c
>> index 2be1f57..2eedf77 100644
>> --- a/drivers/clk/rockchip/clk_rk3368.c
>> +++ b/drivers/clk/rockchip/clk_rk3368.c
>> @@ -12,6 +12,7 @@
>> #include <errno.h>
>> #include <mapmem.h>
>> #include <syscon.h>
>> +#include <bitfield.h>
>> #include <asm/arch/clock.h>
>> #include <asm/arch/cru_rk3368.h>
>> #include <asm/arch/hardware.h>
>> @@ -397,6 +398,31 @@ static ulong rk3368_spi_set_clk(struct rk3368_cru *cru, 
>> ulong clk_id, uint hz)
>>      return rk3368_spi_get_clk(cru, clk_id);
>> }
>> 
>> +static ulong rk3368_saradc_get_clk(struct rk3368_cru *cru)
>> +{
>> +    u32 div, val;
>> +
>> +    val = readl(&cru->clksel_con[25]);
>> +    div = bitfield_extract(val, CLK_SARADC_DIV_CON_SHIFT,
>> +                           CLK_SARADC_DIV_CON_WIDTH);
> 
> Please reuse the functions from bitfield.h.

It’s apparently too late to do code reviews: please ignore this comment.

> 
>> +
>> +    return DIV_TO_RATE(OSC_HZ, div);
>> +}
>> +
>> +static ulong rk3368_saradc_set_clk(struct rk3368_cru *cru, uint hz)
>> +{
>> +    int src_clk_div;
>> +
>> +    src_clk_div = DIV_ROUND_UP(OSC_HZ, hz) - 1;
>> +    assert(src_clk_div < 128);
>> +
>> +    rk_clrsetreg(&cru->clksel_con[25],
>> +                 CLK_SARADC_DIV_CON_MASK,
>> +                 src_clk_div << CLK_SARADC_DIV_CON_SHIFT);
>> +
>> +    return rk3368_saradc_get_clk(cru);
>> +}
>> +
>> static ulong rk3368_clk_get_rate(struct clk *clk)
>> {
>>      struct rk3368_clk_priv *priv = dev_get_priv(clk->dev);
>> @@ -419,6 +445,9 @@ static ulong rk3368_clk_get_rate(struct clk *clk)
>>              rate = rk3368_mmc_get_clk(priv->cru, clk->id);
>>              break;
>> #endif
>> +    case SCLK_SARADC:
>> +            rate = rk3368_saradc_get_clk(priv->cru);
>> +            break;
>>      default:
>>              return -ENOENT;
>>      }
>> @@ -453,6 +482,9 @@ static ulong rk3368_clk_set_rate(struct clk *clk, ulong 
>> rate)
>>              ret = rk3368_gmac_set_clk(priv->cru, clk->id, rate);
>>              break;
>> #endif
>> +    case SCLK_SARADC:
>> +            ret =  rk3368_saradc_set_clk(priv->cru, rate);
>> +            break;
>>      default:
>>              return -ENOENT;
>>      }

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to