Messages in this thread | | | Date | Thu, 17 Jun 2010 16:28:10 -0700 | From | Greg KH <> | Subject | Re: [PATCH] staging:comedi: Fixed coding convention issues. |
| |
On Thu, Jun 17, 2010 at 04:15:58PM -0700, Joe Perches wrote: > On Thu, 2010-06-17 at 15:51 -0700, Greg KH wrote: > > On Sat, Jun 12, 2010 at 10:30:50PM -0700, Joe Perches wrote: > > > On Sat, 2010-06-12 at 22:07 -0700, Joe Perches wrote: > > > > 2: Create some comedi logging functions or macros like: > > > > comedi_<level>(fmt, arg...) (ie: comedi_info, comedi_err, etc) > > > > where "comedi:" is always prefixed and an > > > > optional #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt > > > > could be used. > > > I would prefer the conversion of everything over to the dev_printk() > > versions instead of creating a new macro for every individual subsystem. > > That way you get the advantage of logging everything in the common > > format and the dynamic debug functionality as well. > > What I posted has dynamic_debug. > > +#elif defined(CONFIG_DYNAMIC_DEBUG) > +/* dynamic_pr_debug() uses pr_fmt() internally so we don't need it here */ > +#define comedi_debug(fmt, ...) \ > + dynamic_pr_debug(fmt, ##__VA_ARGS__) > > As far as I know, comedi doesn't always take a struct device *. > I believe it's only used when there's a DMA.
No, there's a struct device down in the device almost always.
> In struct comedi_device, there are two struct device *'s. > > struct device *class_dev; > ... > struct device *hw_dev;
hw_dev is what we want to use.
thanks,
greg k-h
| |