Re: [PATCH edk2-platforms 2/2] Platform/AMD/OverdriveBoard: Re-enable the second SATA Controller

[Ard Biesheuvel <ard.biesheuvel@linaro.org>]

On 19 August 2017 at 22:41, Alan Ott <alan@softiron.com> wrote:
> The comment indicating that only the first SATA controller is operational
> on SoftIron-branded OverDrive 3000 boards is incorrect. Re-enable the
> second SATA controller.
>
> Signed-off-by: Alan Ott <alan@softiron.com>
> Contributed-under: TianoCore Contribution Agreement 1.0

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

> ---
>  Platform/AMD/OverdriveBoard/OverdriveBoard.dsc | 10 +++-------
>  1 file changed, 3 insertions(+), 7 deletions(-)
>
> diff --git a/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc b/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc
> index f256ffb..2881de3 100644
> --- a/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc
> +++ b/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc
> @@ -409,14 +409,10 @@ DEFINE DO_FLASHER   = FALSE
>    gArmTokenSpaceGuid.PcdGicDistributorBase|0xE1110000
>    gArmTokenSpaceGuid.PcdGicInterruptInterfaceBase|0xE112F000
>
> -  #
> -  # AMD's B1 based Overdrive has 14 SATA ports across 2 controllers. However,
> -  # it appears that Softiron's Overdrive 3000, which is also B1 based, does
> -  # not have the second SATA controller enabled, and any attempts to use it
> -  # will crash the firmware. So use the first controller only.
> -  #
> +  # SATA Ports
>    gAmdStyxTokenSpaceGuid.PcdSata0PortCount|8
> -  gAmdStyxTokenSpaceGuid.PcdSataPortMode|0xffff
> +  gAmdStyxTokenSpaceGuid.PcdSata1PortCount|6
> +  gAmdStyxTokenSpaceGuid.PcdSataPortMode|0x0fffffff
>
>    # PCIe Support
>    gEfiMdePkgTokenSpaceGuid.PcdPciExpressBaseAddress|0xF0000000
> --
> 2.9.3
>