[U-Boot] [PATCH] ARMV7: Overo: Automatically set clock rate to maximum if mpurate env variable is "auto"

[Nishanth Menon]

just a minor crib - $subject length is around 88 characters, it'd look 
better with around 50 character length.

Steve Sakoman wrote, on 11/05/2010 10:59 PM:
> The maximum clock rate for the OMAP3 processors on Overo depends on the
> processor type and revision.  This patch sets the clock rate to the
> spec sheet maximum if the mpurate environment variable is set to
> "auto".  Otherwise it passes the mpurate variable unchanged on the
> kernel command line.
>
> Signed-off-by: Steve Sakoman<steve.sakoman at linaro.org>
> ---
>
> diff --git a/board/overo/overo.c b/board/overo/overo.c
> index f917e40..3c9e4a6 100644
> --- a/board/overo/overo.c
> +++ b/board/overo/overo.c
> @@ -281,6 +281,22 @@ int misc_init_r(void)
>
>   	dieid_num_r();
>
> +	if (strcmp(getenv("mpurate"), "auto") == 0)
> +		switch (get_cpu_family()) {
> +		case CPU_OMAP34XX:
> +			if ((get_cpu_rev()>= CPU_3XX_ES31)&&
> +			    (get_sku_id() == SKUID_CLK_720MHZ))
> +				setenv("mpurate", "720");
> +			else
> +				setenv("mpurate", "600");
> +			break;
> +		case CPU_OMAP36XX:
> +			setenv("mpurate", "720");
> +			break;
> +		default:
> +			setenv("mpurate", "500");
> +		}
> +
>   	return 0;
>   }
>
> diff --git a/include/configs/omap3_overo.h b/include/configs/omap3_overo.h
> index 79a5b85..dbdfd9a 100644
> --- a/include/configs/omap3_overo.h
> +++ b/include/configs/omap3_overo.h
> @@ -156,7 +156,7 @@
>   #define CONFIG_EXTRA_ENV_SETTINGS \
>   	"loadaddr=0x82000000\0" \
>   	"console=ttyS2,115200n8\0" \
> -	"mpurate=500\0" \
> +	"mpurate=auto\0" \
>   	"vram=12M\0" \
>   	"dvimode=1024x768MR-16 at 60\0" \
>   	"defaultdisplay=dvi\0" \

yep, this does look like a nice way to do it. thanks.

-- 
Regards,
Nishanth Menon