[U-Boot] [PATCH v2 3/4] imx: mx6: add display of CPU temperature grade in print_cpuinfo()

[Nikolay Dimitrov]

On 05/14/2015 08:11 AM, Tim Harvey wrote:
> When CONFIG_IMX6_THERMAL is defined print the CPU temperature grade info
> along with the current temperature.
>
> Before:
>   CPU:   Temperature 42 C
>
> After:
>   CPU:   Automotive temperature grade (-40C to 125C) at 42C
>   CPU:   Industiral temperature grade (-40C to 105C) at 42C
>   CPU:   Extended Commercial temperature grade (-20C to 105C) at 42C
>
> Signed-off-by: Tim Harvey <tharvey at gateworks.com>
> ---
> v2:
>   - moved display of CPU temperature grade to own patch and combined with the
>     current temperature from the thermal sensor driver
>   - add example output to description
>
> Signed-off-by: Tim Harvey <tharvey at gateworks.com>
> ---
>   arch/arm/imx-common/cpu.c | 27 +++++++++++++++++++++++----
>   1 file changed, 23 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm/imx-common/cpu.c b/arch/arm/imx-common/cpu.c
> index 6b20482..f1b49bb 100644
> --- a/arch/arm/imx-common/cpu.c
> +++ b/arch/arm/imx-common/cpu.c
> @@ -16,6 +16,7 @@
>   #include <asm/arch/clock.h>
>   #include <asm/arch/sys_proto.h>
>   #include <asm/arch/crm_regs.h>
> +#include <imx_thermal.h>
>   #include <ipu_pixfmt.h>
>   #include <thermal.h>
>   #include <sata.h>
> @@ -146,7 +147,7 @@ int print_cpuinfo(void)
>
>   #if defined(CONFIG_MX6) && defined(CONFIG_IMX6_THERMAL)
>   	struct udevice *thermal_dev;
> -	int cpu_tmp, ret;
> +	int cpu_tmp, minc, maxc, ret;
>   #endif
>
>   	cpurev = get_cpu_rev();
> @@ -172,16 +173,34 @@ int print_cpuinfo(void)
>   #endif
>
>   #if defined(CONFIG_MX6) && defined(CONFIG_IMX6_THERMAL)
> +	puts("CPU:   ");
> +	if (!is_cpu_type(MXC_CPU_MX6SX)) {
> +		switch (get_cpu_temp_grade(&minc, &maxc)) {
> +		case TEMP_AUTOMOTIVE:
> +			puts("Automotive temperature grade ");
> +			break;
> +		case TEMP_INDUSTRIAL:
> +			puts("Industrial temperature grade ");
> +			break;
> +		case TEMP_EXTCOMMERCIAL:
> +			puts("Extended Commercial temperature grade ");
> +			break;
> +		default:
> +			puts("Commercial temperature grade ");
> +			break;
> +		}
> +		printf("(%dC to %dC)", minc, maxc);
> +	}
>   	ret = uclass_get_device(UCLASS_THERMAL, 0, &thermal_dev);
>   	if (!ret) {
>   		ret = thermal_get_temp(thermal_dev, &cpu_tmp);
>
>   		if (!ret)
> -			printf("CPU:   Temperature %d C\n", cpu_tmp);
> +			printf(" at %dC\n", cpu_tmp);
>   		else
> -			printf("CPU:   Temperature: invalid sensor data\n");
> +			puts(" - invalid sensor data\n");
>   	} else {
> -		printf("CPU:   Temperature: Can't find sensor device\n");
> +		puts(" - invalid sensor device\n");
>   	}
>   #endif
>
>

Tested-by: Nikolay Dimitrov <picmaster at mail.bg>

Regards,
Nikolay