All of lore.kernel.org
 help / color / mirror / Atom feed
From: Maxime Ripard <maxime.ripard@bootlin.com>
To: Georgi Djakov <georgi.djakov@linaro.org>
Cc: Rob Herring <robh@kernel.org>,
	linux-pm@vger.kernel.org,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	"Rafael J. Wysocki" <rjw@rjwysocki.net>,
	Rob Herring <robh+dt@kernel.org>,
	Mike Turquette <mturquette@baylibre.com>,
	khilman@baylibre.com,
	Vincent Guittot <vincent.guittot@linaro.org>,
	skannan@codeaurora.org,
	Bjorn Andersson <bjorn.andersson@linaro.org>,
	Amit Kucheria <amit.kucheria@linaro.org>,
	seansw@qti.qualcomm.com, daidavid1@codeaurora.org,
	evgreen@chromium.org, Mark Rutland <mark.rutland@arm.com>,
	Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>,
	Alexandre Bailon <abailon@baylibre.com>,
	Arnd Bergmann <arnd@arndb.de>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	linux-arm-kernel <linux-arm-kernel@lists.infradead.org>,
	linux-arm-msm@vger.ke
Subject: Re: [PATCH v7 2/8] dt-bindings: Introduce interconnect provider bindings
Date: Mon, 20 Aug 2018 17:32:07 +0200	[thread overview]
Message-ID: <20180820153207.xx5outviph7ec76p@flea> (raw)
In-Reply-To: <e114ed83-fd54-4952-f375-e1d94826ac6e@linaro.org>

[-- Attachment #1: Type: text/plain, Size: 1137 bytes --]

Hi Georgi,

On Tue, Aug 07, 2018 at 05:54:38PM +0300, Georgi Djakov wrote:
> > There is also a patch series from Maxime Ripard that's addressing the
> > same general area. See "dt-bindings: Add a dma-parent property". We
> > don't need multiple ways to address describing the device to memory
> > paths, so you all had better work out a common solution.
> 
> Looks like this fits exactly into the interconnect API concept. I see
> MBUS as interconnect provider and display/camera as consumers, that
> report their bandwidth needs. I am also planning to add support for
> priority.

Thanks for working on this. After looking at your serie, the one thing
I'm a bit uncertain about (and the most important one to us) is how we
would be able to tell through which interconnect the DMA are done.

This is important to us since our topology is actually quite simple as
you've seen, but the RAM is not mapped on that bus and on the CPU's,
so we need to apply an offset to each buffer being DMA'd.

Maxime

-- 
Maxime Ripard, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: Maxime Ripard <maxime.ripard@bootlin.com>
To: Georgi Djakov <georgi.djakov@linaro.org>
Cc: Rob Herring <robh@kernel.org>,
	linux-pm@vger.kernel.org,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	"Rafael J. Wysocki" <rjw@rjwysocki.net>,
	Rob Herring <robh+dt@kernel.org>,
	Mike Turquette <mturquette@baylibre.com>,
	khilman@baylibre.com,
	Vincent Guittot <vincent.guittot@linaro.org>,
	skannan@codeaurora.org,
	Bjorn Andersson <bjorn.andersson@linaro.org>,
	Amit Kucheria <amit.kucheria@linaro.org>,
	seansw@qti.qualcomm.com, daidavid1@codeaurora.org,
	evgreen@chromium.org, Mark Rutland <mark.rutland@arm.com>,
	Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>,
	Alexandre Bailon <abailon@baylibre.com>,
	Arnd Bergmann <arnd@arndb.de>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	linux-arm-kernel <linux-arm-kernel@lists.infradead.org>,
	linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org
Subject: Re: [PATCH v7 2/8] dt-bindings: Introduce interconnect provider bindings
Date: Mon, 20 Aug 2018 17:32:07 +0200	[thread overview]
Message-ID: <20180820153207.xx5outviph7ec76p@flea> (raw)
In-Reply-To: <e114ed83-fd54-4952-f375-e1d94826ac6e@linaro.org>

[-- Attachment #1: Type: text/plain, Size: 1137 bytes --]

Hi Georgi,

On Tue, Aug 07, 2018 at 05:54:38PM +0300, Georgi Djakov wrote:
> > There is also a patch series from Maxime Ripard that's addressing the
> > same general area. See "dt-bindings: Add a dma-parent property". We
> > don't need multiple ways to address describing the device to memory
> > paths, so you all had better work out a common solution.
> 
> Looks like this fits exactly into the interconnect API concept. I see
> MBUS as interconnect provider and display/camera as consumers, that
> report their bandwidth needs. I am also planning to add support for
> priority.

Thanks for working on this. After looking at your serie, the one thing
I'm a bit uncertain about (and the most important one to us) is how we
would be able to tell through which interconnect the DMA are done.

This is important to us since our topology is actually quite simple as
you've seen, but the RAM is not mapped on that bus and on the CPU's,
so we need to apply an offset to each buffer being DMA'd.

Maxime

-- 
Maxime Ripard, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: maxime.ripard@bootlin.com (Maxime Ripard)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v7 2/8] dt-bindings: Introduce interconnect provider bindings
Date: Mon, 20 Aug 2018 17:32:07 +0200	[thread overview]
Message-ID: <20180820153207.xx5outviph7ec76p@flea> (raw)
In-Reply-To: <e114ed83-fd54-4952-f375-e1d94826ac6e@linaro.org>

Hi Georgi,

On Tue, Aug 07, 2018 at 05:54:38PM +0300, Georgi Djakov wrote:
> > There is also a patch series from Maxime Ripard that's addressing the
> > same general area. See "dt-bindings: Add a dma-parent property". We
> > don't need multiple ways to address describing the device to memory
> > paths, so you all had better work out a common solution.
> 
> Looks like this fits exactly into the interconnect API concept. I see
> MBUS as interconnect provider and display/camera as consumers, that
> report their bandwidth needs. I am also planning to add support for
> priority.

Thanks for working on this. After looking at your serie, the one thing
I'm a bit uncertain about (and the most important one to us) is how we
would be able to tell through which interconnect the DMA are done.

This is important to us since our topology is actually quite simple as
you've seen, but the RAM is not mapped on that bus and on the CPU's,
so we need to apply an offset to each buffer being DMA'd.

Maxime

-- 
Maxime Ripard, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20180820/66f72c97/attachment.sig>

  reply	other threads:[~2018-08-20 15:32 UTC|newest]

Thread overview: 72+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-07-31 16:13 [PATCH v7 0/8] Introduce on-chip interconnect API Georgi Djakov
2018-07-31 16:13 ` Georgi Djakov
2018-07-31 16:13 ` [PATCH v7 1/8] interconnect: Add generic " Georgi Djakov
2018-07-31 16:13   ` Georgi Djakov
2018-08-02  0:05   ` Randy Dunlap
2018-08-02  0:05     ` Randy Dunlap
2018-08-02 11:58     ` Georgi Djakov
2018-08-02 11:58       ` Georgi Djakov
2018-08-03 22:59   ` Evan Green
2018-08-03 22:59     ` Evan Green
2018-07-31 16:13 ` [PATCH v7 2/8] dt-bindings: Introduce interconnect provider bindings Georgi Djakov
2018-07-31 16:13   ` Georgi Djakov
2018-08-02 21:02   ` Rob Herring
2018-08-02 21:02     ` Rob Herring
2018-08-07 14:54     ` Georgi Djakov
2018-08-07 14:54       ` [PATCH " Georgi Djakov
2018-08-07 14:54       ` Georgi Djakov
2018-08-20 15:32       ` Maxime Ripard [this message]
2018-08-20 15:32         ` [PATCH " Maxime Ripard
2018-08-20 15:32         ` Maxime Ripard
2018-08-24 14:51         ` Georgi Djakov
2018-08-24 14:51           ` Georgi Djakov
2018-08-24 14:51           ` Georgi Djakov
2018-08-24 15:35           ` Rob Herring
2018-08-24 15:35             ` Rob Herring
2018-08-24 15:35             ` Rob Herring
2018-08-27 15:11             ` Maxime Ripard
2018-08-27 15:11               ` Maxime Ripard
2018-08-27 15:11               ` Maxime Ripard
2018-08-29 12:33               ` Georgi Djakov
2018-08-29 12:33                 ` Georgi Djakov
2018-08-29 12:33                 ` Georgi Djakov
2018-08-30  7:47                 ` Maxime Ripard
2018-08-30  7:47                   ` Maxime Ripard
2018-08-30  7:47                   ` Maxime Ripard
2018-08-27 15:08           ` Maxime Ripard
2018-08-27 15:08             ` Maxime Ripard
2018-08-27 15:08             ` Maxime Ripard
2018-08-29 12:31             ` Georgi Djakov
2018-08-29 12:31               ` Georgi Djakov
2018-08-29 12:31               ` Georgi Djakov
2018-07-31 16:13 ` [PATCH v7 3/8] interconnect: Add debugfs support Georgi Djakov
2018-07-31 16:13   ` Georgi Djakov
2018-08-03 22:59   ` Evan Green
2018-08-03 22:59     ` Evan Green
2018-07-31 16:13 ` [PATCH v7 4/8] interconnect: qcom: Add RPM communication Georgi Djakov
2018-07-31 16:13   ` Georgi Djakov
2018-08-03 22:59   ` Evan Green
2018-08-03 22:59     ` Evan Green
2018-08-03 22:59     ` Evan Green
2018-07-31 16:13 ` [PATCH v7 5/8] dt-bindings: interconnect: Document qcom, msm8916 NoC bindings Georgi Djakov
2018-07-31 16:13   ` Georgi Djakov
2018-07-31 16:13   ` [PATCH v7 5/8] dt-bindings: interconnect: Document qcom,msm8916 " Georgi Djakov
2018-07-31 16:13 ` [PATCH v7 6/8] interconnect: qcom: Add msm8916 interconnect provider driver Georgi Djakov
2018-07-31 16:13   ` Georgi Djakov
2018-08-03 22:59   ` Evan Green
2018-08-03 22:59     ` Evan Green
2018-07-31 16:13 ` [PATCH v7 7/8] dt-bindings: Introduce interconnect consumers bindings Georgi Djakov
2018-07-31 16:13   ` Georgi Djakov
2018-07-31 16:13 ` [PATCH v7 8/8] interconnect: Allow endpoints translation via DT Georgi Djakov
2018-07-31 16:13   ` Georgi Djakov
2018-08-01 22:57   ` skannan
2018-08-01 22:57     ` skannan at codeaurora.org
2018-08-02 12:07     ` Georgi Djakov
2018-08-02 12:07       ` Georgi Djakov
2018-08-02 19:12       ` skannan
2018-08-02 19:12         ` skannan at codeaurora.org
2018-08-09 14:17         ` Georgi Djakov
2018-08-09 14:17           ` Georgi Djakov
2018-08-03 22:59   ` Evan Green
2018-08-03 22:59     ` Evan Green
2018-08-03 22:59     ` Evan Green

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180820153207.xx5outviph7ec76p@flea \
    --to=maxime.ripard@bootlin.com \
    --cc=abailon@baylibre.com \
    --cc=amit.kucheria@linaro.org \
    --cc=arnd@arndb.de \
    --cc=bjorn.andersson@linaro.org \
    --cc=daidavid1@codeaurora.org \
    --cc=evgreen@chromium.org \
    --cc=georgi.djakov@linaro.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=khilman@baylibre.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-arm-msm@vger.ke \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=lorenzo.pieralisi@arm.com \
    --cc=mark.rutland@arm.com \
    --cc=mturquette@baylibre.com \
    --cc=rjw@rjwysocki.net \
    --cc=robh+dt@kernel.org \
    --cc=robh@kernel.org \
    --cc=seansw@qti.qualcomm.com \
    --cc=skannan@codeaurora.org \
    --cc=vincent.guittot@linaro.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.