Nishanth Menon <nm@xxxxxx> wrote: > On 05/10/2010 07:31 AM, Venkatraman S wrote: >> >> Nishanth Menon<menon.nishanth@xxxxxxxxx> wrote: > >>>>> Please see [1] for SOC specific feature handling. any reasons we can't >>>>> handle it by adding a new feature? >>>>> >>>>> [1] >>>>> >>>>> >>>>> http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=blob;f=arch/arm/plat-omap/include/plat/cpu.h#l439 >>>>> >>>> >>>> Thanks. I can add a new feature here, but I see that the API is tied >>>> to OMAP3, whereas the DMA feature is common >>>> to 3630, OMAP4 and mostly everything after that. I can work on an >>>> upgrade, but do you see that >>>> as a dependency and done on the context of this patch ? >>>> Regards, >>>> Venkat. >>> >>> Yes, I am aware that the current APIs are tied to OMAP3, no reason that >>> we >>> cant introduce a OMAP version independent feature.. Yes, IMHO, this is an >>> SOC specific feature that has no place in a platform data.. lets not >>> misuse >>> that. >>> Regards, >>> NM >> Just a note that I have updated my patch series based on all the comments except that the 'features' framework has to be created to be utilised by this patch series. That would be a separate patch, which I am working on, and will post on monday. Regards, Venkat. -- 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
