diff mbox

[tpmdd-devel,08/10] devicetree: Add Trusted Computing Group to vendor-prefix.txt

Message ID 1460323386-16892-9-git-send-email-christophe-h.ricard@st.com
State New
Headers show

Commit Message

Christophe Ricard April 10, 2016, 9:23 p.m. UTC
Add missing vendor to vendor-prefix.txt.

Trusted Computing Group design common specifications for
TPM (Trusted Platform Module) vendors.
TCG designates a TPM answering to a public specification.

Cc: devicetree@vger.kernel.org
Signed-off-by: Christophe Ricard <christophe-h.ricard@st.com>
---
 Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
 1 file changed, 1 insertion(+)

Comments

Rob Herring April 11, 2016, 8:06 p.m. UTC | #1
On Sun, Apr 10, 2016 at 11:23:04PM +0200, Christophe Ricard wrote:
> Add missing vendor to vendor-prefix.txt.
> 
> Trusted Computing Group design common specifications for
> TPM (Trusted Platform Module) vendors.
> TCG designates a TPM answering to a public specification.

Not sure this should be a vendor prefix. Does TCG actually make h/w?

> 
> Cc: devicetree@vger.kernel.org
> Signed-off-by: Christophe Ricard <christophe-h.ricard@st.com>
> ---
>  Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
> index 6c3d970..717bae9 100644
> --- a/Documentation/devicetree/bindings/vendor-prefixes.txt
> +++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
> @@ -230,6 +230,7 @@ stericsson	ST-Ericsson
>  synology	Synology, Inc.
>  tbs	TBS Technologies
>  tcl	Toby Churchill Ltd.
> +tcg	Trusted Computing Group
>  technologic	Technologic Systems
>  thine	THine Electronics, Inc.
>  ti	Texas Instruments
> -- 
> 2.1.4
> 
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

------------------------------------------------------------------------------
Find and fix application performance issues faster with Applications Manager
Applications Manager provides deep performance insights into multiple tiers of
your business applications. It resolves application problems quickly and
reduces your MTTR. Get your free trial!
https://ad.doubleclick.net/ddm/clk/302982198;130105516;z
Jason Gunthorpe April 11, 2016, 8:21 p.m. UTC | #2
On Mon, Apr 11, 2016 at 03:06:00PM -0500, Rob Herring wrote:
> On Sun, Apr 10, 2016 at 11:23:04PM +0200, Christophe Ricard wrote:
> > Add missing vendor to vendor-prefix.txt.
> > 
> > Trusted Computing Group design common specifications for
> > TPM (Trusted Platform Module) vendors.
> > TCG designates a TPM answering to a public specification.
> 
> Not sure this should be a vendor prefix. Does TCG actually make h/w?

They define the common multi-vendor register API, similar to AHCI.

I would imagine compatible tags like

 compatible = 'winbond,xxxx','tcg,tis_iomap';

Since there actually is a legal entity administering the spec (unlike
AHCI?) 'tgc,' seems like a better prefix to me than 'generic-' ?

Jason

------------------------------------------------------------------------------
Find and fix application performance issues faster with Applications Manager
Applications Manager provides deep performance insights into multiple tiers of
your business applications. It resolves application problems quickly and
reduces your MTTR. Get your free trial!
https://ad.doubleclick.net/ddm/clk/302982198;130105516;z
Rob Herring April 11, 2016, 10:37 p.m. UTC | #3
On Mon, Apr 11, 2016 at 3:21 PM, Jason Gunthorpe
<jgunthorpe@obsidianresearch.com> wrote:
> On Mon, Apr 11, 2016 at 03:06:00PM -0500, Rob Herring wrote:
>> On Sun, Apr 10, 2016 at 11:23:04PM +0200, Christophe Ricard wrote:
>> > Add missing vendor to vendor-prefix.txt.
>> >
>> > Trusted Computing Group design common specifications for
>> > TPM (Trusted Platform Module) vendors.
>> > TCG designates a TPM answering to a public specification.
>>
>> Not sure this should be a vendor prefix. Does TCG actually make h/w?
>
> They define the common multi-vendor register API, similar to AHCI.
>
> I would imagine compatible tags like
>
>  compatible = 'winbond,xxxx','tcg,tis_iomap';
>
> Since there actually is a legal entity administering the spec (unlike
> AHCI?) 'tgc,' seems like a better prefix to me than 'generic-' ?

Well, Intel "manages" the AHCI and EHCI specs. All the generic strings
are somewhat useless in the end IMO.

As long as you aren't planning to use it alone, it is fine.

Acked-by: Rob Herring <robh@kernel.org>

Rob

------------------------------------------------------------------------------
Find and fix application performance issues faster with Applications Manager
Applications Manager provides deep performance insights into multiple tiers of
your business applications. It resolves application problems quickly and
reduces your MTTR. Get your free trial!
https://ad.doubleclick.net/ddm/clk/302982198;130105516;z
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
index 6c3d970..717bae9 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.txt
+++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
@@ -230,6 +230,7 @@  stericsson	ST-Ericsson
 synology	Synology, Inc.
 tbs	TBS Technologies
 tcl	Toby Churchill Ltd.
+tcg	Trusted Computing Group
 technologic	Technologic Systems
 thine	THine Electronics, Inc.
 ti	Texas Instruments