On Sat, Jun 30, 2012 at 09:38:41AM +0530, Shilimkar, Santosh wrote: > (+ Greg, By mistake the your name got dropped cc from list) > > On Fri, Jun 29, 2012 at 7:13 PM, Santosh Shilimkar > <santosh.shilimkar@xxxxxx> wrote: > > From: Aneesh V <aneesh@xxxxxx> > > > > Device tree support for the EMIF driver. > > > > Reviewed-by: Benoit Cousson <b-cousson@xxxxxx> > > Reviewed-by: Grant Likely <grant.likely@xxxxxxxxxxxx> > > Tested-by: Lokesh Vutla <lokeshvutla@xxxxxx> > > Signed-off-by: Aneesh V <aneesh@xxxxxx> > > [santosh.shilimkar@xxxxxx: Rebased against 3.5-rc] > > Signed-off-by: Santosh Shilimkar <santosh.shilimkar@xxxxxx> > > --- > > drivers/memory/emif.c | 291 ++++++++++++++++++++++++++++++++++++++++++++++++- > > 1 file changed, 290 insertions(+), 1 deletion(-) Wouldn't this just be better off as a separate file that only gets build if CONFIG_OF is set, as I'm sure that other systems are going to want access to these "read the device tree values" functions, right? thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html