On 2014-10-22 16:01:29,  wrote:
> From: Reza Jelveh <[email protected]>
> 
> The SataController is copied to OvmfPkg where the IdeController
> resides and enabled for OvmfPkg

In http://permalink.gmane.org/gmane.comp.bios.tianocore.devel/10187
you wrote: "The PcAtChipsetPkg/DuetPkg remains untouched. The copy of
SataController is in OVMF as requested by the Intel guys."

Where was this requested?

Could this driver be made reasonably generic enough for DuetPkg and
OvmfPkg to both be able to use a common version residing in
PcAtChipsetPkg? Maybe the PcAtChipsetPkg owner doesn't think it would
be a good idea? (Or, maybe it is just not a good idea? I don't
know...)

One thing I don't want is for OVMF to depend on DuetPkg.

-Jordan

> Because SataController needs AE to be enabled before reading Ahci
> registers, GetChannelInfo is used to accomodate for initializations
> after AhciMode initialization.
> 
> IdeMode of AtaAtapiPassThru is also adjust to provide more compatibility
> with custom Ide Controllers.
> 
> Reza Jelveh (5):
>   OvmfPkg: copy SataControllerDxe to OvmfPkg
>   OvmfPkg: enable SATA controller
>   OvmfPkg: read host capabilities on ChannelInfo
>   MdeModulePkg: AhciMode calls GetChannelInfo when enumerating
>   MdeModulePkg; IdeMode select master/slave before diagnostics command
> 
>  MdeModulePkg/Bus/Ata/AtaAtapiPassThru/AhciMode.c |   16 +
>  MdeModulePkg/Bus/Ata/AtaAtapiPassThru/IdeMode.c  |    9 +-
>  OvmfPkg/OvmfPkgIa32.dsc                          |    5 +-
>  OvmfPkg/OvmfPkgIa32.fdf                          |    5 +-
>  OvmfPkg/OvmfPkgIa32X64.dsc                       |    5 +-
>  OvmfPkg/OvmfPkgIa32X64.fdf                       |    5 +-
>  OvmfPkg/OvmfPkgX64.dsc                           |    5 +-
>  OvmfPkg/OvmfPkgX64.fdf                           |    5 +-
>  OvmfPkg/SataControllerDxe/ComponentName.c        |  176 ++++
>  OvmfPkg/SataControllerDxe/SataController.c       | 1019 
> ++++++++++++++++++++++
>  OvmfPkg/SataControllerDxe/SataController.h       |  542 ++++++++++++
>  OvmfPkg/SataControllerDxe/SataControllerDxe.inf  |   49 ++
>  12 files changed, 1825 insertions(+), 16 deletions(-)
>  create mode 100644 OvmfPkg/SataControllerDxe/ComponentName.c
>  create mode 100644 OvmfPkg/SataControllerDxe/SataController.c
>  create mode 100644 OvmfPkg/SataControllerDxe/SataController.h
>  create mode 100644 OvmfPkg/SataControllerDxe/SataControllerDxe.inf
> 
> -- 
> 2.1.1
> 
> 
> ------------------------------------------------------------------------------
> _______________________________________________
> edk2-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/edk2-devel

------------------------------------------------------------------------------
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to