* [PATCH] compile fix for 2.5 kdev_t compatibility macros
@ 2002-06-28 13:50 Stephen Lord
2002-06-28 14:37 ` Steven Cole
0 siblings, 1 reply; 6+ messages in thread
From: Stephen Lord @ 2002-06-28 13:50 UTC (permalink / raw)
To: Marcelo Tosatti; +Cc: Linux Kernel Mailing List
Marcelo,
We started using these for XFS, and found a missing bracket, patch
against 2.4.19-rc1.
Steve
*** linux-2.4.19-rc1/include/linux/kdev_t.h Fri Jun 28 08:40:22 2002
--- linux/include/linux/kdev_t.h Fri Jun 28 07:05:32 2002
***************
*** 81,87 ****
#define minor(d) MINOR(d)
#define kdev_same(a,b) ((a) == (b))
#define kdev_none(d) (!(d))
! #define kdev_val(d) ((unsigned int)(d)
#define val_to_kdev(d) ((kdev_t(d))
/*
--- 81,87 ----
#define minor(d) MINOR(d)
#define kdev_same(a,b) ((a) == (b))
#define kdev_none(d) (!(d))
! #define kdev_val(d) ((unsigned int)(d))
#define val_to_kdev(d) ((kdev_t(d))
/*
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] compile fix for 2.5 kdev_t compatibility macros
2002-06-28 13:50 [PATCH] compile fix for 2.5 kdev_t compatibility macros Stephen Lord
@ 2002-06-28 14:37 ` Steven Cole
2002-06-28 14:42 ` Stephen Lord
2002-06-28 15:10 ` Martin Josefsson
0 siblings, 2 replies; 6+ messages in thread
From: Steven Cole @ 2002-06-28 14:37 UTC (permalink / raw)
To: Stephen Lord; +Cc: Marcelo Tosatti, Linux Kernel Mailing List
On Fri, 2002-06-28 at 07:50, Stephen Lord wrote:
>
>
> Marcelo,
>
> We started using these for XFS, and found a missing bracket, patch
> against 2.4.19-rc1.
>
> Steve
>
> *** linux-2.4.19-rc1/include/linux/kdev_t.h Fri Jun 28 08:40:22 2002
> --- linux/include/linux/kdev_t.h Fri Jun 28 07:05:32 2002
> ***************
> *** 81,87 ****
> #define minor(d) MINOR(d)
> #define kdev_same(a,b) ((a) == (b))
> #define kdev_none(d) (!(d))
> ! #define kdev_val(d) ((unsigned int)(d)
> #define val_to_kdev(d) ((kdev_t(d))
>
> /*
> --- 81,87 ----
> #define minor(d) MINOR(d)
> #define kdev_same(a,b) ((a) == (b))
> #define kdev_none(d) (!(d))
> ! #define kdev_val(d) ((unsigned int)(d))
> #define val_to_kdev(d) ((kdev_t(d))
>
> /*
That's an odd-looking patch. Is this what you meant?
Steven
--- linux-2.4.19-rc1/include/linux/kdev_t.h.orig Fri Jun 28 08:31:27 2002
+++ linux-2.4.19-rc1/include/linux/kdev_t.h Fri Jun 28 08:32:36 2002
@@ -81,7 +81,7 @@
#define minor(d) MINOR(d)
#define kdev_same(a,b) ((a) == (b))
#define kdev_none(d) (!(d))
-#define kdev_val(d) ((unsigned int)(d)
+#define kdev_val(d) ((unsigned int)(d))
#define val_to_kdev(d) ((kdev_t(d))
/*
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] compile fix for 2.5 kdev_t compatibility macros
2002-06-28 14:37 ` Steven Cole
@ 2002-06-28 14:42 ` Stephen Lord
2002-06-28 15:02 ` Andries Brouwer
2002-06-28 15:10 ` Martin Josefsson
1 sibling, 1 reply; 6+ messages in thread
From: Stephen Lord @ 2002-06-28 14:42 UTC (permalink / raw)
To: Steven Cole; +Cc: Marcelo Tosatti, Linux Kernel Mailing List
On Fri, 2002-06-28 at 09:37, Steven Cole wrote:
> On Fri, 2002-06-28 at 07:50, Stephen Lord wrote:
> >
> >
> > Marcelo,
> >
> > We started using these for XFS, and found a missing bracket, patch
> > against 2.4.19-rc1.
> >
> > Steve
> >
> > *** linux-2.4.19-rc1/include/linux/kdev_t.h Fri Jun 28 08:40:22 2002
> > --- linux/include/linux/kdev_t.h Fri Jun 28 07:05:32 2002
> > ***************
> > *** 81,87 ****
> > #define minor(d) MINOR(d)
> > #define kdev_same(a,b) ((a) == (b))
> > #define kdev_none(d) (!(d))
> > ! #define kdev_val(d) ((unsigned int)(d)
> > #define val_to_kdev(d) ((kdev_t(d))
> >
> > /*
> > --- 81,87 ----
> > #define minor(d) MINOR(d)
> > #define kdev_same(a,b) ((a) == (b))
> > #define kdev_none(d) (!(d))
> > ! #define kdev_val(d) ((unsigned int)(d))
> > #define val_to_kdev(d) ((kdev_t(d))
> >
> > /*
>
> That's an odd-looking patch. Is this what you meant?
>
> Steven
>
> --- linux-2.4.19-rc1/include/linux/kdev_t.h.orig Fri Jun 28 08:31:27 2002
> +++ linux-2.4.19-rc1/include/linux/kdev_t.h Fri Jun 28 08:32:36 2002
> @@ -81,7 +81,7 @@
> #define minor(d) MINOR(d)
> #define kdev_same(a,b) ((a) == (b))
> #define kdev_none(d) (!(d))
> -#define kdev_val(d) ((unsigned int)(d)
> +#define kdev_val(d) ((unsigned int)(d))
> #define val_to_kdev(d) ((kdev_t(d))
>
> /*
>
Yes, do not use diff before drinking coffee in the morning.
Steve
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] compile fix for 2.5 kdev_t compatibility macros
2002-06-28 14:42 ` Stephen Lord
@ 2002-06-28 15:02 ` Andries Brouwer
0 siblings, 0 replies; 6+ messages in thread
From: Andries Brouwer @ 2002-06-28 15:02 UTC (permalink / raw)
To: Stephen Lord; +Cc: Steven Cole, Marcelo Tosatti, Linux Kernel Mailing List
On Fri, Jun 28, 2002 at 09:42:42AM -0500, Stephen Lord wrote:
> On Fri, 2002-06-28 at 09:37, Steven Cole wrote:
> > On Fri, 2002-06-28 at 07:50, Stephen Lord wrote:
> > >
> > >
> > > Marcelo,
> > >
> > > We started using these for XFS, and found a missing bracket, patch
> > > against 2.4.19-rc1.
> > --- linux-2.4.19-rc1/include/linux/kdev_t.h.orig Fri Jun 28 08:31:27 2002
> > +++ linux-2.4.19-rc1/include/linux/kdev_t.h Fri Jun 28 08:32:36 2002
> > @@ -81,7 +81,7 @@
> > #define minor(d) MINOR(d)
> > #define kdev_same(a,b) ((a) == (b))
> > #define kdev_none(d) (!(d))
> > -#define kdev_val(d) ((unsigned int)(d)
> > +#define kdev_val(d) ((unsigned int)(d))
> > #define val_to_kdev(d) ((kdev_t(d))
> >
> > /*
But what about the next line?
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] compile fix for 2.5 kdev_t compatibility macros
2002-06-28 14:37 ` Steven Cole
2002-06-28 14:42 ` Stephen Lord
@ 2002-06-28 15:10 ` Martin Josefsson
2002-06-28 15:24 ` Steven Cole
1 sibling, 1 reply; 6+ messages in thread
From: Martin Josefsson @ 2002-06-28 15:10 UTC (permalink / raw)
To: Steven Cole; +Cc: Stephen Lord, Marcelo Tosatti, Linux Kernel Mailing List
On Fri, 2002-06-28 at 16:37, Steven Cole wrote:
[snip]
> That's an odd-looking patch. Is this what you meant?
>
> Steven
>
> --- linux-2.4.19-rc1/include/linux/kdev_t.h.orig Fri Jun 28 08:31:27 2002
> +++ linux-2.4.19-rc1/include/linux/kdev_t.h Fri Jun 28 08:32:36 2002
> @@ -81,7 +81,7 @@
> #define minor(d) MINOR(d)
> #define kdev_same(a,b) ((a) == (b))
> #define kdev_none(d) (!(d))
> -#define kdev_val(d) ((unsigned int)(d)
> +#define kdev_val(d) ((unsigned int)(d))
> #define val_to_kdev(d) ((kdev_t(d))
>
> /*
And here's one more...
--- linux-2.4.19-rc1/include/linux/kdev_t.h.orig Fri Jun 28 16:59:48 2002
+++ linux-2.4.19-rc1/include/linux/kdev_t.h Fri Jun 28 17:01:12 2002
@@ -82,7 +82,7 @@
#define kdev_same(a,b) ((a) == (b))
#define kdev_none(d) (!(d))
#define kdev_val(d) ((unsigned int)(d))
-#define val_to_kdev(d) ((kdev_t(d))
+#define val_to_kdev(d) (kdev_t(d))
/*
As long as device numbers in the outside world have 16 bits only,
--
/Martin
Never argue with an idiot. They drag you down to their level, then beat
you with experience.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] compile fix for 2.5 kdev_t compatibility macros
2002-06-28 15:10 ` Martin Josefsson
@ 2002-06-28 15:24 ` Steven Cole
0 siblings, 0 replies; 6+ messages in thread
From: Steven Cole @ 2002-06-28 15:24 UTC (permalink / raw)
To: Martin Josefsson; +Cc: Stephen Lord, Marcelo Tosatti, Linux Kernel Mailing List
On Fri, 2002-06-28 at 09:10, Martin Josefsson wrote:
[snip]
>
> And here's one more...
>
> --- linux-2.4.19-rc1/include/linux/kdev_t.h.orig Fri Jun 28 16:59:48 2002
> +++ linux-2.4.19-rc1/include/linux/kdev_t.h Fri Jun 28 17:01:12 2002
> @@ -82,7 +82,7 @@
> #define kdev_same(a,b) ((a) == (b))
> #define kdev_none(d) (!(d))
> #define kdev_val(d) ((unsigned int)(d))
> -#define val_to_kdev(d) ((kdev_t(d))
> +#define val_to_kdev(d) (kdev_t(d))
>
> /*
> As long as device numbers in the outside world have 16 bits only,
>
> --
> /Martin
>
Hmm. It looks like (d) should be cast to (kdev_t).
Here are both fixes (I hope).
Steven
--- linux-2.4.19-rc1/include/linux/kdev_t.h.orig Fri Jun 28 08:31:27 2002
+++ linux-2.4.19-rc1/include/linux/kdev_t.h Fri Jun 28 09:11:39 2002
@@ -81,8 +81,8 @@
#define minor(d) MINOR(d)
#define kdev_same(a,b) ((a) == (b))
#define kdev_none(d) (!(d))
-#define kdev_val(d) ((unsigned int)(d)
-#define val_to_kdev(d) ((kdev_t(d))
+#define kdev_val(d) ((unsigned int)(d))
+#define val_to_kdev(d) ((kdev_t)(d))
/*
As long as device numbers in the outside world have 16 bits only,
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2002-06-28 15:24 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2002-06-28 13:50 [PATCH] compile fix for 2.5 kdev_t compatibility macros Stephen Lord
2002-06-28 14:37 ` Steven Cole
2002-06-28 14:42 ` Stephen Lord
2002-06-28 15:02 ` Andries Brouwer
2002-06-28 15:10 ` Martin Josefsson
2002-06-28 15:24 ` Steven Cole
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.