[PATCH] OMAP2/3: SDRC: fix walking of SDRC params list

[PATCH] OMAP2/3: SDRC: fix walking of SDRC params list

[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

Re: [PATCH] OMAP2/3: SDRC: fix walking of SDRC params list

[Paul Walmsley]

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

Re: [PATCH] OMAP2/3: SDRC: fix walking of SDRC params list

[Tony Lindgren]

* 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