On Wed, Feb 23, 2011 at 11:00:25AM +0530, Subhasish Ghosh wrote: > I could not follow the recommendations clearly. > This is just to clarify. > > Currently, I have the following files for the suart implementation: > > drivers/tty/serial/da8xx_pruss/pruss_suart_api.h > drivers/tty/serial/da8xx_pruss/pruss_suart_err.h > drivers/tty/serial/da8xx_pruss/pruss_suart_regs.h > drivers/tty/serial/da8xx_pruss/pruss_suart_board.h > drivers/tty/serial/da8xx_pruss/pruss_suart_mcasp.h > drivers/tty/serial/da8xx_pruss/pruss_suart_utils.h > > drivers/tty/serial/da8xx_pruss/pruss_suart_api.c > drivers/tty/serial/da8xx_pruss/pruss_suart.c > drivers/tty/serial/da8xx_pruss/pruss_suart_utils.c > > Of these, I will be removing pruss_suart_err.h as part of the Linux > error code cleanup. > But, I need to keep at least pruss_suart_board.h as a separate > file, as this defines > configurations which will be often modified by users, I don't want > to mix it with other files.
Why would a .h file ever need to be "modified by users"? That sounds wrong to me. > Should I combine rest of the headers into a single file ? Yes, why would they need to be separate? > and keep the other three .c files under "drivers/tty/serial/" and > remove the da8xx_pruss directory altogether. Yes. thanks, greg k-h _______________________________________________ Davinci-linux-open-source mailing list Davinci-linux-open-source@linux.davincidsp.com http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source