[U-Boot] [PATCH] imx6: disable clock command and print_cpuinfo code in SPL

Stefano Babic
2017-09-20T13:04:40Z

Hi Fabio, Anatolji,

just to save time : this is the list of boards that are affected by this
problem.

cgtqmx6eval
wandboard
colibri_imx6
apalis_imx6
mx6cuboxi
mx6sabresd
imx6qdl_icore_rqs

A good solution is also if the board maintainers (I CCed all of them)
could fix this and I will apply afterwards Anatolji's patch.

Best regards,
Stefano

On 20/09/2017 13:22, Anatolij Gustschin wrote:
> Hi Fabio,
>
> On Wed, 20 Sep 2017 00:18:52 -0300
> Fabio Estevam festevam at gmail.com wrote:
>
>> Hi Stefano,
>>
>> On Tue, Sep 19, 2017 at 4:54 AM, Stefano Babic <sbabic@denx.de> wrote:
>>
>>> Reason is clear: boards have in SPL a setup_display() that calls
>>> enable_ipu_clock. But it makes no sense to have IPU support in SPL.
>>
>> Good catch, thanks!
>>
>> Anatolij,
>>
>> Do you plan to remove IPU code from SPL?
>
> I think we can do it by
>
> diff --git a/include/configs/wandboard.h b/include/configs/wandboard.h
> index 3ba4c29..b0f317b 100644
> --- a/include/configs/wandboard.h
> +++ b/include/configs/wandboard.h
> @@ -63,6 +63,7 @@
> #define CONFIG_PHY_ATHEROS
>
> /* Framebuffer */
> +#if !defined(CONFIG_SPL_BUILD)
> #ifdef CONFIG_VIDEO
> #define CONFIG_VIDEO_IPUV3
> #define CONFIG_VIDEO_BMP_RLE8
> @@ -74,6 +75,7 @@
> #define CONFIG_IMX_HDMI
> #define CONFIG_IMX_VIDEO_SKIP
> #endif
> +#endif
>
> #define CONFIG_ENV_VARS_UBOOT_RUNTIME_CONFIG
> #define CONFIG_EXTRA_ENV_SETTINGS \
>
> I'll try to prepare a patch for other board configs later this week.
>
> Thanks,
>
> Anatolij
>

--
=====================================================================
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================