From: "Gary Lin" <glin@suse.com>
To: Laszlo Ersek <lersek@redhat.com>
Cc: devel@edk2.groups.io, Jordan Justen <jordan.l.justen@intel.com>,
Ard Biesheuvel <ard.biesheuvel@arm.com>
Subject: Re: [PATCH 01/11] OvmfPkg/LsiScsiDxe: Create the empty driver
Date: Tue, 7 Jul 2020 16:24:30 +0800 [thread overview]
Message-ID: <20200707082430.GL18504@GaryWorkstation> (raw)
In-Reply-To: <b589caad-50a0-fd8f-c435-6a74f6779536@redhat.com>
On Tue, Jul 07, 2020 at 09:59:04AM +0200, Laszlo Ersek wrote:
> On 07/01/20 06:04, Gary Lin wrote:
> > Create the driver with only a dummy LsiScsiEntryPoint() for the further
> > implementation of the driver for LSI 53C895A SCSI controller.
> >
> > Cc: Jordan Justen <jordan.l.justen@intel.com>
> > Cc: Laszlo Ersek <lersek@redhat.com>
> > Cc: Ard Biesheuvel <ard.biesheuvel@arm.com>
> > Signed-off-by: Gary Lin <glin@suse.com>
> > ---
> > OvmfPkg/LsiScsiDxe/LsiScsi.c | 25 +++++++++++++++++++++++++
> > OvmfPkg/LsiScsiDxe/LsiScsiDxe.inf | 26 ++++++++++++++++++++++++++
> > OvmfPkg/OvmfPkgIa32.dsc | 4 ++++
> > OvmfPkg/OvmfPkgIa32.fdf | 3 +++
> > OvmfPkg/OvmfPkgIa32X64.dsc | 4 ++++
> > OvmfPkg/OvmfPkgIa32X64.fdf | 3 +++
> > OvmfPkg/OvmfPkgX64.dsc | 4 ++++
> > OvmfPkg/OvmfPkgX64.fdf | 3 +++
> > 8 files changed, 72 insertions(+)
> > create mode 100644 OvmfPkg/LsiScsiDxe/LsiScsi.c
> > create mode 100644 OvmfPkg/LsiScsiDxe/LsiScsiDxe.inf
> >
> > diff --git a/OvmfPkg/LsiScsiDxe/LsiScsi.c b/OvmfPkg/LsiScsiDxe/LsiScsi.c
> > new file mode 100644
> > index 000000000000..9c90941688ed
> > --- /dev/null
> > +++ b/OvmfPkg/LsiScsiDxe/LsiScsi.c
> > @@ -0,0 +1,25 @@
> > +/** @file
> > +
> > + This driver produces Extended SCSI Pass Thru Protocol instances for
> > + LSI 53C895A SCSI devices.
> > +
> > + Copyright (C) 2020, SUSE LLC.
> > +
> > + SPDX-License-Identifier: BSD-2-Clause-Patent
> > +
> > +**/
> > +
> > +#include <Uefi/UefiSpec.h>
> > +
> > +//
> > +// Entry point of this driver
> > +//
> > +EFI_STATUS
> > +EFIAPI
> > +LsiScsiEntryPoint (
> > + IN EFI_HANDLE ImageHandle,
> > + IN EFI_SYSTEM_TABLE *SystemTable
> > + )
> > +{
> > + return EFI_UNSUPPORTED;
> > +}
> > diff --git a/OvmfPkg/LsiScsiDxe/LsiScsiDxe.inf b/OvmfPkg/LsiScsiDxe/LsiScsiDxe.inf
> > new file mode 100644
> > index 000000000000..55b483c2fa20
> > --- /dev/null
> > +++ b/OvmfPkg/LsiScsiDxe/LsiScsiDxe.inf
> > @@ -0,0 +1,26 @@
> > +## @file
> > +# This driver produces Extended SCSI Pass Thru Protocol instances for
> > +# LSI 53C895A SCSI devices.
> > +#
> > +# Copyright (C) 2020, SUSE LLC.
> > +#
> > +# SPDX-License-Identifier: BSD-2-Clause-Patent
> > +#
> > +##
> > +
> > +[Defines]
> > + INF_VERSION = 1.29
> > + BASE_NAME = LsiScsiDxe
> > + FILE_GUID = EB4EB21f-5A3D-40BE-8BD2-F1B0E38E5744
>
> (1) The GUID should not be written in mixed case. Please pick upper case
> or lower case for the whole GUID string.
>
> (I wonder how you created the current (mixed-case) GUID string...
> uuidgen outputs lower case, and I wonder how one can translate
> everything except the *first* "f" from that to upper case. I hope you
> didn't manually update a previously generated GUID!)
Oops, I generated the GUID with uuidgen and converted it to upper case
manually (facepalm). Will fix it.
Gary Lin
>
> with the spelling fixed, or the GUID regenerated:
>
> Reviewed-by: Laszlo Ersek <lersek@redhat.com>
>
> Thanks
> Laszlo
>
> > + MODULE_TYPE = UEFI_DRIVER
> > + VERSION_STRING = 1.0
> > + ENTRY_POINT = LsiScsiEntryPoint
> > +
> > +[Sources]
> > + LsiScsi.c
> > +
> > +[Packages]
> > + MdePkg/MdePkg.dec
> > +
> > +[LibraryClasses]
> > + UefiDriverEntryPoint
> > diff --git a/OvmfPkg/OvmfPkgIa32.dsc b/OvmfPkg/OvmfPkgIa32.dsc
> > index b4ee7376791b..9178ffeb71cb 100644
> > --- a/OvmfPkg/OvmfPkgIa32.dsc
> > +++ b/OvmfPkg/OvmfPkgIa32.dsc
> > @@ -50,6 +50,7 @@ [Defines]
> > #
> > DEFINE PVSCSI_ENABLE = TRUE
> > DEFINE MPT_SCSI_ENABLE = TRUE
> > + DEFINE LSI_SCSI_ENABLE = FALSE
> >
> > #
> > # Flash size selection. Setting FD_SIZE_IN_KB on the command line directly to
> > @@ -771,6 +772,9 @@ [Components]
> > !endif
> > !if $(MPT_SCSI_ENABLE) == TRUE
> > OvmfPkg/MptScsiDxe/MptScsiDxe.inf
> > +!endif
> > +!if $(LSI_SCSI_ENABLE) == TRUE
> > + OvmfPkg/LsiScsiDxe/LsiScsiDxe.inf
> > !endif
> > MdeModulePkg/Universal/WatchdogTimerDxe/WatchdogTimer.inf
> > MdeModulePkg/Universal/MonotonicCounterRuntimeDxe/MonotonicCounterRuntimeDxe.inf
> > diff --git a/OvmfPkg/OvmfPkgIa32.fdf b/OvmfPkg/OvmfPkgIa32.fdf
> > index e2b759aa8d05..2b9a6b58015f 100644
> > --- a/OvmfPkg/OvmfPkgIa32.fdf
> > +++ b/OvmfPkg/OvmfPkgIa32.fdf
> > @@ -236,6 +236,9 @@ [FV.DXEFV]
> > !if $(MPT_SCSI_ENABLE) == TRUE
> > INF OvmfPkg/MptScsiDxe/MptScsiDxe.inf
> > !endif
> > +!if $(LSI_SCSI_ENABLE) == TRUE
> > +INF OvmfPkg/LsiScsiDxe/LsiScsiDxe.inf
> > +!endif
> >
> > !if $(SECURE_BOOT_ENABLE) == TRUE
> > INF SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigDxe.inf
> > diff --git a/OvmfPkg/OvmfPkgIa32X64.dsc b/OvmfPkg/OvmfPkgIa32X64.dsc
> > index ed68b080f2a2..a665f78f0dc7 100644
> > --- a/OvmfPkg/OvmfPkgIa32X64.dsc
> > +++ b/OvmfPkg/OvmfPkgIa32X64.dsc
> > @@ -49,6 +49,7 @@ [Defines]
> > #
> > DEFINE PVSCSI_ENABLE = TRUE
> > DEFINE MPT_SCSI_ENABLE = TRUE
> > + DEFINE LSI_SCSI_ENABLE = FALSE
> >
> > #
> > # Flash size selection. Setting FD_SIZE_IN_KB on the command line directly to
> > @@ -785,6 +786,9 @@ [Components.X64]
> > !endif
> > !if $(MPT_SCSI_ENABLE) == TRUE
> > OvmfPkg/MptScsiDxe/MptScsiDxe.inf
> > +!endif
> > +!if $(LSI_SCSI_ENABLE) == TRUE
> > + OvmfPkg/LsiScsiDxe/LsiScsiDxe.inf
> > !endif
> > MdeModulePkg/Universal/WatchdogTimerDxe/WatchdogTimer.inf
> > MdeModulePkg/Universal/MonotonicCounterRuntimeDxe/MonotonicCounterRuntimeDxe.inf
> > diff --git a/OvmfPkg/OvmfPkgIa32X64.fdf b/OvmfPkg/OvmfPkgIa32X64.fdf
> > index bfca1eff9e83..83ff6aef2e8c 100644
> > --- a/OvmfPkg/OvmfPkgIa32X64.fdf
> > +++ b/OvmfPkg/OvmfPkgIa32X64.fdf
> > @@ -237,6 +237,9 @@ [FV.DXEFV]
> > !if $(MPT_SCSI_ENABLE) == TRUE
> > INF OvmfPkg/MptScsiDxe/MptScsiDxe.inf
> > !endif
> > +!if $(LSI_SCSI_ENABLE) == TRUE
> > +INF OvmfPkg/LsiScsiDxe/LsiScsiDxe.inf
> > +!endif
> >
> > !if $(SECURE_BOOT_ENABLE) == TRUE
> > INF SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigDxe.inf
> > diff --git a/OvmfPkg/OvmfPkgX64.dsc b/OvmfPkg/OvmfPkgX64.dsc
> > index cb7e8068a3d8..17f345acf4ee 100644
> > --- a/OvmfPkg/OvmfPkgX64.dsc
> > +++ b/OvmfPkg/OvmfPkgX64.dsc
> > @@ -49,6 +49,7 @@ [Defines]
> > #
> > DEFINE PVSCSI_ENABLE = TRUE
> > DEFINE MPT_SCSI_ENABLE = TRUE
> > + DEFINE LSI_SCSI_ENABLE = FALSE
> >
> > #
> > # Flash size selection. Setting FD_SIZE_IN_KB on the command line directly to
> > @@ -781,6 +782,9 @@ [Components]
> > !endif
> > !if $(MPT_SCSI_ENABLE) == TRUE
> > OvmfPkg/MptScsiDxe/MptScsiDxe.inf
> > +!endif
> > +!if $(LSI_SCSI_ENABLE) == TRUE
> > + OvmfPkg/LsiScsiDxe/LsiScsiDxe.inf
> > !endif
> > MdeModulePkg/Universal/WatchdogTimerDxe/WatchdogTimer.inf
> > MdeModulePkg/Universal/MonotonicCounterRuntimeDxe/MonotonicCounterRuntimeDxe.inf
> > diff --git a/OvmfPkg/OvmfPkgX64.fdf b/OvmfPkg/OvmfPkgX64.fdf
> > index bfca1eff9e83..83ff6aef2e8c 100644
> > --- a/OvmfPkg/OvmfPkgX64.fdf
> > +++ b/OvmfPkg/OvmfPkgX64.fdf
> > @@ -237,6 +237,9 @@ [FV.DXEFV]
> > !if $(MPT_SCSI_ENABLE) == TRUE
> > INF OvmfPkg/MptScsiDxe/MptScsiDxe.inf
> > !endif
> > +!if $(LSI_SCSI_ENABLE) == TRUE
> > +INF OvmfPkg/LsiScsiDxe/LsiScsiDxe.inf
> > +!endif
> >
> > !if $(SECURE_BOOT_ENABLE) == TRUE
> > INF SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigDxe.inf
> >
>
next prev parent reply other threads:[~2020-07-07 8:24 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-07-01 4:04 [PATCH 00/11] Introduce LsiScsi driver to OvmfPkg Gary Lin
2020-07-01 4:04 ` [PATCH 01/11] OvmfPkg/LsiScsiDxe: Create the empty driver Gary Lin
2020-07-07 7:59 ` Laszlo Ersek
2020-07-07 8:24 ` Gary Lin [this message]
2020-07-01 4:04 ` [PATCH 02/11] OvmfPkg/LsiScsiDxe: Install the skeleton of driver binding Gary Lin
2020-07-07 8:06 ` Laszlo Ersek
2020-07-07 8:34 ` Gary Lin
2020-07-01 4:04 ` [PATCH 03/11] OvmfPkg/LsiScsiDxe: Report the name of the driver Gary Lin
2020-07-07 8:09 ` Laszlo Ersek
2020-07-01 4:04 ` [PATCH 04/11] OvmfPkg/LsiScsiDxe: Probe PCI devices and look for LsiScsi Gary Lin
2020-07-07 8:15 ` Laszlo Ersek
2020-07-07 8:16 ` Laszlo Ersek
2020-07-01 4:04 ` [PATCH 05/11] OvmfPkg/LsiScsiDxe: Install stubbed EXT_SCSI_PASS_THRU Gary Lin
2020-07-07 8:28 ` Laszlo Ersek
2020-07-01 4:04 ` [PATCH 06/11] OvmfPkg/LsiScsiDxe: Report Targets and LUNs Gary Lin
2020-07-07 9:04 ` Laszlo Ersek
2020-07-08 2:34 ` Gary Lin
2020-07-08 15:26 ` Laszlo Ersek
2020-07-01 4:04 ` [PATCH 07/11] OvmfPkg/LsiScsiDxe: Open PciIo protocol and initialize the device Gary Lin
2020-07-07 9:46 ` Laszlo Ersek
2020-07-08 2:37 ` Gary Lin
2020-07-01 4:04 ` [PATCH 08/11] OvmfPkg/LsiScsiDxe: Map DMA buffer Gary Lin
2020-07-07 9:59 ` Laszlo Ersek
2020-07-08 2:41 ` Gary Lin
2020-07-01 4:04 ` [PATCH 09/11] OvmfPkg/LsiScsiDxe: Examine the incoming SCSI Request Packet Gary Lin
2020-07-07 10:17 ` Laszlo Ersek
2020-07-08 2:43 ` Gary Lin
2020-07-01 4:04 ` [PATCH 10/11] OvmfPkg/LsiScsiDxe: Process the " Gary Lin
2020-07-07 12:46 ` Laszlo Ersek
2020-07-08 6:02 ` Gary Lin
[not found] ` <161FB1B03BD2D339.11266@groups.io>
2020-07-14 8:19 ` [edk2-devel] " Gary Lin
2020-07-01 4:04 ` [PATCH 11/11] Maintainers.txt: Add myself as the reviewer for LsiScsi driver Gary Lin
2020-07-07 12:49 ` Laszlo Ersek
2020-07-08 6:03 ` Gary Lin
2020-07-03 14:08 ` [edk2-devel] [PATCH 00/11] Introduce LsiScsi driver to OvmfPkg Laszlo Ersek
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200707082430.GL18504@GaryWorkstation \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox