From: Dirk Behme <dirk.behme@gmail.com>
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 18:32:58 +0200 [thread overview]
Message-ID: <505F39BA.90500@gmail.com> (raw)
In-Reply-To: <505F313A.1080705@denx.de>
On 23.09.2012 17:56, Stefano Babic wrote:
> 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 ?
I'm no expert on this, so I might be wrong and people will correct me:
No, not always. It's my understanding that at the *moment* we have to
disable the IPU in U-Boot before jumping into the kernel due to a bug
in Sascha's kernel IPU driver:
http://lists.infradead.org/pipermail/linux-arm-kernel/2012-September/121980.html
http://lists.infradead.org/pipermail/linux-arm-kernel/2012-September/122100.html
So from my understanding, this isn't a SOC limitation. But we have to
do it as long as the kernel's IPU driver isn't fixed.
Sorry if this is wrong ;)
Best regards
Dirk
next prev parent reply other threads:[~2012-09-23 16:32 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
2012-09-23 16:32 ` Dirk Behme [this message]
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=505F39BA.90500@gmail.com \
--to=dirk.behme@gmail.com \
--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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.