Re: [PATCH v2 0/3] Add Platform MHU mailbox driver for Amlogic GXBB

2016-09-06 Thread Kevin Hilman
Jassi Brar  writes:

> On Sun, Sep 4, 2016 at 2:15 AM, Kevin Hilman  wrote:
>> On Fri, Sep 2, 2016 at 10:33 PM, Jassi Brar  wrote:
>>> On Sat, Sep 3, 2016 at 5:04 AM, Kevin Hilman  wrote:
 Hi Jassi,

 Neil Armstrong  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-narmstr...@baylibre.com
> And specific MHU discussions at :
> http://lkml.kernel.org/r/CABb+yY3HqJG2+GMWCWF9PomxobrwWGZ=tze5nvxpchmddlh...@mail.gmail.com
>
> Changes since v1 at 
> http://lkml.kernel.org/r/1470732737-18391-1-git-send-email-narmstr...@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


Re: [PATCH v2 0/3] Add Platform MHU mailbox driver for Amlogic GXBB

2016-09-03 Thread Jassi Brar
On Sun, Sep 4, 2016 at 2:15 AM, Kevin Hilman  wrote:
> On Fri, Sep 2, 2016 at 10:33 PM, Jassi Brar  wrote:
>> On Sat, Sep 3, 2016 at 5:04 AM, Kevin Hilman  wrote:
>>> Hi Jassi,
>>>
>>> Neil Armstrong  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-narmstr...@baylibre.com
 And specific MHU discussions at :
 http://lkml.kernel.org/r/CABb+yY3HqJG2+GMWCWF9PomxobrwWGZ=tze5nvxpchmddlh...@mail.gmail.com

 Changes since v1 at 
 http://lkml.kernel.org/r/1470732737-18391-1-git-send-email-narmstr...@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.


Re: [PATCH v2 0/3] Add Platform MHU mailbox driver for Amlogic GXBB

2016-09-03 Thread Kevin Hilman
On Fri, Sep 2, 2016 at 10:33 PM, Jassi Brar  wrote:
> On Sat, Sep 3, 2016 at 5:04 AM, Kevin Hilman  wrote:
>> Hi Jassi,
>>
>> Neil Armstrong  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-narmstr...@baylibre.com
>>> And specific MHU discussions at :
>>> http://lkml.kernel.org/r/CABb+yY3HqJG2+GMWCWF9PomxobrwWGZ=tze5nvxpchmddlh...@mail.gmail.com
>>>
>>> Changes since v1 at 
>>> http://lkml.kernel.org/r/1470732737-18391-1-git-send-email-narmstr...@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


Re: [PATCH v2 0/3] Add Platform MHU mailbox driver for Amlogic GXBB

2016-09-02 Thread Jassi Brar
On Sat, Sep 3, 2016 at 5:04 AM, Kevin Hilman  wrote:
> Hi Jassi,
>
> Neil Armstrong  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-narmstr...@baylibre.com
>> And specific MHU discussions at :
>> http://lkml.kernel.org/r/CABb+yY3HqJG2+GMWCWF9PomxobrwWGZ=tze5nvxpchmddlh...@mail.gmail.com
>>
>> Changes since v1 at 
>> http://lkml.kernel.org/r/1470732737-18391-1-git-send-email-narmstr...@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.


Re: [PATCH v2 0/3] Add Platform MHU mailbox driver for Amlogic GXBB

2016-09-02 Thread Kevin Hilman
Hi Jassi,

Neil Armstrong  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-narmstr...@baylibre.com
> And specific MHU discussions at :
> http://lkml.kernel.org/r/CABb+yY3HqJG2+GMWCWF9PomxobrwWGZ=tze5nvxpchmddlh...@mail.gmail.com
>
> Changes since v1 at 
> http://lkml.kernel.org/r/1470732737-18391-1-git-send-email-narmstr...@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


[PATCH v2 0/3] Add Platform MHU mailbox driver for Amlogic GXBB

2016-08-18 Thread 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-narmstr...@baylibre.com
And specific MHU discussions at :
http://lkml.kernel.org/r/CABb+yY3HqJG2+GMWCWF9PomxobrwWGZ=tze5nvxpchmddlh...@mail.gmail.com

Changes since v1 at 
http://lkml.kernel.org/r/1470732737-18391-1-git-send-email-narmstr...@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