* [Kernel-janitors] min/max macros from kernel.h
@ 2004-07-04 11:40 Clemens Buchacher
2004-07-04 14:00 ` Michael Veeck
` (6 more replies)
0 siblings, 7 replies; 8+ messages in thread
From: Clemens Buchacher @ 2004-07-04 11:40 UTC (permalink / raw)
To: kernel-janitors
[-- Attachment #1: Type: text/plain, Size: 944 bytes --]
Hi,
I've replaced the custom MIN/MAX macros in drivers/ide/ide-timing.h with
the corresponding min/max macros from kernel.h, as suggested by the TODO
list.
The macros are used for the FIT macro, which returns the value of the
first parameter, delimited by the range specified by the second and
third parameters. This is done by a combination of min/max:
#define FIT(v,vmin,vmax) max(min(v,vmax),vmin)
This expands to something like this:
max(
({
typeof(x) _x = (x);
typeof(y) _y = (y);
(void) (&_x == &_y);
_x < _y ? _x : _y;
}),
vmin
);
So min() returns a value of the same type as v and vmax, and vmin should
be of this type as well.
Nevertheless, I get a warning when I cast the vmin value to a short in
the following use of FIT() (drivers/ide/pci/via82cxxx.c):
FIT(timing->setup, 1, (short)4)
timing->setup is of type short, and only this typecast combination
compiles without warning.
What's going on here?
Regards,
Clemens
[-- Attachment #2: minmax.patch --]
[-- Type: text/plain, Size: 6754 bytes --]
# This is a BitKeeper generated diff -Nru style patch.
#
# ChangeSet
# 2004/07/04 03:24:26+02:00 drizzd@aon
# replaced custom MIN/MAX macros with min/max from kernel.h
#
# drivers/ide/pci/via82cxxx.c
# 2004/07/04 03:23:22+02:00 drizzd@aon +7 -7
# added typecasts to satisfy min/max
#
# drivers/ide/pci/amd74xx.c
# 2004/07/04 03:23:22+02:00 drizzd@aon +7 -7
# added typecasts to satisfy min/max
#
# drivers/ide/ide-timing.h
# 2004/07/04 03:23:22+02:00 drizzd@aon +12 -11
# using min/max from kernel.h now
#
# BitKeeper/etc/ignore
# 2004/07/04 03:23:22+02:00 drizzd@aon +2 -0
# Added .exrc Module.symvers to the ignore list
#
# drivers/ide/ide-timing.h
# 2004/07/03 17:18:20+02:00 drizzd@aon +0 -2
# removed unused macros
#
diff -Nru a/drivers/ide/ide-timing.h b/drivers/ide/ide-timing.h
--- a/drivers/ide/ide-timing.h 2004-07-04 12:23:14 +02:00
+++ b/drivers/ide/ide-timing.h 2004-07-04 12:23:14 +02:00
@@ -27,6 +27,7 @@
* Vojtech Pavlik, Simunkova 1594, Prague 8, 182 00 Czech Republic
*/
+#include <linux/kernel.h>
#include <linux/hdreg.h>
#define XFER_PIO_5 0x0d
@@ -96,11 +97,9 @@
#define IDE_TIMING_UDMA 0x80
#define IDE_TIMING_ALL 0xff
-#define MIN(a,b) ((a)<(b)?(a):(b))
-#define MAX(a,b) ((a)>(b)?(a):(b))
-#define FIT(v,min,max) MAX(MIN(v,max),min)
-#define ENOUGH(v,unit) (((v)-1)/(unit)+1)
-#define EZ(v,unit) ((v)?ENOUGH(v,unit):0)
+#define FIT(v,vmin,vmax) max(min(v,vmax),vmin)
+#define ENOUGH(v,unit) (((v)-1)/(unit)+1)
+#define EZ(v,unit) ((v)?ENOUGH(v,unit):0)
#define XFER_MODE 0xf0
#define XFER_UDMA_133 0x48
@@ -188,14 +187,14 @@
static void ide_timing_merge(struct ide_timing *a, struct ide_timing *b, struct ide_timing *m, unsigned int what)
{
- if (what & IDE_TIMING_SETUP ) m->setup = MAX(a->setup, b->setup);
- if (what & IDE_TIMING_ACT8B ) m->act8b = MAX(a->act8b, b->act8b);
- if (what & IDE_TIMING_REC8B ) m->rec8b = MAX(a->rec8b, b->rec8b);
- if (what & IDE_TIMING_CYC8B ) m->cyc8b = MAX(a->cyc8b, b->cyc8b);
- if (what & IDE_TIMING_ACTIVE ) m->active = MAX(a->active, b->active);
- if (what & IDE_TIMING_RECOVER) m->recover = MAX(a->recover, b->recover);
- if (what & IDE_TIMING_CYCLE ) m->cycle = MAX(a->cycle, b->cycle);
- if (what & IDE_TIMING_UDMA ) m->udma = MAX(a->udma, b->udma);
+ if (what & IDE_TIMING_SETUP ) m->setup = max(a->setup, b->setup);
+ if (what & IDE_TIMING_ACT8B ) m->act8b = max(a->act8b, b->act8b);
+ if (what & IDE_TIMING_REC8B ) m->rec8b = max(a->rec8b, b->rec8b);
+ if (what & IDE_TIMING_CYC8B ) m->cyc8b = max(a->cyc8b, b->cyc8b);
+ if (what & IDE_TIMING_ACTIVE ) m->active = max(a->active, b->active);
+ if (what & IDE_TIMING_RECOVER) m->recover = max(a->recover, b->recover);
+ if (what & IDE_TIMING_CYCLE ) m->cycle = max(a->cycle, b->cycle);
+ if (what & IDE_TIMING_UDMA ) m->udma = max(a->udma, b->udma);
}
static struct ide_timing* ide_timing_find_mode(short speed)
diff -Nru a/drivers/ide/pci/amd74xx.c b/drivers/ide/pci/amd74xx.c
--- a/drivers/ide/pci/amd74xx.c 2004-07-04 12:23:14 +02:00
+++ b/drivers/ide/pci/amd74xx.c 2004-07-04 12:23:14 +02:00
@@ -205,20 +205,20 @@
unsigned char t;
pci_read_config_byte(dev, AMD_ADDRESS_SETUP, &t);
- t = (t & ~(3 << ((3 - dn) << 1))) | ((FIT(timing->setup, 1, 4) - 1) << ((3 - dn) << 1));
+ t = (t & ~(3 << ((3 - dn) << 1))) | ((FIT(timing->setup, 1, (short)4) - 1) << ((3 - dn) << 1));
pci_write_config_byte(dev, AMD_ADDRESS_SETUP, t);
pci_write_config_byte(dev, AMD_8BIT_TIMING + (1 - (dn >> 1)),
- ((FIT(timing->act8b, 1, 16) - 1) << 4) | (FIT(timing->rec8b, 1, 16) - 1));
+ ((FIT(timing->act8b, 1, (short)16) - 1) << 4) | (FIT(timing->rec8b, 1, (short)16) - 1));
pci_write_config_byte(dev, AMD_DRIVE_TIMING + (3 - dn),
- ((FIT(timing->active, 1, 16) - 1) << 4) | (FIT(timing->recover, 1, 16) - 1));
+ ((FIT(timing->active, 1, (short)16) - 1) << 4) | (FIT(timing->recover, 1, (short)16) - 1));
switch (amd_config->flags & AMD_UDMA) {
- case AMD_UDMA_33: t = timing->udma ? (0xc0 | (FIT(timing->udma, 2, 5) - 2)) : 0x03; break;
- case AMD_UDMA_66: t = timing->udma ? (0xc0 | amd_cyc2udma[FIT(timing->udma, 2, 10)]) : 0x03; break;
- case AMD_UDMA_100: t = timing->udma ? (0xc0 | amd_cyc2udma[FIT(timing->udma, 1, 10)]) : 0x03; break;
- case AMD_UDMA_133: t = timing->udma ? (0xc0 | amd_cyc2udma[FIT(timing->udma, 1, 15)]) : 0x03; break;
+ case AMD_UDMA_33: t = timing->udma ? (0xc0 | (FIT(timing->udma, 2, (short)5) - 2)) : 0x03; break;
+ case AMD_UDMA_66: t = timing->udma ? (0xc0 | amd_cyc2udma[FIT(timing->udma, 2, (short)10)]) : 0x03; break;
+ case AMD_UDMA_100: t = timing->udma ? (0xc0 | amd_cyc2udma[FIT(timing->udma, 1, (short)10)]) : 0x03; break;
+ case AMD_UDMA_133: t = timing->udma ? (0xc0 | amd_cyc2udma[FIT(timing->udma, 1, (short)15)]) : 0x03; break;
default: return;
}
diff -Nru a/drivers/ide/pci/via82cxxx.c b/drivers/ide/pci/via82cxxx.c
--- a/drivers/ide/pci/via82cxxx.c 2004-07-04 12:23:14 +02:00
+++ b/drivers/ide/pci/via82cxxx.c 2004-07-04 12:23:14 +02:00
@@ -291,21 +291,21 @@
if (~via_config->flags & VIA_BAD_AST) {
pci_read_config_byte(dev, VIA_ADDRESS_SETUP, &t);
- t = (t & ~(3 << ((3 - dn) << 1))) | ((FIT(timing->setup, 1, 4) - 1) << ((3 - dn) << 1));
+ t = (t & ~(3 << ((3 - dn) << 1))) | ((FIT(timing->setup, 1, (short)4) - 1) << ((3 - dn) << 1));
pci_write_config_byte(dev, VIA_ADDRESS_SETUP, t);
}
pci_write_config_byte(dev, VIA_8BIT_TIMING + (1 - (dn >> 1)),
- ((FIT(timing->act8b, 1, 16) - 1) << 4) | (FIT(timing->rec8b, 1, 16) - 1));
+ ((FIT(timing->act8b, 1, (short)16) - 1) << 4) | (FIT(timing->rec8b, 1, (short)16) - 1));
pci_write_config_byte(dev, VIA_DRIVE_TIMING + (3 - dn),
- ((FIT(timing->active, 1, 16) - 1) << 4) | (FIT(timing->recover, 1, 16) - 1));
+ ((FIT(timing->active, 1, (short)16) - 1) << 4) | (FIT(timing->recover, 1, (short)16) - 1));
switch (via_config->flags & VIA_UDMA) {
- case VIA_UDMA_33: t = timing->udma ? (0xe0 | (FIT(timing->udma, 2, 5) - 2)) : 0x03; break;
- case VIA_UDMA_66: t = timing->udma ? (0xe8 | (FIT(timing->udma, 2, 9) - 2)) : 0x0f; break;
- case VIA_UDMA_100: t = timing->udma ? (0xe0 | (FIT(timing->udma, 2, 9) - 2)) : 0x07; break;
- case VIA_UDMA_133: t = timing->udma ? (0xe0 | (FIT(timing->udma, 2, 9) - 2)) : 0x07; break;
+ case VIA_UDMA_33: t = timing->udma ? (0xe0 | (FIT(timing->udma, 2, (short)5) - 2)) : 0x03; break;
+ case VIA_UDMA_66: t = timing->udma ? (0xe8 | (FIT(timing->udma, 2, (short)9) - 2)) : 0x0f; break;
+ case VIA_UDMA_100: t = timing->udma ? (0xe0 | (FIT(timing->udma, 2, (short)9) - 2)) : 0x07; break;
+ case VIA_UDMA_133: t = timing->udma ? (0xe0 | (FIT(timing->udma, 2, (short)9) - 2)) : 0x07; break;
default: return;
}
[-- Attachment #3: Type: text/plain, Size: 167 bytes --]
_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
http://lists.osdl.org/mailman/listinfo/kernel-janitors
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [Kernel-janitors] min/max macros from kernel.h
2004-07-04 11:40 [Kernel-janitors] min/max macros from kernel.h Clemens Buchacher
@ 2004-07-04 14:00 ` Michael Veeck
2004-07-05 14:04 ` maximilian attems
` (5 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Michael Veeck @ 2004-07-04 14:00 UTC (permalink / raw)
To: kernel-janitors
[-- Attachment #1: Type: text/plain, Size: 446 bytes --]
Clemens Buchacher wrote:
> I've replaced the custom MIN/MAX macros in drivers/ide/ide-timing.h with
> the corresponding min/max macros from kernel.h, as suggested by the TODO
> list.
One advice: Before doing stuff on the todo list, ask on the mailing list
if anybodys working on it. This could save you some time.
Otherwise the patch looks good to me, but next time also tell against
which version it is. 2.6.7 I presume?
Greetings,
Veeck
[-- Attachment #2: Type: text/plain, Size: 167 bytes --]
_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
http://lists.osdl.org/mailman/listinfo/kernel-janitors
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [Kernel-janitors] min/max macros from kernel.h
2004-07-04 11:40 [Kernel-janitors] min/max macros from kernel.h Clemens Buchacher
2004-07-04 14:00 ` Michael Veeck
@ 2004-07-05 14:04 ` maximilian attems
2004-07-10 13:53 ` Clemens Buchacher
` (4 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: maximilian attems @ 2004-07-05 14:04 UTC (permalink / raw)
To: kernel-janitors
[-- Attachment #1: Type: text/plain, Size: 358 bytes --]
On Sun, 04 Jul 2004, Clemens Buchacher wrote:
> Hi,
>
> I've replaced the custom MIN/MAX macros in drivers/ide/ide-timing.h with
> the corresponding min/max macros from kernel.h, as suggested by the TODO
> list.
please read Documentation/SubmittingPatches for further patches.
no attachments ;-)
1 patch / mail helps your patches to be merged.
a++ maks
[-- Attachment #2: Type: text/plain, Size: 167 bytes --]
_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
http://lists.osdl.org/mailman/listinfo/kernel-janitors
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [Kernel-janitors] min/max macros from kernel.h
2004-07-04 11:40 [Kernel-janitors] min/max macros from kernel.h Clemens Buchacher
2004-07-04 14:00 ` Michael Veeck
2004-07-05 14:04 ` maximilian attems
@ 2004-07-10 13:53 ` Clemens Buchacher
2004-07-10 14:00 ` Clemens Buchacher
` (3 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Clemens Buchacher @ 2004-07-10 13:53 UTC (permalink / raw)
To: kernel-janitors
[-- Attachment #1: Type: text/plain, Size: 832 bytes --]
On Mon, Jul 05, 2004 at 03:57:02PM +0200, Michael Veeck wrote:
> One advice: Before doing stuff on the todo list, ask on the mailing list
> if anybodys working on it. This could save you some time.
I posted this patch to the kernel-janitors mailing list almost a week
ago. As there was no reaction to it I presumed nobody was working on the
MIN/MAX patches. Unfortunately, I received you patches only just today.
I will wait longer next time.
Anyways, as you seem to have taken on this job, will you take in my
patch and submit it collectively with yours?
I will then leave the rest of the MIN/MAX work to you, if that's ok.
> Otherwise the patch looks good to me, but next time also tell against
> which version it is. 2.6.7 I presume?
It's against the http://linux.bkbits.net/linux-2.5 TOT as of that day,
so yes.
Clemens
[-- Attachment #2: Type: text/plain, Size: 167 bytes --]
_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
http://lists.osdl.org/mailman/listinfo/kernel-janitors
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [Kernel-janitors] min/max macros from kernel.h
2004-07-04 11:40 [Kernel-janitors] min/max macros from kernel.h Clemens Buchacher
` (2 preceding siblings ...)
2004-07-10 13:53 ` Clemens Buchacher
@ 2004-07-10 14:00 ` Clemens Buchacher
2004-07-11 10:13 ` Michael Veeck
` (2 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Clemens Buchacher @ 2004-07-10 14:00 UTC (permalink / raw)
To: kernel-janitors
[-- Attachment #1: Type: text/plain, Size: 441 bytes --]
On Mon, Jul 05, 2004 at 04:04:00PM +0200, maximilian attems wrote:
> please read Documentation/SubmittingPatches for further patches.
> no attachments ;-)
Thanks for the hint.
> 1 patch / mail helps your patches to be merged.
As these patches were interdependent and corresponded to a single
logical change I thought it was ok to group them together as per
paragraph 3 "Separate your changes" in Documentation/SubmittingPatches.
Clemens
[-- Attachment #2: Type: text/plain, Size: 167 bytes --]
_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
http://lists.osdl.org/mailman/listinfo/kernel-janitors
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [Kernel-janitors] min/max macros from kernel.h
2004-07-04 11:40 [Kernel-janitors] min/max macros from kernel.h Clemens Buchacher
` (3 preceding siblings ...)
2004-07-10 14:00 ` Clemens Buchacher
@ 2004-07-11 10:13 ` Michael Veeck
2004-07-11 10:55 ` maximilian attems
2004-07-11 11:27 ` maximilian attems
6 siblings, 0 replies; 8+ messages in thread
From: Michael Veeck @ 2004-07-11 10:13 UTC (permalink / raw)
To: kernel-janitors
[-- Attachment #1: Type: text/plain, Size: 1289 bytes --]
Clemens Buchacher schrieb:
> On Mon, Jul 05, 2004 at 03:57:02PM +0200, Michael Veeck wrote:
>
>>One advice: Before doing stuff on the todo list, ask on the mailing list
>>if anybodys working on it. This could save you some time.
>
>
> I posted this patch to the kernel-janitors mailing list almost a week
> ago. As there was no reaction to it I presumed nobody was working on the
> MIN/MAX patches. Unfortunately, I received you patches only just today.
> I will wait longer next time.
I received your mail very late, maybe my provider messed something up...
> Anyways, as you seem to have taken on this job, will you take in my
> patch and submit it collectively with yours?
>
> I will then leave the rest of the MIN/MAX work to you, if that's ok.
Most of the macros are now removed, some arch-specific stuff is left to
do. They just need to be merged by the maintainers and/or
kernel-janitors-patchset.
Question to all: Should I resend the not yet merged patches to this list
or what should happen next?
Thanks for your help, Clemens
Reagrds
eeck
>>Otherwise the patch looks good to me, but next time also tell against
>>which version it is. 2.6.7 I presume?
>
>
> It's against the http://linux.bkbits.net/linux-2.5 TOT as of that day,
> so yes.
>
> Clemens
>
>
[-- Attachment #2: Type: text/plain, Size: 167 bytes --]
_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
http://lists.osdl.org/mailman/listinfo/kernel-janitors
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [Kernel-janitors] min/max macros from kernel.h
2004-07-04 11:40 [Kernel-janitors] min/max macros from kernel.h Clemens Buchacher
` (4 preceding siblings ...)
2004-07-11 10:13 ` Michael Veeck
@ 2004-07-11 10:55 ` maximilian attems
2004-07-11 11:27 ` maximilian attems
6 siblings, 0 replies; 8+ messages in thread
From: maximilian attems @ 2004-07-11 10:55 UTC (permalink / raw)
To: kernel-janitors
[-- Attachment #1: Type: text/plain, Size: 698 bytes --]
On Sat, 10 Jul 2004, Clemens Buchacher wrote:
> On Mon, Jul 05, 2004 at 04:04:00PM +0200, maximilian attems wrote:
> > please read Documentation/SubmittingPatches for further patches.
> > no attachments ;-)
>
> Thanks for the hint.
>
> > 1 patch / mail helps your patches to be merged.
>
> As these patches were interdependent and corresponded to a single
> logical change I thought it was ok to group them together as per
> paragraph 3 "Separate your changes" in Documentation/SubmittingPatches.
that's ok for experienced,
but doesn't speed up merging, as patches might need to go to different
maintainers, anyway i saw that last version touches just the header.
great continue :)
a++ maks
[-- Attachment #2: Type: text/plain, Size: 167 bytes --]
_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
http://lists.osdl.org/mailman/listinfo/kernel-janitors
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [Kernel-janitors] min/max macros from kernel.h
2004-07-04 11:40 [Kernel-janitors] min/max macros from kernel.h Clemens Buchacher
` (5 preceding siblings ...)
2004-07-11 10:55 ` maximilian attems
@ 2004-07-11 11:27 ` maximilian attems
6 siblings, 0 replies; 8+ messages in thread
From: maximilian attems @ 2004-07-11 11:27 UTC (permalink / raw)
To: kernel-janitors
[-- Attachment #1: Type: text/plain, Size: 1298 bytes --]
On Sun, 11 Jul 2004, Michael Veeck wrote:
>
>
> Clemens Buchacher schrieb:
> >On Mon, Jul 05, 2004 at 03:57:02PM +0200, Michael Veeck wrote:
> >
> >>One advice: Before doing stuff on the todo list, ask on the mailing list
> >>if anybodys working on it. This could save you some time.
> >
> >
> >I posted this patch to the kernel-janitors mailing list almost a week
> >ago. As there was no reaction to it I presumed nobody was working on the
> >MIN/MAX patches. Unfortunately, I received you patches only just today.
> >I will wait longer next time.
>
> I received your mail very late, maybe my provider messed something up...
>
> >Anyways, as you seem to have taken on this job, will you take in my
> >patch and submit it collectively with yours?
> >
> >I will then leave the rest of the MIN/MAX work to you, if that's ok.
>
> Most of the macros are now removed, some arch-specific stuff is left to
> do. They just need to be merged by the maintainers and/or
> kernel-janitors-patchset.
>
> Question to all: Should I resend the not yet merged patches to this list
> or what should happen next?
i'll be sending a new kjt patchset today,
inside i merged all your latest min-max patches for drivers/char
(1 or 2 where outside), but yes please send to kj those not yet merged.
thanks maks
[-- Attachment #2: Type: text/plain, Size: 167 bytes --]
_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
http://lists.osdl.org/mailman/listinfo/kernel-janitors
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2004-07-11 11:27 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-07-04 11:40 [Kernel-janitors] min/max macros from kernel.h Clemens Buchacher
2004-07-04 14:00 ` Michael Veeck
2004-07-05 14:04 ` maximilian attems
2004-07-10 13:53 ` Clemens Buchacher
2004-07-10 14:00 ` Clemens Buchacher
2004-07-11 10:13 ` Michael Veeck
2004-07-11 10:55 ` maximilian attems
2004-07-11 11:27 ` maximilian attems
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.