On Mon, 2008-04-14 at 17:34 +0100, Russell King wrote: > On Mon, Apr 14, 2008 at 09:27:12AM -0700, Harvey Harrison wrote: > > On Mon, 2008-04-14 at 12:11 +0100, Russell King wrote: > > > On Mon, Apr 14, 2008 at 11:32:09AM +0100, David Howells wrote: > > > > Harvey Harrison <harvey.harrison@xxxxxxxxx> wrote: > > > > > > > > > -#ifndef _ASM_UNALIGNED_H > > > > > -#define _ASM_UNALIGNED_H > > > > > +#ifndef _ASM_FRV_UNALIGNED_H_ > > > > > +#define _ASM_FRV_UNALIGNED_H_ > > > > > > > > Please don't. Why do you need to change these to make your patch work? > > > > > > Indeed - ditto for the ARM changes. The patch is changing them to a > > > different style to the others in the respective asm/ directories - > > > which is also different from the linux/ directory. > > > > > > IMHO, if Harvey wishes to clean those up, then that should be a separate > > > patch and should be part of a set converting all includes to use the > > > same style. > > > > > > > OK, I'll revise without these changes. I am curious what the preferred > > 'linux' style is (if there is one). > > The one that's already established by the majority is one answer. 8) > > $ grep '^#ifndef LINUX_.*_H$' include/linux/*.h | wc -l > 33 > $ grep '^#ifndef _LINUX_.*_H$' include/linux/*.h | wc -l > 388 > $ grep '^#ifndef __LINUX_.*_H$' include/linux/*.h | wc -l > 110 > $ grep '^#ifndef LINUX_.*_H_$' include/linux/*.h | wc -l > 1 > $ grep '^#ifndef LINUX_.*_H__$' include/linux/*.h | wc -l > 0 > $ grep '^#ifndef _LINUX_.*_H_$' include/linux/*.h | wc -l > 20 > $ grep '^#ifndef __LINUX_.*_H__$' include/linux/*.h | wc -l > 20 > > So it seems _LINUX_FOO_H for linux/foo.h wins. > > But... honestly, I wouldn't bother with such a cleanup - at the end of > the day, it's personal choice and people are always going to do something > slightly different. So it's probably best all round to just accept what's > already in the files you're modifying. > > Let's not add to the expanding beaurocracy by demanding a format for these > macro names. > Well, I'm not interested in a comprehensive cleanup here, but as I am changing every arch's asm/unaligned.h, may as well at least do those ones. Would you accept my patch with ARM's changed to _ASM_ARM_UNALIGNED_H? Harvey -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html