Re: [PATCH 7/9] TILER-DMM: Main TILER driver implementation

From: Tony Lindgren
Date: Mon Dec 06 2010 - 18:22:23 EST


Hi,

* David Sin <davidsin@xxxxxx> [101206 14:09]:
> From: Lajos Molnar <molnar@xxxxxx>
>
> 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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/