From: akshay.bhat@timesys•com (Akshay Bhat)
To: linux-arm-kernel@lists•infradead.org
Subject: [PATCH v4 3/3] ARM: imx6: Fix procedure to switch the parent of LDB_DI_CLK
Date: Fri, 8 Jul 2016 17:21:21 -0400 [thread overview]
Message-ID: <57801951.30305@timesys.com> (raw)
In-Reply-To: <CAOMZO5C3eu6W2hHF40giuTgRaP8y9KAZOihr+7kPn627vCkVhA@mail.gmail.com>
On 04/04/2016 08:21 PM, Fabio Estevam wrote:
> Hi Philipp,
>
> On Wed, Mar 30, 2016 at 1:18 PM, Philipp Zabel <p.zabel@pengutronix•de> wrote:
>
>>>> 1. NXP 3.10 kernel switches clock source as follows:
>>>> current_parent -> 7 -> 4 -> new_parent
>>>
>>> Yes, we need to follow this requirement.
>>
>> Thanks. Does anybody know why?
>
> The SoC folks say this needs to be done to make sure the glitch is not
> propagated.
>
>> (Does this work for current_parent == 4 and new_parent != 0)?
>
> They say it works for all combinations.
>
>> Is this also true for starting values other than 3?
>
> Yes.
>
Hi Philipp,
NXP has updated the EB821 document and made it public. As per the
updated document your v4 patch matches the NXP recommendation for the
clock switch. The only change I see is allowing to switch to any new
parent (when starting at a value other than 3). Is it possible for you
to resubmit this useful patch after reviewing the EB821 document?
http://www.nxp.com/files/32bit/doc/eng_bulletin/EB821.pdf?fasp=1&WT_TYPE=Engineering%20Bulletins&WT_VENDOR=FREESCALE&WT_FILE_FORMAT=pdf&WT_ASSET=Documentation&fileExt=.pdf
Thanks,
Akshay
next prev parent reply other threads:[~2016-07-08 21:21 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-26 8:51 [PATCH v4 0/3] i.MX6 LDB mux/divider glitch workaround Philipp Zabel
2016-02-26 8:51 ` [PATCH v4 1/3] ARM: imx6: Mask mmdc_ch1 handshake for periph2_sel and mmdc_ch1_axi_podf Philipp Zabel
2016-02-26 8:51 ` [PATCH v4 2/3] ARM: imx6: Make the LDB_DI0 and LDB_DI1 clocks read-only Philipp Zabel
2016-02-26 8:51 ` [PATCH v4 3/3] ARM: imx6: Fix procedure to switch the parent of LDB_DI_CLK Philipp Zabel
2016-03-01 21:41 ` Akshay Bhat
2016-03-23 15:48 ` Akshay Bhat
2016-03-28 18:53 ` Fabio Estevam
2016-03-28 19:26 ` Akshay Bhat
2016-03-28 19:33 ` Fabio Estevam
2016-03-30 16:18 ` Philipp Zabel
2016-04-05 0:21 ` Fabio Estevam
2016-07-08 21:21 ` Akshay Bhat [this message]
2016-07-11 11:14 ` Philipp Zabel
2016-03-28 18:48 ` Fabio Estevam
2016-03-30 16:12 ` Philipp Zabel
2016-03-30 16:02 ` Philipp Zabel
2016-04-12 23:28 ` Fabio Estevam
2016-07-11 11:13 ` Philipp Zabel
2016-04-13 14:48 ` Akshay Bhat
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=57801951.30305@timesys.com \
--to=akshay.bhat@timesys$(echo .)com \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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