Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S936349Ab0B1S7J (ORCPT ); Sun, 28 Feb 2010 13:59:09 -0500 Received: from mga10.intel.com ([192.55.52.92]:5621 "EHLO fmsmga102.fm.intel.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S936412Ab0B1S7F (ORCPT ); Sun, 28 Feb 2010 13:59:05 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.49,556,1262592000"; d="scan'208";a="544698358" Date: Sun, 28 Feb 2010 20:00:16 +0100 From: Samuel Ortiz To: Denis Turischev Cc: LKML , linux-i2c@vger.kernel.org Subject: Re: [PATCH v3 2/3] i2c: convert i2c-isch to platform_device Message-ID: <20100228190015.GB3589@sortiz.org> References: <4B73DAEE.5080400@compulab.co.il> <4B73DBB2.7020601@compulab.co.il> <4B7C0E5D.3070909@compulab.co.il> <4B812B42.2090405@compulab.co.il> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4B812B42.2090405@compulab.co.il> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 5549 Lines: 195 Hi Denis, On Sun, Feb 21, 2010 at 02:46:58PM +0200, Denis Turischev wrote: > v2: there is no acpi_check_region, it will be implemented in mfd-core > v3: patch refreshed against the latest Linus tree Still failing to apply against Linus' latest tree: patching file drivers/i2c/busses/Kconfig Hunk #1 FAILED at 104. 1 out of 1 hunk FAILED -- saving rejects to file drivers/i2c/busses/Kconfig.rej patching file drivers/i2c/busses/i2c-isch.c Hunk #1 FAILED at 27. Hunk #2 FAILED at 46. Hunk #3 FAILED at 57. Hunk #4 FAILED at 249. Hunk #5 FAILED at 277. Hunk #6 FAILED at 322. 6 out of 6 hunks FAILED -- saving rejects to file drivers/i2c/busses/i2c-isch.c.rej Cheers, Samuel. > Signed-off-by: Denis Turischev > --- > drivers/i2c/busses/Kconfig | 2 +- > drivers/i2c/busses/i2c-isch.c | 68 ++++++++++++++++------------------------ > 2 files changed, 28 insertions(+), 42 deletions(-) > > diff --git a/drivers/i2c/busses/Kconfig b/drivers/i2c/busses/Kconfig > index 5f318ce..d15b6d3 100644 > --- a/drivers/i2c/busses/Kconfig > +++ b/drivers/i2c/busses/Kconfig > @@ -104,7 +104,7 @@ config I2C_I801 > > config I2C_ISCH > tristate "Intel SCH SMBus 1.0" > - depends on PCI > + select LPC_SCH > help > Say Y here if you want to use SMBus controller on the Intel SCH > based systems. > diff --git a/drivers/i2c/busses/i2c-isch.c b/drivers/i2c/busses/i2c-isch.c > index dba6eb0..ddc258e 100644 > --- a/drivers/i2c/busses/i2c-isch.c > +++ b/drivers/i2c/busses/i2c-isch.c > @@ -27,7 +27,7 @@ > */ > > #include > -#include > +#include > #include > #include > #include > @@ -46,12 +46,6 @@ > #define SMBHSTDAT1 (7 + sch_smba) > #define SMBBLKDAT (0x20 + sch_smba) > > -/* count for request_region */ > -#define SMBIOSIZE 64 > - > -/* PCI Address Constants */ > -#define SMBBA_SCH 0x40 > - > /* Other settings */ > #define MAX_TIMEOUT 500 > > @@ -63,7 +57,6 @@ > #define SCH_BLOCK_DATA 0x05 > > static unsigned short sch_smba; > -static struct pci_driver sch_driver; > static struct i2c_adapter sch_adapter; > > /* > @@ -256,37 +249,23 @@ static struct i2c_adapter sch_adapter = { > .algo = &smbus_algorithm, > }; > > -static struct pci_device_id sch_ids[] = { > - { PCI_DEVICE(PCI_VENDOR_ID_INTEL, PCI_DEVICE_ID_INTEL_SCH_LPC) }, > - { 0, } > -}; > - > -MODULE_DEVICE_TABLE(pci, sch_ids); > - > -static int __devinit sch_probe(struct pci_dev *dev, > - const struct pci_device_id *id) > +static int __devinit smbus_sch_probe(struct platform_device *dev) > { > + struct resource *res; > int retval; > - unsigned int smba; > > - pci_read_config_dword(dev, SMBBA_SCH, &smba); > - if (!(smba & (1 << 31))) { > - dev_err(&dev->dev, "SMBus I/O space disabled!\n"); > - return -ENODEV; > - } > + res = platform_get_resource(dev, IORESOURCE_IO, 0); > + if (!res) > + return -EBUSY; > > - sch_smba = (unsigned short)smba; > - if (sch_smba == 0) { > - dev_err(&dev->dev, "SMBus base address uninitialized!\n"); > - return -ENODEV; > - } > - if (acpi_check_region(sch_smba, SMBIOSIZE, sch_driver.name)) > - return -ENODEV; > - if (!request_region(sch_smba, SMBIOSIZE, sch_driver.name)) { > + if (!request_region(res->start, resource_size(res), dev->name)) { > dev_err(&dev->dev, "SMBus region 0x%x already in use!\n", > sch_smba); > return -EBUSY; > } > + > + sch_smba = res->start; > + > dev_dbg(&dev->dev, "SMBA = 0x%X\n", sch_smba); > > /* set up the sysfs linkage to our parent device */ > @@ -298,37 +277,43 @@ static int __devinit sch_probe(struct pci_dev *dev, > retval = i2c_add_adapter(&sch_adapter); > if (retval) { > dev_err(&dev->dev, "Couldn't register adapter!\n"); > - release_region(sch_smba, SMBIOSIZE); > + release_region(res->start, resource_size(res)); > sch_smba = 0; > } > > return retval; > } > > -static void __devexit sch_remove(struct pci_dev *dev) > +static int __devexit smbus_sch_remove(struct platform_device *pdev) > { > + struct resource *res; > if (sch_smba) { > i2c_del_adapter(&sch_adapter); > - release_region(sch_smba, SMBIOSIZE); > + res = platform_get_resource(pdev, IORESOURCE_IO, 0); > + release_region(res->start, resource_size(res)); > sch_smba = 0; > } > + > + return 0; > } > > -static struct pci_driver sch_driver = { > - .name = "isch_smbus", > - .id_table = sch_ids, > - .probe = sch_probe, > - .remove = __devexit_p(sch_remove), > +static struct platform_driver smbus_sch_driver = { > + .driver = { > + .name = "isch_smbus", > + .owner = THIS_MODULE, > + }, > + .probe = smbus_sch_probe, > + .remove = __devexit_p(smbus_sch_remove), > }; > > static int __init i2c_sch_init(void) > { > - return pci_register_driver(&sch_driver); > + return platform_driver_register(&smbus_sch_driver); > } > > static void __exit i2c_sch_exit(void) > { > - pci_unregister_driver(&sch_driver); > + platform_driver_unregister(&smbus_sch_driver); > } > > MODULE_AUTHOR("Jacob Pan "); > @@ -337,3 +322,4 @@ MODULE_LICENSE("GPL"); > > module_init(i2c_sch_init); > module_exit(i2c_sch_exit); > +MODULE_ALIAS("platform:isch_smbus"); > -- > 1.6.3.3 -- Intel Open Source Technology Centre http://oss.intel.com/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/