* [PATCH v2 0/3] Add Platform MHU mailbox driver for Amlogic GXBB
@ 2016-08-18 10:10 Neil Armstrong
2016-08-18 10:10 ` [PATCH v2 1/3] mailbox: Add Platform Message-Handling-Unit variant driver Neil Armstrong
` (3 more replies)
0 siblings, 4 replies; 10+ messages in thread
From: Neil Armstrong @ 2016-08-18 10:10 UTC (permalink / raw)
To: linux-kernel, linux-amlogic, linux-arm-kernel, jassisinghbrar
Cc: Neil Armstrong
In order to support Mailbox links for the Amlogic GXBB SoC, add a generic
platform MHU driver based on arm_mhu.c.
This patchset follows a RFC thread along the GXBB SCPI support at :
http://lkml.kernel.org/r/1466503374-28841-1-git-send-email-narmstrong@baylibre.com
And specific MHU discussions at :
http://lkml.kernel.org/r/CABb+yY3HqJG2+GMWCWF9PomxobrwWGZ=TZe5NvXPcHMDdLhkcQ@mail.gmail.com
Changes since v1 at http://lkml.kernel.org/r/1470732737-18391-1-git-send-email-narmstrong@baylibre.com :
- Fix irq to signed to detect platform_get_irq() failures
- Introduced back the secure channel
- Fixed indexes
Changes since RFC v2 :
- Rename to platform_mhu
- Sync all link functions with arm_mhu
Neil Armstrong (3):
mailbox: Add Platform Message-Handling-Unit variant driver
dt-bindings: mailbox: Add Amlogic Meson MHU Bindings
ARM64: dts: meson-gxbb: Add Meson MHU Node
.../devicetree/bindings/mailbox/meson-mhu.txt | 34 ++++
arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi | 9 +
drivers/mailbox/Kconfig | 10 +
drivers/mailbox/Makefile | 2 +
drivers/mailbox/platform_mhu.c | 205 +++++++++++++++++++++
5 files changed, 260 insertions(+)
create mode 100644 Documentation/devicetree/bindings/mailbox/meson-mhu.txt
create mode 100644 drivers/mailbox/platform_mhu.c
--
1.9.1
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH v2 1/3] mailbox: Add Platform Message-Handling-Unit variant driver
2016-08-18 10:10 [PATCH v2 0/3] Add Platform MHU mailbox driver for Amlogic GXBB Neil Armstrong
@ 2016-08-18 10:10 ` Neil Armstrong
2016-08-18 10:10 ` [PATCH v2 2/3] dt-bindings: mailbox: Add Amlogic Meson MHU Bindings Neil Armstrong
` (2 subsequent siblings)
3 siblings, 0 replies; 10+ messages in thread
From: Neil Armstrong @ 2016-08-18 10:10 UTC (permalink / raw)
To: linux-kernel, linux-amlogic, linux-arm-kernel, jassisinghbrar
Cc: Neil Armstrong
Add Message-Handling-Unit driver for platform variants as mailbox controller.
Actually, only the Amlogic Meson GXBB SoC MHU is supported.
Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
---
drivers/mailbox/Kconfig | 10 ++
drivers/mailbox/Makefile | 2 +
drivers/mailbox/platform_mhu.c | 205 +++++++++++++++++++++++++++++++++++++++++
3 files changed, 217 insertions(+)
create mode 100644 drivers/mailbox/platform_mhu.c
diff --git a/drivers/mailbox/Kconfig b/drivers/mailbox/Kconfig
index 97c3729..4cc93c6 100644
--- a/drivers/mailbox/Kconfig
+++ b/drivers/mailbox/Kconfig
@@ -15,6 +15,16 @@ config ARM_MHU
The controller has 3 mailbox channels, the last of which can be
used in Secure mode only.
+config PLATFORM_MHU
+ tristate "Platform MHU Mailbox"
+ depends on OF
+ depends on HAS_IOMEM
+ help
+ Say Y here if you want to build a platform specific variant MHU
+ controller driver.
+ The controller has a maximum of 3 mailbox channels, the last of
+ which can be used in Secure mode only.
+
config PL320_MBOX
bool "ARM PL320 Mailbox"
depends on ARM_AMBA
diff --git a/drivers/mailbox/Makefile b/drivers/mailbox/Makefile
index 66c38e3..ace6fed 100644
--- a/drivers/mailbox/Makefile
+++ b/drivers/mailbox/Makefile
@@ -6,6 +6,8 @@ obj-$(CONFIG_MAILBOX_TEST) += mailbox-test.o
obj-$(CONFIG_ARM_MHU) += arm_mhu.o
+obj-$(CONFIG_PLATFORM_MHU) += platform_mhu.o
+
obj-$(CONFIG_PL320_MBOX) += pl320-ipc.o
obj-$(CONFIG_OMAP2PLUS_MBOX) += omap-mailbox.o
diff --git a/drivers/mailbox/platform_mhu.c b/drivers/mailbox/platform_mhu.c
new file mode 100644
index 0000000..e13201a
--- /dev/null
+++ b/drivers/mailbox/platform_mhu.c
@@ -0,0 +1,205 @@
+/*
+ * Copyright (C) 2016 BayLibre SAS.
+ * Author: Neil Armstrong <narmstrong@baylibre.com>
+ * Synchronised with arm_mhu.c from :
+ * Copyright (C) 2013-2015 Fujitsu Semiconductor Ltd.
+ * Copyright (C) 2015 Linaro Ltd.
+ * Author: Jassi Brar <jaswinder.singh@linaro.org>
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, version 2 of the License.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ */
+
+#include <linux/interrupt.h>
+#include <linux/spinlock.h>
+#include <linux/mutex.h>
+#include <linux/delay.h>
+#include <linux/slab.h>
+#include <linux/err.h>
+#include <linux/io.h>
+#include <linux/module.h>
+#include <linux/platform_device.h>
+#include <linux/mailbox_controller.h>
+
+#define INTR_SET_OFS 0x0
+#define INTR_STAT_OFS 0x4
+#define INTR_CLR_OFS 0x8
+
+#define MHU_SEC_OFFSET 0x0
+#define MHU_LP_OFFSET 0xc
+#define MHU_HP_OFFSET 0x18
+#define TX_REG_OFFSET 0x24
+
+#define MHU_CHANS 3
+
+struct platform_mhu_link {
+ int irq;
+ void __iomem *tx_reg;
+ void __iomem *rx_reg;
+};
+
+struct platform_mhu {
+ void __iomem *base;
+ struct platform_mhu_link mlink[MHU_CHANS];
+ struct mbox_chan chan[MHU_CHANS];
+ struct mbox_controller mbox;
+};
+
+static irqreturn_t platform_mhu_rx_interrupt(int irq, void *p)
+{
+ struct mbox_chan *chan = p;
+ struct platform_mhu_link *mlink = chan->con_priv;
+ u32 val;
+
+ val = readl_relaxed(mlink->rx_reg + INTR_STAT_OFS);
+ if (!val)
+ return IRQ_NONE;
+
+ mbox_chan_received_data(chan, (void *)&val);
+
+ writel_relaxed(val, mlink->rx_reg + INTR_CLR_OFS);
+
+ return IRQ_HANDLED;
+}
+
+static bool platform_mhu_last_tx_done(struct mbox_chan *chan)
+{
+ struct platform_mhu_link *mlink = chan->con_priv;
+ u32 val = readl_relaxed(mlink->tx_reg + INTR_STAT_OFS);
+
+ return (val == 0);
+}
+
+static int platform_mhu_send_data(struct mbox_chan *chan, void *data)
+{
+ struct platform_mhu_link *mlink = chan->con_priv;
+ u32 *arg = data;
+
+ writel_relaxed(*arg, mlink->tx_reg + INTR_SET_OFS);
+
+ return 0;
+}
+
+static int platform_mhu_startup(struct mbox_chan *chan)
+{
+ struct platform_mhu_link *mlink = chan->con_priv;
+ u32 val;
+ int ret;
+
+ val = readl_relaxed(mlink->tx_reg + INTR_STAT_OFS);
+ writel_relaxed(val, mlink->tx_reg + INTR_CLR_OFS);
+
+ ret = request_irq(mlink->irq, platform_mhu_rx_interrupt,
+ IRQF_SHARED, "platform_mhu_link", chan);
+ if (ret) {
+ dev_err(chan->mbox->dev,
+ "Unable to acquire IRQ %d\n", mlink->irq);
+ return ret;
+ }
+
+ return 0;
+}
+
+static void platform_mhu_shutdown(struct mbox_chan *chan)
+{
+ struct platform_mhu_link *mlink = chan->con_priv;
+
+ free_irq(mlink->irq, chan);
+}
+
+static const struct mbox_chan_ops platform_mhu_ops = {
+ .send_data = platform_mhu_send_data,
+ .startup = platform_mhu_startup,
+ .shutdown = platform_mhu_shutdown,
+ .last_tx_done = platform_mhu_last_tx_done,
+};
+
+static int platform_mhu_probe(struct platform_device *pdev)
+{
+ int i, err;
+ struct platform_mhu *mhu;
+ struct device *dev = &pdev->dev;
+ struct resource *res;
+ int platform_mhu_reg[MHU_CHANS] = {
+ MHU_SEC_OFFSET, MHU_LP_OFFSET, MHU_HP_OFFSET
+ };
+
+ /* Allocate memory for device */
+ mhu = devm_kzalloc(dev, sizeof(*mhu), GFP_KERNEL);
+ if (!mhu)
+ return -ENOMEM;
+
+ res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
+ mhu->base = devm_ioremap_resource(dev, res);
+ if (IS_ERR(mhu->base)) {
+ dev_err(dev, "ioremap failed\n");
+ return PTR_ERR(mhu->base);
+ }
+
+ for (i = 0; i < MHU_CHANS; i++) {
+ mhu->chan[i].con_priv = &mhu->mlink[i];
+ mhu->mlink[i].irq = platform_get_irq(pdev, i);
+ if (mhu->mlink[i].irq < 0) {
+ dev_err(dev, "failed to get irq%d\n", i);
+ return mhu->mlink[i].irq;
+ }
+ mhu->mlink[i].rx_reg = mhu->base + platform_mhu_reg[i];
+ mhu->mlink[i].tx_reg = mhu->mlink[i].rx_reg + TX_REG_OFFSET;
+ }
+
+ mhu->mbox.dev = dev;
+ mhu->mbox.chans = &mhu->chan[0];
+ mhu->mbox.num_chans = MHU_CHANS;
+ mhu->mbox.ops = &platform_mhu_ops;
+ mhu->mbox.txdone_irq = false;
+ mhu->mbox.txdone_poll = true;
+ mhu->mbox.txpoll_period = 1;
+
+ platform_set_drvdata(pdev, mhu);
+
+ err = mbox_controller_register(&mhu->mbox);
+ if (err) {
+ dev_err(dev, "Failed to register mailboxes %d\n", err);
+ return err;
+ }
+
+ dev_info(dev, "Platform MHU Mailbox registered\n");
+ return 0;
+}
+
+static int platform_mhu_remove(struct platform_device *pdev)
+{
+ struct platform_mhu *mhu = platform_get_drvdata(pdev);
+
+ mbox_controller_unregister(&mhu->mbox);
+
+ return 0;
+}
+
+static const struct of_device_id platform_mhu_dt_ids[] = {
+ { .compatible = "amlogic,meson-gxbb-mhu", },
+ { /* sentinel */ },
+};
+MODULE_DEVICE_TABLE(of, platform_mhu_dt_ids);
+
+static struct platform_driver platform_mhu_driver = {
+ .probe = platform_mhu_probe,
+ .remove = platform_mhu_remove,
+ .driver = {
+ .name = "platform-mhu",
+ .of_match_table = platform_mhu_dt_ids,
+ },
+};
+
+module_platform_driver(platform_mhu_driver);
+
+MODULE_LICENSE("GPL v2");
+MODULE_ALIAS("platform:platform-mhu");
+MODULE_DESCRIPTION("Platform MHU Driver");
+MODULE_AUTHOR("Neil Armstrong <narmstrong@baylibre.com>");
--
1.9.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH v2 2/3] dt-bindings: mailbox: Add Amlogic Meson MHU Bindings
2016-08-18 10:10 [PATCH v2 0/3] Add Platform MHU mailbox driver for Amlogic GXBB Neil Armstrong
2016-08-18 10:10 ` [PATCH v2 1/3] mailbox: Add Platform Message-Handling-Unit variant driver Neil Armstrong
@ 2016-08-18 10:10 ` Neil Armstrong
2016-08-18 10:10 ` [PATCH v2 3/3] ARM64: dts: meson-gxbb: Add Meson MHU Node Neil Armstrong
2016-09-02 23:34 ` [PATCH v2 0/3] Add Platform MHU mailbox driver for Amlogic GXBB Kevin Hilman
3 siblings, 0 replies; 10+ messages in thread
From: Neil Armstrong @ 2016-08-18 10:10 UTC (permalink / raw)
To: linux-kernel, linux-amlogic, linux-arm-kernel, jassisinghbrar,
devicetree
Cc: Neil Armstrong
Acked-by: Rob Herring <robh@kernel.org>
Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
---
.../devicetree/bindings/mailbox/meson-mhu.txt | 34 ++++++++++++++++++++++
1 file changed, 34 insertions(+)
create mode 100644 Documentation/devicetree/bindings/mailbox/meson-mhu.txt
diff --git a/Documentation/devicetree/bindings/mailbox/meson-mhu.txt b/Documentation/devicetree/bindings/mailbox/meson-mhu.txt
new file mode 100644
index 0000000..a530310
--- /dev/null
+++ b/Documentation/devicetree/bindings/mailbox/meson-mhu.txt
@@ -0,0 +1,34 @@
+Amlogic Meson MHU Mailbox Driver
+================================
+
+The Amlogic's Meson SoCs Message-Handling-Unit (MHU) is a mailbox controller
+that has 3 independent channels/links to communicate with remote processor(s).
+MHU links are hardwired on a platform. A link raises interrupt for any
+received data. However, there is no specified way of knowing if the sent
+data has been read by the remote. This driver assumes the sender polls
+STAT register and the remote clears it after having read the data.
+
+Mailbox Device Node:
+====================
+
+Required properties:
+--------------------
+- compatible: Shall be "amlogic,meson-gxbb-mhu"
+- reg: Contains the mailbox register address range (base
+ address and length)
+- #mbox-cells Shall be 1 - the index of the channel needed.
+- interrupts: Contains the interrupt information corresponding to
+ each of the 2 links of MHU.
+
+Example:
+--------
+
+ mailbox: mailbox@c883c404 {
+ #mbox-cells = <1>;
+ compatible = "amlogic,meson-gxbb-mhu";
+ reg = <0 0xc883c404 0 0x4c>;
+ interrupts = <0 208 IRQ_TYPE_EDGE_RISING>,
+ <0 209 IRQ_TYPE_EDGE_RISING>,
+ <0 210 IRQ_TYPE_EDGE_RISING>;
+ #mbox-cells = <1>;
+ };
--
1.9.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH v2 3/3] ARM64: dts: meson-gxbb: Add Meson MHU Node
2016-08-18 10:10 [PATCH v2 0/3] Add Platform MHU mailbox driver for Amlogic GXBB Neil Armstrong
2016-08-18 10:10 ` [PATCH v2 1/3] mailbox: Add Platform Message-Handling-Unit variant driver Neil Armstrong
2016-08-18 10:10 ` [PATCH v2 2/3] dt-bindings: mailbox: Add Amlogic Meson MHU Bindings Neil Armstrong
@ 2016-08-18 10:10 ` Neil Armstrong
2016-09-07 20:08 ` Kevin Hilman
2016-09-02 23:34 ` [PATCH v2 0/3] Add Platform MHU mailbox driver for Amlogic GXBB Kevin Hilman
3 siblings, 1 reply; 10+ messages in thread
From: Neil Armstrong @ 2016-08-18 10:10 UTC (permalink / raw)
To: linux-kernel, linux-amlogic, linux-arm-kernel, jassisinghbrar,
devicetree
Cc: Neil Armstrong
Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
---
arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi | 9 +++++++++
1 file changed, 9 insertions(+)
diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi
index e502c24..bdbf6e7 100644
--- a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi
+++ b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi
@@ -321,6 +321,15 @@
#clock-cells = <1>;
reg = <0x0 0x0 0x0 0x3db>;
};
+
+ mailbox: mailbox@404 {
+ compatible = "amlogic,meson-gxbb-mhu";
+ reg = <0 0x404 0 0x4c>;
+ interrupts = <0 208 IRQ_TYPE_EDGE_RISING>,
+ <0 209 IRQ_TYPE_EDGE_RISING>,
+ <0 210 IRQ_TYPE_EDGE_RISING>;
+ #mbox-cells = <1>;
+ };
};
apb: apb@d0000000 {
--
1.9.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH v2 0/3] Add Platform MHU mailbox driver for Amlogic GXBB
2016-08-18 10:10 [PATCH v2 0/3] Add Platform MHU mailbox driver for Amlogic GXBB Neil Armstrong
` (2 preceding siblings ...)
2016-08-18 10:10 ` [PATCH v2 3/3] ARM64: dts: meson-gxbb: Add Meson MHU Node Neil Armstrong
@ 2016-09-02 23:34 ` Kevin Hilman
2016-09-03 5:33 ` Jassi Brar
3 siblings, 1 reply; 10+ messages in thread
From: Kevin Hilman @ 2016-09-02 23:34 UTC (permalink / raw)
To: jassisinghbrar
Cc: linux-kernel, linux-amlogic, linux-arm-kernel, Neil Armstrong
Hi Jassi,
Neil Armstrong <narmstrong@baylibre.com> writes:
> In order to support Mailbox links for the Amlogic GXBB SoC, add a generic
> platform MHU driver based on arm_mhu.c.
>
> This patchset follows a RFC thread along the GXBB SCPI support at :
> http://lkml.kernel.org/r/1466503374-28841-1-git-send-email-narmstrong@baylibre.com
> And specific MHU discussions at :
> http://lkml.kernel.org/r/CABb+yY3HqJG2+GMWCWF9PomxobrwWGZ=TZe5NvXPcHMDdLhkcQ@mail.gmail.com
>
> Changes since v1 at http://lkml.kernel.org/r/1470732737-18391-1-git-send-email-narmstrong@baylibre.com :
> - Fix irq to signed to detect platform_get_irq() failures
> - Introduced back the secure channel
> - Fixed indexes
How is this version looking to you?
With your review/ack on the driver, I'd be happy to take it via the
amlogic tree as there shouldn't be any conflicts with your tree.
Kevin
> Changes since RFC v2 :
> - Rename to platform_mhu
> - Sync all link functions with arm_mhu
>
> Neil Armstrong (3):
> mailbox: Add Platform Message-Handling-Unit variant driver
> dt-bindings: mailbox: Add Amlogic Meson MHU Bindings
> ARM64: dts: meson-gxbb: Add Meson MHU Node
>
> .../devicetree/bindings/mailbox/meson-mhu.txt | 34 ++++
> arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi | 9 +
> drivers/mailbox/Kconfig | 10 +
> drivers/mailbox/Makefile | 2 +
> drivers/mailbox/platform_mhu.c | 205 +++++++++++++++++++++
> 5 files changed, 260 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/mailbox/meson-mhu.txt
> create mode 100644 drivers/mailbox/platform_mhu.c
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2 0/3] Add Platform MHU mailbox driver for Amlogic GXBB
2016-09-02 23:34 ` [PATCH v2 0/3] Add Platform MHU mailbox driver for Amlogic GXBB Kevin Hilman
@ 2016-09-03 5:33 ` Jassi Brar
2016-09-03 20:45 ` Kevin Hilman
0 siblings, 1 reply; 10+ messages in thread
From: Jassi Brar @ 2016-09-03 5:33 UTC (permalink / raw)
To: Kevin Hilman
Cc: Linux Kernel Mailing List, open list:ARM/Amlogic Meson...,
linux-arm-kernel@lists.infradead.org, Neil Armstrong
On Sat, Sep 3, 2016 at 5:04 AM, Kevin Hilman <khilman@baylibre.com> wrote:
> Hi Jassi,
>
> Neil Armstrong <narmstrong@baylibre.com> writes:
>
>> In order to support Mailbox links for the Amlogic GXBB SoC, add a generic
>> platform MHU driver based on arm_mhu.c.
>>
>> This patchset follows a RFC thread along the GXBB SCPI support at :
>> http://lkml.kernel.org/r/1466503374-28841-1-git-send-email-narmstrong@baylibre.com
>> And specific MHU discussions at :
>> http://lkml.kernel.org/r/CABb+yY3HqJG2+GMWCWF9PomxobrwWGZ=TZe5NvXPcHMDdLhkcQ@mail.gmail.com
>>
>> Changes since v1 at http://lkml.kernel.org/r/1470732737-18391-1-git-send-email-narmstrong@baylibre.com :
>> - Fix irq to signed to detect platform_get_irq() failures
>> - Introduced back the secure channel
>> - Fixed indexes
>
> How is this version looking to you?
>
> With your review/ack on the driver, I'd be happy to take it via the
> amlogic tree as there shouldn't be any conflicts with your tree.
>
I am ok with the driver, but I don't understand how it helps going via
amlogic tree. I would like to send a pull request every release ;)
cheers.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2 0/3] Add Platform MHU mailbox driver for Amlogic GXBB
2016-09-03 5:33 ` Jassi Brar
@ 2016-09-03 20:45 ` Kevin Hilman
2016-09-04 5:35 ` Jassi Brar
0 siblings, 1 reply; 10+ messages in thread
From: Kevin Hilman @ 2016-09-03 20:45 UTC (permalink / raw)
To: Jassi Brar
Cc: Linux Kernel Mailing List, open list:ARM/Amlogic Meson...,
linux-arm-kernel@lists.infradead.org, Neil Armstrong
On Fri, Sep 2, 2016 at 10:33 PM, Jassi Brar <jassisinghbrar@gmail.com> wrote:
> On Sat, Sep 3, 2016 at 5:04 AM, Kevin Hilman <khilman@baylibre.com> wrote:
>> Hi Jassi,
>>
>> Neil Armstrong <narmstrong@baylibre.com> writes:
>>
>>> In order to support Mailbox links for the Amlogic GXBB SoC, add a generic
>>> platform MHU driver based on arm_mhu.c.
>>>
>>> This patchset follows a RFC thread along the GXBB SCPI support at :
>>> http://lkml.kernel.org/r/1466503374-28841-1-git-send-email-narmstrong@baylibre.com
>>> And specific MHU discussions at :
>>> http://lkml.kernel.org/r/CABb+yY3HqJG2+GMWCWF9PomxobrwWGZ=TZe5NvXPcHMDdLhkcQ@mail.gmail.com
>>>
>>> Changes since v1 at http://lkml.kernel.org/r/1470732737-18391-1-git-send-email-narmstrong@baylibre.com :
>>> - Fix irq to signed to detect platform_get_irq() failures
>>> - Introduced back the secure channel
>>> - Fixed indexes
>>
>> How is this version looking to you?
>>
>> With your review/ack on the driver, I'd be happy to take it via the
>> amlogic tree as there shouldn't be any conflicts with your tree.
>>
> I am ok with the driver, but I don't understand how it helps going via
> amlogic tree. I would like to send a pull request every release ;)
That's fine with me too. I offered tot take int just in case you had
already sent pull requests for this cycle, and also since I'll take
the DT changes through arm-soc.
Either way is fine with me. Just let me know.
Thanks,
Kevin
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2 0/3] Add Platform MHU mailbox driver for Amlogic GXBB
2016-09-03 20:45 ` Kevin Hilman
@ 2016-09-04 5:35 ` Jassi Brar
2016-09-06 17:56 ` Kevin Hilman
0 siblings, 1 reply; 10+ messages in thread
From: Jassi Brar @ 2016-09-04 5:35 UTC (permalink / raw)
To: Kevin Hilman
Cc: Linux Kernel Mailing List, open list:ARM/Amlogic Meson...,
linux-arm-kernel@lists.infradead.org, Neil Armstrong
On Sun, Sep 4, 2016 at 2:15 AM, Kevin Hilman <khilman@baylibre.com> wrote:
> On Fri, Sep 2, 2016 at 10:33 PM, Jassi Brar <jassisinghbrar@gmail.com> wrote:
>> On Sat, Sep 3, 2016 at 5:04 AM, Kevin Hilman <khilman@baylibre.com> wrote:
>>> Hi Jassi,
>>>
>>> Neil Armstrong <narmstrong@baylibre.com> writes:
>>>
>>>> In order to support Mailbox links for the Amlogic GXBB SoC, add a generic
>>>> platform MHU driver based on arm_mhu.c.
>>>>
>>>> This patchset follows a RFC thread along the GXBB SCPI support at :
>>>> http://lkml.kernel.org/r/1466503374-28841-1-git-send-email-narmstrong@baylibre.com
>>>> And specific MHU discussions at :
>>>> http://lkml.kernel.org/r/CABb+yY3HqJG2+GMWCWF9PomxobrwWGZ=TZe5NvXPcHMDdLhkcQ@mail.gmail.com
>>>>
>>>> Changes since v1 at http://lkml.kernel.org/r/1470732737-18391-1-git-send-email-narmstrong@baylibre.com :
>>>> - Fix irq to signed to detect platform_get_irq() failures
>>>> - Introduced back the secure channel
>>>> - Fixed indexes
>>>
>>> How is this version looking to you?
>>>
>>> With your review/ack on the driver, I'd be happy to take it via the
>>> amlogic tree as there shouldn't be any conflicts with your tree.
>>>
>> I am ok with the driver, but I don't understand how it helps going via
>> amlogic tree. I would like to send a pull request every release ;)
>
> That's fine with me too. I offered tot take int just in case you had
> already sent pull requests for this cycle, and also since I'll take
> the DT changes through arm-soc.
>
This revision came after rc2, so it wasn't included. I will queue
patch-1 & 2 for next release, and you the patch-3.
Thanks.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2 0/3] Add Platform MHU mailbox driver for Amlogic GXBB
2016-09-04 5:35 ` Jassi Brar
@ 2016-09-06 17:56 ` Kevin Hilman
0 siblings, 0 replies; 10+ messages in thread
From: Kevin Hilman @ 2016-09-06 17:56 UTC (permalink / raw)
To: Jassi Brar
Cc: Linux Kernel Mailing List, open list:ARM/Amlogic Meson...,
linux-arm-kernel@lists.infradead.org, Neil Armstrong
Jassi Brar <jassisinghbrar@gmail.com> writes:
> On Sun, Sep 4, 2016 at 2:15 AM, Kevin Hilman <khilman@baylibre.com> wrote:
>> On Fri, Sep 2, 2016 at 10:33 PM, Jassi Brar <jassisinghbrar@gmail.com> wrote:
>>> On Sat, Sep 3, 2016 at 5:04 AM, Kevin Hilman <khilman@baylibre.com> wrote:
>>>> Hi Jassi,
>>>>
>>>> Neil Armstrong <narmstrong@baylibre.com> writes:
>>>>
>>>>> In order to support Mailbox links for the Amlogic GXBB SoC, add a generic
>>>>> platform MHU driver based on arm_mhu.c.
>>>>>
>>>>> This patchset follows a RFC thread along the GXBB SCPI support at :
>>>>> http://lkml.kernel.org/r/1466503374-28841-1-git-send-email-narmstrong@baylibre.com
>>>>> And specific MHU discussions at :
>>>>> http://lkml.kernel.org/r/CABb+yY3HqJG2+GMWCWF9PomxobrwWGZ=TZe5NvXPcHMDdLhkcQ@mail.gmail.com
>>>>>
>>>>> Changes since v1 at http://lkml.kernel.org/r/1470732737-18391-1-git-send-email-narmstrong@baylibre.com :
>>>>> - Fix irq to signed to detect platform_get_irq() failures
>>>>> - Introduced back the secure channel
>>>>> - Fixed indexes
>>>>
>>>> How is this version looking to you?
>>>>
>>>> With your review/ack on the driver, I'd be happy to take it via the
>>>> amlogic tree as there shouldn't be any conflicts with your tree.
>>>>
>>> I am ok with the driver, but I don't understand how it helps going via
>>> amlogic tree. I would like to send a pull request every release ;)
>>
>> That's fine with me too. I offered tot take int just in case you had
>> already sent pull requests for this cycle, and also since I'll take
>> the DT changes through arm-soc.
>>
> This revision came after rc2, so it wasn't included. I will queue
> patch-1 & 2 for next release, and you the patch-3.
OK, sounds good. Thanks,
Kevin
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH v2 3/3] ARM64: dts: meson-gxbb: Add Meson MHU Node
2016-08-18 10:10 ` [PATCH v2 3/3] ARM64: dts: meson-gxbb: Add Meson MHU Node Neil Armstrong
@ 2016-09-07 20:08 ` Kevin Hilman
0 siblings, 0 replies; 10+ messages in thread
From: Kevin Hilman @ 2016-09-07 20:08 UTC (permalink / raw)
To: Neil Armstrong
Cc: linux-kernel, linux-amlogic, linux-arm-kernel, jassisinghbrar,
devicetree
Neil Armstrong <narmstrong@baylibre.com> writes:
> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
> ---
> arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi | 9 +++++++++
> 1 file changed, 9 insertions(+)
Applied,
Kevin
> diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi
> index e502c24..bdbf6e7 100644
> --- a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi
> +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi
> @@ -321,6 +321,15 @@
> #clock-cells = <1>;
> reg = <0x0 0x0 0x0 0x3db>;
> };
> +
> + mailbox: mailbox@404 {
> + compatible = "amlogic,meson-gxbb-mhu";
> + reg = <0 0x404 0 0x4c>;
> + interrupts = <0 208 IRQ_TYPE_EDGE_RISING>,
> + <0 209 IRQ_TYPE_EDGE_RISING>,
> + <0 210 IRQ_TYPE_EDGE_RISING>;
> + #mbox-cells = <1>;
> + };
> };
>
> apb: apb@d0000000 {
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2016-09-07 20:08 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-08-18 10:10 [PATCH v2 0/3] Add Platform MHU mailbox driver for Amlogic GXBB Neil Armstrong
2016-08-18 10:10 ` [PATCH v2 1/3] mailbox: Add Platform Message-Handling-Unit variant driver Neil Armstrong
2016-08-18 10:10 ` [PATCH v2 2/3] dt-bindings: mailbox: Add Amlogic Meson MHU Bindings Neil Armstrong
2016-08-18 10:10 ` [PATCH v2 3/3] ARM64: dts: meson-gxbb: Add Meson MHU Node Neil Armstrong
2016-09-07 20:08 ` Kevin Hilman
2016-09-02 23:34 ` [PATCH v2 0/3] Add Platform MHU mailbox driver for Amlogic GXBB Kevin Hilman
2016-09-03 5:33 ` Jassi Brar
2016-09-03 20:45 ` Kevin Hilman
2016-09-04 5:35 ` Jassi Brar
2016-09-06 17:56 ` Kevin Hilman
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox