Messages in this thread | | | From | "Mohammed, Afzal" <> | Subject | RE: [PATCH] da8xx: Allow use by am33xx based devices | Date | Mon, 7 Jan 2013 12:03:16 +0000 |
| |
Hi,
On Wed, Dec 12, 2012 at 13:30:56, Hiremath, Vaibhav wrote: > On Wed, Dec 12, 2012 at 12:50:28, Manjunathappa, Prakash wrote:
> > Agreed, should not result in build error. But is it ok to show this option > > on the platforms which do not have this IP? > > > > You can choose to put machine dependency here, as this patch is already > doing it. The side-effect of this would be, list may grow and you may have > to edit this file everytime.
It seems adding machine dependency is more in line with the approach adopted in similar cases. As this IP is used only in 2 arch's and would not in any immediate future go with other arch's, it would be better not to ask other arch's a question on whether this driver needs to be selected or not.
Regards Afzal
| |