On Mon, Dec 13, 2010 at 12:38:20PM +0530, Mayank Rana wrote: > >>+#include<linux/msm_serial_hs.h> > > > >Please put this file in include/platform_data/ instead. > Greg, > > Thanks for reviewing the patch. > Is there any particular reason not allowing msm_serial_hs.h as part > of include/linux and need of moving same into as part of > include/platform_data? I am seeing multiple driver doing same. > Can you please suggest on this ? Yes, it's not good to clutter up include/linux/ with this kind of platform specific stuff. Create a new directory and we will then move all of these types of files over to it, yours will be one of the first. thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html