* [PATCH V2 0/2] mailbox: imx: support RST channel @ 2022-06-15 13:00 Peng Fan (OSS) 2022-06-15 13:00 ` [PATCH V2 1/2] dt-bindings: mailbox: imx-mu: add " Peng Fan (OSS) 2022-06-15 13:00 ` [PATCH V2 2/2] mailbox: imx: support " Peng Fan (OSS) 0 siblings, 2 replies; 5+ messages in thread From: Peng Fan (OSS) @ 2022-06-15 13:00 UTC (permalink / raw) To: robh+dt, krzysztof.kozlowski+dt, jassisinghbrar, shawnguo, s.hauer Cc: kernel, festevam, linux-imx, devicetree, linux-kernel, linux-arm-kernel, Peng Fan From: Peng Fan <peng.fan@nxp.com> V2: Add R-b tag Missed to add mailbox maintainer in V1, add in V2 To i.MX generic MU, when linux stop remote core, the MU-B side is not reset because of hardware design, so MU-B contains valid configuration when remote core stops. Then when linux start remote core again, linux may notify remote core before remote core is ready for incoming message. So we need make sure MU is in reset state before remote core start. So add a RST channel which is only to reset MU when shutdown the RST mbox channel. Peng Fan (2): dt-bindings: mailbox: imx-mu: add RST channel mailbox: imx: support RST channel .../devicetree/bindings/mailbox/fsl,mu.yaml | 6 ++-- drivers/mailbox/imx-mailbox.c | 29 +++++++++++++++---- 2 files changed, 27 insertions(+), 8 deletions(-) -- 2.25.1 ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH V2 1/2] dt-bindings: mailbox: imx-mu: add RST channel 2022-06-15 13:00 [PATCH V2 0/2] mailbox: imx: support RST channel Peng Fan (OSS) @ 2022-06-15 13:00 ` Peng Fan (OSS) 2022-06-15 13:00 ` [PATCH V2 2/2] mailbox: imx: support " Peng Fan (OSS) 1 sibling, 0 replies; 5+ messages in thread From: Peng Fan (OSS) @ 2022-06-15 13:00 UTC (permalink / raw) To: robh+dt, krzysztof.kozlowski+dt, jassisinghbrar, shawnguo, s.hauer Cc: kernel, festevam, linux-imx, devicetree, linux-kernel, linux-arm-kernel, Peng Fan, Rob Herring From: Peng Fan <peng.fan@nxp.com> i.MX MU has a MUR bit which is to reset both the Processor B and the Processor A sides of the MU module, forcing all control and status registers to return to their default values (except the BHR bit in the ACR register and BHRM bit in BCR register), and all internal states to be cleared. Signed-off-by: Peng Fan <peng.fan@nxp.com> Reviewed-by: Rob Herring <robh@kernel.org> --- Documentation/devicetree/bindings/mailbox/fsl,mu.yaml | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/Documentation/devicetree/bindings/mailbox/fsl,mu.yaml b/Documentation/devicetree/bindings/mailbox/fsl,mu.yaml index 7a86e7926dd2..191c1ce15009 100644 --- a/Documentation/devicetree/bindings/mailbox/fsl,mu.yaml +++ b/Documentation/devicetree/bindings/mailbox/fsl,mu.yaml @@ -72,14 +72,16 @@ properties: type : Channel type channel : Channel number - This MU support 4 type of unidirectional channels, each type - has 4 channels. A total of 16 channels. Following types are + This MU support 5 type of unidirectional channels, each type + has 4 channels except RST channel which only has 1 channel. + A total of 17 channels. Following types are supported: 0 - TX channel with 32bit transmit register and IRQ transmit acknowledgment support. 1 - RX channel with 32bit receive register and IRQ support 2 - TX doorbell channel. Without own register and no ACK support. 3 - RX doorbell channel. + 4 - RST channel const: 2 clocks: -- 2.25.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH V2 2/2] mailbox: imx: support RST channel 2022-06-15 13:00 [PATCH V2 0/2] mailbox: imx: support RST channel Peng Fan (OSS) 2022-06-15 13:00 ` [PATCH V2 1/2] dt-bindings: mailbox: imx-mu: add " Peng Fan (OSS) @ 2022-06-15 13:00 ` Peng Fan (OSS) 2022-08-24 9:36 ` Liu Ying 1 sibling, 1 reply; 5+ messages in thread From: Peng Fan (OSS) @ 2022-06-15 13:00 UTC (permalink / raw) To: robh+dt, krzysztof.kozlowski+dt, jassisinghbrar, shawnguo, s.hauer Cc: kernel, festevam, linux-imx, devicetree, linux-kernel, linux-arm-kernel, Peng Fan From: Peng Fan <peng.fan@nxp.com> i.MX generic MU supports MU-A/B reset feature. When stop/start remotecore, MU is not reset. So when Linux stop remotecore, the MU-B side BCR may contain valid configuration, because MU-B is not reset. So when linux start Mcore again and notify Mcore, Mcore is not ready to handle MU interrupt and cause issues. So need reset MU when stop Mcore. Signed-off-by: Peng Fan <peng.fan@nxp.com> --- drivers/mailbox/imx-mailbox.c | 29 +++++++++++++++++++++++------ 1 file changed, 23 insertions(+), 6 deletions(-) diff --git a/drivers/mailbox/imx-mailbox.c b/drivers/mailbox/imx-mailbox.c index b10239d6ef93..37fc765783f8 100644 --- a/drivers/mailbox/imx-mailbox.c +++ b/drivers/mailbox/imx-mailbox.c @@ -19,7 +19,7 @@ #include <linux/suspend.h> #include <linux/slab.h> -#define IMX_MU_CHANS 16 +#define IMX_MU_CHANS 17 /* TX0/RX0/RXDB[0-3] */ #define IMX_MU_SCU_CHANS 6 /* TX0/RX0 */ @@ -35,9 +35,11 @@ enum imx_mu_chan_type { IMX_MU_TYPE_RX = 1, /* Rx */ IMX_MU_TYPE_TXDB = 2, /* Tx doorbell */ IMX_MU_TYPE_RXDB = 3, /* Rx doorbell */ + IMX_MU_TYPE_RST = 4, /* Reset */ }; enum imx_mu_xcr { + IMX_MU_CR, IMX_MU_GIER, IMX_MU_GCR, IMX_MU_TCR, @@ -50,6 +52,7 @@ enum imx_mu_xsr { IMX_MU_GSR, IMX_MU_TSR, IMX_MU_RSR, + IMX_MU_xSR_MAX, }; struct imx_sc_rpc_msg_max { @@ -85,7 +88,7 @@ struct imx_mu_priv { int irq[IMX_MU_CHANS]; bool suspend; - u32 xcr[4]; + u32 xcr[IMX_MU_xCR_MAX]; bool side_b; }; @@ -105,8 +108,8 @@ struct imx_mu_dcfg { enum imx_mu_type type; u32 xTR; /* Transmit Register0 */ u32 xRR; /* Receive Register0 */ - u32 xSR[4]; /* Status Registers */ - u32 xCR[4]; /* Control Registers */ + u32 xSR[IMX_MU_xSR_MAX]; /* Status Registers */ + u32 xCR[IMX_MU_xCR_MAX]; /* Control Registers */ }; #define IMX_MU_xSR_GIPn(type, x) (type & IMX_MU_V2 ? BIT(x) : BIT(28 + (3 - (x)))) @@ -121,6 +124,9 @@ struct imx_mu_dcfg { #define IMX_MU_xCR_TIEn(type, x) (type & IMX_MU_V2 ? BIT(x) : BIT(20 + (3 - (x)))) /* General Purpose Interrupt Request */ #define IMX_MU_xCR_GIRn(type, x) (type & IMX_MU_V2 ? BIT(x) : BIT(16 + (3 - (x)))) +/* MU reset */ +#define IMX_MU_xCR_RST(type) (type & IMX_MU_V2 ? BIT(0) : BIT(5)) +#define IMX_MU_xSR_RST(type) (type & IMX_MU_V2 ? BIT(0) : BIT(7)) static struct imx_mu_priv *to_imx_mu_priv(struct mbox_controller *mbox) @@ -497,6 +503,8 @@ static irqreturn_t imx_mu_isr(int irq, void *p) val &= IMX_MU_xSR_GIPn(priv->dcfg->type, cp->idx) & (ctrl & IMX_MU_xCR_GIEn(priv->dcfg->type, cp->idx)); break; + case IMX_MU_TYPE_RST: + return IRQ_NONE; default: dev_warn_ratelimited(priv->dev, "Unhandled channel type %d\n", cp->type); @@ -581,6 +589,8 @@ static void imx_mu_shutdown(struct mbox_chan *chan) { struct imx_mu_priv *priv = to_imx_mu_priv(chan->mbox); struct imx_mu_con_priv *cp = chan->con_priv; + int ret; + u32 sr; if (cp->type == IMX_MU_TYPE_TXDB) { tasklet_kill(&cp->txdb_tasklet); @@ -598,6 +608,13 @@ static void imx_mu_shutdown(struct mbox_chan *chan) case IMX_MU_TYPE_RXDB: imx_mu_xcr_rmw(priv, IMX_MU_GIER, 0, IMX_MU_xCR_GIEn(priv->dcfg->type, cp->idx)); break; + case IMX_MU_TYPE_RST: + imx_mu_xcr_rmw(priv, IMX_MU_CR, IMX_MU_xCR_RST(priv->dcfg->type), 0); + ret = readl_poll_timeout(priv->base + priv->dcfg->xSR[IMX_MU_SR], sr, + !(sr & IMX_MU_xSR_RST(priv->dcfg->type)), 1, 5); + if (ret) + dev_warn(priv->dev, "RST channel timeout\n"); + break; default: break; } @@ -865,7 +882,7 @@ static const struct imx_mu_dcfg imx_mu_cfg_imx6sx = { .xTR = 0x0, .xRR = 0x10, .xSR = {0x20, 0x20, 0x20, 0x20}, - .xCR = {0x24, 0x24, 0x24, 0x24}, + .xCR = {0x24, 0x24, 0x24, 0x24, 0x24}, }; static const struct imx_mu_dcfg imx_mu_cfg_imx7ulp = { @@ -888,7 +905,7 @@ static const struct imx_mu_dcfg imx_mu_cfg_imx8ulp = { .xTR = 0x200, .xRR = 0x280, .xSR = {0xC, 0x118, 0x124, 0x12C}, - .xCR = {0x110, 0x114, 0x120, 0x128}, + .xCR = {0x8, 0x110, 0x114, 0x120, 0x128}, }; static const struct imx_mu_dcfg imx_mu_cfg_imx8ulp_s4 = { -- 2.25.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH V2 2/2] mailbox: imx: support RST channel 2022-06-15 13:00 ` [PATCH V2 2/2] mailbox: imx: support " Peng Fan (OSS) @ 2022-08-24 9:36 ` Liu Ying 2022-08-29 0:41 ` Peng Fan 0 siblings, 1 reply; 5+ messages in thread From: Liu Ying @ 2022-08-24 9:36 UTC (permalink / raw) To: Peng Fan (OSS), robh+dt, krzysztof.kozlowski+dt, jassisinghbrar, shawnguo, s.hauer Cc: kernel, festevam, linux-imx, devicetree, linux-kernel, linux-arm-kernel, Peng Fan Hi, On Wed, 2022-06-15 at 21:00 +0800, Peng Fan (OSS) wrote: > From: Peng Fan <peng.fan@nxp.com> > > i.MX generic MU supports MU-A/B reset feature. > > When stop/start remotecore, MU is not reset. So when Linux stop > remotecore, the MU-B side BCR may contain valid configuration, > because MU-B is not reset. So when linux start Mcore > again and notify Mcore, Mcore is not ready to handle MU interrupt > and cause issues. So need reset MU when stop Mcore. > > Signed-off-by: Peng Fan <peng.fan@nxp.com> It looks like this patch causes i.MX8qxp MEK boot failure with v6.0- rc1. After reverting commit 82ab513baed5895c8b0e991557a12fd38d5f25f0, it boots ok. There are a lot imx_mu and imx-scu error logs: ----------------------------8<------------------------------------ [ 1.166567] imx-scu system-controller: RPC send msg timeout [ 1.171843] imx_scu_soc_id: get soc info failed, ret -110 [ 1.177221] imx-scu system-controller: failed to initialize SoC info: -22 [ 1.184019] imx-scu system-controller: NXP i.MX SCU Initialized [ 1.230605] imx-scu system-controller: RPC send msg timeout [ 1.270559] imx-scu system-controller: RPC send msg timeout [ 1.310561] imx-scu system-controller: RPC send msg timeout [ 1.350560] imx-scu system-controller: RPC send msg timeout [ 1.390560] imx-scu system-controller: RPC send msg timeout [ 1.430561] imx-scu system-controller: RPC send msg timeout [ 1.470559] imx-scu system-controller: RPC send msg timeout [ 1.510559] imx-scu system-controller: RPC send msg timeout [ 1.550559] imx-scu system-controller: RPC send msg timeout [ 1.590559] imx-scu system-controller: RPC send msg timeout [ 1.630561] imx-scu system-controller: RPC send msg timeout [ 1.670558] imx-scu system-controller: RPC send msg timeout [ 1.710559] imx-scu system-controller: RPC send msg timeout [ 1.750558] imx-scu system-controller: RPC send msg timeout [ 1.790559] imx-scu system-controller: RPC send msg timeout [ 1.830562] imx-scu system-controller: RPC send msg timeout [ 1.870559] imx-scu system-controller: RPC send msg timeout [ 1.910559] imx-scu system-controller: RPC send msg timeout [ 1.950561] imx-scu system-controller: RPC send msg timeout [ 1.990592] imx-scu system-controller: RPC send msg timeout [ 1.995838] imx_mu 5d1c0000.mailbox: Try increasing MBOX_TX_QUEUE_LEN [ 2.002262] imx-scu system-controller: RPC send msg failed: -105 [ 2.008260] imx_mu 5d1c0000.mailbox: Try increasing MBOX_TX_QUEUE_LEN [ 2.014682] imx-scu system-controller: RPC send msg failed: -105 [ 2.020681] imx_mu 5d1c0000.mailbox: Try increasing MBOX_TX_QUEUE_LEN [ 2.027103] imx-scu system-controller: RPC send msg failed: -105 [ 2.033105] imx_mu 5d1c0000.mailbox: Try increasing MBOX_TX_QUEUE_LEN [ 2.039526] imx-scu system-controller: RPC send msg failed: -105 [ 2.045526] imx_mu 5d1c0000.mailbox: Try increasing MBOX_TX_QUEUE_LEN [ 2.051949] imx-scu system-controller: RPC send msg failed: -105 ----------------------------8<------------------------------------ Regards, Liu Ying ^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH V2 2/2] mailbox: imx: support RST channel 2022-08-24 9:36 ` Liu Ying @ 2022-08-29 0:41 ` Peng Fan 0 siblings, 0 replies; 5+ messages in thread From: Peng Fan @ 2022-08-29 0:41 UTC (permalink / raw) To: Ying Liu, Peng Fan (OSS), robh+dt, krzysztof.kozlowski+dt, jassisinghbrar, shawnguo, s.hauer Cc: kernel, festevam, dl-linux-imx, devicetree, linux-kernel, linux-arm-kernel > Subject: Re: [PATCH V2 2/2] mailbox: imx: support RST channel > > Hi, > > On Wed, 2022-06-15 at 21:00 +0800, Peng Fan (OSS) wrote: > > From: Peng Fan <peng.fan@nxp.com> > > > > i.MX generic MU supports MU-A/B reset feature. > > > > When stop/start remotecore, MU is not reset. So when Linux stop > > remotecore, the MU-B side BCR may contain valid configuration, because > > MU-B is not reset. So when linux start Mcore again and notify Mcore, > > Mcore is not ready to handle MU interrupt and cause issues. So need > > reset MU when stop Mcore. > > > > Signed-off-by: Peng Fan <peng.fan@nxp.com> > > It looks like this patch causes i.MX8qxp MEK boot failure with v6.0- rc1. After > reverting commit 82ab513baed5895c8b0e991557a12fd38d5f25f0, > it boots ok. Thanks for your test. Indeed, I only take generic MU into consideration and broke i.MX8QXP SCU MU. The following patch could resolve the issue, after more test, I will send a formal patch: diff --git a/drivers/mailbox/imx-mailbox.c b/drivers/mailbox/imx-mailbox.c index f36cb0048ce7..e373ea8b4705 100644 --- a/drivers/mailbox/imx-mailbox.c +++ b/drivers/mailbox/imx-mailbox.c @@ -893,7 +893,7 @@ static const struct imx_mu_dcfg imx_mu_cfg_imx7ulp = { .xTR = 0x20, .xRR = 0x40, .xSR = {0x60, 0x60, 0x60, 0x60}, - .xCR = {0x64, 0x64, 0x64, 0x64}, + .xCR = {0x64, 0x64, 0x64, 0x64, 0x64}, }; static const struct imx_mu_dcfg imx_mu_cfg_imx8ulp = { @@ -916,7 +916,7 @@ static const struct imx_mu_dcfg imx_mu_cfg_imx8ulp_s4 = { .xTR = 0x200, .xRR = 0x280, .xSR = {0xC, 0x118, 0x124, 0x12C}, - .xCR = {0x110, 0x114, 0x120, 0x128}, + .xCR = {0x8, 0x110, 0x114, 0x120, 0x128}, }; static const struct imx_mu_dcfg imx_mu_cfg_imx93_s4 = { @@ -927,7 +927,7 @@ static const struct imx_mu_dcfg imx_mu_cfg_imx93_s4 = { .xTR = 0x200, .xRR = 0x280, .xSR = {0xC, 0x118, 0x124, 0x12C}, - .xCR = {0x110, 0x114, 0x120, 0x128}, + .xCR = {0x8, 0x110, 0x114, 0x120, 0x128}, }; static const struct imx_mu_dcfg imx_mu_cfg_imx8_scu = { @@ -938,7 +938,7 @@ static const struct imx_mu_dcfg imx_mu_cfg_imx8_scu = { .xTR = 0x0, .xRR = 0x10, .xSR = {0x20, 0x20, 0x20, 0x20}, - .xCR = {0x24, 0x24, 0x24, 0x24}, + .xCR = {0x24, 0x24, 0x24, 0x24, 0x24}, }; static const struct imx_mu_dcfg imx_mu_cfg_imx8_seco = { @@ -949,7 +949,7 @@ static const struct imx_mu_dcfg imx_mu_cfg_imx8_seco = { .xTR = 0x0, .xRR = 0x10, .xSR = {0x20, 0x20, 0x20, 0x20}, - .xCR = {0x24, 0x24, 0x24, 0x24}, + .xCR = {0x24, 0x24, 0x24, 0x24, 0x24}, }; static const struct of_device_id imx_mu_dt_ids[] = { -- 2.37.1 Thanks, Peng. > > There are a lot imx_mu and imx-scu error logs: > > ----------------------------8<------------------------------------ > [ 1.166567] imx-scu system-controller: RPC send msg timeout > [ 1.171843] imx_scu_soc_id: get soc info failed, ret -110 > [ 1.177221] imx-scu system-controller: failed to initialize SoC > info: -22 > [ 1.184019] imx-scu system-controller: NXP i.MX SCU Initialized > [ 1.230605] imx-scu system-controller: RPC send msg timeout > [ 1.270559] imx-scu system-controller: RPC send msg timeout > [ 1.310561] imx-scu system-controller: RPC send msg timeout > [ 1.350560] imx-scu system-controller: RPC send msg timeout > [ 1.390560] imx-scu system-controller: RPC send msg timeout > [ 1.430561] imx-scu system-controller: RPC send msg timeout > [ 1.470559] imx-scu system-controller: RPC send msg timeout > [ 1.510559] imx-scu system-controller: RPC send msg timeout > [ 1.550559] imx-scu system-controller: RPC send msg timeout > [ 1.590559] imx-scu system-controller: RPC send msg timeout > [ 1.630561] imx-scu system-controller: RPC send msg timeout > [ 1.670558] imx-scu system-controller: RPC send msg timeout > [ 1.710559] imx-scu system-controller: RPC send msg timeout > [ 1.750558] imx-scu system-controller: RPC send msg timeout > [ 1.790559] imx-scu system-controller: RPC send msg timeout > [ 1.830562] imx-scu system-controller: RPC send msg timeout > [ 1.870559] imx-scu system-controller: RPC send msg timeout > [ 1.910559] imx-scu system-controller: RPC send msg timeout > [ 1.950561] imx-scu system-controller: RPC send msg timeout > [ 1.990592] imx-scu system-controller: RPC send msg timeout > [ 1.995838] imx_mu 5d1c0000.mailbox: Try increasing > MBOX_TX_QUEUE_LEN > [ 2.002262] imx-scu system-controller: RPC send msg failed: -105 > [ 2.008260] imx_mu 5d1c0000.mailbox: Try increasing > MBOX_TX_QUEUE_LEN > [ 2.014682] imx-scu system-controller: RPC send msg failed: -105 > [ 2.020681] imx_mu 5d1c0000.mailbox: Try increasing > MBOX_TX_QUEUE_LEN > [ 2.027103] imx-scu system-controller: RPC send msg failed: -105 > [ 2.033105] imx_mu 5d1c0000.mailbox: Try increasing > MBOX_TX_QUEUE_LEN > [ 2.039526] imx-scu system-controller: RPC send msg failed: -105 > [ 2.045526] imx_mu 5d1c0000.mailbox: Try increasing > MBOX_TX_QUEUE_LEN > [ 2.051949] imx-scu system-controller: RPC send msg failed: -105 > ----------------------------8<------------------------------------ > > Regards, > Liu Ying ^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-08-29 0:41 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-06-15 13:00 [PATCH V2 0/2] mailbox: imx: support RST channel Peng Fan (OSS) 2022-06-15 13:00 ` [PATCH V2 1/2] dt-bindings: mailbox: imx-mu: add " Peng Fan (OSS) 2022-06-15 13:00 ` [PATCH V2 2/2] mailbox: imx: support " Peng Fan (OSS) 2022-08-24 9:36 ` Liu Ying 2022-08-29 0:41 ` Peng Fan
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).