Pull trivial into release branch
[sfrench/cifs-2.6.git] / drivers / char / tpm / tpm_infineon.c
index ec7590951af520378f77759abe0cc274421d6f7f..adfff21beb219f9965d50626dfbc582cc016be31 100644 (file)
@@ -15,6 +15,7 @@
  * License.
  */
 
+#include <linux/init.h>
 #include <linux/pnp.h>
 #include "tpm.h"
 
@@ -33,6 +34,7 @@
 static int TPM_INF_DATA;
 static int TPM_INF_ADDR;
 static int TPM_INF_BASE;
+static int TPM_INF_ADDR_LEN;
 static int TPM_INF_PORT_LEN;
 
 /* TPM header definitions */
@@ -103,7 +105,7 @@ static int empty_fifo(struct tpm_chip *chip, int clear_wrfifo)
 
        if (clear_wrfifo) {
                for (i = 0; i < 4096; i++) {
-                       status = inb(chip->vendor->base + WRFIFO);
+                       status = inb(chip->vendor.base + WRFIFO);
                        if (status == 0xff) {
                                if (check == 5)
                                        break;
@@ -123,8 +125,8 @@ static int empty_fifo(struct tpm_chip *chip, int clear_wrfifo)
         */
        i = 0;
        do {
-               status = inb(chip->vendor->base + RDFIFO);
-               status = inb(chip->vendor->base + STAT);
+               status = inb(chip->vendor.base + RDFIFO);
+               status = inb(chip->vendor.base + STAT);
                i++;
                if (i == TPM_MAX_TRIES)
                        return -EIO;
@@ -137,7 +139,7 @@ static int wait(struct tpm_chip *chip, int wait_for_bit)
        int status;
        int i;
        for (i = 0; i < TPM_MAX_TRIES; i++) {
-               status = inb(chip->vendor->base + STAT);
+               status = inb(chip->vendor.base + STAT);
                /* check the status-register if wait_for_bit is set */
                if (status & 1 << wait_for_bit)
                        break;
@@ -156,7 +158,7 @@ static int wait(struct tpm_chip *chip, int wait_for_bit)
 static void wait_and_send(struct tpm_chip *chip, u8 sendbyte)
 {
        wait(chip, STAT_XFE);
-       outb(sendbyte, chip->vendor->base + WRFIFO);
+       outb(sendbyte, chip->vendor.base + WRFIFO);
 }
 
     /* Note: WTX means Waiting-Time-Extension. Whenever the TPM needs more
@@ -195,6 +197,7 @@ static int tpm_inf_recv(struct tpm_chip *chip, u8 * buf, size_t count)
        int i;
        int ret;
        u32 size = 0;
+       number_of_wtx = 0;
 
 recv_begin:
        /* start receiving header */
@@ -202,7 +205,7 @@ recv_begin:
                ret = wait(chip, STAT_RDA);
                if (ret)
                        return -EIO;
-               buf[i] = inb(chip->vendor->base + RDFIFO);
+               buf[i] = inb(chip->vendor.base + RDFIFO);
        }
 
        if (buf[0] != TPM_VL_VER) {
@@ -217,7 +220,7 @@ recv_begin:
 
                for (i = 0; i < size; i++) {
                        wait(chip, STAT_RDA);
-                       buf[i] = inb(chip->vendor->base + RDFIFO);
+                       buf[i] = inb(chip->vendor.base + RDFIFO);
                }
 
                if ((size == 0x6D00) && (buf[1] == 0x80)) {
@@ -266,7 +269,7 @@ static int tpm_inf_send(struct tpm_chip *chip, u8 * buf, size_t count)
        u8 count_high, count_low, count_4, count_3, count_2, count_1;
 
        /* Disabling Reset, LP and IRQC */
-       outb(RESET_LP_IRQC_DISABLE, chip->vendor->base + CMD);
+       outb(RESET_LP_IRQC_DISABLE, chip->vendor.base + CMD);
 
        ret = empty_fifo(chip, 1);
        if (ret) {
@@ -317,7 +320,7 @@ static void tpm_inf_cancel(struct tpm_chip *chip)
 
 static u8 tpm_inf_status(struct tpm_chip *chip)
 {
-       return inb(chip->vendor->base + STAT);
+       return inb(chip->vendor.base + STAT);
 }
 
 static DEVICE_ATTR(pubek, S_IRUGO, tpm_show_pubek, NULL);
@@ -344,7 +347,7 @@ static struct file_operations inf_ops = {
        .release = tpm_release,
 };
 
-static struct tpm_vendor_specific tpm_inf = {
+static const struct tpm_vendor_specific tpm_inf = {
        .recv = tpm_inf_recv,
        .send = tpm_inf_send,
        .cancel = tpm_inf_cancel,
@@ -373,29 +376,40 @@ static int __devinit tpm_inf_pnp_probe(struct pnp_dev *dev,
        int version[2];
        int productid[2];
        char chipname[20];
+       struct tpm_chip *chip;
 
        /* read IO-ports through PnP */
        if (pnp_port_valid(dev, 0) && pnp_port_valid(dev, 1) &&
            !(pnp_port_flags(dev, 0) & IORESOURCE_DISABLED)) {
                TPM_INF_ADDR = pnp_port_start(dev, 0);
+               TPM_INF_ADDR_LEN = pnp_port_len(dev, 0);
                TPM_INF_DATA = (TPM_INF_ADDR + 1);
                TPM_INF_BASE = pnp_port_start(dev, 1);
                TPM_INF_PORT_LEN = pnp_port_len(dev, 1);
-               if (!TPM_INF_PORT_LEN)
-                       return -EINVAL;
+               if ((TPM_INF_PORT_LEN < 4) || (TPM_INF_ADDR_LEN < 2)) {
+                       rc = -EINVAL;
+                       goto err_last;
+               }
                dev_info(&dev->dev, "Found %s with ID %s\n",
                         dev->name, dev_id->id);
-               if (!((TPM_INF_BASE >> 8) & 0xff))
-                       return -EINVAL;
+               if (!((TPM_INF_BASE >> 8) & 0xff)) {
+                       rc = -EINVAL;
+                       goto err_last;
+               }
                /* publish my base address and request region */
-               tpm_inf.base = TPM_INF_BASE;
                if (request_region
-                   (tpm_inf.base, TPM_INF_PORT_LEN, "tpm_infineon0") == NULL) {
-                       release_region(tpm_inf.base, TPM_INF_PORT_LEN);
-                       return -EINVAL;
+                   (TPM_INF_BASE, TPM_INF_PORT_LEN, "tpm_infineon0") == NULL) {
+                       rc = -EINVAL;
+                       goto err_last;
+               }
+               if (request_region
+                   (TPM_INF_ADDR, TPM_INF_ADDR_LEN, "tpm_infineon0") == NULL) {
+                       rc = -EINVAL;
+                       goto err_last;
                }
        } else {
-               return -EINVAL;
+               rc = -EINVAL;
+               goto err_last;
        }
 
        /* query chip for its vendor, its version number a.s.o. */
@@ -429,9 +443,9 @@ static int __devinit tpm_inf_pnp_probe(struct pnp_dev *dev,
 
                /* configure TPM with IO-ports */
                outb(IOLIMH, TPM_INF_ADDR);
-               outb(((tpm_inf.base >> 8) & 0xff), TPM_INF_DATA);
+               outb(((TPM_INF_BASE >> 8) & 0xff), TPM_INF_DATA);
                outb(IOLIML, TPM_INF_ADDR);
-               outb((tpm_inf.base & 0xff), TPM_INF_DATA);
+               outb((TPM_INF_BASE & 0xff), TPM_INF_DATA);
 
                /* control if IO-ports are set correctly */
                outb(IOLIMH, TPM_INF_ADDR);
@@ -439,12 +453,12 @@ static int __devinit tpm_inf_pnp_probe(struct pnp_dev *dev,
                outb(IOLIML, TPM_INF_ADDR);
                iol = inb(TPM_INF_DATA);
 
-               if ((ioh << 8 | iol) != tpm_inf.base) {
+               if ((ioh << 8 | iol) != TPM_INF_BASE) {
                        dev_err(&dev->dev,
-                               "Could not set IO-ports to 0x%lx\n",
-                               tpm_inf.base);
-                       release_region(tpm_inf.base, TPM_INF_PORT_LEN);
-                       return -EIO;
+                               "Could not set IO-ports to 0x%x\n",
+                               TPM_INF_BASE);
+                       rc = -EIO;
+                       goto err_release_region;
                }
 
                /* activate register */
@@ -453,15 +467,15 @@ static int __devinit tpm_inf_pnp_probe(struct pnp_dev *dev,
                outb(DISABLE_REGISTER_PAIR, TPM_INF_ADDR);
 
                /* disable RESET, LP and IRQC */
-               outb(RESET_LP_IRQC_DISABLE, tpm_inf.base + CMD);
+               outb(RESET_LP_IRQC_DISABLE, TPM_INF_BASE + CMD);
 
                /* Finally, we're done, print some infos */
                dev_info(&dev->dev, "TPM found: "
                         "config base 0x%x, "
                         "io base 0x%x, "
-                        "chip version %02x%02x, "
-                        "vendor id %x%x (Infineon), "
-                        "product id %02x%02x"
+                        "chip version 0x%02x%02x, "
+                        "vendor id 0x%x%x (Infineon), "
+                        "product id 0x%02x%02x"
                         "%s\n",
                         TPM_INF_ADDR,
                         TPM_INF_BASE,
@@ -469,16 +483,22 @@ static int __devinit tpm_inf_pnp_probe(struct pnp_dev *dev,
                         vendorid[0], vendorid[1],
                         productid[0], productid[1], chipname);
 
-               rc = tpm_register_hardware(&dev->dev, &tpm_inf);
-               if (rc < 0) {
-                       release_region(tpm_inf.base, TPM_INF_PORT_LEN);
-                       return -ENODEV;
+               if (!(chip = tpm_register_hardware(&dev->dev, &tpm_inf))) {
+                       goto err_release_region;
                }
+               chip->vendor.base = TPM_INF_BASE;
                return 0;
        } else {
-               dev_info(&dev->dev, "No Infineon TPM found!\n");
-               return -ENODEV;
+               rc = -ENODEV;
+               goto err_release_region;
        }
+
+err_release_region:
+       release_region(TPM_INF_BASE, TPM_INF_PORT_LEN);
+       release_region(TPM_INF_ADDR, TPM_INF_ADDR_LEN);
+
+err_last:
+       return rc;
 }
 
 static __devexit void tpm_inf_pnp_remove(struct pnp_dev *dev)
@@ -486,7 +506,8 @@ static __devexit void tpm_inf_pnp_remove(struct pnp_dev *dev)
        struct tpm_chip *chip = pnp_get_drvdata(dev);
 
        if (chip) {
-               release_region(chip->vendor->base, TPM_INF_PORT_LEN);
+               release_region(TPM_INF_BASE, TPM_INF_PORT_LEN);
+               release_region(TPM_INF_ADDR, TPM_INF_ADDR_LEN);
                tpm_remove_hardware(chip->dev);
        }
 }
@@ -500,7 +521,7 @@ static struct pnp_driver tpm_inf_pnp = {
        },
        .id_table = tpm_pnp_tbl,
        .probe = tpm_inf_pnp_probe,
-       .remove = tpm_inf_pnp_remove,
+       .remove = __devexit_p(tpm_inf_pnp_remove),
 };
 
 static int __init init_inf(void)
@@ -518,5 +539,5 @@ module_exit(cleanup_inf);
 
 MODULE_AUTHOR("Marcel Selhorst <selhorst@crypto.rub.de>");
 MODULE_DESCRIPTION("Driver for Infineon TPM SLD 9630 TT 1.1 / SLB 9635 TT 1.2");
-MODULE_VERSION("1.6");
+MODULE_VERSION("1.8");
 MODULE_LICENSE("GPL");