mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Ahmad Fatoum <a.fatoum@pengutronix.de>
To: barebox@lists.infradead.org
Subject: Re: [PATCH 0/6] clk: imx6: work around LDB hang caused by ERR009219
Date: Wed, 3 Apr 2019 11:52:00 +0200	[thread overview]
Message-ID: <bbe97cb3-2e21-2eb2-a727-5b1014d3fd73@pengutronix.de> (raw)
In-Reply-To: <20190401201210.17975-1-a.fatoum@pengutronix.de>

On 1/4/19 22:12, Ahmad Fatoum wrote:
> The erratum already has a Linux workaround. On barebox, it can currently
> happen if there's an appropriate assigned-clock-parents property or the
> reparenting happens (i.e. CONFIG_DRIVER_VIDEO_IMX_IPUV3 is selected on a
> board with the aforementioned CPU/Revision pairs). Linux has removed the
> reparenting along with the bug fix. If barebox did this, boards that
> depended on the reparenting may be broken.

After a discussion with Philipp and Sascha, I'll rearrange this patch series:
Patch 6/6 will wander to the start of the series and will remove the quirk of
the ldb _sometimes_ having pll5_video_div as parent and instead never set it
by default, like the kernel does now but potentially breaking boards that
depended on barebox setting this. 

Now with the need for the funny conditionals removed, I can squash 1/6, 3/6, 4/6,
without much loss of comprehensibility (not that it was particularly comprehensible
before..).


-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

      parent reply	other threads:[~2019-04-03  9:52 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-04-01 20:12 Ahmad Fatoum
2019-04-01 20:12 ` [PATCH 1/6] clk: imx6: provide helper to check if video PLL post dividers work Ahmad Fatoum
2019-04-01 20:12 ` [PATCH 2/6] clk: imx6: Mask mmdc_ch1 handshake for periph2_sel and mmdc_ch1_axi_podf Ahmad Fatoum
2019-04-01 20:12 ` [PATCH 3/6] clk: imx6: Fix procedure to switch the parent of LDB_DI_CLK Ahmad Fatoum
2019-04-01 20:12 ` [PATCH 4/6] clk: imx6: define an enum for ldb mux inputs Ahmad Fatoum
2019-04-01 20:12 ` [PATCH 5/6] clk: imx6: Make the LDB_DI0 and LDB_DI1 clocks read-only Ahmad Fatoum
2019-04-01 20:15 ` [PATCH] clk: imx6: remove quirky clk_set_parent(LDB_diN_sel, pll5_video_div) Ahmad Fatoum
2019-04-01 20:16   ` Ahmad Fatoum
2019-04-03  9:52 ` Ahmad Fatoum [this message]

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=bbe97cb3-2e21-2eb2-a727-5b1014d3fd73@pengutronix.de \
    --to=a.fatoum@pengutronix.de \
    --cc=barebox@lists.infradead.org \
    /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