public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] [PATCH 0/9 v2] Fix and Re-organise PWM Timer
@ 2013-02-28 10:59 Akshay Saraswat
  2013-02-28 10:59 ` [U-Boot] [PATCH 1/9 v2] Exynos: Change get_timer() to work correctly Akshay Saraswat
                   ` (8 more replies)
  0 siblings, 9 replies; 19+ messages in thread
From: Akshay Saraswat @ 2013-02-28 10:59 UTC (permalink / raw)
  To: u-boot

This patch set tries to fix few bugs in timer and re-organises PWM
clock code.

Changes since v1:
        - Patch-1: Added "Acked-by: Simon Glass".
        - Patch-2: Added "Acked-by: Simon Glass".
        - Patch-3: Added "Acked-by: Simon Glass".
        - Patch-4: Added "Acked-by: Simon Glass".
        - Patch-5: Added "Acked-by: Simon Glass".
        - Patch-6: None.
        - Patch-7: Fixed few nits.
        - Patch-8: Added "Acked-by: Simon Glass".
	- Patch-9: Restored get_pwm_clk call in case of non-exynos5 cpu.

Akshay Saraswat (9):
  Exynos: Change get_timer() to work correctly
  Exynos: Add timer_get_us function
  Exynos: pwm: Fix two bugs in the exynos pwm configuration code
  Exynos: Avoid a divide by zero by specifying a non-zero period for
    pwm 4
  Exynos: Tidy up the pwm_config function in the exynos pwm driver
  Exynos: Add peripherial id for pwm
  Exynos: clock: Add generic api to get the clk freq
  Exynos: clock: Correct pwm source clk selection
  Exynos: pwm: Use generic api to get pwm clk freq

 arch/arm/cpu/armv7/exynos/clock.c         | 138 ++++++++++++++++++++++++++++++
 arch/arm/cpu/armv7/s5p-common/pwm.c       |  57 +++++++-----
 arch/arm/cpu/armv7/s5p-common/timer.c     | 117 ++++++++++++-------------
 arch/arm/include/asm/arch-exynos/clk.h    |  17 ++++
 arch/arm/include/asm/arch-exynos/periph.h |   5 ++
 board/samsung/smdk5250/setup.h            |   2 +-
 6 files changed, 249 insertions(+), 87 deletions(-)

-- 
1.8.0

^ permalink raw reply	[flat|nested] 19+ messages in thread
* [U-Boot] [PATCH 1/9 v2] Exynos: Change get_timer() to work correctly
@ 2013-03-28 14:14 Akshay Saraswat
  0 siblings, 0 replies; 19+ messages in thread
From: Akshay Saraswat @ 2013-03-28 14:14 UTC (permalink / raw)
  To: u-boot

Thanks for your comments.
Please find the reply corresponding to comments below.

>Dear Akshay,
>
>On 28/02/13 19:59, Akshay Saraswat wrote:
>> At present get_timer() does not return sane values. It should count up
>> smoothly in milliscond intervals.
>> 
>> diff --git a/arch/arm/cpu/armv7/s5p-common/pwm.c b/arch/arm/cpu/armv7/s5p-common/pwm.c
>> index 44d7bc3..3147f59 100644
>> --- a/arch/arm/cpu/armv7/s5p-common/pwm.c
>> +++ b/arch/arm/cpu/armv7/s5p-common/pwm.c
>> @@ -174,6 +174,12 @@ int pwm_init(int pwm_id, int div, int invert)
>>  
>>  	/* set count value */
>>  	offset = pwm_id * 3;
>> +
>> +	/*
>> +	 * TODO(sjg): Use this as a countdown timer for now. We count down
>> +	 * from the maximum value to 0, then reset.
>> +	 */
>> +	timer_rate_hz = -1;
>
>is it workaround?

Yes, this is a part of workaround to get appropriate ticks.

>
>>  	writel(timer_rate_hz, &pwm->tcntb0 + offset);
>>  
>>  	val = readl(&pwm->tcon) & ~(0xf << TCON_OFFSET(pwm_id));
>> diff --git a/arch/arm/cpu/armv7/s5p-common/timer.c b/arch/arm/cpu/armv7/s5p-common/timer.c
>> index e78c716..c48a297 100644
>> --- a/arch/arm/cpu/armv7/s5p-common/timer.c
>> +++ b/arch/arm/cpu/armv7/s5p-common/timer.c
>> @@ -39,13 +39,33 @@ static inline struct s5p_timer *s5p_get_base_timer(void)
>>  	return (struct s5p_timer *)samsung_get_base_timer();
>>  }
>>  
>> +/**
>> + * Read the countdown timer.
>> + *
>> + * This operates at 1MHz and counts downwards. It will wrap about every
>> + * hour (2^32 microseconds).
>> + *
>> + * @return current value of timer
>> + */
>> +static unsigned long timer_get_us_down(void)
>> +{
>> +	struct s5p_timer *const timer = s5p_get_base_timer();
>> +
>> +	return readl(&timer->tcnto4);
>> +}
>> +
>>  int timer_init(void)
>>  {
>>  	/* PWM Timer 4 */
>> -	pwm_init(4, MUX_DIV_2, 0);
>> +	pwm_init(4, MUX_DIV_4, 0);
>
>There are special reason to change the div value?
>Please let me know.

We wish to count down at 1MHz, providing a resolution of 1us.
This becomes possible with MUX_DIV_4 for timer 4, since,
exynos 5250 manual says that for 4-bit divider settings at
1/4 (PCLK = 66 MHz ), we get 66637.07 seconds interval. 

>
>>  /* delay x useconds */
>>  void __udelay(unsigned long usec)
>>  {
>> -	struct s5p_timer *const timer = s5p_get_base_timer();
>> -	unsigned long tmo, tmp, count_value;
>> -
>> -	count_value = readl(&timer->tcntb4);
>> -
>> -	if (usec >= 1000) {
>> -		/*
>> -		 * if "big" number, spread normalization
>> -		 * to seconds
>> -		 * 1. start to normalize for usec to ticks per sec
>> -		 * 2. find number of "ticks" to wait to achieve target
>> -		 * 3. finish normalize.
>> -		 */
>> -		tmo = usec / 1000;
>> -		tmo *= (CONFIG_SYS_HZ * count_value);
>> -		tmo /= 1000;
>> -	} else {
>> -		/* else small number, don't kill it prior to HZ multiply */
>> -		tmo = usec * CONFIG_SYS_HZ * count_value;
>> -		tmo /= (1000 * 1000);
>> -	}
>> -
>> -	/* get current timestamp */
>> -	tmp = get_current_tick();
>> -
>> -	/* if setting this fordward will roll time stamp */
>> -	/* reset "advancing" timestamp to 0, set lastinc value */
>> -	/* else, set advancing stamp wake up time */
>> -	if ((tmo + tmp + 1) < tmp)
>> -		reset_timer_masked();
>> -	else
>> -		tmo += tmp;
>> -
>> -	/* loop till event */
>> -	while (get_current_tick() < tmo)
>> -		;	/* nop */
>> +	unsigned long count_value;
>> +
>> +	count_value = timer_get_us_down();
>> +	while ((int)(count_value - timer_get_us_down()) < (int)usec)
>
>why convert to int?

To avoid side effects of compiler optimizations.

>
>> +		;
>>  }
>>  
>>  void reset_timer_masked(void)
>> @@ -109,30 +109,6 @@ void reset_timer_masked(void)
>>  	gd->arch.tbl = 0;
>>  }
>>  
>> -unsigned long get_timer_masked(void)
>> -{
>> -	struct s5p_timer *const timer = s5p_get_base_timer();
>> -	unsigned long count_value = readl(&timer->tcntb4);
>> -
>> -	return get_current_tick() / count_value;
>> -}
>>   * This function is derived from PowerPC code (read timebase as long long).
>>   * On ARM it just returns the timer value.
>> 
>
>Thanks,
>Minkyu Kang.


Regards,
Akshay Saraswat

^ permalink raw reply	[flat|nested] 19+ messages in thread
[parent not found: <mailman.5850.1364484106.1812.u-boot@lists.denx.de>]

end of thread, other threads:[~2013-03-28 16:41 UTC | newest]

Thread overview: 19+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-02-28 10:59 [U-Boot] [PATCH 0/9 v2] Fix and Re-organise PWM Timer Akshay Saraswat
2013-02-28 10:59 ` [U-Boot] [PATCH 1/9 v2] Exynos: Change get_timer() to work correctly Akshay Saraswat
2013-03-28  5:52   ` Minkyu Kang
2013-02-28 10:59 ` [U-Boot] [PATCH 2/9 v2] Exynos: Add timer_get_us function Akshay Saraswat
2013-02-28 10:59 ` [U-Boot] [PATCH 3/9 v2] Exynos: pwm: Fix two bugs in the exynos pwm configuration code Akshay Saraswat
2013-03-28  5:53   ` Minkyu Kang
2013-02-28 10:59 ` [U-Boot] [PATCH 4/9 v2] Exynos: Avoid a divide by zero by specifying a non-zero period for pwm 4 Akshay Saraswat
2013-03-28  5:53   ` Minkyu Kang
2013-02-28 10:59 ` [U-Boot] [PATCH 5/9 v2] Exynos: Tidy up the pwm_config function in the exynos pwm driver Akshay Saraswat
2013-02-28 10:59 ` [U-Boot] [PATCH 6/9 v2] Exynos: Add peripherial id for pwm Akshay Saraswat
2013-02-28 10:59 ` [U-Boot] [PATCH 7/9 v2] Exynos: clock: Add generic api to get the clk freq Akshay Saraswat
2013-02-28 13:37   ` Simon Glass
2013-03-28  5:52   ` Minkyu Kang
2013-02-28 10:59 ` [U-Boot] [PATCH 8/9 v2] Exynos: clock: Correct pwm source clk selection Akshay Saraswat
2013-02-28 10:59 ` [U-Boot] [PATCH 9/9 v2] Exynos: pwm: Use generic api to get pwm clk freq Akshay Saraswat
2013-02-28 13:40   ` Simon Glass
2013-03-28  5:52   ` Minkyu Kang
  -- strict thread matches above, loose matches on Subject: below --
2013-03-28 14:14 [U-Boot] [PATCH 1/9 v2] Exynos: Change get_timer() to work correctly Akshay Saraswat
     [not found] <mailman.5850.1364484106.1812.u-boot@lists.denx.de>
2013-03-28 16:41 ` Alexei Fedorov

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox