From mboxrd@z Thu Jan  1 00:00:00 1970
From: Max Krummenacher <max.krummenacher@toradex.com>
Date: Thu, 21 Sep 2017 09:10:32 +0000
Subject: [U-Boot] [PATCH] imx6: disable clock command and print_cpuinfo
 code in SPL
In-Reply-To: <08dc3fee-b03d-bd37-d3ae-692ef77c5e03@denx.de>
References: <1503949586-24936-1-git-send-email-agust@denx.de>
 <58344258-849d-2eb0-39f4-444178f2953e@denx.de>
 <CAOMZO5AfLjF5_Xq8zp6suzoT7=0cHvAu=-_0T0MKaR40y4-L1Q@mail.gmail.com>
 <20170920132234.636e4e5e@crub>
 <08dc3fee-b03d-bd37-d3ae-692ef77c5e03@denx.de>
Message-ID: <1505985031.1653.1.camel@toradex.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi

On Wed, 2017-09-20 at 15:04 +0200, Stefano Babic wrote:
> 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

I will prepare a patch for these two.
Max

>       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
> > 
> 
>