From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr0-x234.google.com (mail-wr0-x234.google.com [IPv6:2a00:1450:400c:c0c::234]) (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 A004A21D2E625 for ; Mon, 21 Aug 2017 07:10:17 -0700 (PDT) Received: by mail-wr0-x234.google.com with SMTP id f8so64201457wrf.3 for ; Mon, 21 Aug 2017 07:12:49 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=9jKPVreHkJMdtzvNyCFUAs4AxkA/V67sS+BNSqSSydw=; b=O/iN6RiECiNoZx4ZJ9q90jISxB03QwcSwdisVIFq9An7Ny9UkCp+/O0nZaRde8mbgo OL0kHVFDScGhog7gW9wARWUb247sZUwb9PJfE5vs5KWKjVNc1ONTy75L6lHPdAFtMujd TXmwR3ziCa9CYwIWx+Fo8bX8majfHXnM0femM= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=9jKPVreHkJMdtzvNyCFUAs4AxkA/V67sS+BNSqSSydw=; b=NiuBv84lXHucgA76CZck4j4aK17dVfKAo7JkRUU2rw7fbfdGmSNM4bAoLKnZT8RDuy rfcMTXl/dPUg40S+16gAvPDYUcjBCfppmwq2oJnT/NanuU0c/Nu9yrAktUyRliqRtkpE 4/GycngVb1tjYynHx8172rC43LzmaUrC6gXU1s/gI8Dy3XuCcbr0j+tDHid2QrYQhFXW hWAhET5CNueRqaI8gHePAMOtAzyENOksWEaABifogneTzDXm6CkSz1v+BBxVniXjIZDH ePHytNNgu0DDeB0rI0Nyb9xDTg5W3VYUMXllug2VqA9KwyUAq3KXE8MJF3EuOIgBDHXF Mh9A== X-Gm-Message-State: AHYfb5iGYmrVWK6YtcOktvP9iok9SR3vYRelSKnBVF/67jKERR9732LX 5gF0po5hrLD6m5KD X-Received: by 10.223.128.47 with SMTP id 44mr11887316wrk.175.1503324767049; Mon, 21 Aug 2017 07:12:47 -0700 (PDT) Received: from bivouac.eciton.net (bivouac.eciton.net. [2a00:1098:0:86:1000:23:0:2]) by smtp.gmail.com with ESMTPSA id t13sm2205032wrg.27.2017.08.21.07.12.45 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Mon, 21 Aug 2017 07:12:46 -0700 (PDT) Date: Mon, 21 Aug 2017 15:12:43 +0100 From: Leif Lindholm To: Alan Ott Cc: ard.biesheuvel@linaro.org, edk2-devel@lists.01.org, linaro-uefi@lists.linaro.org Message-ID: <20170821141243.e5kgwd433ecur7gi@bivouac.eciton.net> References: <20170820193335.21471-1-alan@softiron.com> MIME-Version: 1.0 In-Reply-To: <20170820193335.21471-1-alan@softiron.com> User-Agent: NeoMutt/20170113 (1.7.2) 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 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 21 Aug 2017 14:10:18 -0000 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Sun, Aug 20, 2017 at 03:33:32PM -0400, Alan Ott wrote: > Without the PCD for the second SATA Controller being specified, the boot > will hang. These patches fix it. Given Ard's RB, I'm happy for this to go in. However, would you be happy to contribute these under TianoCore Contribution Agreement 1.1 rather than 1.0? If so, please confirm here and I can fold in the change before pushing. / Leif > 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 > > Platform/AMD/OverdriveBoard/OverdriveBoard.dsc | 10 +++------- > Silicon/AMD/Styx/AmdStyx.dec | 2 +- > .../AMD/Styx/Drivers/StyxSataPlatformDxe/InitController.c | 13 +++++++++---- > 3 files changed, 13 insertions(+), 12 deletions(-) > > -- > 2.9.3 >