From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-1.mimecast.com (us-smtp-1.mimecast.com [205.139.110.120]) by mx.groups.io with SMTP id smtpd.web12.786.1595014941644297761 for ; Fri, 17 Jul 2020 12:42:21 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=gfX5bLH6; spf=pass (domain: redhat.com, ip: 205.139.110.120, mailfrom: lersek@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1595014940; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=tqlj0+lACZInLg9299DX0SZouyjBuP1fy9qswgZWMhc=; b=gfX5bLH6mRpXS9KJSyfH2XE62AwVuqEiWaK4dFurFk4fFTgqYSSRlFX4Dy7ovSjLAwsITz TR3P/8B6ewQ5ijy9CdTwGrupjf4jnvgrhOWn0hSshMOMfsPFy0V/xYw9mmlUZEUjrFR6fR xkb8PzOFNkW6X+JePe4FBsqFpQ8VdPs= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-176-XBSsxEuONuqQqAoEKEg9Pw-1; Fri, 17 Jul 2020 15:42:19 -0400 X-MC-Unique: XBSsxEuONuqQqAoEKEg9Pw-1 Received: from smtp.corp.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id E0480100AA21; Fri, 17 Jul 2020 19:42:17 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-112-68.ams2.redhat.com [10.36.112.68]) by smtp.corp.redhat.com (Postfix) with ESMTP id EC45F619B5; Fri, 17 Jul 2020 19:42:14 +0000 (UTC) Subject: Re: [PATCH v3 07/11] OvmfPkg/LsiScsiDxe: Open PciIo protocol and initialize the device To: Gary Lin , devel@edk2.groups.io Cc: Jordan Justen , Ard Biesheuvel References: <20200717061130.8881-1-glin@suse.com> <20200717061130.8881-8-glin@suse.com> From: "Laszlo Ersek" Message-ID: <18afdd63-736b-4ae0-75c2-bcb400cc291b@redhat.com> Date: Fri, 17 Jul 2020 21:42:10 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Firefox/52.0 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: <20200717061130.8881-8-glin@suse.com> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.12 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Language: en-US Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit On 07/17/20 08:11, Gary Lin wrote: > Open PciIo protocol and use it to initialize the device. The > initialization of LSI 53C895A is simple: just set the SRST bit in > Interrupt Status Zero register to reset the device. > > v2: > - Use the BITx macros for the bit constants > - Add the closing of PciIo protocol in LsiScsiControllerStop() > > Cc: Jordan Justen > Cc: Laszlo Ersek > Cc: Ard Biesheuvel > Signed-off-by: Gary Lin > --- > OvmfPkg/Include/IndustryStandard/LsiScsi.h | 21 ++++ > OvmfPkg/LsiScsiDxe/LsiScsi.c | 136 ++++++++++++++++++++- > OvmfPkg/LsiScsiDxe/LsiScsi.h | 3 + > 3 files changed, 159 insertions(+), 1 deletion(-) You forgot to pick up my Reviewed-by: Laszlo Ersek from v2. Thanks Laszlo > > diff --git a/OvmfPkg/Include/IndustryStandard/LsiScsi.h b/OvmfPkg/Include/IndustryStandard/LsiScsi.h > index c09e864a1f39..185e553c8eb4 100644 > --- a/OvmfPkg/Include/IndustryStandard/LsiScsi.h > +++ b/OvmfPkg/Include/IndustryStandard/LsiScsi.h > @@ -17,4 +17,25 @@ > #define LSI_LOGIC_PCI_VENDOR_ID 0x1000 > #define LSI_53C895A_PCI_DEVICE_ID 0x0012 > > +// > +// LSI 53C895A Registers > +// > +#define LSI_REG_DSTAT 0x0C > +#define LSI_REG_ISTAT0 0x14 > +#define LSI_REG_DSP 0x2C > +#define LSI_REG_SIST0 0x42 > +#define LSI_REG_SIST1 0x43 > + > +// > +// The status bits for Interrupt Status Zero (ISTAT0) > +// > +#define LSI_ISTAT0_DIP BIT0 > +#define LSI_ISTAT0_SIP BIT1 > +#define LSI_ISTAT0_INTF BIT2 > +#define LSI_ISTAT0_CON BIT3 > +#define LSI_ISTAT0_SEM BIT4 > +#define LSI_ISTAT0_SIGP BIT5 > +#define LSI_ISTAT0_SRST BIT6 > +#define LSI_ISTAT0_ABRT BIT7 > + > #endif // _LSI_SCSI_H_ > diff --git a/OvmfPkg/LsiScsiDxe/LsiScsi.c b/OvmfPkg/LsiScsiDxe/LsiScsi.c > index 172779240636..a9c107c04ef1 100644 > --- a/OvmfPkg/LsiScsiDxe/LsiScsi.c > +++ b/OvmfPkg/LsiScsiDxe/LsiScsi.c > @@ -25,6 +25,33 @@ > > #include "LsiScsi.h" > > +STATIC > +EFI_STATUS > +Out8 ( > + IN LSI_SCSI_DEV *Dev, > + IN UINT32 Addr, > + IN UINT8 Data > + ) > +{ > + return Dev->PciIo->Io.Write ( > + Dev->PciIo, > + EfiPciIoWidthUint8, > + PCI_BAR_IDX0, > + Addr, > + 1, > + &Data > + ); > +} > + > +STATIC > +EFI_STATUS > +LsiScsiReset ( > + IN LSI_SCSI_DEV *Dev > + ) > +{ > + return Out8 (Dev, LSI_REG_ISTAT0, LSI_ISTAT0_SRST); > +} > + > // > // The next seven functions implement EFI_EXT_SCSI_PASS_THRU_PROTOCOL > // for the LSI 53C895A SCSI Controller. Refer to UEFI Spec 2.3.1 + Errata C, > @@ -243,6 +270,21 @@ LsiScsiGetNextTarget ( > return EFI_NOT_FOUND; > } > > +STATIC > +VOID > +EFIAPI > +LsiScsiExitBoot ( > + IN EFI_EVENT Event, > + IN VOID *Context > + ) > +{ > + LSI_SCSI_DEV *Dev; > + > + Dev = Context; > + DEBUG ((DEBUG_VERBOSE, "%a: Context=0x%p\n", __FUNCTION__, Context)); > + LsiScsiReset (Dev); > +} > + > // > // Probe, start and stop functions of this driver, called by the DXE core for > // specific devices. > @@ -333,6 +375,58 @@ LsiScsiControllerStart ( > Dev->MaxTarget = PcdGet8 (PcdLsiScsiMaxTargetLimit); > Dev->MaxLun = PcdGet8 (PcdLsiScsiMaxLunLimit); > > + Status = gBS->OpenProtocol ( > + ControllerHandle, > + &gEfiPciIoProtocolGuid, > + (VOID **)&Dev->PciIo, > + This->DriverBindingHandle, > + ControllerHandle, > + EFI_OPEN_PROTOCOL_BY_DRIVER > + ); > + if (EFI_ERROR (Status)) { > + goto FreePool; > + } > + > + Status = Dev->PciIo->Attributes ( > + Dev->PciIo, > + EfiPciIoAttributeOperationGet, > + 0, > + &Dev->OrigPciAttrs > + ); > + if (EFI_ERROR (Status)) { > + goto CloseProtocol; > + } > + > + // > + // Enable I/O Space & Bus-Mastering > + // > + Status = Dev->PciIo->Attributes ( > + Dev->PciIo, > + EfiPciIoAttributeOperationEnable, > + (EFI_PCI_IO_ATTRIBUTE_IO | > + EFI_PCI_IO_ATTRIBUTE_BUS_MASTER), > + NULL > + ); > + if (EFI_ERROR (Status)) { > + goto CloseProtocol; > + } > + > + Status = LsiScsiReset (Dev); > + if (EFI_ERROR (Status)) { > + goto RestoreAttributes; > + } > + > + Status = gBS->CreateEvent ( > + EVT_SIGNAL_EXIT_BOOT_SERVICES, > + TPL_CALLBACK, > + &LsiScsiExitBoot, > + Dev, > + &Dev->ExitBoot > + ); > + if (EFI_ERROR (Status)) { > + goto UninitDev; > + } > + > // > // Host adapter channel, doesn't exist > // > @@ -357,11 +451,33 @@ LsiScsiControllerStart ( > &Dev->PassThru > ); > if (EFI_ERROR (Status)) { > - goto FreePool; > + goto CloseExitBoot; > } > > return EFI_SUCCESS; > > +CloseExitBoot: > + gBS->CloseEvent (Dev->ExitBoot); > + > +UninitDev: > + LsiScsiReset (Dev); > + > +RestoreAttributes: > + Dev->PciIo->Attributes ( > + Dev->PciIo, > + EfiPciIoAttributeOperationSet, > + Dev->OrigPciAttrs, > + NULL > + ); > + > +CloseProtocol: > + gBS->CloseProtocol ( > + ControllerHandle, > + &gEfiPciIoProtocolGuid, > + This->DriverBindingHandle, > + ControllerHandle > + ); > + > FreePool: > FreePool (Dev); > > @@ -404,6 +520,24 @@ LsiScsiControllerStop ( > return Status; > } > > + gBS->CloseEvent (Dev->ExitBoot); > + > + LsiScsiReset (Dev); > + > + Dev->PciIo->Attributes ( > + Dev->PciIo, > + EfiPciIoAttributeOperationSet, > + Dev->OrigPciAttrs, > + NULL > + ); > + > + gBS->CloseProtocol ( > + ControllerHandle, > + &gEfiPciIoProtocolGuid, > + This->DriverBindingHandle, > + ControllerHandle > + ); > + > FreePool (Dev); > > return Status; > diff --git a/OvmfPkg/LsiScsiDxe/LsiScsi.h b/OvmfPkg/LsiScsiDxe/LsiScsi.h > index 6c6ed25f1c33..8c2acff6e86f 100644 > --- a/OvmfPkg/LsiScsiDxe/LsiScsi.h > +++ b/OvmfPkg/LsiScsiDxe/LsiScsi.h > @@ -14,6 +14,9 @@ > > typedef struct { > UINT32 Signature; > + UINT64 OrigPciAttrs; > + EFI_EVENT ExitBoot; > + EFI_PCI_IO_PROTOCOL *PciIo; > UINT8 MaxTarget; > UINT8 MaxLun; > EFI_EXT_SCSI_PASS_THRU_MODE PassThruMode; >