From: Stefano Babic <sbabic@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/2] i.MX: mxc_ipuv3_fb: add ipuv3_fb_shutdown() routine to stop IPU before bootm
Date: Sun, 23 Sep 2012 17:56:42 +0200 [thread overview]
Message-ID: <505F313A.1080705@denx.de> (raw)
In-Reply-To: <CAOMZO5BezBv+hpFBTaEwoeOuz0g8ncEGon6KyAqgx4+g+oz-Pw@mail.gmail.com>
On 22/09/2012 16:37, Fabio Estevam wrote:
> On Sat, Sep 22, 2012 at 10:42 AM, Otavio Salvador
> <otavio@ossystems.com.br> wrote:
>> Hello Eric,
>>
>> On Fri, Sep 21, 2012 at 5:36 PM, Eric Nelson
>> <eric.nelson@boundarydevices.com> wrote:
>>> Signed-off-by: Eric Nelson <eric.nelson@boundarydevices.com>
>>
>> Did you test it in mx5 too? We seem to need to handle it in mx5 too as
>> we had hungs in FSL kernel when using framebuffer in U-Boot. We're
>> using a patch in kernel for workaround it but it seems your fix does
>> what is need.
>
> I have just tested Eric's series on a mx53loco and it does fix the
> kernel hang issue.
>
> I made some comments on this series and hopefully Eric's v2 can get
> into 2012.10, since this is a bug fix.
Ok, I am waiting for V2 and I will push it.
Anyway, a question about the issue. It seems to me that it is not
possible with IPUV3 (I have not tested myself, so my question) to get
the u-boot splashscreen displayed on the LCD until the kernel has
finished to boot. This could be possible (and it is possible on other
SOC) if the IPU is loaded as module instead of statically linked to the
kernel, and if the kernel does not touch the IPU setup. This means also
that it should not disable the clocks used by U-Boot for the IPU.
But I understand from your patch that this way is not possible on
iMX53/MX6, and IPU must be always disabled. Is this correct ?
Best regards,
Stefano
--
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
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
=====================================================================
next prev parent reply other threads:[~2012-09-23 15:56 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-09-21 16:38 [U-Boot] Disabling video in bootm (cleanup_before_linux) Eric Nelson
2012-09-21 17:22 ` Anatolij Gustschin
2012-09-21 17:44 ` Eric Nelson
2012-09-22 13:35 ` Otavio Salvador
2012-09-22 15:35 ` Eric Nelson
2012-09-22 15:37 ` Otavio Salvador
2012-09-21 20:36 ` [U-Boot] [PATCH 0/2] i.MX: ipufb: shut down IPU frame buffer before booting O/S Eric Nelson
2012-09-21 20:36 ` [U-Boot] [PATCH 1/2] i.MX: mxc_ipuv3_fb: add ipuv3_fb_shutdown() routine to stop IPU before bootm Eric Nelson
2012-09-22 13:42 ` Otavio Salvador
2012-09-22 14:37 ` Fabio Estevam
2012-09-22 14:39 ` Otavio Salvador
2012-09-23 15:56 ` Stefano Babic [this message]
2012-09-23 16:32 ` Dirk Behme
2012-09-23 16:46 ` Eric Nelson
2012-09-23 16:59 ` Fabio Estevam
2012-09-23 17:27 ` Stefano Babic
2012-09-23 18:13 ` [U-Boot] Displays on i.MX (was Re: [PATCH 1/2]) Eric Nelson
2012-09-23 20:12 ` [U-Boot] [PATCH] i2c_probe: update for use in scripting Eric Nelson
2012-10-16 3:58 ` Heiko Schocher
2012-09-23 23:02 ` [U-Boot] [PATCH 0/3] i.MX6: implement hdmidet command Eric Nelson
2012-09-23 23:02 ` [U-Boot] [PATCH 1/3] i.MX6: add HDMI transmitter register declarations from kernel WIP Eric Nelson
2012-09-23 23:02 ` [U-Boot] [PATCH 2/3] i.MX6: Add hdmidet command to detect attached HDMI monitor Eric Nelson
2012-09-23 23:02 ` [U-Boot] [PATCH 3/3] i.MX6: mx6qsabrelite: Include hdmidet command Eric Nelson
2012-10-02 8:28 ` [U-Boot] [PATCH 0/3] i.MX6: implement " Stefano Babic
2012-09-23 18:15 ` [U-Boot] [PATCH 1/2] i.MX: mxc_ipuv3_fb: add ipuv3_fb_shutdown() routine to stop IPU before bootm Sascha Hauer
2012-09-23 17:30 ` [U-Boot] [PATCH V2 0/2] i.MX: ipufb: shut down IPU frame buffer before booting Eric Nelson
2012-09-23 17:30 ` [U-Boot] [PATCH V2 1/2] i.MX: mxc_ipuv3_fb: add ipuv3_fb_shutdown() routine to stop IPU frame buffer Eric Nelson
2012-09-24 8:50 ` Stefano Babic
2012-09-23 17:30 ` [U-Boot] [PATCH V2 2/2] i.MX: shut down video before launch of O/S Eric Nelson
2012-09-24 8:50 ` Stefano Babic
2012-09-21 20:36 ` [U-Boot] [PATCH " Eric Nelson
2012-09-21 22:41 ` Fabio Estevam
2012-09-21 23:02 ` Eric Nelson
-- strict thread matches above, loose matches on Subject: below --
2012-09-21 20:56 [U-Boot] [PATCH] i.MX6: set drive strength for parallel RGB pads Eric Nelson
2012-09-21 20:56 ` [U-Boot] [PATCH 1/2] i.MX: mxc_ipuv3_fb: add ipuv3_fb_shutdown() routine to stop IPU before bootm Eric Nelson
2012-09-21 21:00 ` Eric Nelson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=505F313A.1080705@denx.de \
--to=sbabic@denx.de \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox