From: davidsin@ti.com (David Sin)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 7/9] TILER-DMM: Main TILER driver implementation
Date: Mon, 13 Dec 2010 11:32:40 -0600 [thread overview]
Message-ID: <20101213173239.GC23421@lba0869738> (raw)
In-Reply-To: <20101206232159.GS8345@atomide.com>
On Mon, Dec 06, 2010 at 03:21:59PM -0800, Tony Lindgren wrote:
> Hi,
>
> * David Sin <davidsin@ti.com> [101206 14:09]:
> > From: Lajos Molnar <molnar@ti.com>
> >
> > This patch contains the TILER driver and implementation of the TILER
> > block manipulation and mapping functions.
> >
> > It also contains the makefile and config file for the TILER driver.
>
> ...
> > +config TILER_GRANULARITY
> > + int "Allocation granularity"
> > + range 1 4096
> > + default 128
> ...
> > +config TILER_ALIGNMENT
> > + int "Allocation alignment"
> > + range 1 4096
> > + default 4096
> ...
> > +config TILER_CACHE_LIMIT
> > + int "Memory limit to cache free pages in MBytes"
> > + range 0 128
> > + default 40
> ...
>
> Do you really need these Kconfig options? To me it seems you
> should pass these in platform_data.
>
> Regards,
>
> Tony
Hi Tony,
No -- we don't really need these here, but we thought that
it would be an easy way for the user to set them -- esp the
CACHE_LIMIT. We can actually remove themi, since it's not
going to change very often, and add it to plat data later.
I'll revise the patch. BTW, Are you going to be the one to
merge this code?
Regards,
--
David Sin
next prev parent reply other threads:[~2010-12-13 17:32 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-12-06 22:27 [PATCH 0/9] TI DMM-TILER driver David Sin
2010-12-06 22:27 ` [PATCH 1/9] TILER-DMM: DMM-PAT driver for TI TILER David Sin
2010-12-06 22:27 ` [PATCH 2/9] TILER-DMM: Container manager interface and utility definitons David Sin
2010-12-06 22:27 ` [PATCH 3/9] TILER-DMM: Sample TCM implementation: Simple TILER Allocator (SiTA) David Sin
2010-12-06 22:27 ` [PATCH 4/9] TILER-DMM: TILER Memory Manager interface and implementation David Sin
2010-12-06 22:27 ` [PATCH 5/9] TILER-DMM: TILER interface file and documentation David Sin
2010-12-06 22:27 ` [PATCH 6/9] TILER-DMM: Geometry and view manipulation functions David Sin
2010-12-06 22:27 ` [PATCH 7/9] TILER-DMM: Main TILER driver implementation David Sin
2010-12-06 23:21 ` Tony Lindgren
2010-12-13 17:32 ` David Sin [this message]
2010-12-06 22:27 ` [PATCH 8/9] TILER-DMM: Linking TILER driver into the Linux kernel build David Sin
2010-12-06 22:27 ` [PATCH 9/9] TILER-DMM: Device support for OMAP David Sin
2010-12-16 13:34 ` [PATCH 0/9] TI DMM-TILER driver Arnd Bergmann
2010-12-16 17:25 ` David Sin
2010-12-16 17:37 ` David Sin
2010-12-16 17:43 ` Arnd Bergmann
2010-12-16 19:02 ` David Sin
2010-12-16 23:28 ` Dave Airlie
2010-12-22 14:33 ` David Sin
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=20101213173239.GC23421@lba0869738 \
--to=davidsin@ti.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).