The patch titled
i2c: SMbus i2c-i801 ioports request
has been removed from the -mm tree. Its filename was
smbus-i2c-i801-ioports-request.patch
This patch was dropped because it was nacked
The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/
------------------------------------------------------
Subject: i2c: SMbus i2c-i801 ioports request
From: Maxim Uvarov <muvarov@xxxxxxxxxxxxx>
MPCBL0040 target has buggy DC752HRA bios with wrong length of SMBus region
(SMBL field). That makes acpi pnp code reserve incorrect region. i2c-801
driver fails on requesting ioport region because half of this region is alredy
held by acpi pnp and regions lengths are not match. This patch requests each
ioport separately so it doesn't matter now if ioport is reserved by acpi pnp
or not.
This patch makes extra records in /proc/ioport. One record per ioport = 20
lines. In case with DC752HRA bios we can request ioports on 2 commands and we
will have only 2 lines in /proc/ioport. But this solution is more universal.
Signed-off-by: Maxim Uvarov <muvarov@xxxxxxxxxxxxx>
Cc: Jean Delvare <khali@xxxxxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
---
drivers/i2c/busses/i2c-i801.c | 19 ++++++++++++-------
1 file changed, 12 insertions(+), 7 deletions(-)
diff -puN drivers/i2c/busses/i2c-i801.c~smbus-i2c-i801-ioports-request drivers/i2c/busses/i2c-i801.c
--- a/drivers/i2c/busses/i2c-i801.c~smbus-i2c-i801-ioports-request
+++ a/drivers/i2c/busses/i2c-i801.c
@@ -602,6 +602,7 @@ static int __devinit i801_probe(struct p
{
unsigned char temp;
int err;
+ int idx;
I801_dev = dev;
i801_features = 0;
@@ -644,12 +645,15 @@ static int __devinit i801_probe(struct p
if (err)
goto exit;
- err = pci_request_region(dev, SMBBAR, i801_driver.name);
- if (err) {
- dev_err(&dev->dev, "Failed to request SMBus region "
- "0x%lx-0x%Lx\n", i801_smba,
- (unsigned long long)pci_resource_end(dev, SMBBAR));
- goto exit;
+ for (idx = 0; idx < pci_resource_len(dev, SMBBAR); idx++) {
+ if (!request_region(i801_smba + idx, 1, i801_driver.name)) {
+ while (idx--)
+ release_region(i801_smba + idx, 1);
+ dev_err(&dev->dev, "Failed to request SMBus region "
+ "0x%lx-0x%Lx\n", i801_smba,
+ (unsigned long long)pci_resource_end(dev, SMBBAR));
+ goto exit;
+ }
}
pci_read_config_byte(I801_dev, SMBHSTCFG, &temp);
@@ -684,7 +688,8 @@ static int __devinit i801_probe(struct p
return 0;
exit_release:
- pci_release_region(dev, SMBBAR);
+ for (idx = 0; idx < pci_resource_len(dev, SMBBAR); idx++)
+ release_region(i801_smba + idx, 1);
exit:
return err;
}
_
Patches currently in -mm which might be from muvarov@xxxxxxxxxxxxx are
smbus-i2c-i801-ioports-request.patch
--
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
