[PATCH] ALI1563 SMBus driver fix

Jean Delvare khali at linux-fr.org
Tue Apr 12 21:49:32 CEST 2005


Hi Ruik,

> -/**
> +/*
>   *	i2c-ali1563.c - i2c driver for the ALi 1563 Southbridge
>   *
>   *	Copyright (C) 2004 Patrick Mochel

Please avoid noise in patches, it doesn't help review.

> +	/* device error - probably missing ACK, from autdetection I2C_QUICK */
> +	if (data & HST_STS_DEVERR) {
> +		dev_dbg(&a->dev, "Device error!\n");
> +	}

I don't like this. If this "error" is normal for I2C_QUICK, then don't
issue any message and return 0 in this case, and issue an *error*, not
debug, message in the other cases.

As far as I can see, this means adding a second parameter to
ali1563_transaction, but I don't think this is a problem.

>  	/* Write the command register */
> +
>  	switch(size) {

Noise :p

I'm otherwise fine with your patch, thanks a lot for your great
investigation work on this.

-- 
Jean Delvare



More information about the lm-sensors mailing list