* typo in include/linux/irqflags.h ?
@ 2007-02-24 11:45 Nicholas Mc Guire
2007-02-24 13:14 ` Russell King
0 siblings, 1 reply; 3+ messages in thread
From: Nicholas Mc Guire @ 2007-02-24 11:45 UTC (permalink / raw)
To: linux-kernel
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Hi !
Seems like there was a cut&past error in include/linux/riqflags.h.
- --- linux-2.6.20/include/linux/irqflags.h 2007-02-04 09:44:54.000000000 -0900
+++ linux-2.6.20xm/include/linux/irqflags.h 2008-01-27 20:29:26.000000000 -0900
@@ -91,6 +91,6 @@
})
#define irqs_disabled_flags(flags) raw_irqs_disabled_flags(flags)
- -#endif /* CONFIG_X86 */
+#endif /* CONFIG_TRACE_IRQFLAGS_SUPPORT */
#endif
thx !
hofrat
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.4 (GNU/Linux)
iD8DBQFF4CVPnU7rXZKfY2oRAmrDAJ9eFu3APdl0Y5LoYoUAPGXzipx2wwCgl3gM
28KVNI1v+DM/LyTWEv2XhQc=
=aAnC
-----END PGP SIGNATURE-----
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: typo in include/linux/irqflags.h ?
2007-02-24 13:14 ` Russell King
@ 2007-02-24 12:56 ` Nicholas Mc Guire
0 siblings, 0 replies; 3+ messages in thread
From: Nicholas Mc Guire @ 2007-02-24 12:56 UTC (permalink / raw)
To: Russell King; +Cc: Nicholas Mc Guire, linux-kernel
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
> On Sat, Feb 24, 2007 at 12:45:16PM +0100, Nicholas Mc Guire wrote:
>>
>> Hi !
>>
>> Seems like there was a cut&past error in include/linux/riqflags.h.
>>
>>
>> --- linux-2.6.20/include/linux/irqflags.h 2007-02-04 09:44:54.000000000 -0900
>> +++ linux-2.6.20xm/include/linux/irqflags.h 2008-01-27 20:29:26.000000000 -0900
>> @@ -91,6 +91,6 @@
>> })
>>
>> #define irqs_disabled_flags(flags) raw_irqs_disabled_flags(flags)
>> -#endif /* CONFIG_X86 */
>> +#endif /* CONFIG_TRACE_IRQFLAGS_SUPPORT */
>
> Just yet rid of the comment - you can see the corresponding #ifdef on
> the screen without paging up or down so it seems pointless to have such
> a comment - it just adds additional unnecessary clutter to the screen.
>
well not for people that only have one line on the braille-screen,
for it helps but it better be right or we get lost completly ;)
hofrat
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.4 (GNU/Linux)
iD8DBQFF4DX6nU7rXZKfY2oRAg6zAJ4kK0c6T+fEXavRELt5vJnpLQxbYACfQhvR
bWg4oR0/6ng+1Cmlzwn7Igc=
=hQd0
-----END PGP SIGNATURE-----
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: typo in include/linux/irqflags.h ?
2007-02-24 11:45 typo in include/linux/irqflags.h ? Nicholas Mc Guire
@ 2007-02-24 13:14 ` Russell King
2007-02-24 12:56 ` Nicholas Mc Guire
0 siblings, 1 reply; 3+ messages in thread
From: Russell King @ 2007-02-24 13:14 UTC (permalink / raw)
To: Nicholas Mc Guire; +Cc: linux-kernel
On Sat, Feb 24, 2007 at 12:45:16PM +0100, Nicholas Mc Guire wrote:
>
> Hi !
>
> Seems like there was a cut&past error in include/linux/riqflags.h.
>
>
> --- linux-2.6.20/include/linux/irqflags.h 2007-02-04 09:44:54.000000000 -0900
> +++ linux-2.6.20xm/include/linux/irqflags.h 2008-01-27 20:29:26.000000000 -0900
> @@ -91,6 +91,6 @@
> })
>
> #define irqs_disabled_flags(flags) raw_irqs_disabled_flags(flags)
> -#endif /* CONFIG_X86 */
> +#endif /* CONFIG_TRACE_IRQFLAGS_SUPPORT */
Just yet rid of the comment - you can see the corresponding #ifdef on
the screen without paging up or down so it seems pointless to have such
a comment - it just adds additional unnecessary clutter to the screen.
--
Russell King
Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/
maintainer of:
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2007-02-24 13:40 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-02-24 11:45 typo in include/linux/irqflags.h ? Nicholas Mc Guire
2007-02-24 13:14 ` Russell King
2007-02-24 12:56 ` Nicholas Mc Guire
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.