From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jassi Brar Subject: Re: [PATCH v4 3/5] soc: qcom: Introduce APCS IPC driver Date: Tue, 16 May 2017 16:55:09 +0530 Message-ID: References: <20170506011920.GH15143@minitux> <20170508055439.GJ15143@minitux> <20170508191154.GM15143@minitux> <20170509191106.GQ15143@minitux> <20170510190046.GT15143@minitux> <20170512224840.GW15143@minitux> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: In-Reply-To: <20170512224840.GW15143@minitux> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Bjorn Andersson Cc: Jeffrey Hugo , Andy Gross , Rob Herring , Mark Rutland , Ohad Ben-Cohen , linux-arm-msm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-soc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Devicetree List , Linux Kernel Mailing List , linux-remoteproc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-arm-msm@vger.kernel.org On Sat, May 13, 2017 at 4:18 AM, Bjorn Andersson wrote: > On Wed 10 May 19:07 PDT 2017, Jassi Brar wrote: > >> On Thu, May 11, 2017 at 12:30 AM, Bjorn Andersson >> wrote: >> > On Tue 09 May 19:33 PDT 2017, Jassi Brar wrote: > [..] >> > So please let me know what you think about [1], if you don't like it >> > I'll fix the things pointed to by Stephen and we'll have to live with >> > the two calls. >> > >> My last reply was about [1]. Other platforms call >> mbox_send_data()+mbox_client_txdone() see >> drivers/firmware/tegra/bpmp.c, but you want to introduce another API >> in the innards of the framework. > > Okay, lets go with that then. I will incorporate the changes requested > by Stephen and post a final version and then add the > mbox_client_txdone() in the clients. > OK cool. And please update the commit log discarding "The driver implements the "doorbell" binding and could be used as basis for a new Linux framework, if found useful outside Qualcomm." Cheers! -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html