* [PATCH v6 1/5] nvmem: imx-ocotp: Elongate OCOTP_CTRL ADDR field to eight bits
2019-05-03 16:53 [RESEND PATCH v6 0/5] Add i.MX8MM OCOTP support Bryan O'Donoghue
@ 2019-05-03 16:53 ` Bryan O'Donoghue
2019-05-03 16:53 ` [PATCH v6 2/5] nvmem: imx-ocotp: Ensure WAIT bits are preserved when setting timing Bryan O'Donoghue
` (4 subsequent siblings)
5 siblings, 0 replies; 11+ messages in thread
From: Bryan O'Donoghue @ 2019-05-03 16:53 UTC (permalink / raw)
To: gregkh, l.stach, peng.fan, shawnguo, srinivas.kandagatla,
leonard.crestez
Cc: devel, aisheng.dong, abel.vesa, anson.huang, linux-imx, kernel,
fabio.estevam, Bryan O'Donoghue, linux-arm-kernel
i.MX6 defines OCOTP_CTRLn:ADDR as seven bit address-field with a one bit
RSVD0 field, i.MX7 defines OCOTP_CTRLn:ADDR as a four bit address-field
with a four bit RSVD0 field.
i.MX8 defines the OCOTP_CTRLn:ADDR bit-field as a full range eight bits.
i.MX6 and i.MX7 should return zero for their respective RSVD0 bits and
ignore a write-back of zero where i.MX8 will make use of the full range.
This patch expands the bit-field definition for all users to eight bits,
which is safe due to RSVD0 being a no-op for the i.MX6 and i.MX7.
Signed-off-by: Bryan O'Donoghue <pure.logic@nexus-software.ie>
Reviewed-by: Leonard Crestez <leonard.crestez@nxp.com>
---
drivers/nvmem/imx-ocotp.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/nvmem/imx-ocotp.c b/drivers/nvmem/imx-ocotp.c
index 4cf7b61e4bf5..6600c4ddeb51 100644
--- a/drivers/nvmem/imx-ocotp.c
+++ b/drivers/nvmem/imx-ocotp.c
@@ -45,7 +45,7 @@
#define IMX_OCOTP_ADDR_DATA2 0x0040
#define IMX_OCOTP_ADDR_DATA3 0x0050
-#define IMX_OCOTP_BM_CTRL_ADDR 0x0000007F
+#define IMX_OCOTP_BM_CTRL_ADDR 0x000000FF
#define IMX_OCOTP_BM_CTRL_BUSY 0x00000100
#define IMX_OCOTP_BM_CTRL_ERROR 0x00000200
#define IMX_OCOTP_BM_CTRL_REL_SHADOWS 0x00000400
--
2.21.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 11+ messages in thread* [PATCH v6 2/5] nvmem: imx-ocotp: Ensure WAIT bits are preserved when setting timing
2019-05-03 16:53 [RESEND PATCH v6 0/5] Add i.MX8MM OCOTP support Bryan O'Donoghue
2019-05-03 16:53 ` [PATCH v6 1/5] nvmem: imx-ocotp: Elongate OCOTP_CTRL ADDR field to eight bits Bryan O'Donoghue
@ 2019-05-03 16:53 ` Bryan O'Donoghue
2019-05-03 16:53 ` [PATCH v6 3/5] nvmem: imx-ocotp: Change TIMING calculation to u-boot algorithm Bryan O'Donoghue
` (3 subsequent siblings)
5 siblings, 0 replies; 11+ messages in thread
From: Bryan O'Donoghue @ 2019-05-03 16:53 UTC (permalink / raw)
To: gregkh, l.stach, peng.fan, shawnguo, srinivas.kandagatla,
leonard.crestez
Cc: devel, aisheng.dong, abel.vesa, anson.huang, linux-imx, kernel,
fabio.estevam, Bryan O'Donoghue, linux-arm-kernel
The i.MX6 and i.MX8 both have a bit-field spanning bits 27:22 called the
WAIT field.
The WAIT field according to the documentation for both parts "specifies
time interval between auto read and write access in one time program. It is
given in number of ipg_clk periods."
This patch ensures that the relevant field is read and written back to the
timing register.
Fixes: 0642bac7da42 ("nvmem: imx-ocotp: add write support")
Signed-off-by: Bryan O'Donoghue <pure.logic@nexus-software.ie>
Reviewed-by: Leonard Crestez <leonard.crestez@nxp.com>
---
drivers/nvmem/imx-ocotp.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/nvmem/imx-ocotp.c b/drivers/nvmem/imx-ocotp.c
index 6600c4ddeb51..85a7d0da3abb 100644
--- a/drivers/nvmem/imx-ocotp.c
+++ b/drivers/nvmem/imx-ocotp.c
@@ -189,7 +189,8 @@ static void imx_ocotp_set_imx6_timing(struct ocotp_priv *priv)
strobe_prog = clk_rate / (1000000000 / 10000) + 2 * (DEF_RELAX + 1) - 1;
strobe_read = clk_rate / (1000000000 / 40) + 2 * (DEF_RELAX + 1) - 1;
- timing = strobe_prog & 0x00000FFF;
+ timing = readl(priv->base + IMX_OCOTP_ADDR_TIMING) & 0x0FC00000;
+ timing |= strobe_prog & 0x00000FFF;
timing |= (relax << 12) & 0x0000F000;
timing |= (strobe_read << 16) & 0x003F0000;
--
2.21.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 11+ messages in thread* [PATCH v6 3/5] nvmem: imx-ocotp: Change TIMING calculation to u-boot algorithm
2019-05-03 16:53 [RESEND PATCH v6 0/5] Add i.MX8MM OCOTP support Bryan O'Donoghue
2019-05-03 16:53 ` [PATCH v6 1/5] nvmem: imx-ocotp: Elongate OCOTP_CTRL ADDR field to eight bits Bryan O'Donoghue
2019-05-03 16:53 ` [PATCH v6 2/5] nvmem: imx-ocotp: Ensure WAIT bits are preserved when setting timing Bryan O'Donoghue
@ 2019-05-03 16:53 ` Bryan O'Donoghue
2019-05-03 16:53 ` [PATCH v6 4/5] nvmem: imx-ocotp: Add i.MX8MM support Bryan O'Donoghue
` (2 subsequent siblings)
5 siblings, 0 replies; 11+ messages in thread
From: Bryan O'Donoghue @ 2019-05-03 16:53 UTC (permalink / raw)
To: gregkh, l.stach, peng.fan, shawnguo, srinivas.kandagatla,
leonard.crestez
Cc: devel, aisheng.dong, abel.vesa, anson.huang, linux-imx, kernel,
fabio.estevam, Bryan O'Donoghue, linux-arm-kernel
The RELAX field of the OCOTP block is turning out as a zero on i.MX8MM.
This messes up the subsequent re-load of the fuse shadow registers.
After some discussion with people @ NXP its clear we have missed a trick
here in Linux.
The OCOTP fuse programming time has a physical minimum 'burn time' that is
not related to the ipg_clk.
We need to define the RELAX, STROBE_READ and STROBE_PROG fields in terms of
desired timings to allow for the burn-in to safely complete. Right now only
the RELAX field is calculated in terms of an absolute time and we are
ending up with a value of zero.
This patch inherits the u-boot timings for the OCOTP_TIMING calculation on
the i.MX6 and i.MX8. Those timings are known to work and critically specify
values such as STROBE_PROG as a minimum timing.
Fixes: 0642bac7da42 ("nvmem: imx-ocotp: add write support")
Signed-off-by: Bryan O'Donoghue <pure.logic@nexus-software.ie>
Suggested-by: Leonard Crestez <leonard.crestez@nxp.com>
Reviewed-by: Leonard Crestez <leonard.crestez@nxp.com>
---
drivers/nvmem/imx-ocotp.c | 36 ++++++++++++++++++++++++++++++++----
1 file changed, 32 insertions(+), 4 deletions(-)
diff --git a/drivers/nvmem/imx-ocotp.c b/drivers/nvmem/imx-ocotp.c
index 85a7d0da3abb..2c5009691dd6 100644
--- a/drivers/nvmem/imx-ocotp.c
+++ b/drivers/nvmem/imx-ocotp.c
@@ -50,7 +50,9 @@
#define IMX_OCOTP_BM_CTRL_ERROR 0x00000200
#define IMX_OCOTP_BM_CTRL_REL_SHADOWS 0x00000400
-#define DEF_RELAX 20 /* > 16.5ns */
+#define TIMING_STROBE_PROG_US 10 /* Min time to blow a fuse */
+#define TIMING_STROBE_READ_NS 37 /* Min time before read */
+#define TIMING_RELAX_NS 17
#define DEF_FSOURCE 1001 /* > 1000 ns */
#define DEF_STROBE_PROG 10000 /* IPG clocks */
#define IMX_OCOTP_WR_UNLOCK 0x3E770000
@@ -182,12 +184,38 @@ static void imx_ocotp_set_imx6_timing(struct ocotp_priv *priv)
* fields with timing values to match the current frequency of the
* ipg_clk. OTP writes will work at maximum bus frequencies as long
* as the HW_OCOTP_TIMING parameters are set correctly.
+ *
+ * Note: there are minimum timings required to ensure an OTP fuse burns
+ * correctly that are independent of the ipg_clk. Those values are not
+ * formally documented anywhere however, working from the minimum
+ * timings given in u-boot we can say:
+ *
+ * - Minimum STROBE_PROG time is 10 microseconds. Intuitively 10
+ * microseconds feels about right as representative of a minimum time
+ * to physically burn out a fuse.
+ *
+ * - Minimum STROBE_READ i.e. the time to wait post OTP fuse burn before
+ * performing another read is 37 nanoseconds
+ *
+ * - Minimum RELAX timing is 17 nanoseconds. This final RELAX minimum
+ * timing is not entirely clear the documentation says "This
+ * count value specifies the time to add to all default timing
+ * parameters other than the Tpgm and Trd. It is given in number
+ * of ipg_clk periods." where Tpgm and Trd refer to STROBE_PROG
+ * and STROBE_READ respectively. What the other timing parameters
+ * are though, is not specified. Experience shows a zero RELAX
+ * value will mess up a re-load of the shadow registers post OTP
+ * burn.
*/
clk_rate = clk_get_rate(priv->clk);
- relax = clk_rate / (1000000000 / DEF_RELAX) - 1;
- strobe_prog = clk_rate / (1000000000 / 10000) + 2 * (DEF_RELAX + 1) - 1;
- strobe_read = clk_rate / (1000000000 / 40) + 2 * (DEF_RELAX + 1) - 1;
+ relax = DIV_ROUND_UP(clk_rate * TIMING_RELAX_NS, 1000000000) - 1;
+ strobe_read = DIV_ROUND_UP(clk_rate * TIMING_STROBE_READ_NS,
+ 1000000000);
+ strobe_read += 2 * (relax + 1) - 1;
+ strobe_prog = DIV_ROUND_CLOSEST(clk_rate * TIMING_STROBE_PROG_US,
+ 1000000);
+ strobe_prog += 2 * (relax + 1) - 1;
timing = readl(priv->base + IMX_OCOTP_ADDR_TIMING) & 0x0FC00000;
timing |= strobe_prog & 0x00000FFF;
--
2.21.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 11+ messages in thread* [PATCH v6 4/5] nvmem: imx-ocotp: Add i.MX8MM support
2019-05-03 16:53 [RESEND PATCH v6 0/5] Add i.MX8MM OCOTP support Bryan O'Donoghue
` (2 preceding siblings ...)
2019-05-03 16:53 ` [PATCH v6 3/5] nvmem: imx-ocotp: Change TIMING calculation to u-boot algorithm Bryan O'Donoghue
@ 2019-05-03 16:53 ` Bryan O'Donoghue
2019-05-03 16:53 ` [PATCH v6 5/5] dt-bindings: imx-ocotp: Add i.MX8MM compatible Bryan O'Donoghue
2019-05-04 8:39 ` [RESEND PATCH v6 0/5] Add i.MX8MM OCOTP support Greg KH
5 siblings, 0 replies; 11+ messages in thread
From: Bryan O'Donoghue @ 2019-05-03 16:53 UTC (permalink / raw)
To: gregkh, l.stach, peng.fan, shawnguo, srinivas.kandagatla,
leonard.crestez
Cc: devel, aisheng.dong, abel.vesa, anson.huang, linux-imx, kernel,
fabio.estevam, Bryan O'Donoghue, linux-arm-kernel
This patch adds support to burn the fuses on the i.MX8MM.
https://www.nxp.com/webapp/Download?colCode=IMX8MMRM
The i.MX8MM is similar to i.MX6 processors in terms of addressing and clock
setup.
The documentation specifies 60 discreet OTP registers but, the fusemap
address space encompasses up to 256 registers. We map the entire putative
256 OTP registers.
Signed-off-by: Bryan O'Donoghue <pure.logic@nexus-software.ie>
---
drivers/nvmem/imx-ocotp.c | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/drivers/nvmem/imx-ocotp.c b/drivers/nvmem/imx-ocotp.c
index 2c5009691dd6..189fd5f334f4 100644
--- a/drivers/nvmem/imx-ocotp.c
+++ b/drivers/nvmem/imx-ocotp.c
@@ -479,6 +479,12 @@ static const struct ocotp_params imx8mq_params = {
.set_timing = imx_ocotp_set_imx7_timing,
};
+static const struct ocotp_params imx8mm_params = {
+ .nregs = 256,
+ .bank_address_words = 0,
+ .set_timing = imx_ocotp_set_imx6_timing,
+};
+
static const struct of_device_id imx_ocotp_dt_ids[] = {
{ .compatible = "fsl,imx6q-ocotp", .data = &imx6q_params },
{ .compatible = "fsl,imx6sl-ocotp", .data = &imx6sl_params },
@@ -489,6 +495,7 @@ static const struct of_device_id imx_ocotp_dt_ids[] = {
{ .compatible = "fsl,imx6sll-ocotp", .data = &imx6sll_params },
{ .compatible = "fsl,imx7ulp-ocotp", .data = &imx7ulp_params },
{ .compatible = "fsl,imx8mq-ocotp", .data = &imx8mq_params },
+ { .compatible = "fsl,imx8mm-ocotp", .data = &imx8mm_params },
{ },
};
MODULE_DEVICE_TABLE(of, imx_ocotp_dt_ids);
--
2.21.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 11+ messages in thread* [PATCH v6 5/5] dt-bindings: imx-ocotp: Add i.MX8MM compatible
2019-05-03 16:53 [RESEND PATCH v6 0/5] Add i.MX8MM OCOTP support Bryan O'Donoghue
` (3 preceding siblings ...)
2019-05-03 16:53 ` [PATCH v6 4/5] nvmem: imx-ocotp: Add i.MX8MM support Bryan O'Donoghue
@ 2019-05-03 16:53 ` Bryan O'Donoghue
2019-05-04 8:39 ` [RESEND PATCH v6 0/5] Add i.MX8MM OCOTP support Greg KH
5 siblings, 0 replies; 11+ messages in thread
From: Bryan O'Donoghue @ 2019-05-03 16:53 UTC (permalink / raw)
To: gregkh, l.stach, peng.fan, shawnguo, srinivas.kandagatla,
leonard.crestez
Cc: devel, aisheng.dong, abel.vesa, anson.huang, Rob Herring,
linux-imx, kernel, fabio.estevam, Bryan O'Donoghue,
linux-arm-kernel
Add compatible for i.MX8MM as per arch/arm64/boot/dts/freescale/imx8mm.dtsi
Signed-off-by: Bryan O'Donoghue <pure.logic@nexus-software.ie>
Cc: Rob Herring <robh@kernel.org>
Reviewed-by: Leonard Crestez <leonard.crestez@nxp.com>
---
Documentation/devicetree/bindings/nvmem/imx-ocotp.txt | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/nvmem/imx-ocotp.txt b/Documentation/devicetree/bindings/nvmem/imx-ocotp.txt
index 68f7d6fdd140..96ffd06d2ca8 100644
--- a/Documentation/devicetree/bindings/nvmem/imx-ocotp.txt
+++ b/Documentation/devicetree/bindings/nvmem/imx-ocotp.txt
@@ -15,6 +15,7 @@ Required properties:
"fsl,imx6sll-ocotp" (i.MX6SLL),
"fsl,imx7ulp-ocotp" (i.MX7ULP),
"fsl,imx8mq-ocotp" (i.MX8MQ),
+ "fsl,imx8mm-ocotp" (i.MX8MM),
followed by "syscon".
- #address-cells : Should be 1
- #size-cells : Should be 1
--
2.21.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 11+ messages in thread* Re: [RESEND PATCH v6 0/5] Add i.MX8MM OCOTP support
2019-05-03 16:53 [RESEND PATCH v6 0/5] Add i.MX8MM OCOTP support Bryan O'Donoghue
` (4 preceding siblings ...)
2019-05-03 16:53 ` [PATCH v6 5/5] dt-bindings: imx-ocotp: Add i.MX8MM compatible Bryan O'Donoghue
@ 2019-05-04 8:39 ` Greg KH
2019-05-04 14:49 ` Bryan O'Donoghue
5 siblings, 1 reply; 11+ messages in thread
From: Greg KH @ 2019-05-04 8:39 UTC (permalink / raw)
To: Bryan O'Donoghue
Cc: devel, aisheng.dong, peng.fan, abel.vesa, anson.huang,
srinivas.kandagatla, linux-imx, kernel, fabio.estevam,
leonard.crestez, shawnguo, linux-arm-kernel, l.stach
On Fri, May 03, 2019 at 05:53:37PM +0100, Bryan O'Donoghue wrote:
> V6 RESEND:
> - Adding Greg to sender list. Greg looks like you are the right person to
> apply this.
$ ./scripts/get_maintainer.pl --file drivers/nvmem/imx-ocotp.c
Srinivas Kandagatla <srinivas.kandagatla@linaro.org> (maintainer:NVMEM FRAMEWORK)
Shawn Guo <shawnguo@kernel.org> (maintainer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
Sascha Hauer <s.hauer@pengutronix.de> (maintainer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
Pengutronix Kernel Team <kernel@pengutronix.de> (reviewer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
Fabio Estevam <festevam@gmail.com> (reviewer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
NXP Linux Team <linux-imx@nxp.com> (reviewer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
linux-arm-kernel@lists.infradead.org (moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
linux-kernel@vger.kernel.org (open list)
Why me???
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 11+ messages in thread* Re: [RESEND PATCH v6 0/5] Add i.MX8MM OCOTP support
2019-05-04 8:39 ` [RESEND PATCH v6 0/5] Add i.MX8MM OCOTP support Greg KH
@ 2019-05-04 14:49 ` Bryan O'Donoghue
2019-05-04 15:34 ` Greg KH
2019-05-04 17:10 ` Srinivas Kandagatla
0 siblings, 2 replies; 11+ messages in thread
From: Bryan O'Donoghue @ 2019-05-04 14:49 UTC (permalink / raw)
To: Greg KH
Cc: devel, aisheng.dong, peng.fan, abel.vesa, anson.huang,
srinivas.kandagatla, linux-imx, kernel, fabio.estevam,
leonard.crestez, shawnguo, linux-arm-kernel, l.stach
On 04/05/2019 09:39, Greg KH wrote:
> On Fri, May 03, 2019 at 05:53:37PM +0100, Bryan O'Donoghue wrote:
>> V6 RESEND:
>> - Adding Greg to sender list. Greg looks like you are the right person to
>> apply this.
>
> $ ./scripts/get_maintainer.pl --file drivers/nvmem/imx-ocotp.c
> Srinivas Kandagatla <srinivas.kandagatla@linaro.org> (maintainer:NVMEM FRAMEWORK)
> Shawn Guo <shawnguo@kernel.org> (maintainer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
> Sascha Hauer <s.hauer@pengutronix.de> (maintainer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
> Pengutronix Kernel Team <kernel@pengutronix.de> (reviewer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
> Fabio Estevam <festevam@gmail.com> (reviewer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
> NXP Linux Team <linux-imx@nxp.com> (reviewer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
> linux-arm-kernel@lists.infradead.org (moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
> linux-kernel@vger.kernel.org (open list)
>
>
> Why me???
>
Looked like you were doing the merges to me.
commit 38e7b6efe997c4eb9a5a809dc2b2fe6c759b7c4b
Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Ping, Srini, any chance you can merge this to your tree ?
---
bod
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [RESEND PATCH v6 0/5] Add i.MX8MM OCOTP support
2019-05-04 14:49 ` Bryan O'Donoghue
@ 2019-05-04 15:34 ` Greg KH
2019-05-04 17:10 ` Srinivas Kandagatla
1 sibling, 0 replies; 11+ messages in thread
From: Greg KH @ 2019-05-04 15:34 UTC (permalink / raw)
To: Bryan O'Donoghue
Cc: devel, aisheng.dong, peng.fan, abel.vesa, anson.huang,
srinivas.kandagatla, linux-imx, kernel, fabio.estevam,
leonard.crestez, shawnguo, linux-arm-kernel, l.stach
On Sat, May 04, 2019 at 03:49:05PM +0100, Bryan O'Donoghue wrote:
> On 04/05/2019 09:39, Greg KH wrote:
> > On Fri, May 03, 2019 at 05:53:37PM +0100, Bryan O'Donoghue wrote:
> > > V6 RESEND:
> > > - Adding Greg to sender list. Greg looks like you are the right person to
> > > apply this.
> >
> > $ ./scripts/get_maintainer.pl --file drivers/nvmem/imx-ocotp.c
> > Srinivas Kandagatla <srinivas.kandagatla@linaro.org> (maintainer:NVMEM FRAMEWORK)
> > Shawn Guo <shawnguo@kernel.org> (maintainer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
> > Sascha Hauer <s.hauer@pengutronix.de> (maintainer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
> > Pengutronix Kernel Team <kernel@pengutronix.de> (reviewer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
> > Fabio Estevam <festevam@gmail.com> (reviewer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
> > NXP Linux Team <linux-imx@nxp.com> (reviewer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
> > linux-arm-kernel@lists.infradead.org (moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
> > linux-kernel@vger.kernel.org (open list)
> >
> >
> > Why me???
> >
>
> Looked like you were doing the merges to me.
I might do merges, but that does not mean I do the initial
review/acceptance of the patches. That's up to the real maintainer of
the driver/subsystem. I do not circumvent them unless something really
odd happens.
thanks,
greg k-h
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [RESEND PATCH v6 0/5] Add i.MX8MM OCOTP support
2019-05-04 14:49 ` Bryan O'Donoghue
2019-05-04 15:34 ` Greg KH
@ 2019-05-04 17:10 ` Srinivas Kandagatla
2019-05-04 22:38 ` Bryan O'Donoghue
1 sibling, 1 reply; 11+ messages in thread
From: Srinivas Kandagatla @ 2019-05-04 17:10 UTC (permalink / raw)
To: Bryan O'Donoghue, Greg KH
Cc: devel, aisheng.dong, peng.fan, abel.vesa, anson.huang, linux-imx,
kernel, fabio.estevam, leonard.crestez, shawnguo,
linux-arm-kernel, l.stach
On 04/05/2019 15:49, Bryan O'Donoghue wrote:
> On 04/05/2019 09:39, Greg KH wrote:
>> On Fri, May 03, 2019 at 05:53:37PM +0100, Bryan O'Donoghue wrote:
>>> V6 RESEND:
>>> - Adding Greg to sender list. Greg looks like you are the right
>>> person to
>>> apply this.
>>
>> $ ./scripts/get_maintainer.pl --file drivers/nvmem/imx-ocotp.c
>> Srinivas Kandagatla <srinivas.kandagatla@linaro.org> (maintainer:NVMEM
>> FRAMEWORK)
>> Shawn Guo <shawnguo@kernel.org> (maintainer:ARM/FREESCALE IMX / MXC
>> ARM ARCHITECTURE)
>> Sascha Hauer <s.hauer@pengutronix.de> (maintainer:ARM/FREESCALE IMX /
>> MXC ARM ARCHITECTURE)
>> Pengutronix Kernel Team <kernel@pengutronix.de>
>> (reviewer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
>> Fabio Estevam <festevam@gmail.com> (reviewer:ARM/FREESCALE IMX / MXC
>> ARM ARCHITECTURE)
>> NXP Linux Team <linux-imx@nxp.com> (reviewer:ARM/FREESCALE IMX / MXC
>> ARM ARCHITECTURE)
>> linux-arm-kernel@lists.infradead.org (moderated list:ARM/FREESCALE IMX
>> / MXC ARM ARCHITECTURE)
>> linux-kernel@vger.kernel.org (open list)
>>
>>
>> Why me???
>>
>
> Looked like you were doing the merges to me.
>
> commit 38e7b6efe997c4eb9a5a809dc2b2fe6c759b7c4b
> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
>
> Ping, Srini, any chance you can merge this to your tree ?
Thankyou for your patience.
Normally I don't take patches that are sent after rc5 into next merge
window. Unless there is an urgent fix. In this case I will be applying
these series to nvmem next branch once rc1 is released for 5.3 merge window.
Also any device tree bindings changes need to Reviewed by DT maintainer
before I pick up these patches.
--srini
>
> ---
> bod
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [RESEND PATCH v6 0/5] Add i.MX8MM OCOTP support
2019-05-04 17:10 ` Srinivas Kandagatla
@ 2019-05-04 22:38 ` Bryan O'Donoghue
0 siblings, 0 replies; 11+ messages in thread
From: Bryan O'Donoghue @ 2019-05-04 22:38 UTC (permalink / raw)
To: Srinivas Kandagatla, Greg KH
Cc: devel, aisheng.dong, peng.fan, abel.vesa, anson.huang, linux-imx,
kernel, fabio.estevam, leonard.crestez, shawnguo,
linux-arm-kernel, l.stach
On 04/05/2019 18:10, Srinivas Kandagatla wrote:
> Normally I don't take patches that are sent after rc5 into next merge
> window. Unless there is an urgent fix. In this case I will be applying
> these series to nvmem next branch once rc1 is released for 5.3 merge
> window.
Great, that'll done fine Srini no rush.
I sometimes do a round of pings of outstanding patches I have to make
sure they don't get lost in the ether.
So long as its on your radar that's good enough.
---
bod
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 11+ messages in thread