All of lore.kernel.org
 help / color / mirror / Atom feed
From: Simon Glass <sjg@chromium.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 9/9] Exynos: pwm: Use generic api to get pwm clk freq
Date: Wed, 27 Feb 2013 17:36:27 -0800	[thread overview]
Message-ID: <CAPnjgZ2nnkdMbHS3L2B75SziODNaON8ZTryyAX5mGez9qP_4Wg@mail.gmail.com> (raw)
In-Reply-To: <1361959366-27634-10-git-send-email-akshay.s@samsung.com>

On Wed, Feb 27, 2013 at 2:02 AM, Akshay Saraswat <akshay.s@samsung.com> wrote:
> Use generic api to get the pwm clock frequency
>
> TEST=sf probe 1:0; time sf read 40008000 0 1000
> Try with different numbers of bytes and see that sane values are obtained
> Build and boot U-boot with this patch, backlight works properly.
>
> Signed-off-by: Padmavathi Venna <padma.v@samsung.com>
> Signed-off-by: Akshay Saraswat <akshay.s@samsung.com>

Acked-by: Simon Glass <sjg@chromium.org>

> ---
>  arch/arm/cpu/armv7/s5p-common/pwm.c | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/arch/arm/cpu/armv7/s5p-common/pwm.c b/arch/arm/cpu/armv7/s5p-common/pwm.c
> index 6f401b8..06e0351 100644
> --- a/arch/arm/cpu/armv7/s5p-common/pwm.c
> +++ b/arch/arm/cpu/armv7/s5p-common/pwm.c
> @@ -28,6 +28,7 @@
>  #include <asm/io.h>
>  #include <asm/arch/pwm.h>
>  #include <asm/arch/clk.h>
> +#include <asm/arch/periph.h>
>
>  int pwm_enable(int pwm_id)
>  {
> @@ -60,7 +61,7 @@ static unsigned long pwm_calc_tin(int pwm_id, unsigned long freq)
>         unsigned long tin_parent_rate;
>         unsigned int div;
>
> -       tin_parent_rate = get_pwm_clk();
> +       tin_parent_rate = clock_get_periph_rate(PERIPH_ID_PWM0);
>
>         for (div = 2; div <= 16; div *= 2) {
>                 if ((tin_parent_rate / (div << 16)) < freq)
> @@ -165,8 +166,8 @@ int pwm_init(int pwm_id, int div, int invert)
>                 ticks_per_period = -1UL;
>         } else {
>                 const unsigned long pwm_hz = 1000;
> -               unsigned long timer_rate_hz = get_pwm_clk() /
> -                       ((prescaler + 1) * (1 << div));
> +               unsigned long timer_rate_hz = clock_get_periph_rate(
> +                       PERIPH_ID_PWM0) / ((prescaler + 1) * (1 << div));
>
>                 ticks_per_period = timer_rate_hz / pwm_hz;
>         }
> --
> 1.8.0
>

  reply	other threads:[~2013-02-28  1:36 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-27 10:02 [U-Boot] [PATCH 0/9] Fix and Re-organise PWM Timer Akshay Saraswat
2013-02-27 10:02 ` [U-Boot] [PATCH 1/9] Exynos: Change get_timer() to work correctly Akshay Saraswat
2013-02-28  1:26   ` Simon Glass
2013-02-27 10:02 ` [U-Boot] [PATCH 2/9] Exynos: Add timer_get_us function Akshay Saraswat
2013-02-28  1:27   ` Simon Glass
2013-02-27 10:02 ` [U-Boot] [PATCH 3/9] Exynos: pwm: Fix two bugs in the exynos pwm configuration code Akshay Saraswat
2013-02-28  1:29   ` Simon Glass
2013-02-27 10:02 ` [U-Boot] [PATCH 4/9] Exynos: Avoid a divide by zero by specifying a non-zero period for pwm 4 Akshay Saraswat
2013-02-28  1:29   ` Simon Glass
2013-02-27 10:02 ` [U-Boot] [PATCH 5/9] Exynos: Tidy up the pwm_config function in the exynos pwm driver Akshay Saraswat
2013-02-28  1:30   ` Simon Glass
2013-02-27 10:02 ` [U-Boot] [PATCH 6/9] Exynos: Add peripherial id for pwm Akshay Saraswat
2013-02-28  1:31   ` Simon Glass
2013-02-27 10:02 ` [U-Boot] [PATCH 7/9] Exynos: clock: Add generic api to get the clk freq Akshay Saraswat
2013-02-28  1:35   ` Simon Glass
2013-02-27 10:02 ` [U-Boot] [PATCH 8/9] Exynos: clock: Correct pwm source clk selection Akshay Saraswat
2013-02-27 10:02 ` [U-Boot] [PATCH 9/9] Exynos: pwm: Use generic api to get pwm clk freq Akshay Saraswat
2013-02-28  1:36   ` Simon Glass [this message]
2013-02-28  5:33   ` Rajeshwari Birje

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAPnjgZ2nnkdMbHS3L2B75SziODNaON8ZTryyAX5mGez9qP_4Wg@mail.gmail.com \
    --to=sjg@chromium.org \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.