public inbox for linux-omap@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] OMAP2/3: SDRC: fix walking of SDRC params list
@ 2008-12-15 23:31 Kevin Hilman
  2008-12-15 23:37 ` Paul Walmsley
  0 siblings, 1 reply; 3+ messages in thread
From: Kevin Hilman @ 2008-12-15 23:31 UTC (permalink / raw)
  To: linux-omap; +Cc: Kevin Hilman

This patch fixes a bug where omap2_sdrc_get_params() does not properly
check for a valid table entry before continuing.

Signed-off-by: Kevin Hilman <khilman@deeprootsystems.com>
---
 arch/arm/mach-omap2/sdrc.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/arm/mach-omap2/sdrc.c b/arch/arm/mach-omap2/sdrc.c
index 2a30060..7da6587 100644
--- a/arch/arm/mach-omap2/sdrc.c
+++ b/arch/arm/mach-omap2/sdrc.c
@@ -58,7 +58,7 @@ struct omap_sdrc_params *omap2_sdrc_get_params(unsigned long r)
 
 	sp = sdrc_init_params;
 
-	while (sp->rate != r)
+	while (sp->rate && sp->rate != r)
 		sp++;
 
 	if (!sp->rate)
-- 
1.6.0.3


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] OMAP2/3: SDRC: fix walking of SDRC params list
  2008-12-15 23:31 [PATCH] OMAP2/3: SDRC: fix walking of SDRC params list Kevin Hilman
@ 2008-12-15 23:37 ` Paul Walmsley
  2008-12-16  0:26   ` Tony Lindgren
  0 siblings, 1 reply; 3+ messages in thread
From: Paul Walmsley @ 2008-12-15 23:37 UTC (permalink / raw)
  To: Kevin Hilman; +Cc: linux-omap

On Mon, 15 Dec 2008, Kevin Hilman wrote:

> This patch fixes a bug where omap2_sdrc_get_params() does not properly
> check for a valid table entry before continuing.
> 
> Signed-off-by: Kevin Hilman <khilman@deeprootsystems.com>

Acked-by: Paul Walmsley <paul@pwsan.com>


- Paul

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] OMAP2/3: SDRC: fix walking of SDRC params list
  2008-12-15 23:37 ` Paul Walmsley
@ 2008-12-16  0:26   ` Tony Lindgren
  0 siblings, 0 replies; 3+ messages in thread
From: Tony Lindgren @ 2008-12-16  0:26 UTC (permalink / raw)
  To: Paul Walmsley; +Cc: Kevin Hilman, linux-omap

* Paul Walmsley <paul@booyaka.com> [081215 15:37]:
> On Mon, 15 Dec 2008, Kevin Hilman wrote:
> 
> > This patch fixes a bug where omap2_sdrc_get_params() does not properly
> > check for a valid table entry before continuing.
> > 
> > Signed-off-by: Kevin Hilman <khilman@deeprootsystems.com>
> 
> Acked-by: Paul Walmsley <paul@pwsan.com>

Pushing to l-o tree today.

Tony

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2008-12-16  0:26 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-12-15 23:31 [PATCH] OMAP2/3: SDRC: fix walking of SDRC params list Kevin Hilman
2008-12-15 23:37 ` Paul Walmsley
2008-12-16  0:26   ` Tony Lindgren

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