[PATCH] thermal: sandbox: Don't build sandbox thermal driver for sandbox64

[Robert Marko]

On Tue, Oct 18, 2022 at 10:04 AM Sughosh Ganu <sughosh.ganu at linaro.org>
wrote:

> The sandbox64 defconfig does not enable the thermal uclass, but builds
> the thermal sandbox driver. This breaks booting the sandbox64 variant
> with the test device tree. Do not build the driver for the sandbox64
> variant.
>

Hi,

It would probably be better to just enable the driver for sandbox64.
It is weird that sandbox64 is not covered by tests as well.

Regards,
Robert

>
> Signed-off-by: Sughosh Ganu <sughosh.ganu at linaro.org>
> ---
>  drivers/thermal/Makefile | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/drivers/thermal/Makefile b/drivers/thermal/Makefile
> index 8acc7d20cb..04559dbaf4 100644
> --- a/drivers/thermal/Makefile
> +++ b/drivers/thermal/Makefile
> @@ -4,7 +4,9 @@
>  # Author: Nitin Garg <nitin.garg at freescale.com>
>
>  obj-$(CONFIG_DM_THERMAL) += thermal-uclass.o
> +ifeq ($(CONFIG_SANDBOX64),)
>  obj-$(CONFIG_SANDBOX) += thermal_sandbox.o
> +endif
>  obj-$(CONFIG_IMX_THERMAL) += imx_thermal.o
>  obj-$(CONFIG_IMX_SCU_THERMAL) += imx_scu_thermal.o
>  obj-$(CONFIG_TI_DRA7_THERMAL) += ti-bandgap.o
> --
> 2.34.1
>
>

-- 
Robert Marko
Staff Embedded Linux Engineer
Sartura Ltd.
Lendavska ulica 16a
10000 Zagreb, Croatia
Email:  <http://goog_1951392730>robert.marko at sartura.hr
Web: www.sartura.hr