Date: Mon, 13 Dec 2010 11:32:40 -0600
From: David Sin <davidsin@ti.com>
To: Tony Lindgren <tony@atomide.com>
Cc: Greg KH <greg@kroah.com>, Russell King <rmk@arm.linux.org.uk>,
        Andrew Morton <akpm@linux-foundation.org>,
        linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, Lajos Molnar <molnar@ti.com>
Subject: Re: [PATCH 7/9] TILER-DMM: Main TILER driver implementation
Message-ID: <20101213173239.GC23421@lba0869738>
References: <1291674446-10766-1-git-send-email-davidsin@ti.com>
 <1291674446-10766-8-git-send-email-davidsin@ti.com>
 <20101206232159.GS8345@atomide.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20101206232159.GS8345@atomide.com>
User-Agent: Mutt/1.5.20 (2009-06-14)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1501
Lines: 50

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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/