Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1330198

Re: [PATCH 1/2] Documentation: dt: mailbox: Add TI Message Manager

Show all headers | View raw

On 02/08/2016 10:14 PM, Jassi Brar wrote:

Thanks for the review.

> On Fri, Feb 5, 2016 at 10:04 PM, Nishanth Menon <nm@ti.com> wrote:
>> +
>> +       msgmgr: msgmgr@02a00000 {
>> +               compatible = "ti,k2g-message-manager", "ti,message-manager";
>> +               #mbox-cells = <1>;
>> +               reg-names = "queue_proxy_region", "queue_state_debug_region";
>> +               reg = <0x02a00000 0x400000>, <0x028c3400 0x400>;
>> +
>> +               msgmgr_proxy_pmmc_tx_prio0: pmmc_tx_prio0 {
>> +                       ti,queue-id = <0>;
>> +                       ti,proxy-id = <0>;
>> +               };
>> +
>> +               msgmgr_proxy_pmmc_rx: pmmc_rx {
>> +                       ti,queue-id = <5>;
>> +                       ti,proxy-id = <2>;
>> +                       interrupt-names = "rx";
>> +                       interrupts = <GIC_SPI 324 IRQ_TYPE_EDGE_RISING>;
>> +               };
>> +       };
>> +
> I think we should get rid of consumer specifics from the provider node...


If I get rid of the consumer nodes, how do you propose I describe the rx
queue interrupt(s) in the msmgr dt node (Every Rx queue will have it's
own interrupt - and it cannot be reverse computed from queue ID, proxy ID)?


>> +...
>> +       pmmc {
>> +               ...
>> +               mbox-names = "tx", "rx";
>> +               mboxes = <&msgmgr &msgmgr_proxy_pmmc_tx>
>> +                        <&msgmgr &msgmgr_proxy_pmmc_rx>;
>> +               ...
>> +       };
>>
> ... and have consumers like
>        pmmc {
>                ...
>                mbox-names = "tx", "rx";
>                mboxes = <&msgmgr 0 0>
>                         <&msgmgr 5 2>;
>        };
> 
> I leave the IRQ for you to decide how to specify - a 'dummy' or
> 'valid' always provided as last cell in mboxes or some other way.
> (I'll review other patches in detail later)

What do we do with the issues that Suman pointed out in the mailbox
framework itself? Could you respond to that thread[1] as well?


[1] http://marc.info/?l=devicetree&m=145496308418123&w=2

-- 
Regards,
Nishanth Menon

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 1/2] Documentation: dt: mailbox: Add TI Message Manager Nishanth Menon <nm@ti.com> - 2016-02-05 17:40 +0100
  Re: [PATCH 1/2] Documentation: dt: mailbox: Add TI Message Manager Rob Herring <robh@kernel.org> - 2016-02-08 20:40 +0100
    Re: [PATCH 1/2] Documentation: dt: mailbox: Add TI Message Manager Suman Anna <s-anna@ti.com> - 2016-02-08 21:30 +0100
    Re: [PATCH 1/2] Documentation: dt: mailbox: Add TI Message Manager Nishanth Menon <nm@ti.com> - 2016-02-08 21:40 +0100
  Re: [PATCH 1/2] Documentation: dt: mailbox: Add TI Message Manager Jassi Brar <jassisinghbrar@gmail.com> - 2016-02-09 05:20 +0100
    Re: [PATCH 1/2] Documentation: dt: mailbox: Add TI Message Manager Nishanth Menon <nm@ti.com> - 2016-02-09 13:40 +0100
      Re: [PATCH 1/2] Documentation: dt: mailbox: Add TI Message Manager Jassi Brar <jassisinghbrar@gmail.com> - 2016-02-09 16:00 +0100
        Re: [PATCH 1/2] Documentation: dt: mailbox: Add TI Message Manager Jassi Brar <jassisinghbrar@gmail.com> - 2016-02-09 16:40 +0100
        Re: [PATCH 1/2] Documentation: dt: mailbox: Add TI Message Manager Nishanth Menon <nm@ti.com> - 2016-02-09 16:50 +0100
          Re: [PATCH 1/2] Documentation: dt: mailbox: Add TI Message Manager Nishanth Menon <nm@ti.com> - 2016-02-09 19:20 +0100
            Re: [PATCH 1/2] Documentation: dt: mailbox: Add TI Message Manager Suman Anna <s-anna@ti.com> - 2016-02-10 21:20 +0100
              Re: [PATCH 1/2] Documentation: dt: mailbox: Add TI Message Manager Nishanth Menon <nm@ti.com> - 2016-02-10 22:00 +0100
            Re: [PATCH 1/2] Documentation: dt: mailbox: Add TI Message Manager Jassi Brar <jassisinghbrar@gmail.com> - 2016-02-11 05:30 +0100
              Re: [PATCH 1/2] Documentation: dt: mailbox: Add TI Message Manager Nishanth Menon <nm@ti.com> - 2016-02-11 06:10 +0100

csiph-web