From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sascha Hauer Subject: Re: [PATCH V2 2/4] dt-bindings: arm: fsl: add mu binding doc Date: Fri, 22 Jun 2018 07:49:32 +0200 Message-ID: <20180622054932.ysqve7o7pvdnlp5k@pengutronix.de> References: <1529239789-26849-1-git-send-email-aisheng.dong@nxp.com> <1529239789-26849-3-git-send-email-aisheng.dong@nxp.com> <20180620194310.GA28983@rob-hp-laptop> <20180621074641.sqqvudxebt3hd43m@pengutronix.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: "A.s. Dong" Cc: Mark Rutland , Rob Herring , "dongas86@gmail.com" , "devicetree@vger.kernel.org" , dl-linux-imx , "kernel@pengutronix.de" , Fabio Estevam , "shawnguo@kernel.org" , "linux-arm-kernel@lists.infradead.org" List-Id: devicetree@vger.kernel.org On Thu, Jun 21, 2018 at 05:11:33PM +0000, A.s. Dong wrote: > Hi Sascha, > > > -----Original Message----- > > From: Sascha Hauer [mailto:s.hauer@pengutronix.de] > > Sent: Thursday, June 21, 2018 3:47 PM > > To: Rob Herring > > Cc: A.s. Dong ; linux-arm- > > kernel@lists.infradead.org; dongas86@gmail.com; kernel@pengutronix.de; > > shawnguo@kernel.org; Fabio Estevam ; dl-linux- > > imx ; Mark Rutland ; > > devicetree@vger.kernel.org > > Subject: Re: [PATCH V2 2/4] dt-bindings: arm: fsl: add mu binding doc > > > > On Wed, Jun 20, 2018 at 01:43:10PM -0600, Rob Herring wrote: > > > On Sun, Jun 17, 2018 at 08:49:47PM +0800, Dong Aisheng wrote: > > > > The Messaging Unit module enables two processors within the SoC to > > > > communicate and coordinate by passing messages (e.g. data, status > > > > and control) through the MU interface. > > > > > > > > Cc: Shawn Guo > > > > Cc: Sascha Hauer > > > > Cc: Fabio Estevam > > > > Cc: Rob Herring > > > > Cc: Mark Rutland > > > > Cc: devicetree@vger.kernel.org > > > > Signed-off-by: Dong Aisheng > > > > --- > > > > v1->v2: > > > > * typo fixes > > > > * remove status property > > > > * remove imx6&7 compatible string which may be added later for > > > > the generic mailbox binding > > > > > > > > Note: Because MU used by SCU is not implemented as a mailbox driver, > > > > Instead, they're provided in library calls to gain higher performance. > > > > > > Using a binding doesn't mean you have to use an OS's subsystem. > > > > > > What needs higher performance? What's the performance difference? > > Why > > > can't the mailbox framework be improved? > > > > From what I see the performance is improved by polling the interrupt > > registers rather than using interrupts. > > I see no reason though why this can't be implemented with the mailbox > > framework as is. > > > > I thought you've agreed to not write generic MU(mailbox) driver for SCU. > https://www.spinics.net/lists/arm-kernel/msg650217.html > But seems it's still not quite certain... My suggestion was that we change the compatible of the MU unit to something that the SCU driver matches. But since we do not do that and instead use a fsl,imx8qxp-mu compatible for the MU we have to come up with a MU driver that handles both the SCU case and the regular usecase. Otherwise we'll end up with a generic driver attaching to the device you are using under the hoods. Sascha -- 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 | From mboxrd@z Thu Jan 1 00:00:00 1970 From: s.hauer@pengutronix.de (Sascha Hauer) Date: Fri, 22 Jun 2018 07:49:32 +0200 Subject: [PATCH V2 2/4] dt-bindings: arm: fsl: add mu binding doc In-Reply-To: References: <1529239789-26849-1-git-send-email-aisheng.dong@nxp.com> <1529239789-26849-3-git-send-email-aisheng.dong@nxp.com> <20180620194310.GA28983@rob-hp-laptop> <20180621074641.sqqvudxebt3hd43m@pengutronix.de> Message-ID: <20180622054932.ysqve7o7pvdnlp5k@pengutronix.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, Jun 21, 2018 at 05:11:33PM +0000, A.s. Dong wrote: > Hi Sascha, > > > -----Original Message----- > > From: Sascha Hauer [mailto:s.hauer at pengutronix.de] > > Sent: Thursday, June 21, 2018 3:47 PM > > To: Rob Herring > > Cc: A.s. Dong ; linux-arm- > > kernel at lists.infradead.org; dongas86 at gmail.com; kernel at pengutronix.de; > > shawnguo at kernel.org; Fabio Estevam ; dl-linux- > > imx ; Mark Rutland ; > > devicetree at vger.kernel.org > > Subject: Re: [PATCH V2 2/4] dt-bindings: arm: fsl: add mu binding doc > > > > On Wed, Jun 20, 2018 at 01:43:10PM -0600, Rob Herring wrote: > > > On Sun, Jun 17, 2018 at 08:49:47PM +0800, Dong Aisheng wrote: > > > > The Messaging Unit module enables two processors within the SoC to > > > > communicate and coordinate by passing messages (e.g. data, status > > > > and control) through the MU interface. > > > > > > > > Cc: Shawn Guo > > > > Cc: Sascha Hauer > > > > Cc: Fabio Estevam > > > > Cc: Rob Herring > > > > Cc: Mark Rutland > > > > Cc: devicetree at vger.kernel.org > > > > Signed-off-by: Dong Aisheng > > > > --- > > > > v1->v2: > > > > * typo fixes > > > > * remove status property > > > > * remove imx6&7 compatible string which may be added later for > > > > the generic mailbox binding > > > > > > > > Note: Because MU used by SCU is not implemented as a mailbox driver, > > > > Instead, they're provided in library calls to gain higher performance. > > > > > > Using a binding doesn't mean you have to use an OS's subsystem. > > > > > > What needs higher performance? What's the performance difference? > > Why > > > can't the mailbox framework be improved? > > > > From what I see the performance is improved by polling the interrupt > > registers rather than using interrupts. > > I see no reason though why this can't be implemented with the mailbox > > framework as is. > > > > I thought you've agreed to not write generic MU(mailbox) driver for SCU. > https://www.spinics.net/lists/arm-kernel/msg650217.html > But seems it's still not quite certain... My suggestion was that we change the compatible of the MU unit to something that the SCU driver matches. But since we do not do that and instead use a fsl,imx8qxp-mu compatible for the MU we have to come up with a MU driver that handles both the SCU case and the regular usecase. Otherwise we'll end up with a generic driver attaching to the device you are using under the hoods. Sascha -- 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 |