![]() | |||||||||||||
Messages in this thread |
On Nov 20 2006 01:01, Chris Snook wrote: > Jan Engelhardt wrote: >> > +#define AT_READ_REG_ARRAY(a, reg, offset) ( \ >> > + readl(((a)->hw_addr + reg) + ((offset) << 2))) >> >> Possibly similarly. > > Yeah, we'll inline these as well. Would you say that level of cosmetic > cleanliness is required for merging, or should we focus solely on the > functional issues for now? Required? No I do not think so, there is worse code than this. (Don't take that as an excuse to write bad code! :-) It's just that it's easier to read if there are less parentheses. These defines are a good example of border (define vs inline) cases. -`J' -- - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ | ||||||||||||
| Last update: 2006-11-20 11:21 [W:0.166 / U:0.020 seconds] ©2003-2008 Jasper Spaans | |||||||||||||