[U-Boot] [PATCH v2 10/15] cgtqmx6eval: Add thermal support

[Simon Glass]

Hi,

On 13 July 2015 at 13:01, Otavio Salvador <otavio at ossystems.com.br> wrote:
> Add thermal support so that we can see the following message on boot:
>
> CPU:   Industrial temperature grade (-40C to 105C) at 33C
>
> Signed-off-by: Otavio Salvador <otavio at ossystems.com.br>
> ---
>
> Changes in v2: None
>
>  configs/cgtqmx6qeval_defconfig | 3 +++
>  include/configs/cgtqmx6eval.h  | 8 ++++++++
>  2 files changed, 11 insertions(+)

Reviewed-by: Simon Glass <sjg at chromium.org>

While you are in there, do you think you could fix the comments in
thermal.h, which are mostly bogus?

>
> diff --git a/configs/cgtqmx6qeval_defconfig b/configs/cgtqmx6qeval_defconfig
> index e1eb871..6fd29a0 100644
> --- a/configs/cgtqmx6qeval_defconfig
> +++ b/configs/cgtqmx6qeval_defconfig
> @@ -1,6 +1,9 @@
>  CONFIG_ARM=y
>  CONFIG_TARGET_CGTQMX6EVAL=y
>  CONFIG_SYS_EXTRA_OPTIONS="IMX_CONFIG=board/congatec/cgtqmx6eval/imximage.cfg,MX6Q"
> +CONFIG_CMD_NET=y
> +CONFIG_DM=y
> +CONFIG_DM_THERMAL=y
>  # CONFIG_CMD_IMLS is not set
>  # CONFIG_CMD_FLASH is not set
>  # CONFIG_CMD_SETEXPR is not set
> diff --git a/include/configs/cgtqmx6eval.h b/include/configs/cgtqmx6eval.h
> index 31ad366..f031940 100644
> --- a/include/configs/cgtqmx6eval.h
> +++ b/include/configs/cgtqmx6eval.h
> @@ -56,6 +56,14 @@
>  #define CONFIG_PHYLIB
>  #define CONFIG_PHY_ATHEROS
>
> +/* Thermal support */
> +#define CONFIG_IMX6_THERMAL

This could move to Kconfig.

> +
> +#define CONFIG_CMD_FUSE
> +#if defined(CONFIG_CMD_FUSE) || defined(CONFIG_IMX6_THERMAL)
> +#define CONFIG_MXC_OCOTP
> +#endif
> +
>  #define CONFIG_DEFAULT_FDT_FILE "imx6q-congatec.dtb"
>
>  #define CONFIG_EXTRA_ENV_SETTINGS \
> --
> 2.4.5
>

Regards,
Simon