From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <ard.biesheuvel@linaro.org>
Received: from mail-io0-x22c.google.com (mail-io0-x22c.google.com
 [IPv6:2607:f8b0:4001:c06::22c])
 (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 DA34F21D2E637
 for <edk2-devel@lists.01.org>; Sun, 20 Aug 2017 12:49:56 -0700 (PDT)
Received: by mail-io0-x22c.google.com with SMTP id y4so286756iod.4
 for <edk2-devel@lists.01.org>; Sun, 20 Aug 2017 12:52:27 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google;
 h=mime-version:in-reply-to:references:from:date:message-id:subject:to
 :cc; bh=HK1h+plnsqb3OsEkUvlldVUF+OYQWfVKR7sWIrAjdaA=;
 b=CWSQOTOCKwsyfBmsPSB2uIWhX80hdjO3z560SJ0YG2AQHLHiR3FbQd9YhOJOUADWeF
 8uNe+m7Kg5A7pWxyGR5eMD+/lklqPQklPpbXj/9BZuOGRc0cGv/K+/i69mVCaZQ2bYfp
 kwZ72GaPRnD8muor0XL590XD76xNHFTN/S0Es=
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=1e100.net; s=20161025;
 h=x-gm-message-state:mime-version:in-reply-to:references:from:date
 :message-id:subject:to:cc;
 bh=HK1h+plnsqb3OsEkUvlldVUF+OYQWfVKR7sWIrAjdaA=;
 b=LBDqUNQJzwLFnsW5/fbVsKh5AAk6GSgPj0JY2VEbn1q5mZR8JnbyMdEbPtpf1F+yHD
 r5vKXHGvcTE3fuTvC/09hUqm8FiXfhkquzpSrM+jcHhzsylvSEWJBsAZij8JsdYCuVC/
 LKpKwulu91otiQcvK0lhd5xGUM072iu9wRsN4GEe78KoQLIwGcLb1OQKzl9MpUc9hNdI
 yNe1bsL3dfL1X04hxjgOlR4Oz6N2IFW4B9jDSpRn0XMhmrEaB/uxNMGdXyjp6ntmySmo
 CONV51X9/NVn/kzS/SqiZoSf/EwKV072gUiVDAPLcDwyBanFEk5BTReov/D2ukMnVULJ
 1ifQ==
X-Gm-Message-State: AHYfb5h81uclDnRf+7Pz68ZdaM8jhBJM/ixYxQU4R1w9b36K8iTDgyB8
 qQPB133WqRyoq5rXVtTy87CoK1Ai5aqMUBE=
X-Received: by 10.107.175.167 with SMTP id p39mr13132076ioo.83.1503258747063; 
 Sun, 20 Aug 2017 12:52:27 -0700 (PDT)
MIME-Version: 1.0
Received: by 10.107.162.1 with HTTP; Sun, 20 Aug 2017 12:52:26 -0700 (PDT)
In-Reply-To: <20170820193335.21471-1-alan@softiron.com>
References: <20170820193335.21471-1-alan@softiron.com>
From: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Date: Sun, 20 Aug 2017 20:52:26 +0100
Message-ID: <CAKv+Gu9isON_wa-dcKSyJzCZ-aV+Zp=8H_+cQ7Z-mZzKqqSqVQ@mail.gmail.com>
To: Alan Ott <alan@softiron.com>
Cc: Leif Lindholm <leif.lindholm@linaro.org>, 
 "edk2-devel@lists.01.org" <edk2-devel@lists.01.org>,
 linaro-uefi <linaro-uefi@lists.linaro.org>
Subject: Re: [PATCH v2 edk2-platforms 0/3] Re-enable the second SATA controller on Overdrive
X-BeenThere: edk2-devel@lists.01.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: EDK II Development  <edk2-devel.lists.01.org>
List-Unsubscribe: <https://lists.01.org/mailman/options/edk2-devel>,
 <mailto:edk2-devel-request@lists.01.org?subject=unsubscribe>
List-Archive: <http://lists.01.org/pipermail/edk2-devel/>
List-Post: <mailto:edk2-devel@lists.01.org>
List-Help: <mailto:edk2-devel-request@lists.01.org?subject=help>
List-Subscribe: <https://lists.01.org/mailman/listinfo/edk2-devel>,
 <mailto:edk2-devel-request@lists.01.org?subject=subscribe>
X-List-Received-Date: Sun, 20 Aug 2017 19:49:57 -0000
Content-Type: text/plain; charset="UTF-8"

On 20 August 2017 at 20:33, Alan Ott <alan@softiron.com> wrote:
> Without the PCD for the second SATA Controller being specified, the boot
> will hang. These patches fix it.
>
> Alan Ott (3):
>   Silicon/AMD/Styx: Make PcdSataPortMode 32 bits
>   Silicon/AMD/Styx: Use PcdSataPortMode properly for two controllers
>   Platform/AMD/OverdriveBoard: Re-enable the second SATA Controller
>

Reviewed-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>