[PATCH v3 4/4] memory: emif: add device tree support to emif driver

[Greg Kroah-Hartman]

On Fri, Jun 29, 2012 at 04:28:36PM -0500, Jon Hunter wrote:
> 
> On 06/29/2012 04:07 PM, Greg Kroah-Hartman wrote:
> > On Fri, Jun 29, 2012 at 07:16:19PM +0530, Shilimkar, Santosh wrote:
> >> Greg,
> >>
> >> On Tue, Jun 26, 2012 at 1:19 PM, Shilimkar, Santosh
> >> <santosh.shilimkar at ti.com> wrote:
> >>>
> >>> On Tue, Jun 26, 2012 at 10:56 AM, Rajendra Nayak <rnayak at ti.com> wrote:
> >>>> On Tuesday 26 June 2012 10:53 AM, Shilimkar, Santosh wrote:
> >>>>>
> >>>>> On Tue, Jun 26, 2012 at 10:49 AM, Rajendra Nayak<rnayak at ti.com>  wrote:
> >>>>>>
> >>>>>> On Tuesday 26 June 2012 10:39 AM, Shilimkar, Santosh wrote:
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>> +#if defined(CONFIG_OF)
> >>>>>>>>>>
> >>>>>>>>>>  >    +static const struct of_device_id emif_of_match[] = {
> >>>>>>>>>>  >    +             { .compatible = "ti,emif-4d" },
> >>>>>>>>>>  >    +             { .compatible = "ti,emif-4d5" },
> >>>>>>>>>>  >    +             {},
> >>>>>>>>>>  >    +};
> >>>>>>>>>>  >    +MODULE_DEVICE_TABLE(of, emif_of_match);
> >>>>>>>>>>  >    +#endif
> >>>>>>>>>>  >    +
> >>>>>>>>>>  >      static struct platform_driver emif_driver = {
> >>>>>>>>>>  >           .remove         = __exit_p(emif_remove),
> >>>>>>>>>>  >           .shutdown       = emif_shutdown,
> >>>>>>>>>>  >           .driver = {
> >>>>>>>>>>  >                   .name = "emif",
> >>>>>>>>>>  >    +#if defined(CONFIG_OF)
> >>>>>>>>>>  >    +             .of_match_table = of_match_ptr(emif_of_match),
> >>>>>>>>>>  >    +#endif
> >>>>>>>>
> >>>>>>>>
> >>>>>>>>>
> >>>>>>>>>  Are these two still #if defined needed?
> >>>>>>>>>
> >>>>>>>>>  Also, what about the IS_DEFINED macro?
> >>>>>>>>>
> >>>>>>>
> >>>>>>> Yes. To avoid the build break in !DT build.
> >>>>>>>
> >>>>>>
> >>>>>> No, infact of_match_ptr is there just so you can avoid a
> >>>>>> #ifdef around and not break !CONFIG_OF.
> >>>>>
> >>>>>
> >>>>> Need to recheck but the build was throwing error becasue of
> >>>>> "emif_of_match"
> >>>>> mostly.
> >>>>
> >>>>
> >>>> You still need the first one. I meant the second one around
> >>>> 'of_match_ptr' should be removed.
> >>>>
> >>>> With !CONFIG_OF, of_match_ptr(_ptr) is just defined as NULL.
> >>>>
> >>> Yep. That make sense too.
> >>> Will drop the second check.
> >>>
> >> Posted the entire series again with updated $subject patch.
> >> With your ack, I will ask Tony to pull this series for 3.6.
> > 
> > Sent them where?  I didn't see a copy here :(
> 
> Hi Greg, should have hit both linux-omap and linux-arm. See:
> 
> http://marc.info/?l=linux-omap&m=134097764625471&w=2

Ah, two lists I am not on, so how would I know to be able to ack them?
Please always cc: people you wish to review patches, otherwise it could
get lost, or as in this case, the people might not even be on the lists
you send the patches to.

greg k-h