From: khilman@baylibre.com (Kevin Hilman)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 6/6] ARM: da850: adjust memory settings for tilcdc
Date: Tue, 04 Oct 2016 06:02:48 -0700 [thread overview]
Message-ID: <7hvax8ferr.fsf@baylibre.com> (raw)
In-Reply-To: <0125defa-5c34-85d0-f62a-133235c5094c@ti.com> (Peter Ujfalusi's message of "Mon, 3 Oct 2016 10:13:25 +0300")
Peter Ujfalusi <peter.ujfalusi@ti.com> writes:
> On 10/01/16 12:24, Sekhar Nori wrote:
[...]
>> In any case, to configure the PBBR, you will have to introduce a driver
>> for it in drivers/memory. Then you can set it up per board using a DT
>> parameter.
>
> and we can reuse the introduced bindings for am335x and OMAP1/2 as well. On
> OMAP the legacy DMA API provided a call to raise the priority of the sDMA in
> EMIF :o That needs to be removed and replaced.
Can you point us to the bindings you're referring to?
Also, a new driver in drivers/memory is fine for setting the PBBR, but
what about the SYSCFG0 registers. Are you OK with leaving those in the
init code as proposed in $SUBJECT patch?
Kevin
WARNING: multiple messages have this Message-ID (diff)
From: Kevin Hilman <khilman@baylibre.com>
To: Peter Ujfalusi <peter.ujfalusi@ti.com>
Cc: Sekhar Nori <nsekhar@ti.com>,
Bartosz Golaszewski <bgolaszewski@baylibre.com>,
Michael Turquette <mturquette@baylibre.com>,
Rob Herring <robh+dt@kernel.org>,
Mark Rutland <mark.rutland@arm.com>,
Russell King <linux@armlinux.org.uk>,
LKML <linux-kernel@vger.kernel.org>,
arm-soc <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 6/6] ARM: da850: adjust memory settings for tilcdc
Date: Tue, 04 Oct 2016 06:02:48 -0700 [thread overview]
Message-ID: <7hvax8ferr.fsf@baylibre.com> (raw)
In-Reply-To: <0125defa-5c34-85d0-f62a-133235c5094c@ti.com> (Peter Ujfalusi's message of "Mon, 3 Oct 2016 10:13:25 +0300")
Peter Ujfalusi <peter.ujfalusi@ti.com> writes:
> On 10/01/16 12:24, Sekhar Nori wrote:
[...]
>> In any case, to configure the PBBR, you will have to introduce a driver
>> for it in drivers/memory. Then you can set it up per board using a DT
>> parameter.
>
> and we can reuse the introduced bindings for am335x and OMAP1/2 as well. On
> OMAP the legacy DMA API provided a call to raise the priority of the sDMA in
> EMIF :o That needs to be removed and replaced.
Can you point us to the bindings you're referring to?
Also, a new driver in drivers/memory is fine for setting the PBBR, but
what about the SYSCFG0 registers. Are you OK with leaving those in the
init code as proposed in $SUBJECT patch?
Kevin
next prev parent reply other threads:[~2016-10-04 13:02 UTC|newest]
Thread overview: 66+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-29 16:31 [PATCH 0/6] ARM: davinci: da850-lcdk: add support for tilcdc Bartosz Golaszewski
2016-09-29 16:31 ` Bartosz Golaszewski
2016-09-29 16:31 ` [PATCH 1/6] ARM: davinci: da8xx-dt: add OF_DEV_AUXDATA entry for lcdc Bartosz Golaszewski
2016-09-29 16:31 ` Bartosz Golaszewski
2016-09-29 16:31 ` [PATCH 2/6] ARM: dts: da850: add a node for the LCD controller Bartosz Golaszewski
2016-09-29 16:31 ` Bartosz Golaszewski
2016-09-30 9:34 ` Sekhar Nori
2016-09-30 9:34 ` Sekhar Nori
2016-09-30 13:03 ` Bartosz Golaszewski
2016-09-30 13:03 ` Bartosz Golaszewski
2016-09-29 16:31 ` [PATCH 3/6] ARM: dts: da850-lcdk: enable " Bartosz Golaszewski
2016-09-29 16:31 ` Bartosz Golaszewski
2016-09-29 18:40 ` Karl Beldan
2016-09-29 18:40 ` Karl Beldan
2016-09-30 9:42 ` Bartosz Golaszewski
2016-09-30 9:42 ` Bartosz Golaszewski
2016-09-30 13:15 ` Karl Beldan
2016-09-30 13:15 ` Karl Beldan
2016-09-30 14:21 ` Sekhar Nori
2016-09-30 14:21 ` Sekhar Nori
2016-09-29 16:31 ` [PATCH 4/6] ARM: dts: da850-lcdk: add support for 1024x768 resolution Bartosz Golaszewski
2016-09-29 16:31 ` Bartosz Golaszewski
2016-09-29 18:58 ` Karl Beldan
2016-09-29 18:58 ` Karl Beldan
2016-09-30 9:37 ` Bartosz Golaszewski
2016-09-30 9:37 ` Bartosz Golaszewski
2016-09-30 11:47 ` Sekhar Nori
2016-09-30 11:47 ` Sekhar Nori
2016-09-30 12:49 ` Karl Beldan
2016-09-30 12:49 ` Karl Beldan
2016-09-30 13:48 ` Bartosz Golaszewski
2016-09-30 13:48 ` Bartosz Golaszewski
2016-10-01 9:35 ` Sekhar Nori
2016-10-01 9:35 ` Sekhar Nori
2016-09-29 16:31 ` [PATCH 5/6] ARM: davinci: enable the LCDC DRM driver in defconfig Bartosz Golaszewski
2016-09-29 16:31 ` Bartosz Golaszewski
2016-10-01 12:39 ` Sekhar Nori
2016-10-01 12:39 ` Sekhar Nori
2016-09-29 16:31 ` [PATCH 6/6] ARM: da850: adjust memory settings for tilcdc Bartosz Golaszewski
2016-09-29 16:31 ` Bartosz Golaszewski
2016-09-29 19:07 ` Karl Beldan
2016-09-29 19:07 ` Karl Beldan
2016-09-30 9:31 ` Bartosz Golaszewski
2016-09-30 9:31 ` Bartosz Golaszewski
2016-09-30 9:39 ` Karl Beldan
2016-09-30 9:39 ` Karl Beldan
2016-09-30 12:59 ` Peter Ujfalusi
2016-09-30 12:59 ` Peter Ujfalusi
2016-09-30 15:06 ` Bartosz Golaszewski
2016-09-30 15:06 ` Bartosz Golaszewski
2016-09-30 19:19 ` Peter Ujfalusi
2016-09-30 19:19 ` Peter Ujfalusi
2016-10-01 9:24 ` Sekhar Nori
2016-10-01 9:24 ` Sekhar Nori
2016-10-03 7:13 ` Peter Ujfalusi
2016-10-03 7:13 ` Peter Ujfalusi
2016-10-04 13:02 ` Kevin Hilman [this message]
2016-10-04 13:02 ` Kevin Hilman
2016-10-05 8:22 ` Peter Ujfalusi
2016-10-05 8:22 ` Peter Ujfalusi
2016-10-06 17:57 ` Sekhar Nori
2016-10-06 17:57 ` Sekhar Nori
2016-10-04 9:20 ` Bartosz Golaszewski
2016-10-04 9:20 ` Bartosz Golaszewski
2016-10-05 8:02 ` Peter Ujfalusi
2016-10-05 8:02 ` Peter Ujfalusi
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=7hvax8ferr.fsf@baylibre.com \
--to=khilman@baylibre.com \
--cc=linux-arm-kernel@lists.infradead.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.