diff options
| author | Jean Delvare <khali@linux-fr.org> | 2007-09-09 22:29:13 +0200 |
|---|---|---|
| committer | Jean Delvare <khali@hyperion.delvare> | 2007-09-09 22:29:13 +0200 |
| commit | 51e5709ad46127da9476f31336113b5401f94dba (patch) | |
| tree | b5ac346545ef80576843143108060125f9c6bbd2 /drivers/i2c | |
| parent | 9a3180e70e35ff69586ada5421b5e7bad52f60e4 (diff) | |
| download | op-kernel-dev-51e5709ad46127da9476f31336113b5401f94dba.zip op-kernel-dev-51e5709ad46127da9476f31336113b5401f94dba.tar.gz | |
i2c-pxa: Fix adapter number
It turns out that platform_device.id is a "u32" so testing it for being
nonnegative is useless when setting up an i2c adapte. Instead,
do what the platform_bus code does and test it against the value "-1".
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Diffstat (limited to 'drivers/i2c')
| -rw-r--r-- | drivers/i2c/busses/i2c-pxa.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/i2c/busses/i2c-pxa.c b/drivers/i2c/busses/i2c-pxa.c index 9d6b790..bb5466b 100644 --- a/drivers/i2c/busses/i2c-pxa.c +++ b/drivers/i2c/busses/i2c-pxa.c @@ -926,7 +926,7 @@ static int i2c_pxa_probe(struct platform_device *dev) * The reason to do so is to avoid sysfs names that only make * sense when there are multiple adapters. */ - i2c->adap.nr = dev->id >= 0 ? dev->id : 0; + i2c->adap.nr = dev->id != -1 ? dev->id : 0; ret = i2c_add_numbered_adapter(&i2c->adap); if (ret < 0) { |
