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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxx> wrote:
> >>>
> >>> On Tue, Jun 26, 2012 at 10:56 AM, Rajendra Nayak <rnayak@xxxxxx> wrote:
> >>>> On Tuesday 26 June 2012 10:53 AM, Shilimkar, Santosh wrote:
> >>>>>
> >>>>> On Tue, Jun 26, 2012 at 10:49 AM, Rajendra Nayak<rnayak@xxxxxx>  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
--
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


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux