On Thu, Jun 14, 2012 at 9:18 PM, Greg Kroah-Hartman <[email protected]> wrote: > > On Thu, Jun 14, 2012 at 03:59:24PM +0530, Santosh Shilimkar wrote: > > From: Aneesh V <[email protected]> > > > > Device tree support for the EMIF driver. > > > > Reviewed-by: Benoit Cousson <[email protected]> > > Reviewed-by: Grant Likely <[email protected]> > > Tested-by: Lokesh Vutla <[email protected]> > > Signed-off-by: Aneesh V <[email protected]> > > [[email protected]: Rebased against 3.5-rc] > > Signed-off-by: Santosh Shilimkar <[email protected]> > > Cc: Greg Kroah-Hartman <[email protected]> > > Am I the memory maintainer now taking these patches? > I assumed that. With your ack, $subject patch should go along with other 3 OMAP platform patches because of dependency.
> > @@ -1267,7 +1534,13 @@ static int __init_or_module emif_probe(struct > > platform_device *pdev) > > struct resource *res; > > int irq; > > > > - emif = get_device_details(pdev); > > +#if defined(CONFIG_OF) > > + if (pdev->dev.of_node) > > + emif = of_get_device_details(pdev->dev.of_node, > > &pdev->dev); > > + else > > +#endif > > + emif = get_device_details(pdev); > > + > > Can't you do this without the #ifdef? > Can be done. Just need to have empty of_get_device_details() for the !CONFIG_OF build to avoid build break. Will fix that. Regards santosh -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
