From patchwork Wed Dec 6 20:31:30 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Leif Lindholm X-Patchwork-Id: 120905 Delivered-To: patch@linaro.org Received: by 10.80.152.193 with SMTP id j59csp3908963edb; Wed, 6 Dec 2017 12:31:38 -0800 (PST) X-Google-Smtp-Source: AGs4zMZNaS6foL9q589bT6F9dihCVKSrvWEpOEsm0Z/a7VB9VSVy47ePHnb0D+dcE9qsh1S0aF24 X-Received: by 10.84.168.101 with SMTP id e92mr23152917plb.34.1512592298074; Wed, 06 Dec 2017 12:31:38 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1512592298; cv=none; d=google.com; s=arc-20160816; b=S4pHC6N6PyBLR8M0P2IIGsHsBy4MJE3cQTXEazBAXRa2YrMwd1Rv5P4rlRvk0tvE9v 2XfPg6JwrUQniw/sF6tCVBcJ1a++aYmczZUXSGQkukaFQxZDXacC9OH7dt1wtzJvLkzU +ouZfwPgIAqs/n9WDHQD0VvepOcdVzoi4XNszjmICqvrTfei5mLNkJjlghn2C4rNFtCQ dLKwRxIQZOEHnas549DR2PAIUyissKGBe+L8WA9sZT+0LdY4l8tKamXPtFqslcmebjk0 sGiEpQzAs7zTqi5VMHRpFmA0PFSkI7JpTcIFWYKI9ZiYWzsRJYbvReWow3WGmqyglFcm 6MMw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:content-transfer-encoding:mime-version:cc :list-subscribe:list-help:list-post:list-archive:list-unsubscribe :list-id:precedence:subject:message-id:date:to:from:dkim-signature :delivered-to:arc-authentication-results; bh=DWrXw68iFT1pcmf+AlFf9zo88nJq5xcBEbSbUlVw12s=; b=QEK+7mmW0pTm06F18//xehtUp1K7aw7D4XF4nDH9wY0rt4CHjKjzjss3IixinTZiCB g2Aux5Ea8ubqaJ1hccsBKQ60dgFxJrdRbOMxkH5TqjkIe/yzG2kvhDbtVSmV+Ex+m6Hk 0Tg1h4KgkhN2BbscpBUsBlWtvCoZCM/Cneij8iHmzjdOt+EuUpbYm6P9CFSM007cHVSQ RCaxBqnosPcpUl+ToIIUELUqU4bnZB4KKiF/w2P5Izmd2BqBFBeCTiqya52Kt3k93tI7 rUHb1beHr4kr3s5OOSyQn30dBQFSu+zLu2te7Q++wPc385Xg7LYt8359Qxw6fxja79Qa /N+A== ARC-Authentication-Results: i=1; mx.google.com; dkim=neutral (body hash did not verify) header.i=@linaro.org header.s=google header.b=jChXovON; spf=pass (google.com: best guess record for domain of edk2-devel-bounces@lists.01.org designates 2001:19d0:306:5::1 as permitted sender) smtp.mailfrom=edk2-devel-bounces@lists.01.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from ml01.01.org (ml01.01.org. [2001:19d0:306:5::1]) by mx.google.com with ESMTPS id 74si2697612pfk.175.2017.12.06.12.31.37 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 06 Dec 2017 12:31:38 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of edk2-devel-bounces@lists.01.org designates 2001:19d0:306:5::1 as permitted sender) client-ip=2001:19d0:306:5::1; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@linaro.org header.s=google header.b=jChXovON; spf=pass (google.com: best guess record for domain of edk2-devel-bounces@lists.01.org designates 2001:19d0:306:5::1 as permitted sender) smtp.mailfrom=edk2-devel-bounces@lists.01.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id 7EDFF2217CE5F; Wed, 6 Dec 2017 12:27:04 -0800 (PST) X-Original-To: edk2-devel@lists.01.org Delivered-To: edk2-devel@lists.01.org Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2a00:1450:400c:c0c::243; helo=mail-wr0-x243.google.com; envelope-from=leif.lindholm@linaro.org; receiver=edk2-devel@lists.01.org Received: from mail-wr0-x243.google.com (mail-wr0-x243.google.com [IPv6:2a00:1450:400c:c0c::243]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 95A792217CE28 for ; Wed, 6 Dec 2017 12:27:02 -0800 (PST) Received: by mail-wr0-x243.google.com with SMTP id l22so5221019wrc.11 for ; Wed, 06 Dec 2017 12:31:34 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id; bh=dKST8WWlzo1lxxMcxHcw1q8pnl/rmbxc5xwcDqYi5Xw=; b=jChXovONlB7GYIZrnRYkddSeGmj/rAEpp3wIKZhbOss+Z9LVEvnMoi46d0Opl1KsV3 aOSu5Rl8DlT4o1sCrpRfwLMbZd66WY/kG4PKquKymSL1bve6aHKCFwGHKVKq8NrcFCru xvPSOSF+pvRsN44Mnm2K4oKXBovQgOyO8P1bI= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=dKST8WWlzo1lxxMcxHcw1q8pnl/rmbxc5xwcDqYi5Xw=; b=ZvauQytLJ7ehcyrdc4KuFTipU1hyqMEve0P6IC6I4RizKeDdxfIHSDNn+qitUpdG0K ccJssvjS+30nx3CtiZ2fZ/xoCseXl90nth2UEOTMJyHFe/kz6JAWqfl0CqI4wFCwt4ms YUPSF+k31KC+5btC4sBYYGPP8e+TXNz7uZ5Zc/oq+VNn8DjN1+sDL1u17x06AbTtpeI5 8phXC4IZVSIVolIKLltKwEbEb32hBtrTG6PM2XgARL0NAVhw++O4/ISCRO5XMRiv7Xah WKyhKXvhDXQ9YqpV1Ao6TOFIWWsXui8vtnMJlBdupKNL4EQ8FNLJtANGYRnpzIWsQWHS uUGA== X-Gm-Message-State: AJaThX4Xr5pRTkUIuVODocEctBnUUAjiSte6/s6y1PnW1m8SbH1f/iRu 28IvyL/zkzkC09aGlhd+fTL+J1Hq7Dw= X-Received: by 10.223.174.211 with SMTP id y77mr20281444wrc.196.1512592292658; Wed, 06 Dec 2017 12:31:32 -0800 (PST) Received: from vanye.hemma.eciton.net (cpc92316-cmbg19-2-0-cust118.5-4.cable.virginm.net. [82.12.0.119]) by smtp.gmail.com with ESMTPSA id x75sm4009085wme.44.2017.12.06.12.31.31 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 06 Dec 2017 12:31:31 -0800 (PST) From: Leif Lindholm To: edk2-devel@lists.01.org Date: Wed, 6 Dec 2017 20:31:30 +0000 Message-Id: <20171206203130.8219-1-leif.lindholm@linaro.org> X-Mailer: git-send-email 2.11.0 Subject: [edk2] [PATCH] EmbeddedPkg: AndroidBootApp: fix clang compilation X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: ard.biesheuvel@linaro.org MIME-Version: 1.0 Errors-To: edk2-devel-bounces@lists.01.org Sender: "edk2-devel" Address an incorrect function prototype (using ; instead of ,) in AndroidBootImg.h. Also restructure code slightly to avoid a "may be used uninitialized" warning. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Leif Lindholm --- EmbeddedPkg/Include/Protocol/AndroidBootImg.h | 4 ++-- EmbeddedPkg/Library/AndroidBootImgLib/AndroidBootImgLib.c | 9 +++++---- 2 files changed, 7 insertions(+), 6 deletions(-) -- 2.11.0 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel Reviewed-by: Ard Biesheuvel diff --git a/EmbeddedPkg/Include/Protocol/AndroidBootImg.h b/EmbeddedPkg/Include/Protocol/AndroidBootImg.h index 1c458d0255..cd3c42e858 100644 --- a/EmbeddedPkg/Include/Protocol/AndroidBootImg.h +++ b/EmbeddedPkg/Include/Protocol/AndroidBootImg.h @@ -33,8 +33,8 @@ EFI_STATUS typedef EFI_STATUS (EFIAPI *ANDROID_BOOTIMG_UPDATE_DTB) ( - IN EFI_PHYSICAL_ADDRESS OrigDtbBase; - OUT EFI_PHYSICAL_ADDRESS *NewDtbBase; + IN EFI_PHYSICAL_ADDRESS OrigDtbBase, + OUT EFI_PHYSICAL_ADDRESS *NewDtbBase ); struct _ANDROID_BOOTIMG_PROTOCOL { diff --git a/EmbeddedPkg/Library/AndroidBootImgLib/AndroidBootImgLib.c b/EmbeddedPkg/Library/AndroidBootImgLib/AndroidBootImgLib.c index 09c4d924f1..2e50cedf6a 100644 --- a/EmbeddedPkg/Library/AndroidBootImgLib/AndroidBootImgLib.c +++ b/EmbeddedPkg/Library/AndroidBootImgLib/AndroidBootImgLib.c @@ -357,12 +357,13 @@ AndroidBootImgUpdateFdt ( if (EFI_ERROR (Status)) { goto Fdt_Exit; } + + Status = gBS->InstallConfigurationTable ( + &gFdtTableGuid, + (VOID *)(UINTN)NewFdtBase + ); } - Status = gBS->InstallConfigurationTable ( - &gFdtTableGuid, - (VOID *)(UINTN)NewFdtBase - ); if (!EFI_ERROR (Status)) { return EFI_SUCCESS; }