Hi Mathias,
please note:
I'm writing this email on behalf of myself only and nobody else, especially not
my employer - and I'm doing this in my spare time.
I'm working for a direct competitor of yours, but I'm not using any knowledge
that I've picked up at work or that is considered secret in a
Am Donnerstag, 29. November 2012, 23:08:11 schrieb Kent Yoder:
> On Thu, Nov 29, 2012 at 01:54:52PM -0800, Joe Perches wrote:
> > >
> > > - itpm = (probe == 0) ? 0 : 1;
> > > + itpm = !!probe;
> >
> > Even more trivia:
> >
> > This !! isn't necessary as ints assigned to bool are
Am Freitag, 25. Januar 2013, 21:25:38 schrieb Jason Gunthorpe:
> > https://github.com/shpedoikal/linux.git tpmdd-01-22-13
>
> Thanks Kent, I will try to test your branch next week, if I am able.
>
> Can you also grab
>
> https://github.com/jgunthorpe/linux/commit/98b2a198b43b41b0535200bf47516078
Am Mittwoch, 13. Februar 2013, 03:54:26 schrieb Peter Huewe:
> Local variable AcmCtrl is never read/used after assignment so we can remove
> all assignments to it and the related code around the assignments.
>
> Signed-off-by: Peter Huewe
Sorry wrong distribution list.
Seems my script went a bit
Hi,
while analyzing the comedi drivers, I noticed that quite a lot of them use a
more or less similar find_boardinfo function.
e.g.:
cb_pcidas64.c
static const struct pcidas64_board
*cb_pcidas64_find_pci_board(struct pci_dev *pcidev)
{
unsigned int i;
for (i = 0; i < ARRAY_SIZ
Hi,
thanks for resending.
Am Donnerstag, 28. Februar 2013, 11:06:11 schrieb Jean Delvare:
> From: Shubhrajyoti Datta
>
> Convert the struct i2c_msg initialization to C99 format. This makes
> maintaining and editing the code simpler. Also helps once other fields
> like transferred are added in
> > I'd suggest not to. Mixing actual changes with cleanups is generally
> > discouraged.
>
> Yes, please don't do this. Also because it makes it really
> confusing for me to figure out whether something has been applied or
> not and if I have the right people on the sign off.
>
> Peter, can
Hi,
loading iio_dummy results in kernel panic as the call to
iio_buffer_register in iio_dummy_probe is performed with indio_dev->buffer ==
NULL and thus the access to indio_dev->buffer->attrs results in this
oops/panic.
Thanks,
Peter
Steps to reproduce:
#modprobe iio_dummy
iio_dummy: module
Am Montag, 23. September 2013, 20:14:31 schrieb Jason Gunthorpe:
> This suppresses compile warnings on 32 bit builds.
>
> Signed-off-by: Jason Gunthorpe
Reviewed-by: Peter Huewe
Signed-off-by: Peter Huewe
Staged here
https://github.com/PeterHuewe/linux-tpmdd for-james
--
To unsubscribe fr
Hi Jason,
Am Mittwoch, 2. Oktober 2013, 00:21:13 schrieb Jason Gunthorpe:
> On Wed, Oct 02, 2013 at 12:09:22AM +0200, Peter H?we wrote:
> >Since the tpm_spi_stm_st33, tpm_i2c_nuvoton and tpm_i2c_atmel drivers
> >are not yet merged and were heavily improved by you anyway, please
> >incl
Am Montag, 23. September 2013, 20:14:38 schrieb Jason Gunthorpe:
> CLASS-dev.c is a common idiom for Linux subsystems
>
> This pulls all the code related to the miscdev into tpm-dev.c and makes it
> static. The identical file_operation structs in the drivers are purged and
> the tpm common code un
Am Mittwoch, 2. Oktober 2013, 00:57:33 schrieb Jason Gunthorpe:
> > When compiling the tpm drivers as modules I get
> > ERROR: "tpm_sysfs_del_device" [drivers/char/tpm/tpm.ko] undefined!
> > ERROR: "tpm_dev_add_device" [drivers/char/tpm/tpm.ko] undefined!
> > ERROR: "tpm_dev_del_device" [drivers/ch
Am Donnerstag, 3. Oktober 2013, 07:05:04 schrieb Jason Gunthorpe:
> On Wed, Oct 02, 2013 at 01:14:18AM +0200, Peter H?we wrote:
> > > I botched the makefile changes for the new .c files.
> > >
> > > I believe it should be like this:
> > >
> > > obj-$(CONFIG_TCG_TPM) += tpm-core.o
> > > tpm-core-y
Am Freitag, 4. Oktober 2013, 21:17:36 schrieb Stefan Berger:
> On 10/04/2013 01:08 PM, Jason Gunthorpe wrote:
> > On Mon, Sep 30, 2013 at 05:09:51PM -0500, Joel Schopp wrote:
> >>> So far, nobody I have talked to has offered any strong opinions on
> >>> what locality should be used or how it should
Hi,
thanks for your patch
> static struct pnp_device_id tpm_pnp_tbl[] = {
> {"PNP0C31", 0}, /* TPM */
> {"ATM1200", 0}, /* Atmel */
> @@ -835,9 +834,12 @@ static struct pnp_driver tis_pnp_driver = {
> .name = "tpm_tis",
> .id_table = tpm_pnp_tbl,
> .p
Hi Shuah,
thanks for your reply.
> >> +#ifdef CONFIG_PM_SLEEP
> >> + .driver = {
> >> + .pm = &tpm_tis_pm,
> >> + },
> >> +#endif
> >>
> >> };
> >
> > I don't think the #if CONFIG_PM_SLEEP is required here.
In this case, the SIMPLE_DEV_PM_OPS macro handles the case internally - i.e
When compiling it I get
WARNING: modpost: missing MODULE_LICENSE() in
drivers/char/tpm/tpm_spi_stm_st33.o
Thanks,
Peter
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/m
Hi Kent,
>
> Heh, duh, well of course it is. I've now staged everything I'm
> planning on pushing at:
>
> git://github.com/shpedoikal/linux.git tpmdd-12-05-12
>
> Please test and let me know if I missed anything.
>
> Thanks,
> Kent
>
> > Kent
> >
(I'm still writing this on behalf of myself
Hi Kent, Matthias,
Am Mittwoch, 5. Dezember 2012, 19:07:20 schrieb Kent Yoder:
> Heh, duh, well of course it is. I've now staged everything I'm
> planning on pushing at:
>
> git://github.com/shpedoikal/linux.git tpmdd-12-05-12
>
> Please test and let me know if I missed anything.
I just chec
Hi Kent,
Am Mittwoch, 5. Dezember 2012, 22:00:48 schrieb Kent Yoder:
> This is already at least v3 of the driver IIRC. And its not fully
> committed at all, its just in a staging tree. I can blow it away at any
> time if we find problems.
>
> Not a problem. usually I'd attach any updates I pl
Am Donnerstag, 6. Dezember 2012, 00:09:41 schrieb Kent Yoder:
> >
> > > static int tpm_st33_i2c_pm_suspend(struct i2c_client *client,
> > > pm_message_t mesg)
> > >
> > >...
> > >
> > > static int tpm_st33_i2c_pm_resume(struct i2c_client *client)
> > >
> > >,,,
> > >static struct i2c_driver tpm_st
Am Donnerstag, 6. Dezember 2012, 17:27:02 schrieb Kent Yoder:
> On Thu, Dec 06, 2012 at 01:20:51AM +0100, Peter Huewe wrote:
> > This patch converts the suspend and resume functions for
> > tpm_i2c_stm_st33 to the new dev_pm_ops.
> >
> > Signed-off-by: Peter Huewe
>
> One minor tweak, the PM f
Hi,
since I don't have a tpm I simply tried out whether the handling without a tpm
is correct, which it unfortunately isn't as my kernel oopses.
Steps to reproduce:
# insmod /data/data-old/linux-2.6/drivers/char/tpm/tpm_i2c_stm_st33.ko
# echo st33zp24_i2c 0x20 /sys/bus/i2c/devices/i2c-1/new_dev
Am Mittwoch, 14. November 2012, 23:18:26 schrieb Kent Yoder:
> On Mon, Nov 12, 2012 at 11:37:18PM +0100, Peter Huewe wrote:> >
> > This patch also gets rid of the (false positive) sparse warning:
> > drivers/char/tpm/tpm.c:360 tpm_calc_ordinal_duration() error: buffer
> > overflow 'tpm_protected
Hi Daniel,
thanks for this v3.
It's really nice to see the progress and I really like that
sparse/smatch/clang/coccicheck do not complain at all - nice job!
Konrad already did an excellent job at reviewing the driver (thanks for that),
and all previously pointed out issues are fixed.
Unfortuna
Am Samstag, 15. Juni 2013, 16:23:24 schrieb Lars-Peter Clausen:
> The I2C client driver is not supposed to modify the client's driver
> pointer, this is handled by the I2C core.
>
> Signed-off-by: Lars-Peter Clausen
> ---
Nice catch.
Staged here:
https://github.com/PeterHuewe/linux-tpmdd/commit
Hi Daniel,
thanks for the rework.
I'll try to review it this week, so Konrad might take it through his tree.
Thanks,
Peter
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.or
Am Montag, 1. Juli 2013, 23:34:28 schrieb Daniel De Graaf:
> This is a complete rewrite of the Xen TPM frontend driver, taking
> advantage of a simplified frontend/backend interface and adding support
> for cancellation and timeouts. The backend for this driver is provided
> by a vTPM stub domain
Hi Kent,
short reply from my private account - so only talking on behalf of myself.
Am Montag, 4. März 2013, 18:41:04 schrieb Kent Yoder:
> Hi Peter,
>
> Sorry for the long delay in getting this reviewed...
No problem.
>
> > - for (count = 0; count < MAX_COUNT; count++) {
> > - u
Am Montag, 4. März 2013, 20:42:00 schrieb Geert Uytterhoeven:
> drivers/staging/vt6656/card.c: In function ‘CARDqGetNextTBTT’:
> drivers/staging/vt6656/card.c:793: warning: integer constant is too large
> for ‘unsigned long’ type
>
> Commit c7b7cad0d8df823ea063c86a54316bbcbfa04a7c ("staging/vt6656
Hi Matthias,
Am Mittwoch, 15. Mai 2013, 15:53:19 schrieb Matthias Leblanc:
> +static inline int spi_read_write(struct spi_device *spi,
> + struct spi_transfer xfer) {
> + struct spi_message msg;
> + spi_message_init(&msg);
> + spi_message_add_tail(&xfer, &msg);
> + return spi
Hi Matthias,
Am Dienstag, 9. April 2013, 16:42:42 schrieb Matthias Leblanc:
> From: Mathias leblanc
>
> * STMicroelectronics version 1.2.0, Copyright (C) 2010
> * STMicroelectronics comes with ABSOLUTELY NO WARRANTY.
> * This is free software, and you are welcome to redistribute it
> * under
Hi Matthias,
it's nice to see that you consider most of the comments, unfortunately I still
have some left ;)
> +/*
> + * tpm_st33_spi_init initialize driver
> + * @return: 0 if successful, else non zero value.
> + */
> +static int __init tpm_st33_spi_init(void)
> +{
> + return spi_register_
Hi Matthias,
Am Donnerstag, 11. April 2013, 10:58:59 schrieb Mathias LEBLANC:
> Hi Peter,
>
> Yes, you have right, these error are generated by #ifndef
> __STM_ST33_TPM_SPI_H__ Just remove it and the #endif next the structure.
Yes, I know that - for the submission to be included this has to be cha
Hi Matthias:
> Regarding the while loop, I don't see how can I check the number of dummy
> byte differently?
Checking the number of dummy bytes is fine -
in line #144 you check it with a while loop which looks fine
in line #198 you check it with a for loop with an empty body, which in my
opinio
Am Montag, 20. Mai 2013, 21:56:16 schrieb Peter Huewe:
> Removing some boilerplate by using module_spi_driver instead of calling
> register and unregister in the otherwise empty init/exit functions
>
> Signed-off-by: Peter Huewe
Sorry for the noise
already exists in 3.10-rc1 by Lars-Peter Cla
Am Montag, 20. Mai 2013, 21:28:35 schrieb Peter Huewe:
> Removing some boilerplate by using module_i2c_driver instead of calling
> register and unregister in the otherwise empty init/exit functions
>
> Signed-off-by: Peter Huewe
Sorry for the noise
099bb952b064 mfd: adp5520: Use module_i2c_dr
Am Montag, 20. Mai 2013, 21:31:57 schrieb Peter Huewe:
> Removing some boilerplate by using module_i2c_driver instead of calling
> register and unregister in the otherwise empty init/exit functions
>
> Signed-off-by: Peter Huewe
> ---
Sorry for the noise
already exists in 3.10-rc1 by Lars-Peter
Am Montag, 20. Mai 2013, 21:54:46 schrieb Peter Huewe:
> Removing some boilerplate by using module_spi_driver instead of calling
> register and unregister in the otherwise empty init/exit functions
>
> Signed-off-by: Peter Huewe
Sorry for the noise
already exists in 3.10-rc1
3d4a131 mrf24j40: u
Am Montag, 20. Mai 2013, 21:53:31 schrieb Marc Kleine-Budde:
> On 05/20/2013 09:49 PM, Peter Huewe wrote:
> > Removing some boilerplate by using module_spi_driver instead of calling
> > register and unregister in the otherwise empty init/exit functions
> >
> > Signed-off-by: Peter Huewe
>
> A si
Hi James,
The following changes since commit 5b88e270253db6d817e6a2f61909d1e53620e990:
maintainers: Remove Kent from maintainers
are available in the git repository at:
git://github.com/PeterHuewe/linux-tpmdd.git for-james
for you to fetch changes up to e2fa3d799b7471f00adae59ff36260a
Hi Daniel,
> Daniel De Graaf wrote:
> I wasn't sure if this patch goes in via a Xen tree or TPM driver
> maintainers. I'll CC the full list from scripts/get_maintainer.pl
> on my v3 submission.
I'd suggest to get at least an ack from both subsystems, as most people aren't
expert in both fields -
> > As far as I know, none of these people have
> > previously submitted a single patch.
> >
> Works for me... I'll submit a new patch removing myself. The tpmdd
> list will be monitored by the same folks in the meantime.
>
> If prior experience is an issue I'd nominate Peter Huewe if he's
>
Hi Peter,
> > Also the flags etc would perhaps be handy, they can be retrieved via
> > sysfs - on you machine it _should_ be
> > # cat /sys/devices/pnp0/00:0a/*
> > and also post the output.
>
> root@zepto:/home/peter# ls /sys/devices/pnp0/00\:0a/
Okay it seems it's not 00:0a ;/
Can you check t
Am Montag, 21. Januar 2013, 21:56:22 schrieb H Hartley Sweeten:
> On Monday, January 21, 2013 1:52 PM, Peter Huewe wrote:
> > This patch adds the comedi_driver.name to pci_driver.name
> > to the skel driver, so we can remove the workaround in drivers.c
> >
> > Signed-off-by: Peter Huewe
> > ---
>
Am Dienstag, 22. Januar 2013, 23:49:53 schrieb H Hartley Sweeten:
> On Tuesday, January 22, 2013 3:40 PM, Peter Huewe wrote:
> > (Almost) all comedi pci drivers have some wrapper for their
> >
> > pci_driver.remove function which simply calls comedi_pci_auto_unconfig
> > which has the same functi
Am Freitag, 15. Februar 2013, 17:11:02 schrieb Dan Carpenter:
> Are you using this version of clang?:
>
> http://git.linuxfoundation.org/llvmlinux.git/
>
> regards,
> dan carpenter
Hi,
nope - plain vanilla llvm/clang, but I use some of the kernel patches from
there:
http://git.linuxfoundation.
Hi Jason,
Thanks for the updated patch!
Sorry, I have one really minor remark left:
> + if (rc) {
> + dev_err(chip->dev,
> + "A TPM error (%d) occurred attempting to determine the
> timeouts\n",
rc is a ssize_t here and when compiling with C=1 I get
drivers/
Hi,
> Do you have the I2C configuration activated ?
> I have this error when I add the driver in a kernel with no I2C activated
I2C works fine on my beaglebone, however I did not add the platform data to my
boardfile.
a) as I don't use board files anymore but device trees ;)
b) it's perfectly l
Am Freitag, 16. Oktober 2015, 20:40:22 schrieb Jarkko Sakkinen:
> Added a new function __compat_only_sysfs_link_group_to_kobj() that adds
> a symlink from attribute or group to a kobject. This needed for
> maintaining backwards compatibility with PPI attributes in the TPM
> driver.
>
> Signed-off-
Am Freitag, 16. Oktober 2015, 20:40:20 schrieb Jarkko Sakkinen:
> + pa = ((u64) le32_to_cpu(ioread32(&priv->cca->cmd_pa_high)) << 32) +
> + (u64) le32_to_cpu(ioread32(&priv->cca->cmd_pa_low));
The canonical form would be
> + pa = ((u64) le32_to_cpu(ioread32(&priv->cca->c
Am Freitag, 16. Oktober 2015, 20:40:25 schrieb Jarkko Sakkinen:
> This patch introduces struct tpm_buf that provides a string buffer for
> constructing TPM commands. This allows to construct variable sized TPM
> commands. For the buffer a page is allocated and mapped, which limits
> maximum size to
Am Donnerstag, 4. Dezember 2014, 06:55:17 schrieb Jarkko Sakkinen:
> tpm_crb is a driver for TPM 2.0 Command Response Buffer (CRB) Interface
> as defined in PC Client Platform TPM Profile (PTP) Specification.
>
> Only polling and single locality is supported as these are the limitations
> of the a
Am Donnerstag, 4. Dezember 2014, 06:55:13 schrieb Jarkko Sakkinen:
> Traversal of the ACPI device tree was not done right. PPI interface
> should be looked up only from the ACPI device that is the platform
> device for the TPM. This could cause problems with systems with
> two TPM chips such as 4th
Am Donnerstag, 4. Dezember 2014, 06:55:14 schrieb Jarkko Sakkinen:
> Rename chip->dev to chip->pdev to make it explicit that this not the
> character device but actually represents the platform device.
>
> Signed-off-by: Jarkko Sakkinen
> ---
> drivers/char/tpm/tpm-chip.c | 4 ++--
> dr
Am Donnerstag, 4. Dezember 2014, 06:55:18 schrieb Jarkko Sakkinen:
> From: Will Arthur
>
> Detect TPM 2.0 by using the extended STS (STS3) register. For TPM 2.0,
> instead of calling tpm_get_timeouts(), assign duration and timeout
> values defined in the TPM 2.0 PTP specification.
>
> Signed-off
Hi Jarkko,
Am Donnerstag, 4. Dezember 2014, 06:55:18 schrieb Jarkko Sakkinen:
> From: Will Arthur
>
> Detect TPM 2.0 by using the extended STS (STS3) register. For TPM 2.0,
> instead of calling tpm_get_timeouts(), assign duration and timeout
> values defined in the TPM 2.0 PTP specification.
>
Am Donnerstag, 4. Dezember 2014, 23:18:33 schrieb Peter Hüwe:
> Hi Jarkko,
>
> Am Donnerstag, 4. Dezember 2014, 06:55:18 schrieb Jarkko Sakkinen:
> > From: Will Arthur
> >
> > Detect TPM 2.0 by using the extended STS (STS3) register. For TPM 2.0,
> > in
Am Donnerstag, 4. Dezember 2014, 06:55:17 schrieb Jarkko Sakkinen:
> tpm_crb is a driver for TPM 2.0 Command Response Buffer (CRB) Interface
> as defined in PC Client Platform TPM Profile (PTP) Specification.
>
> Only polling and single locality is supported as these are the limitations
> of the a
Am Mittwoch, 3. Dezember 2014, 03:28:35 schrieb Stefan Berger:
> On 12/02/2014 05:31 PM, Jarkko Sakkinen wrote:
> > +
> > +/**
> > + * tpm2_startup() - send startup command to the TPM chip
> > + * @chip: TPM chip to use.
> > + * @startup_type startup type. The value is either
> > + *
Am Donnerstag, 4. Dezember 2014, 06:55:16 schrieb Jarkko Sakkinen:
> TPM 2.0 devices are separated by adding a field 'flags' to struct
> tpm_chip and defining a flag TPM_CHIP_FLAG_TPM2 for tagging them.
>
> This patch adds the following internal functions:
>
> - tpm2_get_random()
> - tpm2_get_tpm
Hi Bruno,
thanks for your patch!
The patch is fine,
but unfortunately not against the latest version of the tpm subsytem code.
You can find the latest version of the tpm subsystem at
https://github.com/PeterHuewe/linux-tpmdd
the for-james branch is usually the best choice here.
Nevertheless
Am Montag, 30. März 2015, 22:06:08 schrieb Fabian Frederick:
> KERN_ERR is implicitely declared in pr_err()
>
> Signed-off-by: Fabian Frederick
Good catch.
Acked-by: Peter Huewe
If nobody else pulls this one through his tree, I'll apply it probably for
4._2_ (so it might take a while)
Peter
Hey,
Am Freitag, 22. Mai 2015, 19:23:02 schrieb Hon Ching(Vicky) Lo:
> tpm_ibmvtpm_probe() calls ibmvtpm_reset_crq(ibmvtpm) without having yet
> set the virtual device in the ibmvtpm structure. So in ibmvtpm_reset_crq,
> the phype call contains empty unit addresses, ibmvtpm->vdev->unit_address.
>
Hi Jarkko
> > > >Betreff: [PATCH] tpm, tpm_crb: migrate to struct acpi_table_tpm2 and
> > > >acpi_tpm2_control
> > > but are the names there still correct? Isn't this information outdated?
> > > The acpi spec refers to the MS spec which is not present anymore, and
> > > MS refers to the TCG -- and
Hi,
Am Mittwoch, 3. Juni 2015, 18:39:14 schrieb Jarkko Sakkinen:
> This patch introduces struct tpm_buf that provides a string buffer for
> constructing TPM commands. This allows to construct variable sized TPM
> commands. This feature is needed for TPM 2.0 commands in order to allow
> policy aut
Am Dienstag, 21. Januar 2014, 18:15:01 schrieb Linus Torvalds:
> is that this is a callback for acpi_walk_namespace(), and returning a
> failure status means that the walk will be interrupted.
>
> So you actually want to return AE_OK if the acpi_get_name() call
> fails, because that just skips the
Hi Greg,
> >
> > CID: 986658
> What is this field for?
That's the scan id in the coverity database.
If you think that's just noise I can leave it out.
Thanks,
Peter
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
M
Hi Joe, Jason
> fyi: using
> if (strcmp(foo, bar) < 0)
> is canonical.
Yeah of course you're both right -
if (strcmp(foo, bar) < 0)
is the correct version.
Sorry about the rubbish patch.
> lib/string.c:strcmp returns only -1,0,1
> so that's what the arch versions should do too.
> How
Hi Greg,
> It [the CID] is noise as not everyone can see it or make anything out of it,
> so please don't include it.
I'll remove it from my tree before sending out the pull request to James.
Thanks for your opinion.
Peter
--
To unsubscribe from this list: send the line "unsubscribe linux-ker
Am Dienstag, 22. Juli 2014, 01:43:30 schrieb Jason Gunthorpe:
> On Tue, Jul 22, 2014 at 01:26:05AM +0200, Peter Huewe wrote:
> > Unfortunately the tpm_tis driver did never call request_region for its
> > adress space - now since we've got devm_ioremap_resource we can simply
> > remove all the stuff
Am Montag, 22. September 2014, 19:13:38 schrieb Jason Gunthorpe:
>
> This does look much nicer, lets use this version.
>
> I think Peter were prefer a new clean patch that superceeds the
> original.
>
> > + if (!priv->irq_tested) {
>
> I think the sleep and check is still needed here, the IR
Am Mittwoch, 24. September 2014, 19:13:38 schrieb Jason Gunthorpe:
> On Wed, Sep 24, 2014 at 12:06:02PM +0300, Jarkko Sakkinen wrote:
> > Added tpm2-sysfs.c that implements sysfs attributes for a TPM2
> > device.
>
> You need to document in Documentation every new sysfs that is added.
>
> The exi
Am Mittwoch, 24. September 2014, 21:43:23 schrieb Jarkko Sakkinen:
> On Wed, Sep 24, 2014 at 10:53:20AM -0600, Jason Gunthorpe wrote:
> > On Wed, Sep 24, 2014 at 12:05:53PM +0300, Jarkko Sakkinen wrote:
> > > +static struct tpm_input_header tpm2_pcrread_header = {
> >
> > Missing const - all of th
Am Mittwoch, 24. September 2014, 22:19:38 schrieb Jason Gunthorpe:
> On Wed, Sep 24, 2014 at 10:02:34PM +0300, Jarkko Sakkinen wrote:
> > > The pcrs file never conformed to the sysfs rules, if TPM2 is getting a
> > > whole new file set, I wouldn't mind seeing it not include the
> > > non-conformant
Am Mittwoch, 24. September 2014, 19:59:40 schrieb Jason Gunthorpe:
> On Wed, Sep 24, 2014 at 01:34:11PM -0400, Stefan Berger wrote:
> > On 09/24/2014 01:13 PM, Jason Gunthorpe wrote:
> > >On Wed, Sep 24, 2014 at 12:06:02PM +0300, Jarkko Sakkinen wrote:
> > >>+static ssize_t durations_show(struct de
Am Freitag, 12. September 2014, 21:09:47 schrieb Peter Huewe:
> If adapter->dev.parent == NULL there is a NULL pointer dereference in
> acpi_i2c_install_space_handler and acpi_i2c_remove_space_handler.
>
> This is present since introduction of this code:
> 366047515c6e "i2c: rework kernel config I
Hi
Am Samstag, 13. September 2014, 19:35:33 schrieb Jarkko Sakkinen:
> Replaced transmit_cmd() functions in tpm-interface.c and tpm-sysfs.c
> with a single tpm_transmit_cmd() that can be used in both files.
>
> This patch is preliminary clean up work for the TPM2 support. This
> function is need
Am Freitag, 29. November 2013, 22:12:52 schrieb Jason Gunthorpe:
> On Fri, Nov 29, 2013 at 06:00:18PM +0100, Michal Nazarewicz wrote:
> > From: Michal Nazarewicz
> >
> > “wait” wait queue is defined but never used in the function, thus
> > it can be removed.
> >
> > Signed-off-by: Michal Nazarew
Hi Jason
did push everything to
https://github.com/PeterHuewe/linux-tpmdd/tree/testing-and-review
against latest rc, with alls signed-offs and reviewed-bys
I'll move it to for-james after giving it a final test run.
Thanks,
Peter
--
To unsubscribe from this list: send the line "unsubscr
Hi James,
since the next merge window approaches, can you please pull the following TPM
Changes for 3.14?
I'll send the patches as replies to this email.
The following changes since commit 802eee95bde72fd0cd0f3a5b2098375a487d1eda:
Linux 3.13-rc6 (2013-12-29 16:01:33 -0800)
are available in
Hi James,
> Please generate this against my next branch.
Here it is - note:
I cherry-picked the changes that went through the xen tree before applying my
changes.
The following changes since commit dcf4e392867bf98d50ad108ed7c2bfb941e8c33d:
ima: remove unneeded size_limit argument from ima_eve
Hi James,
The following changes since commit 38fd2c202a3d82bc12430bce5789fa2c2a406f71:
Merge to v3.13-rc7 for prerequisite changes in the Xen code for TPM
(2014-01-06 22:23:01 +1100)
are available in the git repository at:
https://github.com/PeterHuewe/linux-tpmdd for-james
for you to fe
CC: Luigi, he works at Google and is responsible for the TPMs in
Chromebooks ;)
Thanks,
Peter
Am Freitag, 22. August 2014, 22:32:41 schrieb Jason Gunthorpe:
> On Fri, Aug 22, 2014 at 08:17:27PM +, Scot Doyle wrote:
> > On Fri, 22 Aug 2014, Jason Gunthorpe wrote:
> > > On Fri, Aug 22, 2014 at
Hi Scot,
Am Montag, 25. August 2014, 09:14:03 schrieb Scot Doyle:
> commit 4c663cfc523a88d97a8309b04a089c27dc57fd7e
> wait: fix false timeouts when using wait_event_timeout()
>
> changed the semantics of wait_event_interruptible_timeout so that a
> condition check is performed after timeout an
Hi,
as far as I remember everything marked with __exit or __exit_data will only be
used/called when unloading a module, and gets moved to the .exit.text or
.exit.data sections.
Why are these sections present in the vmlinux/vmlinux.bin/bzImage and not
dropped by the linker or at least objdump?
Hi Vicky,
Am Donnerstag, 16. Juli 2015, 19:54:15 schrieb Hon Ching(Vicky) Lo:
> Hi Peter,
>
> On Mon, 2015-07-13 at 23:08 +0200, Peter Hüwe wrote:
> > Hi Vicky,
> >
> > sorry for the late reply
> >
> > > This patch makes the code endian
Applied to my tree:
https://github.com/PeterHuewe/linux-tpmdd for-james
Will be included in the next pull-request.
Thanks,
Peter
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.ker
Hi Scot,
Am Montag, 29. September 2014, 19:24:57 schrieb Jason Gunthorpe:
> On Wed, Sep 24, 2014 at 10:41:10PM +, Scot Doyle wrote:
> > Some machines, such as the Acer C720 and Toshiba CB35, have TPMs that do
> > not send IRQs while also having an ACPI TPM entry indicating that they
> > will b
Hi,
so shall I apply this patch? or not?
Thanks,
Peter
Am Dienstag, 2. Dezember 2014, 20:22:35 schrieb Ashley Lai:
> > @@ -63,9 +63,9 @@ static int ibmvtpm_send_crq(struct vio_dev *vdev, u64
> > w1, u64 w2) static struct ibmvtpm_dev *ibmvtpm_get_data(const struct
> > device *dev) {
> >
> >
Hi Anton,
is this patchset still needed after Vicky's patch
"[tpmdd-devel] Fix NULL return in tpm_ibmvtpm_get_desired_dma"
https://patchwork.ozlabs.org/patch/402315/
Ashley raised some concerns.
Since merge window is coming up, a fast reply is appreciated.
Thanks,
Peter
Am Freitag, 19. Septe
Am Freitag, 19. September 2014, 09:14:39 schrieb Kiran Padwal:
> Currently these driver are missing a check on the return value of
> devm_kzalloc, which would cause a NULL pointer dereference in a OOM
> situation.
>
> This patch adds a missing check for tpm_i2c_atmel.c and tpm_i2c_nuvoton.c
>
> S
Hi Jarkko,
Am Dienstag, 2. Dezember 2014, 23:31:12 schrieb Jarkko Sakkinen:
> This patch set enables TPM2 protocol and provides drivers for FIFO and
> CRB interfaces. This patch set does not export any sysfs attributes for
> TPM 2.0 because existing sysfs attributes have three non-trivial issues:
Am Dienstag, 2. Dezember 2014, 23:31:19 schrieb Jarkko Sakkinen:
> tpm_crb is a driver for TPM 2.0 Command Response Buffer (CRB) Interface
> as defined in PC Client Platform TPM Profile (PTP) Specification.
>
> Only polling and single locality is supported as these are the limitations
> of the ava
Am Dienstag, 2. Dezember 2014, 23:31:12 schrieb Jarkko Sakkinen:
> This patch set enables TPM2 protocol and provides drivers for FIFO and
> CRB interfaces. This patch set does not export any sysfs attributes for
> TPM 2.0 because existing sysfs attributes have three non-trivial issues:
>
> - They
Am Mittwoch, 3. Dezember 2014, 00:16:19 schrieb Peter Hüwe:
> Am Dienstag, 2. Dezember 2014, 23:31:12 schrieb Jarkko Sakkinen:
> > This patch set enables TPM2 protocol and provides drivers for FIFO and
> > CRB interfaces. This patch set does not export any sysfs attributes for
>
Am Mittwoch, 3. Dezember 2014, 00:24:43 schrieb Aaro Koskinen:
> Hi,
>
> On Wed, Dec 03, 2014 at 12:21:07AM +0100, Peter Hüwe wrote:
> > --- a/drivers/char/tpm/tpm_i2c_nuvoton.c
> > +++ b/drivers/char/tpm/tpm_i2c_nuvoton.c
> > @@ -605,10 +605,8 @@ static int i2c_nuvo
Am Dienstag, 5. Mai 2015, 18:32:19 schrieb Geert Uytterhoeven:
> The GPIO subsystem provides dummy GPIO consumer functions if GPIOLIB is
> not enabled. Hence drivers that depend on GPIOLIB, but use GPIO consumer
> functionality only, can still be compiled if GPIOLIB is not enabled.
>
> Relax the d
Am Mittwoch, 4. Februar 2015, 15:21:09 schrieb Jarkko Sakkinen:
> If during transmission system error was returned, the logic was to
> incorrectly deduce that chip is a TPM 1.x chip. This patch fixes this
> issue. Also, this patch changes probing so that message tag is used as the
> measure for TPM
Am Sonntag, 1. März 2015, 22:55:47 schrieb Jarkko Sakkinen:
> - tpm_dev_add_device(): cdev_add() must be done before uevent is
> propagated in order to avoid races.
> - tpm_chip_register(): tpm_dev_add_device() must be done as the
> last step before exposing device to the user space in order to
1 - 100 of 109 matches
Mail list logo
