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


Groups > linux.kernel > #1634842

Re: [PATCH v3 2/4] soc: qcom: Introduce APCS IPC driver

From Jassi Brar <jassisinghbrar@gmail.com>
Newsgroups linux.kernel
Subject Re: [PATCH v3 2/4] soc: qcom: Introduce APCS IPC driver
Date 2017-05-03 12:00 +0200
Message-ID <tCZeF-3tF-7@gated-at.bofh.it> (permalink)
References <tCVb4-sO-3@gated-at.bofh.it> <tCVb4-sO-5@gated-at.bofh.it> <tCYLE-3gi-25@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Loic, thanks for adding me.

On Wed, May 3, 2017 at 2:58 PM, Loic PALLARDY <loic.pallardy@st.com> wrote:
>
>
>> -----Original Message-----
>> From: linux-remoteproc-owner@vger.kernel.org [mailto:linux-remoteproc-
>> owner@vger.kernel.org] On Behalf Of Bjorn Andersson
>> Sent: Wednesday, May 03, 2017 7:29 AM
>> To: Andy Gross <andy.gross@linaro.org>; Rob Herring
>> <robh+dt@kernel.org>; Mark Rutland <mark.rutland@arm.com>; Ohad Ben-
>> Cohen <ohad@wizery.com>
>> Cc: linux-arm-msm@vger.kernel.org; linux-soc@vger.kernel.org;
>> devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; linux-
>> remoteproc@vger.kernel.org
>> Subject: [PATCH v3 2/4] soc: qcom: Introduce APCS IPC driver
>>
>> This implements a driver that exposes the IPC bits found in the APCS Global
>> block in various Qualcomm platforms. The bits are used to signal inter-
>> processor communication signals from the application CPU to other masters.
>>
>> The driver implements the "doorbell" binding and could be used as basis for a
>> new Linux framework, if found useful outside Qualcomm.
>>
> Hi Bjorn,
>
> Even if Qualcom APCS IPC is limited, why don't you rely on existing mailbox framework.
> It is there to gather all IPC management under the same interface.
> No need to create a new one from my pov.
> If you don't provide message data, mailbox framework behaves as doorbell.
>
QCOM RPM reinvented the wheel for what mailbox framework already did,
despite my pointing it out =>
http://lkml.iu.edu/hypermail//linux/kernel/1406.2/03918.html

The driver bypassed mailbox framework and was pushed via another tree.
Same is being attempted now, only now it is more expensive to switch
to generic mailbox framework having spent so much time on QCOM
specific implementation of controller and protocol drivers inside
drivers/soc/qcom/

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


Thread

[PATCH v3 1/4] dt-bindings: Introduce doorbell binding Bjorn Andersson <bjorn.andersson@linaro.org> - 2017-05-03 07:40 +0200
  [PATCH v3 2/4] soc: qcom: Introduce APCS IPC driver Bjorn Andersson <bjorn.andersson@linaro.org> - 2017-05-03 07:40 +0200
    RE: [PATCH v3 2/4] soc: qcom: Introduce APCS IPC driver Loic PALLARDY <loic.pallardy@st.com> - 2017-05-03 11:30 +0200
      Re: [PATCH v3 2/4] soc: qcom: Introduce APCS IPC driver Jassi Brar <jassisinghbrar@gmail.com> - 2017-05-03 12:00 +0200
        Re: [PATCH v3 2/4] soc: qcom: Introduce APCS IPC driver Bjorn Andersson <bjorn.andersson@linaro.org> - 2017-05-04 07:50 +0200
          Re: [PATCH v3 2/4] soc: qcom: Introduce APCS IPC driver Jassi Brar <jassisinghbrar@gmail.com> - 2017-05-04 10:00 +0200
            Re: [PATCH v3 2/4] soc: qcom: Introduce APCS IPC driver Bjorn Andersson <bjorn.andersson@linaro.org> - 2017-05-04 18:20 +0200
              Re: [PATCH v3 2/4] soc: qcom: Introduce APCS IPC driver Jassi Brar <jassisinghbrar@gmail.com> - 2017-05-04 19:10 +0200
  [PATCH v3 3/4] soc: qcom: Add device tree binding for GLINK RPM Bjorn Andersson <bjorn.andersson@linaro.org> - 2017-05-03 07:40 +0200
  [PATCH v3 4/4] rpmsg: Introduce Qualcomm RPM glink driver Bjorn Andersson <bjorn.andersson@linaro.org> - 2017-05-03 07:40 +0200

csiph-web