Re: [PATCH] sata_via VT6421 vendor update

[Mark Lord]

Jeff Garzik wrote:
> +	switch (adev->dma_mode & 0x07) {
> +		case 0:
> +			cfg_byte = 0xee;
> +			break;	
> +		case 1:
> +			cfg_byte = 0xe8;
> +			break;	
> +		case 2:
> +			cfg_byte = 0xe6;
> +			break;	
> +		case 3:
> +			cfg_byte = 0xe4;
> +			break;	
> +		case 4:
> +			cfg_byte = 0xe2;
> +			break;	
> +		case 5:
> +			cfg_byte = 0xe1;
> +			break;	
> +		case 6:
> +			cfg_byte = 0xe0;
> +			break;	
> +		default:
> +			cfg_byte = 0xe0;
> +	}

Mmm.. replace all that with this (?):

	u8 cfg_bytes[8] = {0xee, 0xe8, 0xe6, 0xe4, 0xe2, 0xe1, 0xe0, 0xe0};
	cfg_byte = cfg_bytes[adev->dma_mode & 7];

-ml
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/