From patchwork Tue Jan 19 06:23:08 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: David Gibson X-Patchwork-Id: 366125 Delivered-To: patch@linaro.org Received: by 2002:a02:a60d:0:0:0:0:0 with SMTP id c13csp469130jam; Mon, 18 Jan 2021 22:24:16 -0800 (PST) X-Google-Smtp-Source: ABdhPJzzQfrQmhlLrcizqM4cN+eMNpRb8Ng+WI+sqEE1KV8jEPKUTDLDMDBYmEhvIb3BVBHHdSKI X-Received: by 2002:a25:3457:: with SMTP id b84mr3569373yba.167.1611037456480; Mon, 18 Jan 2021 22:24:16 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1611037456; cv=none; d=google.com; s=arc-20160816; b=NzP3ZLCMmhx3cCeX1eq1X0CQ1yJwzvp0JJaqLbW8MIjnxQCBAdKBk13jjpdriDsNsP ehKakJV1x1O5xJB9wY6WqJ+0WvU5p4tLZlzdRUBYfnEHyjbIK75uxR/vxlPIdFJuaCc+ aHw4akVHfyEfwhPqsio3lRp4BeEGRSKmg1OxdeTfZ066pF+owDjOO1ICDaM7YfeTfGNR TJAf/CGLodDAWj3yoJsKynpd23Eq45NFfc7TQKdHbxTMeDcarb1eU5DAFrgt/o0Tbh1o bxAGs/yZVEfhFhkfR9yblCFQE/1eMXneO2JP73JzS623SgtB3ED+HLLcmcPjVtJSpr6G /acQ== 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:references:in-reply-to:message-id:date:subject:to:from :dkim-signature; bh=BCsSx34zg7/Oku2U1zUWWvdDneM9dlXX9wyiIe4hYWw=; b=ewDsTDA61bpBJZygrMJEQPILFGyFhUYnwzIkrpJUHwq0UFZjniydyc6qflbyQmCP2o V5WI0AY5Bqz++CRgpfh3y3XipKoD4POosa9qqgLDnD9MdG9zyl5hbOU/BQFTIwFh42Zz Ik6AcZzV1z5lEJj8XNSONLmWIWHfRsfCAOvomslunBQuNvTaX/UDqrpNtpDvPNRP1ODi gMoCWTw4XhWE8J7qWKjyY5cU/Ks59PRBl2chxevbd/1PQoIDXC4I5UUhRytiCOkaIypH t4F5JHeCgcfBuD+VkIsYTkKV3zr6eLYFYJOmIFZ/fMhSpre7iSO8ON7ZqyRSCSGvzB8J 03Rw== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@gibson.dropbear.id.au header.s=201602 header.b=MSpQhUPx; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org" Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id b7si19208611ybk.193.2021.01.18.22.24.16 for (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Mon, 18 Jan 2021 22:24:16 -0800 (PST) Received-SPF: pass (google.com: domain of qemu-devel-bounces+patch=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=@gibson.dropbear.id.au header.s=201602 header.b=MSpQhUPx; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org" Received: from localhost ([::1]:58164 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1l1kR5-0001Dp-Tf for patch@linaro.org; Tue, 19 Jan 2021 01:24:15 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:38850) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1l1kQb-0001D6-Dh; Tue, 19 Jan 2021 01:23:45 -0500 Received: from ozlabs.org ([203.11.71.1]:36789) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1l1kQW-0000LS-A3; Tue, 19 Jan 2021 01:23:45 -0500 Received: by ozlabs.org (Postfix, from userid 1007) id 4DKdrW37SDz9sWV; Tue, 19 Jan 2021 17:23:23 +1100 (AEDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gibson.dropbear.id.au; s=201602; t=1611037403; bh=xxmvz4OB5N2cFqIicRajYQ8j7Dwknrym5stSSTgyNMU=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=MSpQhUPx3pS26iGfF5kF8z2fVVCvDI3LY9EEXmJ8FKJ6Am9RpY+sAcUCGQuWxZc37 bhw6GpAgzqt+o26wtM1cGWwIwE1zMepMaSSWrET/tNvlhrbXwRi9f88feGBvLjzRtj eS3bud1KxANymPTf9keQ5IwVC428HX8s/uIhBfl8= From: David Gibson To: peter.maydell@linaro.org, groug@kaod.org Subject: [PULL 03/13] hw/intc/ppc-uic: Make default dcr-base 0xc0, not 0x30 Date: Tue, 19 Jan 2021 17:23:08 +1100 Message-Id: <20210119062318.13857-4-david@gibson.dropbear.id.au> X-Mailer: git-send-email 2.29.2 In-Reply-To: <20210119062318.13857-1-david@gibson.dropbear.id.au> References: <20210119062318.13857-1-david@gibson.dropbear.id.au> MIME-Version: 1.0 Received-SPF: pass client-ip=203.11.71.1; envelope-from=dgibson@ozlabs.org; helo=ozlabs.org X-Spam_score_int: -17 X-Spam_score: -1.8 X-Spam_bar: - X-Spam_report: (-1.8 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HEADER_FROM_DIFFERENT_DOMAINS=0.248, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001 autolearn=no autolearn_force=no X-Spam_action: no action 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: Nathan Chancellor , David Gibson , qemu-ppc@nongnu.org, qemu-devel@nongnu.org Errors-To: qemu-devel-bounces+patch=linaro.org@nongnu.org Sender: "Qemu-devel" From: Peter Maydell In commit 34d0831f38fd8 the ppc-uic device was added, with a dcr-base property. The intention was that the default value of dcr-base should be the one that most of our boards need, so that in the common case they don't need to specify a property value. All QEMU boards with a UIC use a dcr-base of 0xc0, with the exception of sam460ex which has four UICs and so puts them at 0xc0, 0xd0, 0xe0, 0xf0. So 0xc0 is the obvious right choice for the default dcr-base. The board code conversions in commits 0270d74ef88623505 (bamboo) and c5ac9dc64fa552a6 (virtex_ml507) assumed that default was 0xc0. Unfortunately the actual default in 34d0831f38fd8 was 0x30, by mistake, so the bamboo and virtex_ml507 boards were broken as they were converted away from ppcuic_init() (which always specifies the dcr_base property value explicitly). Set the default dcr-base to 0xc0 as was intended, fixing bamboo and virtex_ml507. Fixes: 34d0831f38fd8 Reported-by: Nathan Chancellor Suggested-by: BALATON Zoltan Signed-off-by: Peter Maydell Message-Id: <20210111213007.7381-1-peter.maydell@linaro.org> Signed-off-by: David Gibson --- hw/intc/ppc-uic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.29.2 diff --git a/hw/intc/ppc-uic.c b/hw/intc/ppc-uic.c index b21951eea8..7171de7b35 100644 --- a/hw/intc/ppc-uic.c +++ b/hw/intc/ppc-uic.c @@ -274,7 +274,7 @@ static void ppc_uic_realize(DeviceState *dev, Error **errp) static Property ppc_uic_properties[] = { DEFINE_PROP_LINK("cpu", PPCUIC, cpu, TYPE_CPU, CPUState *), - DEFINE_PROP_UINT32("dcr-base", PPCUIC, dcr_base, 0x30), + DEFINE_PROP_UINT32("dcr-base", PPCUIC, dcr_base, 0xc0), DEFINE_PROP_BOOL("use-vectors", PPCUIC, use_vectors, true), DEFINE_PROP_END_OF_LIST() };