Re: [PATCH v2 0/2] char: tpm: add new driver for tpm i2c ptp

From: Tomer Maimon <tmaimon77@gmail.com>
To: Oshri Alkobi <oshrialkoby85@gmail.com>,
	Alexander Steffen <Alexander.Steffen@infineon.com>,
	Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
Cc: Rob Herring <robh+dt@kernel.org>,
	Mark Rutland <mark.rutland@arm.com>,
	peterhuewe@gmx.de, jgg@ziepe.ca, Arnd Bergmann <arnd@arndb.de>,
	Greg KH <gregkh@linuxfoundation.org>,
	IS20 Oshri Alkoby <oshri.alkoby@nuvoton.com>,
	devicetree <devicetree@vger.kernel.org>,
	AP MS30 Linux Kernel community <linux-kernel@vger.kernel.org>,
	linux-integrity@vger.kernel.org, gcwilson@us.ibm.com,
	kgoldman@us.ibm.com, nayna@linux.vnet.ibm.com,
	IS30 Dan Morav <Dan.Morav@nuvoton.com>,
	eyal.cohen@nuvoton.com
Subject: Re: [PATCH v2 0/2] char: tpm: add new driver for tpm i2c ptp
Date: Mon, 15 Jul 2019 11:08:47 +0300	[thread overview]
Message-ID: <CAP6Zq1hPo9dG71YFyr7z9rjmi-DvoUZJOme4+2uqsfO+7nH+HQ@mail.gmail.com> (raw)
In-Reply-To: <6e7ff1b958d84f6e8e585fd3273ef295@NTILML02.nuvoton.com>

[-- Attachment #1: Type: text/plain, Size: 3418 bytes --]

Hi Jarkko and All,

Thanks for your feedback and sorry for the late response.

Due to the amount of work required to handle this technical feedback and
project constraints we need to put this task on hold for the near future.
In the meantime, anyone from the community is welcome to take over this
code and handle the re-design for the benefit of the entire TPM community.

Thanks again,

Tomer

On Mon, 15 Jul 2019 at 10:54, <tomer.maimon@nuvoton.com> wrote:

>
>
> -----Original Message-----
> From: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
> Sent: Friday, 5 July 2019 14:28
> To: Oshri Alkobi <oshrialkoby85@gmail.com>; Alexander Steffen <
> Alexander.Steffen@infineon.com>
> Cc: robh+dt@kernel.org; mark.rutland@arm.com; peterhuewe@gmx.de;
> jgg@ziepe.ca; arnd@arndb.de; gregkh@linuxfoundation.org; IS20 Oshri
> Alkoby <oshri.alkoby@nuvoton.com>; devicetree@vger.kernel.org; AP MS30
> Linux Kernel community <linux-kernel@vger.kernel.org>;
> linux-integrity@vger.kernel.org; gcwilson@us.ibm.com; kgoldman@us.ibm.com;
> nayna@linux.vnet.ibm.com; IS30 Dan Morav <Dan.Morav@nuvoton.com>; IS20
> Tomer Maimon <tomer.maimon@nuvoton.com>
> Subject: Re: [PATCH v2 0/2] char: tpm: add new driver for tpm i2c ptp
>
> On Thu, 2019-07-04 at 12:48 -0500, Oshri Alkobi wrote:
> > Alex, Jarkko, thank you very much for your feedbacks!
>
> Please configure your email client to use plain text.
>
> > I totally agree, there are some duplications that can be common,
> > indeed it will require some work in tpm_tis_core.
> > Since I believe it is not going to happen soon, I would suggest to
> > examine what duplications can currently be dropped from the new
> > driver, so the kernel will support the PTP I2C interface in the meantime.
> > I will appreciate getting ideas about any tpm_tis_core logic that
> > currently can be used as is by the new drive.
>
> I rather wait for a solution that integrates with our mature stack for TIS
> (or these days FIFO) than integrate something half-baked. If you want
> something in, please do right things right.
>
> What you are proposing would mean maintaining duplicate stacks forever.
>
> > Since the TIS is an old specification that mostly defines FIFO for
> > TPM1.2 I would say the name tpm_tis_i2c does not completely reflect
> > its goal. However we really don't have any problem with any name that
> the group will agree on.
> > Does tpm_ptp_i2c sound better than the current name?
>
> Absolutely not going to use that name. The naming convention is what it is
> for other drivers that are adapt tpm_tis_core to different HW interfaces.
>
> /Jarkko
>
>
>
>
> ===========================================================================================
> The privileged confidential information contained in this email is
> intended for use only by the addressees as indicated by the original sender
> of this email. If you are not the addressee indicated in this email or are
> not responsible for delivery of the email to such a person, please kindly
> reply to the sender indicating this fact and delete all copies of it from
> your computer and network server immediately. Your cooperation is highly
> appreciated. It is advised that any unauthorized use of confidential
> information of Nuvoton is strictly prohibited; and any information in this
> email irrelevant to the official business of Nuvoton shall be deemed as
> neither given nor endorsed by Nuvoton.
>

[-- Attachment #2: Type: text/html, Size: 5821 bytes --]