From: Bryan O'Donoghue <pure.logic@nexus-software.ie>
To: Leonard Crestez <leonard.crestez@nxp.com>, Peng Fan <peng.fan@nxp.com>
Cc: Aisheng Dong <aisheng.dong@nxp.com>,
Abel Vesa <abel.vesa@nxp.com>, Anson Huang <anson.huang@nxp.com>,
"srinivas.kandagatla@linaro.org" <srinivas.kandagatla@linaro.org>,
dl-linux-imx <linux-imx@nxp.com>,
"kernel@pengutronix.de" <kernel@pengutronix.de>,
Fabio Estevam <fabio.estevam@nxp.com>,
"shawnguo@kernel.org" <shawnguo@kernel.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
"l.stach@pengutronix.de" <l.stach@pengutronix.de>
Subject: Re: [PATCH v4 3/5] nvmem: imx-ocotp: Ensure the RELAX field is non-zero
Date: Tue, 23 Apr 2019 15:58:30 +0100 [thread overview]
Message-ID: <93ff5c71-edca-4ef1-7a42-fe5bfd4ced0d@nexus-software.ie> (raw)
In-Reply-To: <AM0PR04MB64347666742FE924AB9C3617EE230@AM0PR04MB6434.eurprd04.prod.outlook.com>
On 23/04/2019 15:42, Leonard Crestez wrote:
>> 17 nanoseconds is literally the defintion of a magic number which
>> _ought_ to have been defined in cycles from the beginning
>
> It makes sense for physical hardware requirements really to be measured
> in seconds. It's just the OCOTP block can only measure time based on the
> bus clock input so it wants the driver to convert "real time" to "ipg
> clock periods".
Exactly. The block only understands clocks, not absolute time values.
If you increase your ipg_clock to 133 Mhz the initial 17 nanosecond
value breaks again or if you reduce your ipg_clk to 6.6 Mhz, again that
value breaks.
But if you specify the right _ratio_ between RELAX and STROBE/READ
cycles - you can push your clock in any direction you like.
Because the block 'ticks over' so to speak, on clock edges.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2019-04-23 14:58 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-04-22 15:04 [PATCH v4 0/5] Add i.MX8MM support Bryan O'Donoghue
2019-04-22 15:04 ` [PATCH v4 1/5] nvmem: imx-ocotp: Elongate OCOTP_CTRL ADDR field to eight bits Bryan O'Donoghue
2019-04-22 15:04 ` [PATCH v4 2/5] nvmem: imx-ocotp: Ensure WAIT bits are preserved when setting timing Bryan O'Donoghue
2019-04-22 15:04 ` [PATCH v4 3/5] nvmem: imx-ocotp: Ensure the RELAX field is non-zero Bryan O'Donoghue
2019-04-23 8:48 ` Leonard Crestez
2019-04-23 14:07 ` Bryan O'Donoghue
2019-04-23 14:42 ` Leonard Crestez
2019-04-23 14:58 ` Bryan O'Donoghue [this message]
2019-04-22 15:04 ` [PATCH v4 4/5] nvmem: imx-ocotp: Add i.MX8MM support Bryan O'Donoghue
2019-04-22 19:06 ` Leonard Crestez
2019-04-22 15:05 ` [PATCH v4 5/5] dt-bindings: imx-ocotp: Add i.MX8MM compatible Bryan O'Donoghue
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=93ff5c71-edca-4ef1-7a42-fe5bfd4ced0d@nexus-software.ie \
--to=pure.logic@nexus-software.ie \
--cc=abel.vesa@nxp.com \
--cc=aisheng.dong@nxp.com \
--cc=anson.huang@nxp.com \
--cc=fabio.estevam@nxp.com \
--cc=kernel@pengutronix.de \
--cc=l.stach@pengutronix.de \
--cc=leonard.crestez@nxp.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-imx@nxp.com \
--cc=peng.fan@nxp.com \
--cc=shawnguo@kernel.org \
--cc=srinivas.kandagatla@linaro.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 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.