All of lore.kernel.org
 help / color / mirror / Atom feed
From: Lokesh Vutla <lokeshvutla@ti.com>
To: Lennart Sorensen <lsorense@csclub.uwaterloo.ca>
Cc: <linux-omap@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	"Kristo, Tero" <t-kristo@ti.com>, Nishanth Menon <nm@ti.com>,
	Sekhar Nori <nsekhar@ti.com>
Subject: Re: [PATCH 1/2] ARM: omap5/dra7xx: Fix frequency typos.
Date: Tue, 16 Dec 2014 17:08:21 +0530	[thread overview]
Message-ID: <549019AD.9050807@ti.com> (raw)
In-Reply-To: <5f7d841d41f6485022e07a22c332d3c8ea985bfa.1418421100.git.lsorense@csclub.uwaterloo.ca>

Hi Len,
On Saturday 13 December 2014 03:38 AM, Lennart Sorensen wrote:
> The switch statement of the possible list of SYSCLK1 frequencies is
> missing a 0 in 4 out of the 7 frequencies.
Indeed a good catch !!
Reviewed-by: Lokesh Vutla <lokeshvutla@ti.com>

Thanks and regards,
Lokesh
> 
> Signed-off-by: Len Sorensen <lsorense@csclub.uwaterloo.ca>
> ---
>  arch/arm/mach-omap2/timer.c |    8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm/mach-omap2/timer.c b/arch/arm/mach-omap2/timer.c
> index 4f61148..fb0cb2b 100644
> --- a/arch/arm/mach-omap2/timer.c
> +++ b/arch/arm/mach-omap2/timer.c
> @@ -513,11 +513,11 @@ static void __init realtime_counter_init(void)
>  	rate = clk_get_rate(sys_clk);
>  	/* Numerator/denumerator values refer TRM Realtime Counter section */
>  	switch (rate) {
> -	case 1200000:
> +	case 12000000:
>  		num = 64;
>  		den = 125;
>  		break;
> -	case 1300000:
> +	case 13000000:
>  		num = 768;
>  		den = 1625;
>  		break;
> @@ -529,11 +529,11 @@ static void __init realtime_counter_init(void)
>  		num = 192;
>  		den = 625;
>  		break;
> -	case 2600000:
> +	case 26000000:
>  		num = 384;
>  		den = 1625;
>  		break;
> -	case 2700000:
> +	case 27000000:
>  		num = 256;
>  		den = 1125;
>  		break;
> 


WARNING: multiple messages have this Message-ID (diff)
From: Lokesh Vutla <lokeshvutla@ti.com>
To: Lennart Sorensen <lsorense@csclub.uwaterloo.ca>
Cc: linux-omap@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org, "Kristo,
	Tero" <t-kristo@ti.com>, Nishanth Menon <nm@ti.com>,
	Sekhar Nori <nsekhar@ti.com>
Subject: Re: [PATCH 1/2] ARM: omap5/dra7xx: Fix frequency typos.
Date: Tue, 16 Dec 2014 17:08:21 +0530	[thread overview]
Message-ID: <549019AD.9050807@ti.com> (raw)
In-Reply-To: <5f7d841d41f6485022e07a22c332d3c8ea985bfa.1418421100.git.lsorense@csclub.uwaterloo.ca>

Hi Len,
On Saturday 13 December 2014 03:38 AM, Lennart Sorensen wrote:
> The switch statement of the possible list of SYSCLK1 frequencies is
> missing a 0 in 4 out of the 7 frequencies.
Indeed a good catch !!
Reviewed-by: Lokesh Vutla <lokeshvutla@ti.com>

Thanks and regards,
Lokesh
> 
> Signed-off-by: Len Sorensen <lsorense@csclub.uwaterloo.ca>
> ---
>  arch/arm/mach-omap2/timer.c |    8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm/mach-omap2/timer.c b/arch/arm/mach-omap2/timer.c
> index 4f61148..fb0cb2b 100644
> --- a/arch/arm/mach-omap2/timer.c
> +++ b/arch/arm/mach-omap2/timer.c
> @@ -513,11 +513,11 @@ static void __init realtime_counter_init(void)
>  	rate = clk_get_rate(sys_clk);
>  	/* Numerator/denumerator values refer TRM Realtime Counter section */
>  	switch (rate) {
> -	case 1200000:
> +	case 12000000:
>  		num = 64;
>  		den = 125;
>  		break;
> -	case 1300000:
> +	case 13000000:
>  		num = 768;
>  		den = 1625;
>  		break;
> @@ -529,11 +529,11 @@ static void __init realtime_counter_init(void)
>  		num = 192;
>  		den = 625;
>  		break;
> -	case 2600000:
> +	case 26000000:
>  		num = 384;
>  		den = 1625;
>  		break;
> -	case 2700000:
> +	case 27000000:
>  		num = 256;
>  		den = 1125;
>  		break;
> 

WARNING: multiple messages have this Message-ID (diff)
From: lokeshvutla@ti.com (Lokesh Vutla)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/2] ARM: omap5/dra7xx: Fix frequency typos.
Date: Tue, 16 Dec 2014 17:08:21 +0530	[thread overview]
Message-ID: <549019AD.9050807@ti.com> (raw)
In-Reply-To: <5f7d841d41f6485022e07a22c332d3c8ea985bfa.1418421100.git.lsorense@csclub.uwaterloo.ca>

Hi Len,
On Saturday 13 December 2014 03:38 AM, Lennart Sorensen wrote:
> The switch statement of the possible list of SYSCLK1 frequencies is
> missing a 0 in 4 out of the 7 frequencies.
Indeed a good catch !!
Reviewed-by: Lokesh Vutla <lokeshvutla@ti.com>

Thanks and regards,
Lokesh
> 
> Signed-off-by: Len Sorensen <lsorense@csclub.uwaterloo.ca>
> ---
>  arch/arm/mach-omap2/timer.c |    8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm/mach-omap2/timer.c b/arch/arm/mach-omap2/timer.c
> index 4f61148..fb0cb2b 100644
> --- a/arch/arm/mach-omap2/timer.c
> +++ b/arch/arm/mach-omap2/timer.c
> @@ -513,11 +513,11 @@ static void __init realtime_counter_init(void)
>  	rate = clk_get_rate(sys_clk);
>  	/* Numerator/denumerator values refer TRM Realtime Counter section */
>  	switch (rate) {
> -	case 1200000:
> +	case 12000000:
>  		num = 64;
>  		den = 125;
>  		break;
> -	case 1300000:
> +	case 13000000:
>  		num = 768;
>  		den = 1625;
>  		break;
> @@ -529,11 +529,11 @@ static void __init realtime_counter_init(void)
>  		num = 192;
>  		den = 625;
>  		break;
> -	case 2600000:
> +	case 26000000:
>  		num = 384;
>  		den = 1625;
>  		break;
> -	case 2700000:
> +	case 27000000:
>  		num = 256;
>  		den = 1125;
>  		break;
> 

  reply	other threads:[~2014-12-16 11:40 UTC|newest]

Thread overview: 60+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-12 22:08 [PATCH 0/2] ARM: omap5/dra7xx counter frequency fixes Lennart Sorensen
2014-12-12 22:08 ` Lennart Sorensen
2014-12-12 22:08 ` [PATCH 1/2] ARM: omap5/dra7xx: Fix frequency typos Lennart Sorensen
2014-12-12 22:08   ` Lennart Sorensen
2014-12-16 11:38   ` Lokesh Vutla [this message]
2014-12-16 11:38     ` Lokesh Vutla
2014-12-16 11:38     ` Lokesh Vutla
2014-12-16 14:06   ` Nishanth Menon
2014-12-16 14:06     ` Nishanth Menon
2014-12-16 14:06     ` Nishanth Menon
2014-12-16 16:39     ` Lennart Sorensen
2014-12-16 16:39       ` Lennart Sorensen
2014-12-12 22:08 ` [PATCH 2/2] ARM: omap5/dra7xx: Fix counter frequency drift for AM572x errata i856 Lennart Sorensen
2014-12-12 22:08   ` Lennart Sorensen
2014-12-14  4:45   ` Lennart Sorensen
2014-12-14  4:45     ` Lennart Sorensen
2014-12-16 11:35     ` Lokesh Vutla
2014-12-16 11:35       ` Lokesh Vutla
2014-12-16 11:35       ` Lokesh Vutla
2014-12-16 14:58       ` Nishanth Menon
2014-12-16 14:58         ` Nishanth Menon
2014-12-16 14:58         ` Nishanth Menon
2014-12-16 16:36         ` Lennart Sorensen
2014-12-16 16:36           ` Lennart Sorensen
2014-12-16 18:59           ` Nishanth Menon
2014-12-16 18:59             ` Nishanth Menon
2014-12-16 18:59             ` Nishanth Menon
2014-12-16 19:27             ` Lennart Sorensen
2014-12-16 19:27               ` Lennart Sorensen
2014-12-16 19:33               ` Nishanth Menon
2014-12-16 19:33                 ` Nishanth Menon
2014-12-16 19:33                 ` Nishanth Menon
2014-12-17 13:21         ` Tero Kristo
2014-12-17 13:21           ` Tero Kristo
2014-12-17 13:21           ` Tero Kristo
2014-12-17 14:55           ` Lennart Sorensen
2014-12-17 14:55             ` Lennart Sorensen
2014-12-17 15:22             ` Nishanth Menon
2014-12-17 15:22               ` Nishanth Menon
2014-12-17 15:22               ` Nishanth Menon
2014-12-17 15:27               ` Lennart Sorensen
2014-12-17 15:27                 ` Lennart Sorensen
2014-12-17 15:45                 ` Tero Kristo
2014-12-17 15:45                   ` Tero Kristo
2014-12-17 15:45                   ` Tero Kristo
2014-12-17 15:49                   ` Lennart Sorensen
2014-12-17 15:49                     ` Lennart Sorensen
2014-12-17 15:53                   ` Nishanth Menon
2014-12-17 15:53                     ` Nishanth Menon
2014-12-17 15:53                     ` Nishanth Menon
2014-12-17 15:56                     ` Tero Kristo
2014-12-17 15:56                       ` Tero Kristo
2014-12-17 15:56                       ` Tero Kristo
2014-12-16 16:16       ` Lennart Sorensen
2014-12-16 16:16         ` Lennart Sorensen
2014-12-16 19:56         ` Nishanth Menon
2014-12-16 19:56           ` Nishanth Menon
2014-12-16 19:56           ` Nishanth Menon
2014-12-16 19:58           ` Lennart Sorensen
2014-12-16 19:58             ` Lennart Sorensen

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=549019AD.9050807@ti.com \
    --to=lokeshvutla@ti.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=lsorense@csclub.uwaterloo.ca \
    --cc=nm@ti.com \
    --cc=nsekhar@ti.com \
    --cc=t-kristo@ti.com \
    /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.