Re: [RFC 5/7] soc: qcom: Add Shared Memory Driver

2014-10-30 Thread Bjorn Andersson
On Thu 30 Oct 06:34 PDT 2014, Ohad Ben-Cohen wrote: > Hi Bjorn, > > On Thu, Oct 30, 2014 at 2:38 AM, Bjorn Andersson > wrote: > > My concern is that merging SMD into rpmsg will still give us two different > > solutions, except that they will be deeply tangled. > > I agree. > > It's very clear

Re: [RFC 5/7] soc: qcom: Add Shared Memory Driver

2014-10-30 Thread Ohad Ben-Cohen
Hi Bjorn, On Thu, Oct 30, 2014 at 2:38 AM, Bjorn Andersson wrote: > My concern is that merging SMD into rpmsg will still give us two different > solutions, except that they will be deeply tangled. I agree. It's very clear that SMD and rpmsg are different. But fundamentally, they do exactly the

Re: [RFC 5/7] soc: qcom: Add Shared Memory Driver

2014-10-29 Thread Bjorn Andersson
On Wed 29 Oct 07:28 PDT 2014, Ohad Ben-Cohen wrote: > [Thanks Kevin for the heads up on this] > > Hi Bjorn, > Hi Ohad, > On Tue, Sep 30, 2014 at 3:34 AM, Bjorn Andersson > wrote: > > == Codeaurora implementation == > > The codeaurora implementation originates from an implementation that was

Re: [RFC 5/7] soc: qcom: Add Shared Memory Driver

2014-10-29 Thread Ohad Ben-Cohen
[Thanks Kevin for the heads up on this] Hi Bjorn, On Tue, Sep 30, 2014 at 3:34 AM, Bjorn Andersson wrote: > == Codeaurora implementation == > The codeaurora implementation originates from an implementation that was > based purely on initcall and global state Do you refer to the in-tree arch/ar

Re: [RFC 5/7] soc: qcom: Add Shared Memory Driver

2014-10-03 Thread Bjorn Andersson
On Thu 02 Oct 15:38 PDT 2014, Stephen Boyd wrote: > On 09/29/14 17:34, Bjorn Andersson wrote: > > + > > +#define GET_RX_CHANNEL_INFO(channel, param) \ > > + (channel->rx_info_word ? \ > > + channel->rx_info_word->param : \ > > + channel->rx_info->param) > > + > > +#define GET

Re: [RFC 5/7] soc: qcom: Add Shared Memory Driver

2014-10-02 Thread Stephen Boyd
On 09/29/14 17:34, Bjorn Andersson wrote: > + > +#define GET_RX_CHANNEL_INFO(channel, param) \ > + (channel->rx_info_word ? \ > + channel->rx_info_word->param : \ > + channel->rx_info->param) > + > +#define GET_TX_CHANNEL_INFO(channel, param) \ > + (channel->rx_info_

[RFC 5/7] soc: qcom: Add Shared Memory Driver

2014-09-29 Thread Bjorn Andersson
This adds the Qualcomm Shared Memory Driver (SMD) providing communication channels to remote processors, ontop of SMEM. Signed-off-by: Bjorn Andersson --- As we've already seen this is fuel for discussions, so I hope to start these off by giving some insight in my research here. == Codeaurora i