Commit e0dd03ca authored by Kylene Jo Hall's avatar Kylene Jo Hall Committed by Linus Torvalds

[PATCH] tpm: return chip from tpm_register_hardware

Changes in the 1.2 TPM Specification make it necessary to update some fields
of the chip structure in the initialization function after it is registered
with tpm.c thus tpm_register_hardware was modified to return a pointer to the
structure.  This patch makes that change and the associated changes in
tpm_atmel and tpm_nsc.  The changes to tpm_infineon will be coming in a patch
from Marcel Selhorst.
Signed-off-by: default avatarKylene Hall <kjhall@us.ibm.com>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 90dda520
...@@ -544,7 +544,8 @@ EXPORT_SYMBOL_GPL(tpm_pm_resume); ...@@ -544,7 +544,8 @@ EXPORT_SYMBOL_GPL(tpm_pm_resume);
* upon errant exit from this function specific probe function should call * upon errant exit from this function specific probe function should call
* pci_disable_device * pci_disable_device
*/ */
int tpm_register_hardware(struct device *dev, struct tpm_vendor_specific *entry) struct tpm_chip *tpm_register_hardware(struct device *dev, const struct tpm_vendor_specific
*entry)
{ {
#define DEVNAME_SIZE 7 #define DEVNAME_SIZE 7
...@@ -555,7 +556,7 @@ int tpm_register_hardware(struct device *dev, struct tpm_vendor_specific *entry) ...@@ -555,7 +556,7 @@ int tpm_register_hardware(struct device *dev, struct tpm_vendor_specific *entry)
/* Driver specific per-device data */ /* Driver specific per-device data */
chip = kzalloc(sizeof(*chip), GFP_KERNEL); chip = kzalloc(sizeof(*chip), GFP_KERNEL);
if (chip == NULL) if (chip == NULL)
return -ENOMEM; return NULL;
init_MUTEX(&chip->buffer_mutex); init_MUTEX(&chip->buffer_mutex);
init_MUTEX(&chip->tpm_mutex); init_MUTEX(&chip->tpm_mutex);
...@@ -584,7 +585,7 @@ int tpm_register_hardware(struct device *dev, struct tpm_vendor_specific *entry) ...@@ -584,7 +585,7 @@ int tpm_register_hardware(struct device *dev, struct tpm_vendor_specific *entry)
if (chip->dev_num < 0) { if (chip->dev_num < 0) {
dev_err(dev, "No available tpm device numbers\n"); dev_err(dev, "No available tpm device numbers\n");
kfree(chip); kfree(chip);
return -ENODEV; return NULL;
} else if (chip->dev_num == 0) } else if (chip->dev_num == 0)
chip->vendor.miscdev.minor = TPM_MINOR; chip->vendor.miscdev.minor = TPM_MINOR;
else else
...@@ -605,7 +606,7 @@ int tpm_register_hardware(struct device *dev, struct tpm_vendor_specific *entry) ...@@ -605,7 +606,7 @@ int tpm_register_hardware(struct device *dev, struct tpm_vendor_specific *entry)
put_device(dev); put_device(dev);
kfree(chip); kfree(chip);
dev_mask[i] &= !(1 << j); dev_mask[i] &= !(1 << j);
return -ENODEV; return NULL;
} }
spin_lock(&driver_lock); spin_lock(&driver_lock);
...@@ -620,7 +621,7 @@ int tpm_register_hardware(struct device *dev, struct tpm_vendor_specific *entry) ...@@ -620,7 +621,7 @@ int tpm_register_hardware(struct device *dev, struct tpm_vendor_specific *entry)
chip->bios_dir = tpm_bios_log_setup(devname); chip->bios_dir = tpm_bios_log_setup(devname);
return 0; return chip;
} }
EXPORT_SYMBOL_GPL(tpm_register_hardware); EXPORT_SYMBOL_GPL(tpm_register_hardware);
......
...@@ -48,9 +48,9 @@ extern ssize_t tpm_store_cancel(struct device *, struct device_attribute *attr, ...@@ -48,9 +48,9 @@ extern ssize_t tpm_store_cancel(struct device *, struct device_attribute *attr,
struct tpm_chip; struct tpm_chip;
struct tpm_vendor_specific { struct tpm_vendor_specific {
u8 req_complete_mask; const u8 req_complete_mask;
u8 req_complete_val; const u8 req_complete_val;
u8 req_canceled; const u8 req_canceled;
void __iomem *iobase; /* ioremapped address */ void __iomem *iobase; /* ioremapped address */
unsigned long base; /* TPM base address */ unsigned long base; /* TPM base address */
...@@ -100,8 +100,8 @@ static inline void tpm_write_index(int base, int index, int value) ...@@ -100,8 +100,8 @@ static inline void tpm_write_index(int base, int index, int value)
outb(value & 0xFF, base+1); outb(value & 0xFF, base+1);
} }
extern int tpm_register_hardware(struct device *, extern struct tpm_chip* tpm_register_hardware(struct device *,
struct tpm_vendor_specific *); const struct tpm_vendor_specific *);
extern int tpm_open(struct inode *, struct file *); extern int tpm_open(struct inode *, struct file *);
extern int tpm_release(struct inode *, struct file *); extern int tpm_release(struct inode *, struct file *);
extern ssize_t tpm_write(struct file *, const char __user *, size_t, extern ssize_t tpm_write(struct file *, const char __user *, size_t,
......
...@@ -140,7 +140,7 @@ static struct attribute* atmel_attrs[] = { ...@@ -140,7 +140,7 @@ static struct attribute* atmel_attrs[] = {
static struct attribute_group atmel_attr_grp = { .attrs = atmel_attrs }; static struct attribute_group atmel_attr_grp = { .attrs = atmel_attrs };
static struct tpm_vendor_specific tpm_atmel = { static const struct tpm_vendor_specific tpm_atmel = {
.recv = tpm_atml_recv, .recv = tpm_atml_recv,
.send = tpm_atml_send, .send = tpm_atml_send,
.cancel = tpm_atml_cancel, .cancel = tpm_atml_cancel,
...@@ -179,18 +179,22 @@ static struct device_driver atml_drv = { ...@@ -179,18 +179,22 @@ static struct device_driver atml_drv = {
static int __init init_atmel(void) static int __init init_atmel(void)
{ {
int rc = 0; int rc = 0;
void __iomem *iobase = NULL;
int have_region, region_size;
unsigned long base;
struct tpm_chip *chip;
driver_register(&atml_drv); driver_register(&atml_drv);
if ((tpm_atmel.iobase = atmel_get_base_addr(&tpm_atmel)) == NULL) { if ((iobase = atmel_get_base_addr(&base, &region_size)) == NULL) {
rc = -ENODEV; rc = -ENODEV;
goto err_unreg_drv; goto err_unreg_drv;
} }
tpm_atmel.have_region = have_region =
(atmel_request_region (atmel_request_region
(tpm_atmel.base, tpm_atmel.region_size, (tpm_atmel.base, region_size, "tpm_atmel0") == NULL) ? 0 : 1;
"tpm_atmel0") == NULL) ? 0 : 1;
if (IS_ERR if (IS_ERR
(pdev = (pdev =
...@@ -199,17 +203,25 @@ static int __init init_atmel(void) ...@@ -199,17 +203,25 @@ static int __init init_atmel(void)
goto err_rel_reg; goto err_rel_reg;
} }
if ((rc = tpm_register_hardware(&pdev->dev, &tpm_atmel)) < 0) if (!(chip = tpm_register_hardware(&pdev->dev, &tpm_atmel))) {
rc = -ENODEV;
goto err_unreg_dev; goto err_unreg_dev;
}
chip->vendor.iobase = iobase;
chip->vendor.base = base;
chip->vendor.have_region = have_region;
chip->vendor.region_size = region_size;
return 0; return 0;
err_unreg_dev: err_unreg_dev:
platform_device_unregister(pdev); platform_device_unregister(pdev);
err_rel_reg: err_rel_reg:
atmel_put_base_addr(&tpm_atmel); atmel_put_base_addr(iobase);
if (tpm_atmel.have_region) if (have_region)
atmel_release_region(tpm_atmel.base, atmel_release_region(base,
tpm_atmel.region_size); region_size);
err_unreg_drv: err_unreg_drv:
driver_unregister(&atml_drv); driver_unregister(&atml_drv);
return rc; return rc;
......
...@@ -28,13 +28,12 @@ ...@@ -28,13 +28,12 @@
#define atmel_request_region request_mem_region #define atmel_request_region request_mem_region
#define atmel_release_region release_mem_region #define atmel_release_region release_mem_region
static inline void atmel_put_base_addr(struct tpm_vendor_specific static inline void atmel_put_base_addr(void __iomem *iobase)
*vendor)
{ {
iounmap(vendor->iobase); iounmap(iobase);
} }
static void __iomem * atmel_get_base_addr(struct tpm_vendor_specific *vendor) static void __iomem * atmel_get_base_addr(unsigned long *base, int *region_size)
{ {
struct device_node *dn; struct device_node *dn;
unsigned long address, size; unsigned long address, size;
...@@ -71,9 +70,9 @@ static void __iomem * atmel_get_base_addr(struct tpm_vendor_specific *vendor) ...@@ -71,9 +70,9 @@ static void __iomem * atmel_get_base_addr(struct tpm_vendor_specific *vendor)
else else
size = reg[naddrc]; size = reg[naddrc];
vendor->base = address; *base = address;
vendor->region_size = size; *region_size = size;
return ioremap(vendor->base, vendor->region_size); return ioremap(*base, *region_size);
} }
#else #else
#define atmel_getb(chip, offset) inb(chip->vendor->base + offset) #define atmel_getb(chip, offset) inb(chip->vendor->base + offset)
...@@ -106,14 +105,12 @@ static int atmel_verify_tpm11(void) ...@@ -106,14 +105,12 @@ static int atmel_verify_tpm11(void)
return 0; return 0;
} }
static inline void atmel_put_base_addr(struct tpm_vendor_specific static inline void atmel_put_base_addr(void __iomem *iobase)
*vendor)
{ {
} }
/* Determine where to talk to device */ /* Determine where to talk to device */
static void __iomem * atmel_get_base_addr(struct tpm_vendor_specific static void __iomem * atmel_get_base_addr(unsigned long *base, int *region_size)
*vendor)
{ {
int lo, hi; int lo, hi;
...@@ -123,9 +120,9 @@ static void __iomem * atmel_get_base_addr(struct tpm_vendor_specific ...@@ -123,9 +120,9 @@ static void __iomem * atmel_get_base_addr(struct tpm_vendor_specific
lo = tpm_read_index(TPM_ADDR, TPM_ATMEL_BASE_ADDR_LO); lo = tpm_read_index(TPM_ADDR, TPM_ATMEL_BASE_ADDR_LO);
hi = tpm_read_index(TPM_ADDR, TPM_ATMEL_BASE_ADDR_HI); hi = tpm_read_index(TPM_ADDR, TPM_ATMEL_BASE_ADDR_HI);
vendor->base = (hi << 8) | lo; *base = (hi << 8) | lo;
vendor->region_size = 2; *region_size = 2;
return ioport_map(vendor->base, vendor->region_size); return ioport_map(*base, *region_size);
} }
#endif #endif
...@@ -250,7 +250,7 @@ static struct attribute * nsc_attrs[] = { ...@@ -250,7 +250,7 @@ static struct attribute * nsc_attrs[] = {
static struct attribute_group nsc_attr_grp = { .attrs = nsc_attrs }; static struct attribute_group nsc_attr_grp = { .attrs = nsc_attrs };
static struct tpm_vendor_specific tpm_nsc = { static const struct tpm_vendor_specific tpm_nsc = {
.recv = tpm_nsc_recv, .recv = tpm_nsc_recv,
.send = tpm_nsc_send, .send = tpm_nsc_send,
.cancel = tpm_nsc_cancel, .cancel = tpm_nsc_cancel,
...@@ -286,7 +286,8 @@ static int __init init_nsc(void) ...@@ -286,7 +286,8 @@ static int __init init_nsc(void)
int rc = 0; int rc = 0;
int lo, hi; int lo, hi;
int nscAddrBase = TPM_ADDR; int nscAddrBase = TPM_ADDR;
struct tpm_chip *chip;
unsigned long base;
/* verify that it is a National part (SID) */ /* verify that it is a National part (SID) */
if (tpm_read_index(TPM_ADDR, NSC_SID_INDEX) != 0xEF) { if (tpm_read_index(TPM_ADDR, NSC_SID_INDEX) != 0xEF) {
...@@ -300,7 +301,7 @@ static int __init init_nsc(void) ...@@ -300,7 +301,7 @@ static int __init init_nsc(void)
hi = tpm_read_index(nscAddrBase, TPM_NSC_BASE0_HI); hi = tpm_read_index(nscAddrBase, TPM_NSC_BASE0_HI);
lo = tpm_read_index(nscAddrBase, TPM_NSC_BASE0_LO); lo = tpm_read_index(nscAddrBase, TPM_NSC_BASE0_LO);
tpm_nsc.base = (hi<<8) | lo; base = (hi<<8) | lo;
/* enable the DPM module */ /* enable the DPM module */
tpm_write_index(nscAddrBase, NSC_LDC_INDEX, 0x01); tpm_write_index(nscAddrBase, NSC_LDC_INDEX, 0x01);
...@@ -320,13 +321,15 @@ static int __init init_nsc(void) ...@@ -320,13 +321,15 @@ static int __init init_nsc(void)
if ((rc = platform_device_register(pdev)) < 0) if ((rc = platform_device_register(pdev)) < 0)
goto err_free_dev; goto err_free_dev;
if (request_region(tpm_nsc.base, 2, "tpm_nsc0") == NULL ) { if (request_region(base, 2, "tpm_nsc0") == NULL ) {
rc = -EBUSY; rc = -EBUSY;
goto err_unreg_dev; goto err_unreg_dev;
} }
if ((rc = tpm_register_hardware(&pdev->dev, &tpm_nsc)) < 0) if (!(chip = tpm_register_hardware(&pdev->dev, &tpm_nsc))) {
rc = -ENODEV;
goto err_rel_reg; goto err_rel_reg;
}
dev_dbg(&pdev->dev, "NSC TPM detected\n"); dev_dbg(&pdev->dev, "NSC TPM detected\n");
dev_dbg(&pdev->dev, dev_dbg(&pdev->dev,
...@@ -361,10 +364,12 @@ static int __init init_nsc(void) ...@@ -361,10 +364,12 @@ static int __init init_nsc(void)
"NSC TPM revision %d\n", "NSC TPM revision %d\n",
tpm_read_index(nscAddrBase, 0x27) & 0x1F); tpm_read_index(nscAddrBase, 0x27) & 0x1F);
chip->vendor.base = base;
return 0; return 0;
err_rel_reg: err_rel_reg:
release_region(tpm_nsc.base, 2); release_region(base, 2);
err_unreg_dev: err_unreg_dev:
platform_device_unregister(pdev); platform_device_unregister(pdev);
err_free_dev: err_free_dev:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment