Messages in this thread | | | Subject | Re: [PATCH 09/12] n_tracesink header file. | From | J Freyensee <> | Date | Thu, 17 Feb 2011 12:10:47 -0800 |
| |
On Thu, 2011-02-17 at 11:54 -0800, Greg KH wrote: > On Thu, Feb 17, 2011 at 11:45:15AM -0800, J Freyensee wrote: > > On Thu, 2011-02-17 at 11:23 -0800, Greg KH wrote: > > > On Tue, Feb 08, 2011 at 11:34:54AM -0800, james_p_freyensee@linux.intel.com wrote: > > > > From: J Freyensee <james_p_freyensee@linux.intel.com> > > > > > > > > This header file allows the n_tracerouter to send it's information > > > > to the n_tracesink ldisc driver. It's part of the Intel-Atom > > > > PTI implementation solution. > > > > > > > > Signed-off-by: J Freyensee <james_p_freyensee@linux.intel.com> > > > > --- > > > > include/linux/n_tracesink.h | 32 ++++++++++++++++++++++++++++++++ > > > > > > Why is this in include/linux/ ? > > > > > > > I thought this was the best place to stick this .h file. I'd be happy > > to change locations based off of your suggestion. > > Why do you need a .h file at all? Who is using it?
The only current module that really uses it is n_tracerouter; however, I thought sticking this in a .h file made things slightly more modular and more re-usable, especially for future device driver writers that could want something like this.
If you would like me to remove this though, I can, just let me know. Thanks.
> > thanks, > > greg k-h
| |