Re: [TRIVIAL PATCH 2.5.51] Remove compile warning from

[Alan Cox]

On Wed, 2002-12-11 at 22:28, Bob Miller wrote:
> The function, cs5520_tune_chipset() is declared to return an int.
> Added a return statement instead of just falling of off the bottom.
> 
> diff -Nru a/drivers/ide/pci/cs5520.c b/drivers/ide/pci/cs5520.c
> --- a/drivers/ide/pci/cs5520.c	Wed Dec 11 13:41:51 2002
> +++ b/drivers/ide/pci/cs5520.c	Wed Dec 11 13:41:51 2002
> @@ -166,6 +166,8 @@
>  	/* ATAPI is harder so leave it for now */
>  	if(!error && drive->media == ide_disk)
>  		error = hwif->ide_dma_on(drive);
> +
> +	return error;

Eep thats a nasty little bug, lucky my compiler left the error value in
%eax anyway good spotting