[PATCH] tpm: Stop taking over the non-unique lpc bus PCI ID, Also timer, stack and enum fixes

2005-04-15 Thread Kylene Hall
? The first person to the lpc bus needs to call pci_enable_device and the last to leave one should call pci_disable_device, how as a device driver on this bus do I know if I am the first or last one and need to make the appropriate call? Thanks, Kylie Hall Signed-off-by: Kylene Hall <[EMAIL PROTEC

Re: [PATCH] char/tpm: use msleep(), clean-up timers, fix typo

2005-04-15 Thread Kylene Hall
I have tested this patch and agree that using msleep is the right. Please apply this patch to the tpm driver. One hunk might fail b/c the typo has been fixed already. Thanks, Kylie Hall On Fri, 11 Mar 2005, Nishanth Aravamudan wrote: > Not sure what happened to the original mail, but I'm

Re: [PATCH] char/tpm: use msleep(), clean-up timers, fix typo

2005-04-15 Thread Kylene Hall
I have tested this patch and agree that using msleep is the right. Please apply this patch to the tpm driver. One hunk might fail b/c the typo has been fixed already. Thanks, Kylie Hall On Fri, 11 Mar 2005, Nishanth Aravamudan wrote: Not sure what happened to the original mail, but I'm not

[PATCH] tpm: Stop taking over the non-unique lpc bus PCI ID, Also timer, stack and enum fixes

2005-04-15 Thread Kylene Hall
? The first person to the lpc bus needs to call pci_enable_device and the last to leave one should call pci_disable_device, how as a device driver on this bus do I know if I am the first or last one and need to make the appropriate call? Thanks, Kylie Hall Signed-off-by: Kylene Hall [EMAIL PROTECTED

Re: [PATCH] Add TPM hardware enablement driver

2005-03-16 Thread Kylene Hall
The patch at the bottom addresses a number of the concerns raised in this email along with a couple of other comments which this generated regarding not needing __force and the need for a MAINTAINERS entry. Signed-off-by: Kylene Hall <[EMAIL PROTECTED]> On Wed, 9 Mar 2005, Jeff Garzik

Re: [PATCH] Add TPM hardware enablement driver

2005-03-16 Thread Kylene Hall
The patch at the bottom addresses a number of the concerns raised in this email along with a couple of other comments which this generated regarding not needing __force and the need for a MAINTAINERS entry. Signed-off-by: Kylene Hall [EMAIL PROTECTED] On Wed, 9 Mar 2005, Jeff Garzik wrote

Re: [tpmdd-devel] Re: [PATCH 1/1] tpm: update tpm sysfs file ownership - updated version

2005-02-10 Thread Kylene Hall
On Wed, 2005-02-09 at 16:04, Chris Wright wrote: > > +#define TPM_DEVICE_ATTRS { \ > > + __ATTR(pubek, S_IRUGO, show_pubek, NULL), \ > > + __ATTR(pcrs, S_IRUGO, show_pcrs, NULL), \ > > + __ATTR(caps, S_IRUGO, show_caps, NULL), \ > > + __ATTR(cancel, S_IWUSR | S_IWGRP, NULL, store_cancel) }

Re: [tpmdd-devel] Re: [PATCH 1/1] tpm: update tpm sysfs file ownership - updated version

2005-02-10 Thread Kylene Hall
On Wed, 2005-02-09 at 16:04, Chris Wright wrote: +#define TPM_DEVICE_ATTRS { \ + __ATTR(pubek, S_IRUGO, show_pubek, NULL), \ + __ATTR(pcrs, S_IRUGO, show_pcrs, NULL), \ + __ATTR(caps, S_IRUGO, show_caps, NULL), \ + __ATTR(cancel, S_IWUSR | S_IWGRP, NULL, store_cancel) } This

Re: [tpmdd-devel] Re: [PATCH 1/1] tpm: update tpm sysfs file ownership - updated version

2005-02-09 Thread Kylene Hall
On Wed, 9 Feb 2005, Greg KH wrote: > On Wed, Feb 09, 2005 at 12:05:42PM -0600, Kylene Hall wrote: > > @@ -539,9 +551,8 @@ void tpm_remove_hardware(struct device * > > dev_set_drvdata(dev, NULL); > > misc_deregister(>vendor->miscdev); > > > >

Re: [PATCH 1/1] tpm: update tpm sysfs file ownership

2005-02-09 Thread Kylene Hall
On Fri, 4 Feb 2005, Greg KH wrote: > On Fri, Feb 04, 2005 at 03:37:20PM -0600, Kylene Hall wrote: > > On Fri, 2005-02-04 at 14:52, Greg KH wrote: > > > On Fri, Feb 04, 2005 at 02:12:50PM -0600, Kylene Hall wrote: > > > > +static struct class tpm_class =

Re: [PATCH 1/1] tpm: update tpm sysfs file ownership

2005-02-09 Thread Kylene Hall
On Fri, 4 Feb 2005, Greg KH wrote: On Fri, Feb 04, 2005 at 03:37:20PM -0600, Kylene Hall wrote: On Fri, 2005-02-04 at 14:52, Greg KH wrote: On Fri, Feb 04, 2005 at 02:12:50PM -0600, Kylene Hall wrote: +static struct class tpm_class = { + .name = tpm

Re: [tpmdd-devel] Re: [PATCH 1/1] tpm: update tpm sysfs file ownership - updated version

2005-02-09 Thread Kylene Hall
On Wed, 9 Feb 2005, Greg KH wrote: On Wed, Feb 09, 2005 at 12:05:42PM -0600, Kylene Hall wrote: @@ -539,9 +551,8 @@ void tpm_remove_hardware(struct device * dev_set_drvdata(dev, NULL); misc_deregister(chip-vendor-miscdev); - device_remove_file(dev, dev_attr_pubek

Re: [PATCH 1/1] tpm: implement use of sysfs classes

2005-02-04 Thread Kylene Hall
On Fri, 2005-02-04 at 14:52, Greg KH wrote: > On Fri, Feb 04, 2005 at 02:12:50PM -0600, Kylene Hall wrote: > > +static struct class tpm_class = { > > + .name = "tpm", > > + .class_dev_attrs = tpm_attrs, > > +}; > > Where is your release function?

[PATCH 1/1] tpm: implement use of sysfs classes

2005-02-04 Thread Kylene Hall
he driver, device and tpm specific files (pcrs, etc) now live. This patch also includes a new sysfs file for each tpm that is needed to support canceling an operation. Thanks, Kylie Signed-off-by: Kylene Hall <[EMAIL PROTECTED]> --- diff -uprN linux-2.6.10/drivers/char/tpm/tpm.c linux-2.

[PATCH 1/1] tpm: implement use of sysfs classes

2005-02-04 Thread Kylene Hall
, device and tpm specific files (pcrs, etc) now live. This patch also includes a new sysfs file for each tpm that is needed to support canceling an operation. Thanks, Kylie Signed-off-by: Kylene Hall [EMAIL PROTECTED] --- diff -uprN linux-2.6.10/drivers/char/tpm/tpm.c linux-2.6.10-tpm/drivers/char

Re: [PATCH 1/1] tpm: implement use of sysfs classes

2005-02-04 Thread Kylene Hall
On Fri, 2005-02-04 at 14:52, Greg KH wrote: On Fri, Feb 04, 2005 at 02:12:50PM -0600, Kylene Hall wrote: +static struct class tpm_class = { + .name = tpm, + .class_dev_attrs = tpm_attrs, +}; Where is your release function? Did you see any warnings from the kernel when you removed

[PATCH 1/1] tpm: remove pci specific stuff from the underlying generic driver

2005-02-03 Thread Kylene Hall
device stuff and future non-pci chips can easily be connected to the architecture. Thanks, Kylie Signed-off-by: Kylene Hall <[EMAIL PROTECTED]> --- diff -uprN linux-2.6.10/drivers/char/tpm/tpm_atmel.c linux-2.6.10-tpm/drivers/char/tpm/tpm_atmel.c --- linux-2.6.10/drivers/char/tpm/tpm_a

[PATCH 1/1] tpm: remove pci specific stuff from the underlying generic driver

2005-02-03 Thread Kylene Hall
device stuff and future non-pci chips can easily be connected to the architecture. Thanks, Kylie Signed-off-by: Kylene Hall [EMAIL PROTECTED] --- diff -uprN linux-2.6.10/drivers/char/tpm/tpm_atmel.c linux-2.6.10-tpm/drivers/char/tpm/tpm_atmel.c --- linux-2.6.10/drivers/char/tpm/tpm_atmel.c 2005

Re: [PATCH 1/1] tpm: insert missing up mutex in an error path, typo build fix -- updated version

2005-01-31 Thread Kylene Hall
There is also a typo in the driver version in the 2.6.11-rc2-mm2 tree that is not fixed by the previous patch. Please apply this upated version to fix the previously acknowledged problems and this typo causing a build error. Thanks, Kylie On Fri, 28 Jan 2005, Kylene Hall wrote: > This pa

Re: [PATCH 1/1] tpm: insert missing up mutex in an error path, typo build fix -- updated version

2005-01-31 Thread Kylene Hall
There is also a typo in the driver version in the 2.6.11-rc2-mm2 tree that is not fixed by the previous patch. Please apply this upated version to fix the previously acknowledged problems and this typo causing a build error. Thanks, Kylie On Fri, 28 Jan 2005, Kylene Hall wrote: This patch

[PATCH 1/1] tpm: insert missing up mutex in an error path

2005-01-28 Thread Kylene Hall
Thanks, Kylie Signed-off-by: Kylene Hall <[EMAIL PROTECTED]> --- diff -uprN linux-2.6.10/drivers/char/tpm/tpm_atmel.c linux-2.6.10-tpm/drivers/char/tpm/tpm_atmel.c --- linux-2.6.10/drivers/char/tpm/tpm_atmel.c 2005-01-18 16:42:17.0 -0600 +++ linux-2.6.10-tpm/drivers/char/tpm

[PATCH 1/1] tpm: insert missing up mutex in an error path

2005-01-28 Thread Kylene Hall
, Kylie Signed-off-by: Kylene Hall [EMAIL PROTECTED] --- diff -uprN linux-2.6.10/drivers/char/tpm/tpm_atmel.c linux-2.6.10-tpm/drivers/char/tpm/tpm_atmel.c --- linux-2.6.10/drivers/char/tpm/tpm_atmel.c 2005-01-18 16:42:17.0 -0600 +++ linux-2.6.10-tpm/drivers/char/tpm/tpm_atmel.c

Re: [PATCH 1/1] tpm: fix cause of SMP stack traces -- updated version

2005-01-18 Thread Kylene Hall
TED]> Thanks, Kylie Signed-off-by: Kylene Hall <[EMAIL PROTECTED]> --- diff -uprN linux-2.6.10/drivers/char/tpm/tpm.c linux-2.6.10-tpm/drivers/char/tpm/tpm.c --- linux-2.6.10/drivers/char/tpm/tpm.c 2005-01-18 18:10:16.0 -0600 +++ linux-2.6.10-tpm/drivers/char/tpm/tpm.c 2005

Re: [PATCH 1/1] tpm: fix cause of SMP stack traces

2005-01-18 Thread Kylene Hall
On Tue, 2005-01-18 at 16:47, Greg KH wrote: > On Tue, Jan 18, 2005 at 04:29:23PM -0600, Kylene Hall wrote: > > There were misplaced spinlock acquires and releases in the probe, open, > > close and release paths which were causing might_sleep and schedule while > >

Re: [PATCH 1/1] tpm: fix cause of SMP stack traces

2005-01-18 Thread Kylene Hall
On Tue, 2005-01-18 at 16:37, Chris Wright wrote: > * Kylene Hall ([EMAIL PROTECTED]) wrote: > > There were misplaced spinlock acquires and releases in the probe, open, > > close and release paths which were causing might_sleep and schedule while > > atomic error messages

[PATCH 1/1] tpm: fix cause of SMP stack traces

2005-01-18 Thread Kylene Hall
TED]> Signed-off-by: Kylene Hall <[EMAIL PROTECTED]> --- diff -uprN linux-2.6.10/drivers/char/tpm/tpm.c linux-2.6.10-tpm/drivers/char/tpm/tpm.c --- linux-2.6.10/drivers/char/tpm/tpm.c 2005-01-18 16:42:17.0 -0600 +++ linux-2.6.10-tpm/drivers/char/tpm/tpm.c 2005-01-18 12:52:53.

[PATCH 1/1] tpm: fix cause of SMP stack traces

2005-01-18 Thread Kylene Hall
] Signed-off-by: Kylene Hall [EMAIL PROTECTED] --- diff -uprN linux-2.6.10/drivers/char/tpm/tpm.c linux-2.6.10-tpm/drivers/char/tpm/tpm.c --- linux-2.6.10/drivers/char/tpm/tpm.c 2005-01-18 16:42:17.0 -0600 +++ linux-2.6.10-tpm/drivers/char/tpm/tpm.c 2005-01-18 12:52:53.0 -0600

Re: [PATCH 1/1] tpm: fix cause of SMP stack traces

2005-01-18 Thread Kylene Hall
On Tue, 2005-01-18 at 16:37, Chris Wright wrote: * Kylene Hall ([EMAIL PROTECTED]) wrote: There were misplaced spinlock acquires and releases in the probe, open, close and release paths which were causing might_sleep and schedule while atomic error messages accompanied by stack traces

Re: [PATCH 1/1] tpm: fix cause of SMP stack traces

2005-01-18 Thread Kylene Hall
On Tue, 2005-01-18 at 16:47, Greg KH wrote: On Tue, Jan 18, 2005 at 04:29:23PM -0600, Kylene Hall wrote: There were misplaced spinlock acquires and releases in the probe, open, close and release paths which were causing might_sleep and schedule while atomic error messages accompanied

Re: [PATCH 1/1] tpm: fix cause of SMP stack traces -- updated version

2005-01-18 Thread Kylene Hall
, Kylie Signed-off-by: Kylene Hall [EMAIL PROTECTED] --- diff -uprN linux-2.6.10/drivers/char/tpm/tpm.c linux-2.6.10-tpm/drivers/char/tpm/tpm.c --- linux-2.6.10/drivers/char/tpm/tpm.c 2005-01-18 18:10:16.0 -0600 +++ linux-2.6.10-tpm/drivers/char/tpm/tpm.c 2005-01-18 18:13:59.0