From mboxrd@z Thu Jan 1 00:00:00 1970 Received: by 2002:ac2:52b1:0:0:0:0:0 with SMTP id r17csp5325678lfm; Tue, 28 Apr 2020 03:37:24 -0700 (PDT) X-Google-Smtp-Source: APiQypJAU8XZJAhfT1gPH+eaCu4j2vnGP0FQBBKT6s3SuJEfgUuobGK/PNfRtb9bcLrzDyPd+RnH X-Received: by 2002:ae9:e214:: with SMTP id c20mr27315169qkc.107.1588070243864; Tue, 28 Apr 2020 03:37:23 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1588070243; cv=none; d=google.com; s=arc-20160816; b=owEwDUW2huVoKVPsLWcrDY4ofiO25wUXeFYY3sXWhezaHGGANKdH4XKgSQpPFV5tcX BzvztCohU+G5JqsVHoy63IxAJlaKFpVk+HzINRbs56dAUHdyN3/6ugnID0F/W5QKqkbt p9fh0vY6sFJOBbRBw77TaQEJogc7eix7m9NP39PA4kygxB4nLJDoc/H2zWY78bWla/lj HTWToOwnMnT+JgCS8iGsQpY33GzuSOqNcGahF7cDtXSwJOSDbzJ9EPU9ksHZNak3oNVI BRwWd08439TPdvearN/5BuEaEq+/r1mKXOp77uY0e5jqxEFEevzb6GrnGdSD8HkxhDPz jYEQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:cc:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:content-transfer-encoding :mime-version:organization:references:in-reply-to:message-id:subject :to:from:date:dkim-signature; bh=AcXSHJlO6VB+UzxHJHspmsvLtz82Ki+Jm/9V2p6dgto=; b=eqbOa49HZ4bsdVaCtTtPiFg0wd1BhwAMERPrcH4orsxCecpkHbRNojBCBeh5ZqLd7f KuZRq6SqbzHMlalqemjAU7mhNEx9ICj01RShXLVzCQuxWFiqSSFYx/kV7QUuimRd2zPc FcwDVgUVu1pmPL18aH9uCFAW8Hdu4vbc8Oe+szzW+WYcBH2aFCD8kYnEedw+AMVOOPqI UL8jvzAysfWn927CjGnni9GotNgF9Z7hH1C6dINyPIcJxK7qlGmssvacBVCa6ceTThht fUu/iwrMZJ4eywtJuGu24KTlfvIK2EG6A7ReuovT8F0SlXXEAjcorjG7F2CraXRqGUVH zRxw== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@redhat.com header.s=mimecast20190719 header.b=DeBX8LiI; spf=pass (google.com: domain of qemu-devel-bounces+alex.bennee=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+alex.bennee=linaro.org@nongnu.org"; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=redhat.com Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id z5si10250781qtd.93.2020.04.28.03.37.23 for (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Tue, 28 Apr 2020 03:37:23 -0700 (PDT) Received-SPF: pass (google.com: domain of qemu-devel-bounces+alex.bennee=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; Authentication-Results: mx.google.com; dkim=fail header.i=@redhat.com header.s=mimecast20190719 header.b=DeBX8LiI; spf=pass (google.com: domain of qemu-devel-bounces+alex.bennee=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+alex.bennee=linaro.org@nongnu.org"; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=redhat.com Received: from localhost ([::1]:52274 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jTNcB-0003QC-9Y for alex.bennee@linaro.org; Tue, 28 Apr 2020 06:37:23 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:44666) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jTNZh-0001e0-VY for qemu-devel@nongnu.org; Tue, 28 Apr 2020 06:34:53 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.90_1) (envelope-from ) id 1jTNZg-0002d2-M1 for qemu-devel@nongnu.org; Tue, 28 Apr 2020 06:34:49 -0400 Received: from us-smtp-1.mimecast.com ([207.211.31.81]:50913 helo=us-smtp-delivery-1.mimecast.com) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.90_1) (envelope-from ) id 1jTNZg-0002T9-46 for qemu-devel@nongnu.org; Tue, 28 Apr 2020 06:34:48 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1588070086; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=AcXSHJlO6VB+UzxHJHspmsvLtz82Ki+Jm/9V2p6dgto=; b=DeBX8LiIKxjC+Y3JNrQ3cOYiUYwBe99Q8uOpIHo9xrbNVlHboPgxjkcYEOXDVvd8miqtM6 /xpCH5+KYll9zsGC+orkCWPhzL7QFKURaerZ3vw41xVYUKQHizoFgDffD45/vPa2XdnlYR qPUf8Qvxqj+nQBY/tULrTvALFEeFZK0= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-467-Ch0k9yhxO_qBiGsUqf7KUA-1; Tue, 28 Apr 2020 06:34:44 -0400 X-MC-Unique: Ch0k9yhxO_qBiGsUqf7KUA-1 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 63DBD80058A; Tue, 28 Apr 2020 10:34:43 +0000 (UTC) Received: from gondolin (ovpn-112-178.ams2.redhat.com [10.36.112.178]) by smtp.corp.redhat.com (Postfix) with ESMTP id C535E614CC; Tue, 28 Apr 2020 10:34:39 +0000 (UTC) Date: Tue, 28 Apr 2020 12:34:36 +0200 From: Cornelia Huck To: Eric Auger Subject: Re: [PATCH 1/2] tpm: tpm-tis-device: set PPI to false by default Message-ID: <20200428123436.75432a8e.cohuck@redhat.com> In-Reply-To: <20200427143145.16251-2-eric.auger@redhat.com> References: <20200427143145.16251-1-eric.auger@redhat.com> <20200427143145.16251-2-eric.auger@redhat.com> Organization: Red Hat GmbH MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Received-SPF: pass client-ip=207.211.31.81; envelope-from=cohuck@redhat.com; helo=us-smtp-delivery-1.mimecast.com X-detected-operating-system: by eggs.gnu.org: First seen = 2020/04/28 04:11:46 X-ACL-Warn: Detected OS = Linux 2.2.x-3.x [generic] [fuzzy] X-Received-From: 207.211.31.81 X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: peter.maydell@linaro.org, drjones@redhat.com, qemu-devel@nongnu.org, qemu-arm@nongnu.org, eric.auger.pro@gmail.com, stefanb@linux.ibm.com Errors-To: qemu-devel-bounces+alex.bennee=linaro.org@nongnu.org Sender: "Qemu-devel" X-TUID: 9eQ276Z7lFWy On Mon, 27 Apr 2020 16:31:44 +0200 Eric Auger wrote: > The tpm-tis-device device does not support PPI. Let's > change the default value for the corresponding property > instead of tricking this latter in the mach-virt machine. > > Signed-off-by: Eric Auger > --- > hw/tpm/tpm_tis_sysbus.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/hw/tpm/tpm_tis_sysbus.c b/hw/tpm/tpm_tis_sysbus.c > index 18c02aed67..eced1fc843 100644 > --- a/hw/tpm/tpm_tis_sysbus.c > +++ b/hw/tpm/tpm_tis_sysbus.c > @@ -91,7 +91,7 @@ static void tpm_tis_sysbus_reset(DeviceState *dev) > static Property tpm_tis_sysbus_properties[] = { > DEFINE_PROP_UINT32("irq", TPMStateSysBus, state.irq_num, TPM_TIS_IRQ), > DEFINE_PROP_TPMBE("tpmdev", TPMStateSysBus, state.be_driver), > - DEFINE_PROP_BOOL("ppi", TPMStateSysBus, state.ppi_enabled, true), > + DEFINE_PROP_BOOL("ppi", TPMStateSysBus, state.ppi_enabled, false), > DEFINE_PROP_END_OF_LIST(), > }; > This looks like a better place to do this than in the virt compat machines, and should get us the same result, leaving compatibility intact. Reviewed-by: Cornelia Huck