Re: [PATCH] Add TPM hardware enablement driver

2005-04-09 Thread Ian Campbell
On Fri, 2005-04-08 at 15:07 -0500, Kylene Jo Hall wrote: > Basically, what I need to figure out is how to solve both issues > simultaneously. I need to not register a pci_driver as I would be > taking over an ID that is not unique to my device as well as get the > hotplugging correct (which i

Re: [PATCH] Add TPM hardware enablement driver

2005-04-09 Thread Ian Campbell
On Fri, 2005-04-08 at 15:07 -0500, Kylene Jo Hall wrote: Basically, what I need to figure out is how to solve both issues simultaneously. I need to not register a pci_driver as I would be taking over an ID that is not unique to my device as well as get the hotplugging correct (which i don't

Re: [PATCH] Add TPM hardware enablement driver

2005-04-08 Thread Kylene Jo Hall
On Tue, 2005-04-05 at 11:14 -0500, Kylene Jo Hall wrote: > On Thu, 2005-03-24 at 13:33 -0800, Greg KH wrote: > > On Thu, Mar 24, 2005 at 04:04:25PM -0500, Jeff Garzik wrote: > > > Greg KH wrote: > > > >On Tue, Mar 22, 2005 at 09:02:24PM -0500, Jeff Garzik wrote: > > > > > > > >>Kylene Hall wrote:

Re: [PATCH] Add TPM hardware enablement driver

2005-04-08 Thread Kylene Jo Hall
On Tue, 2005-04-05 at 11:14 -0500, Kylene Jo Hall wrote: On Thu, 2005-03-24 at 13:33 -0800, Greg KH wrote: On Thu, Mar 24, 2005 at 04:04:25PM -0500, Jeff Garzik wrote: Greg KH wrote: On Tue, Mar 22, 2005 at 09:02:24PM -0500, Jeff Garzik wrote: Kylene Hall wrote: what is the

Re: [PATCH] Add TPM hardware enablement driver

2005-04-05 Thread Kylene Jo Hall
On Thu, 2005-03-24 at 13:33 -0800, Greg KH wrote: > On Thu, Mar 24, 2005 at 04:04:25PM -0500, Jeff Garzik wrote: > > Greg KH wrote: > > >On Tue, Mar 22, 2005 at 09:02:24PM -0500, Jeff Garzik wrote: > > > > > >>Kylene Hall wrote: > > >> > > what is the purpose of this pci_dev_get/put?

Re: [PATCH] Add TPM hardware enablement driver

2005-04-05 Thread Kylene Jo Hall
On Thu, 2005-03-24 at 13:33 -0800, Greg KH wrote: On Thu, Mar 24, 2005 at 04:04:25PM -0500, Jeff Garzik wrote: Greg KH wrote: On Tue, Mar 22, 2005 at 09:02:24PM -0500, Jeff Garzik wrote: Kylene Hall wrote: what is the purpose of this pci_dev_get/put? attempting to prevent

Re: [PATCH] Add TPM hardware enablement driver

2005-03-24 Thread Greg KH
On Thu, Mar 24, 2005 at 04:04:25PM -0500, Jeff Garzik wrote: > Greg KH wrote: > >On Tue, Mar 22, 2005 at 09:02:24PM -0500, Jeff Garzik wrote: > > > >>Kylene Hall wrote: > >> > what is the purpose of this pci_dev_get/put? attempting to prevent > hotplug or > something? > >>> > >>> >

Re: [PATCH] Add TPM hardware enablement driver

2005-03-24 Thread Jeff Garzik
Greg KH wrote: On Tue, Mar 22, 2005 at 09:02:24PM -0500, Jeff Garzik wrote: Kylene Hall wrote: what is the purpose of this pci_dev_get/put? attempting to prevent hotplug or something? Seems that since there is a refernce to the device in the chip structure and I am making the file private data

Re: [PATCH] Add TPM hardware enablement driver

2005-03-24 Thread Jeff Garzik
Greg KH wrote: On Tue, Mar 22, 2005 at 09:02:24PM -0500, Jeff Garzik wrote: Kylene Hall wrote: what is the purpose of this pci_dev_get/put? attempting to prevent hotplug or something? Seems that since there is a refernce to the device in the chip structure and I am making the file private data

Re: [PATCH] Add TPM hardware enablement driver

2005-03-24 Thread Greg KH
On Thu, Mar 24, 2005 at 04:04:25PM -0500, Jeff Garzik wrote: Greg KH wrote: On Tue, Mar 22, 2005 at 09:02:24PM -0500, Jeff Garzik wrote: Kylene Hall wrote: what is the purpose of this pci_dev_get/put? attempting to prevent hotplug or something? Seems that since there is a refernce

Re: [PATCH] Add TPM hardware enablement driver

2005-03-23 Thread Greg KH
On Tue, Mar 22, 2005 at 09:02:24PM -0500, Jeff Garzik wrote: > Kylene Hall wrote: > >>what is the purpose of this pci_dev_get/put? attempting to prevent > >>hotplug or > >>something? > > > > > >Seems that since there is a refernce to the device in the chip structure > >and I am making the file

Re: [PATCH] Add TPM hardware enablement driver

2005-03-23 Thread Greg KH
On Tue, Mar 22, 2005 at 09:02:24PM -0500, Jeff Garzik wrote: Kylene Hall wrote: what is the purpose of this pci_dev_get/put? attempting to prevent hotplug or something? Seems that since there is a refernce to the device in the chip structure and I am making the file private data

Re: [PATCH] Add TPM hardware enablement driver

2005-03-22 Thread Jeff Garzik
Kylene Hall wrote: what is the purpose of this pci_dev_get/put? attempting to prevent hotplug or something? Seems that since there is a refernce to the device in the chip structure and I am making the file private data pointer point to that chip structure this is another reference that must be

Re: [PATCH] Add TPM hardware enablement driver

2005-03-22 Thread Jeff Garzik
Kylene Hall wrote: what is the purpose of this pci_dev_get/put? attempting to prevent hotplug or something? Seems that since there is a refernce to the device in the chip structure and I am making the file private data pointer point to that chip structure this is another reference that must be

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: [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: [PATCH] Add TPM hardware enablement driver

2005-03-15 Thread Kylene Jo Hall
Thanks for the helpful comments I am working on a patch to fix your concerns but I have a couple of questions. On Wed, 2005-03-09 at 22:51 -0500, Jeff Garzik wrote: > > + down(>timer_manipulation_mutex); > > + chip->time_expired = 0; > > + init_timer(>device_timer); > > +

Re: [PATCH] Add TPM hardware enablement driver

2005-03-15 Thread Kylene Jo Hall
Thanks for the helpful comments I am working on a patch to fix your concerns but I have a couple of questions. On Wed, 2005-03-09 at 22:51 -0500, Jeff Garzik wrote: snip + down(chip-timer_manipulation_mutex); + chip-time_expired = 0; + init_timer(chip-device_timer); +

Re: [PATCH] Add TPM hardware enablement driver

2005-03-10 Thread Alexey Dobriyan
On Thursday 10 March 2005 02:42, Greg KH wrote: > [PATCH] Add TPM hardware enablement driver > +static ssize_t tpm_transmit(struct tpm_chip *chip, const char *buf, > + size_t bufsiz) > +{ > + u32 count; > + __be32 *native_size; > + > +

Re: [PATCH] Add TPM hardware enablement driver

2005-03-10 Thread Nish Aravamudan
On Thu, 10 Mar 2005 09:35:36 -0800, Nish Aravamudan <[EMAIL PROTECTED]> wrote: > On Wed, 9 Mar 2005 16:42:01 -0800, Greg KH <[EMAIL PROTECTED]> wrote: > > ChangeSet 1.2035, 2005/03/09 10:12:19-08:00, [EMAIL PROTECTED] > > > > [PATCH] Add TPM hardware ena

Re: [PATCH] Add TPM hardware enablement driver

2005-03-10 Thread Nish Aravamudan
On Wed, 9 Mar 2005 16:42:01 -0800, Greg KH <[EMAIL PROTECTED]> wrote: > ChangeSet 1.2035, 2005/03/09 10:12:19-08:00, [EMAIL PROTECTED] > > [PATCH] Add TPM hardware enablement driver > +void tpm_time_expired(unsigned long ptr) > +{ > + int *exp = (int *)

Re: [PATCH] Add TPM hardware enablement driver

2005-03-10 Thread Nish Aravamudan
On Wed, 9 Mar 2005 16:42:01 -0800, Greg KH [EMAIL PROTECTED] wrote: ChangeSet 1.2035, 2005/03/09 10:12:19-08:00, [EMAIL PROTECTED] [PATCH] Add TPM hardware enablement driver snip +void tpm_time_expired(unsigned long ptr) +{ + int *exp = (int *) ptr; + *exp = 1

Re: [PATCH] Add TPM hardware enablement driver

2005-03-10 Thread Nish Aravamudan
On Thu, 10 Mar 2005 09:35:36 -0800, Nish Aravamudan [EMAIL PROTECTED] wrote: On Wed, 9 Mar 2005 16:42:01 -0800, Greg KH [EMAIL PROTECTED] wrote: ChangeSet 1.2035, 2005/03/09 10:12:19-08:00, [EMAIL PROTECTED] [PATCH] Add TPM hardware enablement driver snip +void tpm_time_expired

Re: [PATCH] Add TPM hardware enablement driver

2005-03-10 Thread Alexey Dobriyan
On Thursday 10 March 2005 02:42, Greg KH wrote: [PATCH] Add TPM hardware enablement driver +static ssize_t tpm_transmit(struct tpm_chip *chip, const char *buf, + size_t bufsiz) +{ + u32 count; + __be32 *native_size; + + native_size = (__force __be32

Re: [PATCH] Add TPM hardware enablement driver

2005-03-09 Thread Jeff Garzik
Greg KH wrote: diff -Nru a/drivers/char/tpm/tpm.c b/drivers/char/tpm/tpm.c --- /dev/null Wed Dec 31 16:00:00 196900 +++ b/drivers/char/tpm/tpm.c 2005-03-09 16:40:26 -08:00 @@ -0,0 +1,697 @@ +/* + * Copyright (C) 2004 IBM Corporation + * + * Authors: + * Leendert van Doorn <[EMAIL PROTECTED]> + *

[PATCH] Add TPM hardware enablement driver

2005-03-09 Thread Greg KH
ChangeSet 1.2035, 2005/03/09 10:12:19-08:00, [EMAIL PROTECTED] [PATCH] Add TPM hardware enablement driver This patch is a device driver to enable new hardware. The new hardware is the TPM chip as described by specifications at <http://www.trustedcomputinggroup.org>. The TPM chip will

[PATCH] Add TPM hardware enablement driver

2005-03-09 Thread Greg KH
ChangeSet 1.2035, 2005/03/09 10:12:19-08:00, [EMAIL PROTECTED] [PATCH] Add TPM hardware enablement driver This patch is a device driver to enable new hardware. The new hardware is the TPM chip as described by specifications at http://www.trustedcomputinggroup.org. The TPM chip will enable you

Re: [PATCH] Add TPM hardware enablement driver

2005-03-09 Thread Jeff Garzik
Greg KH wrote: diff -Nru a/drivers/char/tpm/tpm.c b/drivers/char/tpm/tpm.c --- /dev/null Wed Dec 31 16:00:00 196900 +++ b/drivers/char/tpm/tpm.c 2005-03-09 16:40:26 -08:00 @@ -0,0 +1,697 @@ +/* + * Copyright (C) 2004 IBM Corporation + * + * Authors: + * Leendert van Doorn [EMAIL PROTECTED] + *