Path: csiph.com!news.redatomik.org!aioe.org!bofh.it!news.nic.it!robomod From: Jassi Brar Newsgroups: linux.kernel Subject: Re: [PATCH v4 3/5] soc: qcom: Introduce APCS IPC driver Date: Tue, 16 May 2017 13:30:01 +0200 Message-ID: References: Dkim-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=wjjgeqfRtdrSG53eHJ5hoD8wigena20LXAbJsjs3Lkk=; b=XN2jX7eo0u5Cgm7aHd1uw8/Y1EEzT5CTj54dZoQRRjUEW0NOpJynTXeOS4JuwE8uQD xR1cYIOsWgJ5imDZnvp9ywT1TzS47CSFkZQPkEXOY44kYMEPJ+6ccqMObdM48ruemz0Q XjmAF9LFWay8GYBjplXaWuh4NEIliMjPAYsWwQ59OloxM1TWSsPq+rmnN0UhxrvS2lYW GH9shcIpfxEiO1vDNH0WRQJGRPiOXClLGM6b2FxZ0qju+yCHoYYRGI74829vH48RDrrZ b1YmTksbbrxn34fq5WqKMUkl6iqE/4UhYH+8HV8kKwi4vZua/qsLqxbQiuw2eBvkxf00 /sLw== X-Google-Dkim-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=wjjgeqfRtdrSG53eHJ5hoD8wigena20LXAbJsjs3Lkk=; b=UWx76YwV/Rjc5Br0lGfyo17TrfaBYsBCm6Rx8iVxh1hgfps4Kp1zB4BVtsGFByX2WE WP69f3lxRfUjYV/VZzzsrhYgQSWMngRWm6K0Hz7keQdVy1O0r67kI7oFusI+uHXKc4dv VyoxD4EA3Il6KtWeXiw3zETaVqNRil2L0ow//V9zACsJLUOYTaIBoG2F420EhKHkKCEB AAAJl6n/Uqedp31s36ws446yTMWBMLa4qC8S5hqSPhnMxuO/WurOh0Knb3Bh+QcgzeAI ZkiTdy6b+ouZbTmM6dYYJp3I6Mbe9sMOQx5+JAhwD/X1yGD7Gl9E0BPw9QdJryDGP+QF O0Ug== X-Gm-Message-State: AODbwcC3NfDzK4nKnv6MfXM7NwX8yETDrGGDg2HUMwlSoI8251WQC0V7 4BBcsWx0WfT4C3aI9Qc2mpi1gGxtXg== X-Received: by 10.80.153.43 with SMTP id k40mr8183616edb.63.1494933910370; Tue, 16 May 2017 04:25:10 -0700 (PDT) MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Sender: robomod@news.nic.it List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Approved: robomod@news.nic.it Lines: 26 Organization: linux.* mail to news gateway X-Original-Cc: Jeffrey Hugo , Andy Gross , Rob Herring , Mark Rutland , Ohad Ben-Cohen , linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org, Devicetree List , Linux Kernel Mailing List , linux-remoteproc@vger.kernel.org X-Original-Date: Tue, 16 May 2017 16:55:09 +0530 X-Original-Message-ID: X-Original-References: <20170506011920.GH15143@minitux> <20170508055439.GJ15143@minitux> <20170508191154.GM15143@minitux> <20170509191106.GQ15143@minitux> <20170510190046.GT15143@minitux> <20170512224840.GW15143@minitux> X-Original-Sender: linux-kernel-owner@vger.kernel.org Xref: csiph.com linux.kernel:1642425 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!