Messages in this thread |  | | | From | David Brownell <> | | Subject | Re: [PATCH 1/1] [GPIO]: new arch-independent simple-gpio driver | | Date | Sat, 5 Apr 2008 15:42:44 -0700 | |
On Wednesday 26 March 2008, Ben Nizette wrote:
> > +#define pr_devinit(fmt, args...) ({ static const __devinitdata char __fmt[] = fmt; printk(__fmt, ## args); })
> > +#define pr_init(fmt, args...) ({ static const __initdata char __fmt[] = fmt; printk(__fmt, ## args); })
>
> Should your pr_*init macros be accepted somewhere higher up the tree?
> Either that or dropped, it doesn't seem right wedging them in here.
> Sure it might cost you a few hundred bytes of RAM but would be nice to
> keep it all consistent across the kernel.
Me, I'm all in favor of getting rid of structural code bloat.
A few hundred bytes of such stuff shaved out a dozen drivers
on a given platforms would be almost free page saved! :)
So I'd be interested in seeing those get submitted ... but as
inline functions, not fancy macros. (Once they're submitted,
then let the flamage begin ... much less than kernel I18N!)
- Dave
|  |