Re: [IDE] meaningless #ifndef?

2001-02-20 Thread Hugh Dickins
On Tue, 20 Feb 2001, Vojtech Pavlik wrote: > On Tue, Feb 20, 2001 at 05:45:52PM +, Hugh Dickins wrote: > > > > byte eighty_ninty_three (ide_drive_t *drive) > > > > { > > > > return ((byte) ((HWIF(drive)->udma_four) && > > > > #ifndef CONFIG_IDEDMA_IVB > > > >

Re: [IDE] meaningless #ifndef?

2001-02-20 Thread Vojtech Pavlik
On Tue, Feb 20, 2001 at 05:45:52PM +, Hugh Dickins wrote: > > > > > from drivers/ide/ide-features.c: > > > > > > /* > > > * All hosts that use the 80c ribbon mus use! > > > */ > > > byte eighty_ninty_three (ide_drive_t *drive) > > > { > > > return ((byte)

Re: [IDE] meaningless #ifndef?

2001-02-20 Thread Hugh Dickins
On Mon, 19 Feb 2001, Andre Hedrick wrote: > On Mon, 19 Feb 2001, Pozsar Balazs wrote: > > > from drivers/ide/ide-features.c: > > > > /* > > * All hosts that use the 80c ribbon mus use! > > */ > > byte eighty_ninty_three (ide_drive_t *drive) > > { > > return ((byte)

Re: [IDE] meaningless #ifndef?

2001-02-20 Thread Andrzej Krzysztofowicz
"Pozsar Balazs wrote:" > from drivers/ide/ide-features.c: > > /* > * All hosts that use the 80c ribbon mus use! > */ > byte eighty_ninty_three (ide_drive_t *drive) > { > return ((byte) ((HWIF(drive)->udma_four) && > #ifndef CONFIG_IDEDMA_IVB >

Re: [IDE] meaningless #ifndef?

2001-02-20 Thread Andrzej Krzysztofowicz
"Pozsar Balazs wrote:" from drivers/ide/ide-features.c: /* * All hosts that use the 80c ribbon mus use! */ byte eighty_ninty_three (ide_drive_t *drive) { return ((byte) ((HWIF(drive)-udma_four) #ifndef CONFIG_IDEDMA_IVB (drive-id-hw_config 0x4000)

Re: [IDE] meaningless #ifndef?

2001-02-20 Thread Hugh Dickins
On Mon, 19 Feb 2001, Andre Hedrick wrote: On Mon, 19 Feb 2001, Pozsar Balazs wrote: from drivers/ide/ide-features.c: /* * All hosts that use the 80c ribbon mus use! */ byte eighty_ninty_three (ide_drive_t *drive) { return ((byte) ((HWIF(drive)-udma_four) #ifndef

Re: [IDE] meaningless #ifndef?

2001-02-20 Thread Vojtech Pavlik
On Tue, Feb 20, 2001 at 05:45:52PM +, Hugh Dickins wrote: from drivers/ide/ide-features.c: /* * All hosts that use the 80c ribbon mus use! */ byte eighty_ninty_three (ide_drive_t *drive) { return ((byte) ((HWIF(drive)-udma_four) #ifndef

Re: [IDE] meaningless #ifndef?

2001-02-20 Thread Hugh Dickins
On Tue, 20 Feb 2001, Vojtech Pavlik wrote: On Tue, Feb 20, 2001 at 05:45:52PM +, Hugh Dickins wrote: byte eighty_ninty_three (ide_drive_t *drive) { return ((byte) ((HWIF(drive)-udma_four) #ifndef CONFIG_IDEDMA_IVB (drive-id-hw_config

Re: [IDE] meaningless #ifndef?

2001-02-19 Thread Tom Leete
Bill Wendling wrote: > > The use of the ternary operator is superfluous, though...and makes the > code look ugly IMNSHO :). > You are correct. Please ignore my thinko. Tom -- The Daemons lurk and are dumb. -- Emerson - To unsubscribe from this list: send the line "unsubscribe linux-kernel"

Re: [IDE] meaningless #ifndef?

2001-02-19 Thread Tom Leete
Bill Wendling wrote: > > Also sprach Andre Hedrick: > } On Mon, 19 Feb 2001, Pozsar Balazs wrote: > } > } > from drivers/ide/ide-features.c: > } > > } > /* > } > * All hosts that use the 80c ribbon mus use! > } > */ > } > byte eighty_ninty_three (ide_drive_t *drive) > } > { > } >

Re: [IDE] meaningless #ifndef?

2001-02-19 Thread Andre Hedrick
On Mon, 19 Feb 2001, Bill Wendling wrote: > The use of the ternary operator is superfluous, though...and makes the > code look ugly IMNSHO :). What is ugly is that the commitee can not decide if there is going to be host-side only, device-side only or both-side. Cheers, Andre Hedrick Linux

Re: [IDE] meaningless #ifndef?

2001-02-19 Thread Bill Wendling
Also sprach Andre Hedrick: } On Mon, 19 Feb 2001, Pozsar Balazs wrote: } } > from drivers/ide/ide-features.c: } > } > /* } > * All hosts that use the 80c ribbon mus use! } > */ } > byte eighty_ninty_three (ide_drive_t *drive) } > { } > return ((byte) ((HWIF(drive)->udma_four) && } >

Re: [IDE] meaningless #ifndef?

2001-02-19 Thread Andre Hedrick
On Mon, 19 Feb 2001, Pozsar Balazs wrote: > from drivers/ide/ide-features.c: > > /* > * All hosts that use the 80c ribbon mus use! > */ > byte eighty_ninty_three (ide_drive_t *drive) > { > return ((byte) ((HWIF(drive)->udma_four) && > #ifndef CONFIG_IDEDMA_IVB >

[IDE] meaningless #ifndef?

2001-02-19 Thread Pozsar Balazs
from drivers/ide/ide-features.c: /* * All hosts that use the 80c ribbon mus use! */ byte eighty_ninty_three (ide_drive_t *drive) { return ((byte) ((HWIF(drive)->udma_four) && #ifndef CONFIG_IDEDMA_IVB (drive->id->hw_config & 0x4000) && #endif /*

[IDE] meaningless #ifndef?

2001-02-19 Thread Pozsar Balazs
from drivers/ide/ide-features.c: /* * All hosts that use the 80c ribbon mus use! */ byte eighty_ninty_three (ide_drive_t *drive) { return ((byte) ((HWIF(drive)-udma_four) #ifndef CONFIG_IDEDMA_IVB (drive-id-hw_config 0x4000) #endif /* CONFIG_IDEDMA_IVB */

Re: [IDE] meaningless #ifndef?

2001-02-19 Thread Andre Hedrick
On Mon, 19 Feb 2001, Pozsar Balazs wrote: from drivers/ide/ide-features.c: /* * All hosts that use the 80c ribbon mus use! */ byte eighty_ninty_three (ide_drive_t *drive) { return ((byte) ((HWIF(drive)-udma_four) #ifndef CONFIG_IDEDMA_IVB

Re: [IDE] meaningless #ifndef?

2001-02-19 Thread Bill Wendling
Also sprach Andre Hedrick: } On Mon, 19 Feb 2001, Pozsar Balazs wrote: } } from drivers/ide/ide-features.c: } } /* } * All hosts that use the 80c ribbon mus use! } */ } byte eighty_ninty_three (ide_drive_t *drive) } { } return ((byte) ((HWIF(drive)-udma_four) } #ifndef

Re: [IDE] meaningless #ifndef?

2001-02-19 Thread Andre Hedrick
On Mon, 19 Feb 2001, Bill Wendling wrote: The use of the ternary operator is superfluous, though...and makes the code look ugly IMNSHO :). What is ugly is that the commitee can not decide if there is going to be host-side only, device-side only or both-side. Cheers, Andre Hedrick Linux ATA

Re: [IDE] meaningless #ifndef?

2001-02-19 Thread Tom Leete
Bill Wendling wrote: Also sprach Andre Hedrick: } On Mon, 19 Feb 2001, Pozsar Balazs wrote: } } from drivers/ide/ide-features.c: } } /* } * All hosts that use the 80c ribbon mus use! } */ } byte eighty_ninty_three (ide_drive_t *drive) } { } return ((byte)

Re: [IDE] meaningless #ifndef?

2001-02-19 Thread Tom Leete
Bill Wendling wrote: The use of the ternary operator is superfluous, though...and makes the code look ugly IMNSHO :). You are correct. Please ignore my thinko. Tom -- The Daemons lurk and are dumb. -- Emerson - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in