From nobody Fri Apr 19 13:01:51 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) client-ip=66.175.222.108; envelope-from=bounce+27952+69605+1787277+3901457@groups.io; helo=mail02.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+69605+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=arm.com ARC-Seal: i=1; a=rsa-sha256; t=1609781880; cv=none; d=zohomail.com; s=zohoarc; b=nNTa+52qEwDeBhcPU1YooHNTWFHGWwLTtIYfA8s1UscqGFgbe857AhddfGss/TyDxJtuNbUKKWgqV3VRVdbHuVC1fYW3iTWcssduHwgZJ/4XoyT30UsdrSM/xuNh63spxF25OLezJVLwutMyijmMK40Tphl86yEmzAdGB3D2cxI= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1609781880; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Id:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:References:Sender:Subject:To; bh=Z81tBm3YjMWvnBdu9VJ872QehqUw9O08h3PgA8JMzzY=; b=KZPXztKawxOziEugOS7jkqfyrupPEf3IxGX+G/VMxIwL5oL/FADtavjtTW7zVL0UuMvRdmfnimHMXjbM6HRPv5BvogMtOT7ZYOq9JXHKLwFAQQkDxb4jl3XVXOVex/JnRltaIR4AgHlbehTrdkfqKZcEoZDYRtI4wk6BkO67xrc= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+69605+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) header.from= Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 1609781880546683.0702409022821; Mon, 4 Jan 2021 09:38:00 -0800 (PST) Return-Path: X-Received: by 127.0.0.2 with SMTP id vyTNYY1788612xWKC38fAfA4; Mon, 04 Jan 2021 09:38:00 -0800 X-Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web09.16965.1609781873078172531 for ; Mon, 04 Jan 2021 09:37:53 -0800 X-Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id B45B4101E; Mon, 4 Jan 2021 09:37:52 -0800 (PST) X-Received: from mammon-tx2.austin.arm.com (mammon-tx2.austin.arm.com [10.118.28.62]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id AD1773F66E; Mon, 4 Jan 2021 09:37:52 -0800 (PST) From: "Jeremy Linton" To: devel@edk2.groups.io Cc: ard.biesheuvel@arm.com, leif@nuviainc.com, pete@akeo.ie, samer.el-haj-mahmoud@arm.com, Jeremy Linton , Andrei Warkentin Subject: [edk2-devel] [PATCH v3 1/7] Platform/RaspberryPi: Update VPU mailbox constants Date: Mon, 4 Jan 2021 11:37:25 -0600 Message-Id: <20210104173731.1413044-2-jeremy.linton@arm.com> In-Reply-To: <20210104173731.1413044-1-jeremy.linton@arm.com> References: <20210104173731.1413044-1-jeremy.linton@arm.com> MIME-Version: 1.0 Precedence: Bulk List-Unsubscribe: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,jeremy.linton@arm.com X-Gm-Message-State: Tu1ATvFfZoIx9DVjYLc4bWaNx1787277AA= Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1609781880; bh=Xh8Jn0/vd7c+mmt82Blpaoajxzb/jSeYjpYve7vlSWQ=; h=Cc:Date:From:Reply-To:Subject:To; b=v8fE5xR/IRDL6sjZlqXqtBKLPRR/YHCc/UzxT3lj4/VqHNoUZd6ekG3pMvYPFh4eKAM nTxfKMbK4APCeax/+QA/ej0NY3iRiF2E5DAZjxQskuJReLirr2I68lypQPAkN1Jk6UV6n WWBuqJBvtLcLWTozd92lhBG3aEooUGp+U4E= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" Lets sync our mailbox commands with the known/linux constants so that we have a more complete view of what we can request from the VPU. Signed-off-by: Jeremy Linton Reviewed-by: Andrei Warkentin --- .../RaspberryPi/Include/IndustryStandard/RpiMbox.h | 94 ++++++++++++++++++= ++-- 1 file changed, 89 insertions(+), 5 deletions(-) diff --git a/Platform/RaspberryPi/Include/IndustryStandard/RpiMbox.h b/Plat= form/RaspberryPi/Include/IndustryStandard/RpiMbox.h index 7104068998..551c2b82e5 100644 --- a/Platform/RaspberryPi/Include/IndustryStandard/RpiMbox.h +++ b/Platform/RaspberryPi/Include/IndustryStandard/RpiMbox.h @@ -45,6 +45,10 @@ #define RPI_MBOX_GET_BOARD_SERIAL 0x00010004 #define RPI_MBOX_GET_ARM_MEMSIZE 0x00010005 #define RPI_MBOX_GET_VC_MEMSIZE 0x00010006 +#define RPI_MBOX_GET_CLOCKS 0x00010007 + +#define RPI_MBOX_GET_POWER_STATE 0x00020001 +#define RPI_MBOX_GET_TIMING 0x00020002 =20 #define RPI_MBOX_SET_POWER_STATE 0x00028001 =20 @@ -58,28 +62,92 @@ #define RPI_MBOX_POWER_STATE_SPI 0x00000007 #define RPI_MBOX_POWER_STATE_CCP2TX 0x00000008 =20 +#define RPI_MBOX_GET_CLOCK_STATE 0x00030001 #define RPI_MBOX_GET_CLOCK_RATE 0x00030002 +#define RPI_MBOX_GET_VOLTAGE 0x00030003 #define RPI_MBOX_GET_MAX_CLOCK_RATE 0x00030004 +#define RPI_MBOX_GET_MAX_VOLTAGE 0x00030005 +#define RPI_MBOX_GET_TEMPERATURE 0x00030006 #define RPI_MBOX_GET_MIN_CLOCK_RATE 0x00030007 - +#define RPI_MBOX_GET_MIN_VOLTAGE 0x00030008 +#define RPI_MBOX_GET_TURBO 0x00030009 +#define RPI_MBOX_GET_MAX_TEMPERATURE 0x0003000a +#define RPI_MBOX_GET_STC 0x0003000b +#define RPI_MBOX_ALLOCATE_MEMORY 0x0003000c +#define RPI_MBOX_LOCK_MEMORY 0x0003000d +#define RPI_MBOX_UNLOCK_MEMORY 0x0003000e +#define RPI_MBOX_RELEASE_MEMORY 0x0003000f +#define RPI_MBOX_EXECUTE_CODE 0x00030010 +#define RPI_MBOX_EXECUTE_QPU 0x00030011 +#define RPI_MBOX_SET_ENABLE_QPU 0x00030012 +#define RPI_MBOX_GET_DISPMANX_RESOURCE_MEM_HANDLE 0x00030014 +#define RPI_MBOX_GET_EDID_BLOCK 0x00030020 +#define RPI_MBOX_GET_CUSTOMER_OTP 0x00030021 +#define RPI_MBOX_GET_DOMAIN_STATE 0x00030030 +#define RPI_MBOX_GET_GPIO_STATE 0x00030041 +#define RPI_MBOX_GET_GPIO_CONFIG 0x00030043 +#define RPI_MBOX_GET_PERIPH_REG 0x00030045 +#define RPI_MBOX_GET_THROTTLED 0x00030046 +#define RPI_MBOX_GET_CLOCK_MEASURED 0x00030047 +#define RPI_MBOX_NOTIFY_REBOOT 0x00030048 +#define RPI_MBOX_GET_POE_HAT_VAL 0x00030049 +#define RPI_MBOX_SET_POE_HAT_VAL 0x00030050 #define RPI_MBOX_NOTIFY_XHCI_RESET 0x00030058 =20 +#define RPI_MBOX_SET_CLOCK_STATE 0x00038001 #define RPI_MBOX_SET_CLOCK_RATE 0x00038002 +#define RPI_MBOX_SET_VOLTAGE 0x00038003 +#define RPI_MBOX_SET_TURBO 0x00038009 +#define RPI_MBOX_SET_CUSTOMER_OTP 0x00038021 +#define RPI_MBOX_SET_DOMAIN_STATE 0x00038030 #define RPI_MBOX_SET_GPIO 0x00038041 +#define RPI_MBOX_SET_SDHOST_CLOCK 0x00038042 +#define RPI_MBOX_SET_GPIO_CONFIG 0x00038043 +#define RPI_MBOX_SET_PERIPH_REG 0x00038045 =20 +#define RPI_MBOX_ALLOC_FB 0x00040001 +#define RPI_MBOX_FB_BLANK 0x00040002 #define RPI_MBOX_GET_FB_GEOMETRY 0x00040003 -#define RPI_MBOX_GET_FB_LINELENGTH 0x00040008 +#define RPI_MBOX_GET_FB_VIRTUAL_WIDTH_HEIGHT 0x00040004 #define RPI_MBOX_GET_FB_COLOR_DEPTH 0x00040005 -#define RPI_MBOX_GET_FB_REGION 0x00040001 +#define RPI_MBOX_GET_FB_PIXEL_ORDER 0x00040006 +#define RPI_MBOX_GET_FB_ALPHA_MODE 0x00040007 +#define RPI_MBOX_GET_FB_LINELENGTH 0x00040008 +#define RPI_MBOX_GET_FB_VIRTUAL_OFFSET 0x00040009 +#define RPI_MBOX_GET_FB_OVERSCAN 0x0004000a +#define RPI_MBOX_GET_FB_PALETTE 0x0004000b +#define RPI_MBOX_GET_FB_TOUCHBUF 0x0004000f +#define RPI_MBOX_GET_FB_GPIOVIRTBUF 0x00040010 + +#define RPI_MBOX_TEST_FB_PHYSICAL_WIDTH_HEIGHT 0x00044003 +#define RPI_MBOX_TEST_FB_VIRTUAL_WIDTH_HEIGHT 0x00044004 +#define RPI_MBOX_TEST_FB_DEPTH 0x00044005 +#define RPI_MBOX_TEST_FB_PIXEL_ORDER 0x00044006 +#define RPI_MBOX_TEST_FB_ALPHA_MODE 0x00044007 +#define RPI_MBOX_TEST_FB_VIRTUAL_OFFSET 0x00044009 +#define RPI_MBOX_TEST_FB_OVERSCAN 0x0004400a +#define RPI_MBOX_TEST_FB_PALETTE 0x0004400b +#define RPI_MBOX_TEST_FB_VSYNC 0x0004400e =20 +#define RPI_MBOX_FREE_FB 0x00048001 #define RPI_MBOX_SET_FB_PGEOM 0x00048003 #define RPI_MBOX_SET_FB_VGEOM 0x00048004 #define RPI_MBOX_SET_FB_DEPTH 0x00048005 -#define RPI_MBOX_ALLOC_FB 0x00040001 -#define RPI_MBOX_FREE_FB 0x00048001 +#define RPI_MBOX_SET_FB_PIXEL_ORDER 0x00048006 +#define RPI_MBOX_SET_FB_ALPHA_MODE 0x00048007 +#define RPI_MBOX_SET_FB_VIRTUAL_OFFSET 0x00048009 +#define RPI_MBOX_SET_FB_OVERSCAN 0x0004800a +#define RPI_MBOX_SET_FB_PALET TE 0x0004800b +#define RPI_MBOX_VCHIQ_INIT 0x00048010 +#define RPI_MBOX_SET_FB_TOUCHBUF 0x0004801f +#define RPI_MBOX_SET_FB_GPIOVIRTBUF 0x00048020 +#define RPI_MBOX_SET_FB_VSYNC 0x0004800e +#define RPI_MBOX_SET_FB_BACKLIGHT 0x0004800f =20 #define RPI_MBOX_GET_COMMAND_LINE 0x00050001 =20 +#define RPI_MBOX_GET_DMA_CHANNELS 0x00060001 + #define RPI_MBOX_POWER_STATE_ENABLE BIT0 #define RPI_MBOX_POWER_STATE_WAIT BIT1 =20 @@ -93,5 +161,21 @@ #define RPI_MBOX_CLOCK_RATE_SDRAM 0x000000008 #define RPI_MBOX_CLOCK_RATE_PIXEL 0x000000009 #define RPI_MBOX_CLOCK_RATE_PWM 0x00000000a +#define RPI_MBOX_CLOCK_RATE_HEVC 0x00000000b +#define RPI_MBOX_CLOCK_RATE_EMMC2 0x00000000c +#define RPI_MBOX_CLOCK_RATE_M2MC 0x00000000d +#define RPI_MBOX_CLOCK_RATE_PIXEL_BVB 0x00000000d + +#define RPI_EXP_GPIO_DIR_IN 0 +#define RPI_EXP_GPIO_DIR_OUT 1 + +#define RPI_EXP_GPIO_BT 0 +#define RPI_EXP_GPIO_WIFI 1 +#define RPI_EXP_GPIO_LED 2 +#define RPI_EXP_GPIO_RESET 3 +#define RPI_EXP_GPIO_SD_VOLT 4 +#define RPI_EXP_GPIO_CAMERA 5 +#define RPI_EXP_GPIO_SD_POWER 6 +#define RPI_EXP_GPIO_POWER_LED 7 =20 #endif /* __RASPBERRY_PI_MAILBOX_H__ */ --=20 2.13.7 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#69605): https://edk2.groups.io/g/devel/message/69605 Mute This Topic: https://groups.io/mt/79429620/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- From nobody Fri Apr 19 13:01:51 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) client-ip=66.175.222.108; envelope-from=bounce+27952+69606+1787277+3901457@groups.io; helo=mail02.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+69606+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=arm.com ARC-Seal: i=1; a=rsa-sha256; t=1609781886; cv=none; d=zohomail.com; s=zohoarc; b=MqHcdW0is45rH9tbfBFPrLuKCh+YYzb1816gyYqa6oFDNFnYF2OP7M5zK/YF8LJCNgcmhbOXqrr+E1efBWwJH5aWEORWFx9iGqANlIeyG35B6e1b3TWSC0kMGqIx0QiVaIdtOOtHKfzJWc8BLkRMzqXgQ5DRYBlFDt8hd0MUXbg= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1609781886; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Id:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:References:Sender:Subject:To; bh=f4c3BzmM/dx1cISVAvQlQMwD5vEM+AIX1zOITa2iU9U=; b=hP7Rt4AvvkeL5pnK7VSmVte0gufNXTD2F4UQZ04UeP9aqeIS/sJXfemdMwPhnk/Y0RPyfJ/LOQzODxcvwnp3YWd2yKT+cwmqhBcAyL4/SM82ZhaF9gjk2SLKXczU7T8ctmjKjAeN/GuKW3BXnyrb1xR+ysapp5ud5DtEx9s2xy0= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+69606+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) header.from= Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 1609781886662526.5205736104912; Mon, 4 Jan 2021 09:38:06 -0800 (PST) Return-Path: X-Received: by 127.0.0.2 with SMTP id 35iwYY1788612xNuXekKtYXI; Mon, 04 Jan 2021 09:38:06 -0800 X-Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web11.16938.1609781874121978063 for ; Mon, 04 Jan 2021 09:37:54 -0800 X-Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id C903E1042; Mon, 4 Jan 2021 09:37:53 -0800 (PST) X-Received: from mammon-tx2.austin.arm.com (mammon-tx2.austin.arm.com [10.118.28.62]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id BFEB23F66E; Mon, 4 Jan 2021 09:37:53 -0800 (PST) From: "Jeremy Linton" To: devel@edk2.groups.io Cc: ard.biesheuvel@arm.com, leif@nuviainc.com, pete@akeo.ie, samer.el-haj-mahmoud@arm.com, Jeremy Linton , Andrei Warkentin Subject: [edk2-devel] [PATCH v3 2/7] Platform/RaspberryPi: Add further mailbox helpers Date: Mon, 4 Jan 2021 11:37:26 -0600 Message-Id: <20210104173731.1413044-3-jeremy.linton@arm.com> In-Reply-To: <20210104173731.1413044-1-jeremy.linton@arm.com> References: <20210104173731.1413044-1-jeremy.linton@arm.com> MIME-Version: 1.0 Precedence: Bulk List-Unsubscribe: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,jeremy.linton@arm.com X-Gm-Message-State: iPKDW8K1HV2A9s8UheYDNz6ux1787277AA= Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1609781886; bh=OTMubrLRH8yt0EcRYdUSlP53RIcUMz01mua9lCyBWvE=; h=Cc:Date:From:Reply-To:Subject:To; b=WkjxhnwqH61/VxOtW3lXFDaBeKrFTbpBJLxh6sATLfZAIbYfSMID24CcKMXdP65SScD xM1B0yvDGufO4a4xc6F6h2o8LEJ/rxHKyoZLo40eNUzVviOsRAYQXtb0Ds+KjihZ01USh ZoWlcZoL6R9l/59QwYzENlwVu75O4kHUhrE= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" Lets add some further mailbox helpers and convert the existing RpiFirmwareSetLed into a generic SetGpio() function. Signed-off-by: Jeremy Linton Reviewed-by: Andrei Warkentin --- .../Drivers/RpiFirmwareDxe/RpiFirmwareDxe.c | 240 +++++++++++++++++= +++- .../RaspberryPi/Include/Protocol/RpiFirmware.h | 25 +++ 2 files changed, 255 insertions(+), 10 deletions(-) diff --git a/Platform/RaspberryPi/Drivers/RpiFirmwareDxe/RpiFirmwareDxe.c b= /Platform/RaspberryPi/Drivers/RpiFirmwareDxe/RpiFirmwareDxe.c index ade91c9f34..bf74148bbb 100644 --- a/Platform/RaspberryPi/Drivers/RpiFirmwareDxe/RpiFirmwareDxe.c +++ b/Platform/RaspberryPi/Drivers/RpiFirmwareDxe/RpiFirmwareDxe.c @@ -1090,7 +1090,6 @@ RpiFirmwareSetClockRate ( return EFI_SUCCESS; } =20 - #pragma pack() typedef struct { UINT32 ClockId; @@ -1152,6 +1151,17 @@ RpiFirmwareGetClockRate ( STATIC EFI_STATUS EFIAPI +RpiFirmwareGetCurrentClockState ( + IN UINT32 ClockId, + OUT UINT32 *ClockState + ) +{ + return RpiFirmwareGetClockRate (ClockId, RPI_MBOX_GET_CLOCK_STATE, Clock= State); +} + +STATIC +EFI_STATUS +EFIAPI RpiFirmwareGetCurrentClockRate ( IN UINT32 ClockId, OUT UINT32 *ClockRate @@ -1181,6 +1191,63 @@ RpiFirmwareGetMinClockRate ( { return RpiFirmwareGetClockRate (ClockId, RPI_MBOX_GET_MIN_CLOCK_RATE, Cl= ockRate); } + +#pragma pack() +typedef struct { + UINT32 ClockId; + UINT32 ClockState; +} RPI_FW_GET_CLOCK_STATE_TAG; + +typedef struct { + RPI_FW_BUFFER_HEAD BufferHead; + RPI_FW_TAG_HEAD TagHead; + RPI_FW_GET_CLOCK_STATE_TAG TagBody; + UINT32 EndTag; +} RPI_FW_SET_CLOCK_STATE_CMD; +#pragma pack() + +STATIC +EFI_STATUS +RpiFirmwareSetClockState ( + IN UINT32 ClockId, + IN UINT32 ClockState + ) +{ + RPI_FW_SET_CLOCK_STATE_CMD *Cmd; + EFI_STATUS Status; + UINT32 Result; + + if (!AcquireSpinLockOrFail (&mMailboxLock)) { + DEBUG ((DEBUG_ERROR, "%a: failed to acquire spinlock\n", __FUNCTION__)= ); + return EFI_DEVICE_ERROR; + } + + Cmd =3D mDmaBuffer; + ZeroMem (Cmd, sizeof (*Cmd)); + + Cmd->BufferHead.BufferSize =3D sizeof (*Cmd); + Cmd->BufferHead.Response =3D 0; + Cmd->TagHead.TagId =3D RPI_MBOX_SET_CLOCK_STATE; + Cmd->TagHead.TagSize =3D sizeof (Cmd->TagBody); + Cmd->TagHead.TagValueSize =3D 0; + Cmd->TagBody.ClockId =3D ClockId; + Cmd->TagBody.ClockState =3D ClockState; + Cmd->EndTag =3D 0; + + Status =3D MailboxTransaction (Cmd->BufferHead.BufferSize, RPI_MBOX_VC_C= HANNEL, &Result); + + ReleaseSpinLock (&mMailboxLock); + + if (EFI_ERROR (Status) || + Cmd->BufferHead.Response !=3D RPI_MBOX_RESP_SUCCESS) { + DEBUG ((DEBUG_ERROR, + "%a: mailbox transaction error: Status =3D=3D %r, Response =3D=3D 0x= %x\n", + __FUNCTION__, Status, Cmd->BufferHead.Response)); + return EFI_DEVICE_ERROR; + } + + return EFI_SUCCESS; +} =20 #pragma pack() typedef struct { @@ -1199,8 +1266,9 @@ typedef struct { STATIC VOID EFIAPI -RpiFirmwareSetLed ( - IN BOOLEAN On +RpiFirmwareSetGpio ( + IN UINT32 Gpio, + IN BOOLEAN State ) { RPI_FW_SET_GPIO_CMD *Cmd; @@ -1220,14 +1288,10 @@ RpiFirmwareSetLed ( Cmd->TagHead.TagId =3D RPI_MBOX_SET_GPIO; Cmd->TagHead.TagSize =3D sizeof (Cmd->TagBody); /* - * GPIO_PIN_2 =3D Activity LED - * GPIO_PIN_4 =3D HDMI Detect (Input / Active Low) - * GPIO_PIN_7 =3D Power LED (Input / Active Low) - * * There's also a 128 pin offset. */ - Cmd->TagBody.Pin =3D 128 + 2; - Cmd->TagBody.State =3D On; + Cmd->TagBody.Pin =3D 128 + Gpio; + Cmd->TagBody.State =3D State; Cmd->TagHead.TagValueSize =3D 0; Cmd->EndTag =3D 0; =20 @@ -1242,6 +1306,16 @@ RpiFirmwareSetLed ( __FUNCTION__, Status, Cmd->BufferHead.Response)); } } + +STATIC +VOID +EFIAPI +RpiFirmwareSetLed ( + IN BOOLEAN On + ) +{ + RpiFirmwareSetGpio (RPI_EXP_GPIO_LED, On); +} =20 #pragma pack() typedef struct { @@ -1299,6 +1373,149 @@ RpiFirmwareNotifyXhciReset ( return Status; } =20 +#pragma pack() +typedef struct { + UINT32 Gpio; + UINT32 Direction; + UINT32 Polarity; + UINT32 TermEn; + UINT32 TermPullUp; +} RPI_FW_GPIO_GET_CFG_TAG; + +typedef struct { + RPI_FW_BUFFER_HEAD BufferHead; + RPI_FW_TAG_HEAD TagHead; + RPI_FW_GPIO_GET_CFG_TAG TagBody; + UINT32 EndTag; +} RPI_FW_NOTIFY_GPIO_GET_CFG_CMD; +#pragma pack() + + +STATIC +EFI_STATUS +EFIAPI +RpiFirmwareNotifyGpioGetCfg ( + IN UINTN Gpio, + IN UINT32 *Polarity + ) +{ + RPI_FW_NOTIFY_GPIO_GET_CFG_CMD *Cmd; + EFI_STATUS Status; + UINT32 Result; + + if (!AcquireSpinLockOrFail (&mMailboxLock)) { + DEBUG ((DEBUG_ERROR, "%a: failed to acquire spinlock\n", __FUNCTION__)= ); + return EFI_DEVICE_ERROR; + } + + Cmd =3D mDmaBuffer; + ZeroMem (Cmd, sizeof (*Cmd)); + + Cmd->BufferHead.BufferSize =3D sizeof (*Cmd); + Cmd->BufferHead.Response =3D 0; + Cmd->TagHead.TagId =3D RPI_MBOX_GET_GPIO_CONFIG; + Cmd->TagHead.TagSize =3D sizeof (Cmd->TagBody); + Cmd->TagBody.Gpio =3D 128 + Gpio; + + Cmd->TagHead.TagValueSize =3D 0; + Cmd->EndTag =3D 0; + + Status =3D MailboxTransaction (Cmd->BufferHead.BufferSize, RPI_MBOX_VC_C= HANNEL, &Result); + + *Polarity =3D Cmd->TagBody.Polarity; + + ReleaseSpinLock (&mMailboxLock); + + if (EFI_ERROR (Status) || + Cmd->BufferHead.Response !=3D RPI_MBOX_RESP_SUCCESS) { + DEBUG ((DEBUG_ERROR, + "%a: mailbox transaction error: Status =3D=3D %r, Response =3D=3D 0= x%x\n", + __FUNCTION__, Status, Cmd->BufferHead.Response)); + } + + return Status; +} + + +#pragma pack() +typedef struct { + UINT32 Gpio; + UINT32 Direction; + UINT32 Polarity; + UINT32 TermEn; + UINT32 TermPullUp; + UINT32 State; +} RPI_FW_GPIO_SET_CFG_TAG; + +typedef struct { + RPI_FW_BUFFER_HEAD BufferHead; + RPI_FW_TAG_HEAD TagHead; + RPI_FW_GPIO_SET_CFG_TAG TagBody; + UINT32 EndTag; +} RPI_FW_NOTIFY_GPIO_SET_CFG_CMD; +#pragma pack() + + +STATIC +EFI_STATUS +EFIAPI +RpiFirmwareNotifyGpioSetCfg ( + IN UINTN Gpio, + IN UINTN Direction, + IN UINTN State + ) +{ + RPI_FW_NOTIFY_GPIO_SET_CFG_CMD *Cmd; + EFI_STATUS Status; + UINT32 Result; + + Status =3D RpiFirmwareNotifyGpioGetCfg (Gpio, &Result); + if (EFI_ERROR (Status)) { + DEBUG ((DEBUG_ERROR, "%a: Failed to get GPIO polarity\n", __FUNCTION__)= ); + Result =3D 0; //default polarity + } + + + if (!AcquireSpinLockOrFail (&mMailboxLock)) { + DEBUG ((DEBUG_ERROR, "%a: failed to acquire spinlock\n", __FUNCTION__)= ); + return EFI_DEVICE_ERROR; + } + + Cmd =3D mDmaBuffer; + ZeroMem (Cmd, sizeof (*Cmd)); + + Cmd->BufferHead.BufferSize =3D sizeof (*Cmd); + Cmd->BufferHead.Response =3D 0; + Cmd->TagHead.TagId =3D RPI_MBOX_SET_GPIO_CONFIG; + Cmd->TagHead.TagSize =3D sizeof (Cmd->TagBody); + + Cmd->TagBody.Gpio =3D 128 + Gpio; + Cmd->TagBody.Direction =3D Direction; + Cmd->TagBody.Polarity =3D Result; + Cmd->TagBody.TermEn =3D 0; + Cmd->TagBody.TermPullUp =3D 0; + Cmd->TagBody.State =3D State; + + Cmd->TagHead.TagValueSize =3D 0; + Cmd->EndTag =3D 0; + + Status =3D MailboxTransaction (Cmd->BufferHead.BufferSize, RPI_MBOX_VC_C= HANNEL, &Result); + + ReleaseSpinLock (&mMailboxLock); + + if (EFI_ERROR (Status) || + Cmd->BufferHead.Response !=3D RPI_MBOX_RESP_SUCCESS) { + DEBUG ((DEBUG_ERROR, + "%a: mailbox transaction error: Status =3D=3D %r, Response =3D=3D 0= x%x\n", + __FUNCTION__, Status, Cmd->BufferHead.Response)); + } + + RpiFirmwareSetGpio (Gpio,!State); + + + return Status; +} + STATIC RASPBERRY_PI_FIRMWARE_PROTOCOL mRpiFirmwareProtocol =3D { RpiFirmwareSetPowerState, RpiFirmwareGetMacAddress, @@ -1321,7 +1538,10 @@ STATIC RASPBERRY_PI_FIRMWARE_PROTOCOL mRpiFirmwarePr= otocol =3D { RpiFirmwareGetCpuName, RpiFirmwareGetArmMemory, RPiFirmwareGetModelInstalledMB, - RpiFirmwareNotifyXhciReset + RpiFirmwareNotifyXhciReset, + RpiFirmwareGetCurrentClockState, + RpiFirmwareSetClockState, + RpiFirmwareNotifyGpioSetCfg }; =20 /** diff --git a/Platform/RaspberryPi/Include/Protocol/RpiFirmware.h b/Platform= /RaspberryPi/Include/Protocol/RpiFirmware.h index 56a8d15a38..d841608e57 100644 --- a/Platform/RaspberryPi/Include/Protocol/RpiFirmware.h +++ b/Platform/RaspberryPi/Include/Protocol/RpiFirmware.h @@ -37,6 +37,20 @@ EFI_STATUS =20 typedef EFI_STATUS +(EFIAPI *GET_CLOCK_STATE) ( + IN UINT32 ClockId, + OUT UINT32 *ClockState + ); + +typedef +EFI_STATUS +(EFIAPI *SET_CLOCK_STATE) ( + IN UINT32 ClockId, + IN UINT32 ClockState + ); + +typedef +EFI_STATUS (EFIAPI *GET_CLOCK_RATE) ( IN UINT32 ClockId, OUT UINT32 *ClockRate @@ -149,6 +163,14 @@ EFI_STATUS UINTN FunctionNumber ); =20 +typedef=20 +EFI_STATUS +(EFIAPI *GPIO_SET_CFG) ( + UINTN Gpio, + UINTN Direction, + UINTN State + ); + typedef struct { SET_POWER_STATE SetPowerState; GET_MAC_ADDRESS GetMacAddress; @@ -172,6 +194,9 @@ typedef struct { GET_ARM_MEM GetArmMem; GET_MODEL_INSTALLED_MB GetModelInstalledMB; NOTIFY_XHCI_RESET NotifyXhciReset; + GET_CLOCK_STATE GetClockState; + SET_CLOCK_STATE SetClockState; + GPIO_SET_CFG SetGpioConfig; } RASPBERRY_PI_FIRMWARE_PROTOCOL; =20 extern EFI_GUID gRaspberryPiFirmwareProtocolGuid; --=20 2.13.7 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#69606): https://edk2.groups.io/g/devel/message/69606 Mute This Topic: https://groups.io/mt/79429626/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- From nobody Fri Apr 19 13:01:51 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) client-ip=66.175.222.108; envelope-from=bounce+27952+69607+1787277+3901457@groups.io; helo=mail02.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+69607+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=arm.com ARC-Seal: i=1; a=rsa-sha256; t=1609781876; cv=none; d=zohomail.com; s=zohoarc; b=lAebsDgwR83EXSOpRPHQBq0kXlO7oY9h1u++q1r/h4nMx8Ooe2fEpcALt6PcSOf9UifwQTubS0ywwuakdJECbmSgNG2K0JhOcWxxLL2NtGMZ5LZmgaIAW+xoURjeiWjkg4ngXVSdCLz1ZhBueBSP/BMY9z1S30mh29bqkt4/+7k= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1609781876; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Id:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:References:Sender:Subject:To; bh=CkipaTM6tQb5LmPohWQSpPXrvOuOofU561V3Ii2XsRI=; b=IfaBrolLOr7unNsqNfrcAI4UgkivqG1sNc8O1MNvhy0x7b927+Ae79yA9OzdATGm19hNxvFzECRrY8Dwkr1RwIxaZT+RNUO3dx7ut9K3z05HlvowqrHnA+y3pVORduSKmzJF+ev2TyMwZtdh50MaCyYH+HqgqFAqy+1cW6n02c4= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+69607+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) header.from= Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 1609781876620534.1681808169241; Mon, 4 Jan 2021 09:37:56 -0800 (PST) Return-Path: X-Received: by 127.0.0.2 with SMTP id IybeYY1788612x93Hzmamtrh; Mon, 04 Jan 2021 09:37:56 -0800 X-Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web11.16939.1609781875051065719 for ; Mon, 04 Jan 2021 09:37:55 -0800 X-Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id A023A101E; Mon, 4 Jan 2021 09:37:54 -0800 (PST) X-Received: from mammon-tx2.austin.arm.com (mammon-tx2.austin.arm.com [10.118.28.62]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id 8FC223F66E; Mon, 4 Jan 2021 09:37:54 -0800 (PST) From: "Jeremy Linton" To: devel@edk2.groups.io Cc: ard.biesheuvel@arm.com, leif@nuviainc.com, pete@akeo.ie, samer.el-haj-mahmoud@arm.com, Jeremy Linton , Andrei Warkentin Subject: [edk2-devel] [PATCH v3 3/7] Platform/RaspberryPi: Split MMC register defintions Date: Mon, 4 Jan 2021 11:37:27 -0600 Message-Id: <20210104173731.1413044-4-jeremy.linton@arm.com> In-Reply-To: <20210104173731.1413044-1-jeremy.linton@arm.com> References: <20210104173731.1413044-1-jeremy.linton@arm.com> MIME-Version: 1.0 Precedence: Bulk List-Unsubscribe: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,jeremy.linton@arm.com X-Gm-Message-State: X1Q6DpVnpneRrmbMH0iXm9RPx1787277AA= Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1609781876; bh=uBCnvX6oU4ZhX+4pB1z8w6ls2vepLStOnvSI/ZiRYik=; h=Cc:Date:From:Reply-To:Subject:To; b=C67sgN/z7BQUk2Xxd9G/kSbLN0uAiRi+7VdxTRlY942gOrtlze1RZinnTRdVIxl5Jba ONzPXMXY4mhe3z0iStTiCr6oeHB/5SQoUf7OsRfD8gHl9D2q7wrFYLxP8aGD2/87oVKLh JLVKA2wOwqwgyVZZvOBWp+pC3typxR3W6M8= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" The current MMC (really SDHCI) defintions are tied to the arasan controller. As we intend to reuse the definitions lets make the base address configurable when the driver loads. This assumes we won't ever want to run both the emmc2 and arasan sdhci controller at the same time. Signed-off-by: Jeremy Linton Reviewed-by: Andrei Warkentin --- .../Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe.c | 9 ++++- .../Bcm283x/Include/IndustryStandard/Bcm2836Sdio.h | 42 ++++++++++++------= ---- 2 files changed, 32 insertions(+), 19 deletions(-) diff --git a/Platform/RaspberryPi/Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe= .c b/Platform/RaspberryPi/Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe.c index 88e9126e35..0cb7e85b38 100644 --- a/Platform/RaspberryPi/Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe.c +++ b/Platform/RaspberryPi/Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe.c @@ -16,6 +16,7 @@ STATIC CARD_DETECT_STATE mCardDetectState =3D CardDetectR= equired; UINT32 LastExecutedCommand =3D (UINT32) -1; =20 STATIC RASPBERRY_PI_FIRMWARE_PROTOCOL *mFwProtocol; +STATIC UINTN MMCHS_BASE; =20 /** These SD commands are optional, according to the SD Spec @@ -763,7 +764,13 @@ MMCInitialize ( =20 DEBUG ((DEBUG_MMCHOST_SD, "ArasanMMCHost: MMCInitialize()\n")); =20 - if (!PcdGet32 (PcdSdIsArasan)) { + if (PcdGet32 (PcdSdIsArasan)) { + DEBUG ((DEBUG_INFO, "SD is routed to Arasan\n")); + MMCHS_BASE =3D MMCHS1_BASE; + } else if (RPI_MODEL =3D=3D 4) { + DEBUG ((DEBUG_INFO, "SD is routed to emmc2\n")); + MMCHS_BASE =3D MMCHS2_BASE; + } else { DEBUG ((DEBUG_INFO, "SD is not routed to Arasan\n")); return EFI_REQUEST_UNLOAD_IMAGE; } diff --git a/Silicon/Broadcom/Bcm283x/Include/IndustryStandard/Bcm2836Sdio.= h b/Silicon/Broadcom/Bcm283x/Include/IndustryStandard/Bcm2836Sdio.h index fd07b47170..e6892d36cf 100644 --- a/Silicon/Broadcom/Bcm283x/Include/IndustryStandard/Bcm2836Sdio.h +++ b/Silicon/Broadcom/Bcm283x/Include/IndustryStandard/Bcm2836Sdio.h @@ -13,15 +13,18 @@ =20 // MMC/SD/SDIO1 register definitions. #define MMCHS1_OFFSET 0x00300000 +#define MMCHS2_OFFSET 0x00340000 #define MMCHS1_BASE (BCM2836_SOC_REGISTERS + MMCHS1_OFFSET) +#define MMCHS2_BASE (BCM2836_SOC_REGISTERS + MMCHS2_OFFSET) #define MMCHS1_LENGTH 0x00000100 +#define MMCHS2_LENGTH 0x00000100 =20 -#define MMCHS_BLK (MMCHS1_BASE + 0x4) +#define MMCHS_BLK (MMCHS_BASE + 0x4) #define BLEN_512BYTES (0x200UL << 0) =20 -#define MMCHS_ARG (MMCHS1_BASE + 0x8) +#define MMCHS_ARG (MMCHS_BASE + 0x8) =20 -#define MMCHS_CMD (MMCHS1_BASE + 0xC) +#define MMCHS_CMD (MMCHS_BASE + 0xC) #define BCE_ENABLE BIT1 #define DDIR_READ BIT4 #define DDIR_WRITE (0x0UL << 4) @@ -43,13 +46,13 @@ #define INDX(CMD_INDX) (TYPE(CMD_TYPE_NORMAL) | _INDX(CMD_INDX)) #define INDX_ABORT(CMD_INDX) (TYPE(CMD_TYPE_ABORT) | _INDX(CMD_INDX)) =20 -#define MMCHS_RSP10 (MMCHS1_BASE + 0x10) -#define MMCHS_RSP32 (MMCHS1_BASE + 0x14) -#define MMCHS_RSP54 (MMCHS1_BASE + 0x18) -#define MMCHS_RSP76 (MMCHS1_BASE + 0x1C) -#define MMCHS_DATA (MMCHS1_BASE + 0x20) +#define MMCHS_RSP10 (MMCHS_BASE + 0x10) +#define MMCHS_RSP32 (MMCHS_BASE + 0x14) +#define MMCHS_RSP54 (MMCHS_BASE + 0x18) +#define MMCHS_RSP76 (MMCHS_BASE + 0x1C) +#define MMCHS_DATA (MMCHS_BASE + 0x20) =20 -#define MMCHS_PRES_STATE (MMCHS1_BASE + 0x24) +#define MMCHS_PRES_STATE (MMCHS_BASE + 0x24) #define CMDI_MASK BIT0 #define CMDI_ALLOWED (0x0UL << 0) #define CMDI_NOT_ALLOWED BIT0 @@ -58,17 +61,19 @@ #define DATI_NOT_ALLOWED BIT1 #define WRITE_PROTECT_OFF BIT19 =20 -#define MMCHS_HCTL (MMCHS1_BASE + 0x28) +#define MMCHS_HCTL (MMCHS_BASE + 0x28) #define DTW_1_BIT (0x0UL << 1) #define DTW_4_BIT BIT1 #define SDBP_MASK BIT8 #define SDBP_OFF (0x0UL << 8) #define SDBP_ON BIT8 +#define SDVS_MASK (0x7UL << 9) #define SDVS_1_8_V (0x5UL << 9) #define SDVS_3_0_V (0x6UL << 9) +#define SDVS_3_3_V (0x7UL << 9) #define IWE BIT24 =20 -#define MMCHS_SYSCTL (MMCHS1_BASE + 0x2C) +#define MMCHS_SYSCTL (MMCHS_BASE + 0x2C) #define ICE BIT0 #define ICS_MASK BIT1 #define ICS BIT1 @@ -84,7 +89,7 @@ #define SRC BIT25 #define SRD BIT26 =20 -#define MMCHS_INT_STAT (MMCHS1_BASE + 0x30) +#define MMCHS_INT_STAT (MMCHS_BASE + 0x30) #define CC BIT0 #define TC BIT1 #define BWR BIT4 @@ -96,7 +101,7 @@ #define DCRC BIT21 #define DEB BIT22 =20 -#define MMCHS_IE (MMCHS1_BASE + 0x34) +#define MMCHS_IE (MMCHS_BASE + 0x34) #define CC_EN BIT0 #define TC_EN BIT1 #define BWR_EN BIT4 @@ -112,7 +117,7 @@ #define BADA_EN BIT29 #define ALL_EN 0xFFFFFFFF =20 -#define MMCHS_ISE (MMCHS1_BASE + 0x38) +#define MMCHS_ISE (MMCHS_BASE + 0x38) #define CC_SIGEN BIT0 #define TC_SIGEN BIT1 #define BWR_SIGEN BIT4 @@ -127,14 +132,15 @@ #define CERR_SIGEN BIT28 #define BADA_SIGEN BIT29 =20 -#define MMCHS_AC12 (MMCHS1_BASE + 0x3C) +#define MMCHS_AC12 (MMCHS_BASE + 0x3C) +#define MMCHS_HC2R (MMCHS_BASE + 0x3E) =20 -#define MMCHS_CAPA (MMCHS1_BASE + 0x40) +#define MMCHS_CAPA (MMCHS_BASE + 0x40) #define VS30 BIT25 #define VS18 BIT26 =20 -#define MMCHS_CUR_CAPA (MMCHS1_BASE + 0x48) -#define MMCHS_REV (MMCHS1_BASE + 0xFC) +#define MMCHS_CUR_CAPA (MMCHS_BASE + 0x48) +#define MMCHS_REV (MMCHS_BASE + 0xFC) =20 #define BLOCK_COUNT_SHIFT 16 #define RCA_SHIFT 16 --=20 2.13.7 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#69607): https://edk2.groups.io/g/devel/message/69607 Mute This Topic: https://groups.io/mt/79429630/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- From nobody Fri Apr 19 13:01:51 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) client-ip=66.175.222.108; envelope-from=bounce+27952+69608+1787277+3901457@groups.io; helo=mail02.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+69608+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=arm.com ARC-Seal: i=1; a=rsa-sha256; t=1609781878; cv=none; d=zohomail.com; s=zohoarc; b=hq9Az/V4IPy2xGeJQlUguSxrO9tIvekYGnj2raBKiClVm568p6PfYVUaWx7enCNaPJ2EuhsHq93QRBaTa0gnQZEVyuQUg4HmpKuKwMhVEDr4KaZdcXq44JyEcpJu+hnkw+esvH2LF44LcRRbtTQTTpGIbSlBhhHb4wSx4KcsLy0= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1609781878; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Id:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:References:Sender:Subject:To; bh=3KHehP+/dUZ2hOjLLViOQsSiuUu4NB95y6JGlvqbX44=; b=lVYUn7IVCrCBEbA8q8R3D9fXrC55gqZzljHqFmXF/GHN3qCEJhQU0ONPETp7si62FHAtFQXhgd6pipwWFAygEC+QUjK6OEq9HoNgKxu5Y6wspUNK19h91kvZPIFKqP/vWnVJLMBpPASU1uznNVoAuL5ba0kfKLZeEnrWoLZ5foY= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+69608+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) header.from= Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 1609781878844118.20303996807343; Mon, 4 Jan 2021 09:37:58 -0800 (PST) Return-Path: X-Received: by 127.0.0.2 with SMTP id 2SeGYY1788612xqlH1Kq9YNM; Mon, 04 Jan 2021 09:37:58 -0800 X-Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web12.17289.1609781875558865415 for ; Mon, 04 Jan 2021 09:37:55 -0800 X-Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 3ED461063; Mon, 4 Jan 2021 09:37:55 -0800 (PST) X-Received: from mammon-tx2.austin.arm.com (mammon-tx2.austin.arm.com [10.118.28.62]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id 356B83F66E; Mon, 4 Jan 2021 09:37:55 -0800 (PST) From: "Jeremy Linton" To: devel@edk2.groups.io Cc: ard.biesheuvel@arm.com, leif@nuviainc.com, pete@akeo.ie, samer.el-haj-mahmoud@arm.com, Jeremy Linton , Andrei Warkentin Subject: [edk2-devel] [PATCH v3 4/7] Platform/RaspberryPi/Arasan: Add write delay and voltage/clock config Date: Mon, 4 Jan 2021 11:37:28 -0600 Message-Id: <20210104173731.1413044-5-jeremy.linton@arm.com> In-Reply-To: <20210104173731.1413044-1-jeremy.linton@arm.com> References: <20210104173731.1413044-1-jeremy.linton@arm.com> MIME-Version: 1.0 Precedence: Bulk List-Unsubscribe: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,jeremy.linton@arm.com X-Gm-Message-State: urh9Qypoc3GNZZeKZeQzZl3Kx1787277AA= Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1609781878; bh=EqOthVN1dIV4WW6iHs68vrRKtJUv/rUCZxPY+eBgZ5Q=; h=Cc:Date:From:Reply-To:Subject:To; b=EQMIuHthosRBhj7vbltFP/NQfxCO74H5q38Jgj+pUNg2gZTQinfuaAVcipQWFdP7oUM aq/Pj/eAM/wffdiCJACJRxV5t2gVCZs7IbLXMK8bMEDLbQZ4GdMPZMq9q6xgVuNAjAM8+ VGMXMSn6jD/V2K08qeIYhNqZvklNJ7BsO1o= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" The uboot and linux drivers have notes that there is a clock domain crossing problem that happens with back to back writes to the sd controllers on the rpi. Its not clear if this is still applicable to the rpi4/emmc2 but it seems wise to add it. Futher, we need to assure that the card voltage is set to 3.3V, and we should try and follow some of the SDHCI docs when it comes to changing the clock. Signed-off-by: Jeremy Linton Reviewed-by: Andrei Warkentin --- .../Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe.c | 112 +++++++++++++++++= ---- .../Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe.h | 1 + 2 files changed, 93 insertions(+), 20 deletions(-) diff --git a/Platform/RaspberryPi/Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe= .c b/Platform/RaspberryPi/Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe.c index 0cb7e85b38..a7b538a91a 100644 --- a/Platform/RaspberryPi/Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe.c +++ b/Platform/RaspberryPi/Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe.c @@ -18,6 +18,56 @@ UINT32 LastExecutedCommand =3D (UINT32) -1; STATIC RASPBERRY_PI_FIRMWARE_PROTOCOL *mFwProtocol; STATIC UINTN MMCHS_BASE; =20 +STATIC +UINT32 +EFIAPI +SdMmioWrite32 ( + IN UINTN Address, + IN UINT32 Value + ) +{ + UINT32 ret; + ret =3D (UINT32)MmioWrite32 (Address, Value); + // There is a bug about clock domain crossing on writes, delay to avoid = it + gBS->Stall (STALL_AFTER_REG_WRITE_US); + return ret; +} + +STATIC +UINT32 +EFIAPI +SdMmioOr32 ( + IN UINTN Address, + IN UINT32 OrData + ) +{ + return SdMmioWrite32 (Address, MmioRead32 (Address) | OrData); +} + +STATIC +UINT32 +EFIAPI +SdMmioAnd32 ( + IN UINTN Address, + IN UINT32 AndData + ) +{ + return SdMmioWrite32 (Address, MmioRead32 (Address) & AndData); +} + +STATIC +UINT32 +EFIAPI +SdMmioAndThenOr32 ( + IN UINTN Address, + IN UINT32 AndData, + IN UINT32 OrData + ) +{ + return SdMmioWrite32 (Address, (MmioRead32 (Address) & AndData) | OrData= ); +} + + /** These SD commands are optional, according to the SD Spec **/ @@ -175,7 +225,9 @@ SoftReset ( IN UINT32 Mask ) { - MmioOr32 (MMCHS_SYSCTL, Mask); + DEBUG ((DEBUG_MMCHOST_SD, "SoftReset with mask 0x%x\n", Mask)); + + SdMmioOr32 (MMCHS_SYSCTL, Mask); if (PollRegisterWithMask (MMCHS_SYSCTL, Mask, 0) =3D=3D EFI_TIMEOUT) { DEBUG ((DEBUG_ERROR, "Failed to SoftReset with mask 0x%x\n", Mask)); return EFI_TIMEOUT; @@ -326,29 +378,29 @@ MMCSendCommand ( } =20 if (IsAppCmd && MmcCmd =3D=3D ACMD22) { - MmioWrite32 (MMCHS_BLK, 4); + SdMmioWrite32 (MMCHS_BLK, 4); } else if (IsAppCmd && MmcCmd =3D=3D ACMD51) { - MmioWrite32 (MMCHS_BLK, 8); + SdMmioWrite32 (MMCHS_BLK, 8); } else if (!IsAppCmd && MmcCmd =3D=3D CMD6) { - MmioWrite32 (MMCHS_BLK, 64); + SdMmioWrite32 (MMCHS_BLK, 64); } else if (IsADTCCmd) { - MmioWrite32 (MMCHS_BLK, BLEN_512BYTES); + SdMmioWrite32 (MMCHS_BLK, BLEN_512BYTES); } =20 // Set Data timeout counter value to max value. - MmioAndThenOr32 (MMCHS_SYSCTL, (UINT32) ~DTO_MASK, DTO_VAL); + SdMmioAndThenOr32 (MMCHS_SYSCTL, (UINT32) ~DTO_MASK, DTO_VAL); =20 // // Clear Interrupt Status Register, but not the Card Inserted bit // to avoid messing with card detection logic. // - MmioWrite32 (MMCHS_INT_STAT, ALL_EN & ~(CARD_INS)); + SdMmioWrite32 (MMCHS_INT_STAT, ALL_EN & ~(CARD_INS)); =20 // Set command argument register - MmioWrite32 (MMCHS_ARG, Argument); + SdMmioWrite32 (MMCHS_ARG, Argument); =20 // Send the command - MmioWrite32 (MMCHS_CMD, MmcCmd); + SdMmioWrite32 (MMCHS_CMD, MmcCmd); =20 // Check for the command status. while (RetryCount < MAX_RETRY_COUNT) { @@ -373,7 +425,7 @@ MMCSendCommand ( =20 // Check if command is completed. if ((MmcStatus & CC) =3D=3D CC) { - MmioWrite32 (MMCHS_INT_STAT, CC); + SdMmioWrite32 (MMCHS_INT_STAT, CC); break; } =20 @@ -428,6 +480,21 @@ MMCNotifyState ( return Status; } =20 + DEBUG ((DEBUG_MMCHOST_SD, "ArasanMMCHost: CAP %X CAPH %X\n", MmioRea= d32(MMCHS_CAPA),MmioRead32(MMCHS_CUR_CAPA))); + + // Lets switch to card detect test mode. + SdMmioOr32 (MMCHS_HCTL, BIT7|BIT6); + + // set card voltage + SdMmioAnd32 (MMCHS_HCTL, ~SDBP_ON); + SdMmioAndThenOr32 (MMCHS_HCTL, (UINT32) ~SDBP_MASK, SDVS_3_3_V); + SdMmioOr32 (MMCHS_HCTL, SDBP_ON); + + DEBUG ((DEBUG_MMCHOST_SD, "ArasanMMCHost: AC12 %X HCTL %X\n", MmioRe= ad32(MMCHS_AC12),MmioRead32(MMCHS_HCTL))); + + // First turn off the clock + SdMmioAnd32 (MMCHS_SYSCTL, ~CEN); + // Attempt to set the clock to 400Khz which is the expected initiali= zation speed Status =3D CalculateClockFrequencyDivisor (400000, &Divisor, NULL); if (EFI_ERROR (Status)) { @@ -436,10 +503,15 @@ MMCNotifyState ( } =20 // Set Data Timeout Counter value, set clock frequency, enable inter= nal clock - MmioOr32 (MMCHS_SYSCTL, DTO_VAL | Divisor | CEN | ICS | ICE); + SdMmioOr32 (MMCHS_SYSCTL, DTO_VAL | Divisor | CEN | ICS | ICE); + SdMmioOr32 (MMCHS_HCTL, SDBP_ON); + // wait for ICS + while ((MmioRead32 (MMCHS_SYSCTL) & ICS_MASK) !=3D ICS); + + DEBUG ((DEBUG_MMCHOST_SD, "ArasanMMCHost: AC12 %X HCTL %X\n", MmioRe= ad32(MMCHS_AC12),MmioRead32(MMCHS_HCTL))); =20 // Enable interrupts - MmioWrite32 (MMCHS_IE, ALL_EN); + SdMmioWrite32 (MMCHS_IE, ALL_EN); } break; case MmcIdleState: @@ -452,7 +524,7 @@ MMCNotifyState ( ClockFrequency =3D 25000000; =20 // First turn off the clock - MmioAnd32 (MMCHS_SYSCTL, ~CEN); + SdMmioAnd32 (MMCHS_SYSCTL, ~CEN); =20 Status =3D CalculateClockFrequencyDivisor (ClockFrequency, &Divisor, N= ULL); if (EFI_ERROR (Status)) { @@ -462,13 +534,13 @@ MMCNotifyState ( } =20 // Setup new divisor - MmioAndThenOr32 (MMCHS_SYSCTL, (UINT32) ~CLKD_MASK, Divisor); + SdMmioAndThenOr32 (MMCHS_SYSCTL, (UINT32) ~CLKD_MASK, Divisor); =20 // Wait for the clock to stabilise while ((MmioRead32 (MMCHS_SYSCTL) & ICS_MASK) !=3D ICS); =20 // Set Data Timeout Counter value, set clock frequency, enable interna= l clock - MmioOr32 (MMCHS_SYSCTL, CEN); + SdMmioOr32 (MMCHS_SYSCTL, CEN); break; case MmcTransferState: break; @@ -635,7 +707,7 @@ MMCReadBlockData ( while (RetryCount < MAX_RETRY_COUNT) { MmcStatus =3D MmioRead32 (MMCHS_INT_STAT); if ((MmcStatus & BRR) !=3D 0) { - MmioWrite32 (MMCHS_INT_STAT, BRR); + SdMmioWrite32 (MMCHS_INT_STAT, BRR); /* * Data is ready. */ @@ -662,7 +734,7 @@ MMCReadBlockData ( gBS->Stall (STALL_AFTER_READ_US); } =20 - MmioWrite32 (MMCHS_INT_STAT, BRR); + SdMmioWrite32 (MMCHS_INT_STAT, BRR); return EFI_SUCCESS; } =20 @@ -699,13 +771,13 @@ MMCWriteBlockData ( while (RetryCount < MAX_RETRY_COUNT) { MmcStatus =3D MmioRead32 (MMCHS_INT_STAT); if ((MmcStatus & BWR) !=3D 0) { - MmioWrite32 (MMCHS_INT_STAT, BWR); + SdMmioWrite32 (MMCHS_INT_STAT, BWR); /* * Can write data. */ mFwProtocol->SetLed (TRUE); for (Count =3D 0; Count < BlockLen; Count +=3D 4, Buffer++) { - MmioWrite32 (MMCHS_DATA, *Buffer); + SdMmioWrite32 (MMCHS_DATA, *Buffer); } =20 mFwProtocol->SetLed (FALSE); @@ -726,7 +798,7 @@ MMCWriteBlockData ( gBS->Stall (STALL_AFTER_WRITE_US); } =20 - MmioWrite32 (MMCHS_INT_STAT, BWR); + SdMmioWrite32 (MMCHS_INT_STAT, BWR); return EFI_SUCCESS; } =20 diff --git a/Platform/RaspberryPi/Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe= .h b/Platform/RaspberryPi/Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe.h index 6cd600f738..e94606cc5b 100644 --- a/Platform/RaspberryPi/Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe.h +++ b/Platform/RaspberryPi/Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe.h @@ -37,6 +37,7 @@ #define STALL_AFTER_REC_RESP_US (50) #define STALL_AFTER_WRITE_US (200) #define STALL_AFTER_READ_US (20) +#define STALL_AFTER_REG_WRITE_US (10) #define STALL_AFTER_RETRY_US (20) =20 #define MAX_DIVISOR_VALUE 1023 --=20 2.13.7 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#69608): https://edk2.groups.io/g/devel/message/69608 Mute This Topic: https://groups.io/mt/79429631/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- From nobody Fri Apr 19 13:01:51 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) client-ip=66.175.222.108; envelope-from=bounce+27952+69609+1787277+3901457@groups.io; helo=mail02.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+69609+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=arm.com ARC-Seal: i=1; a=rsa-sha256; t=1609781887; cv=none; d=zohomail.com; s=zohoarc; b=XPngcegr4KS34PKrnuDNKSvTcgezgmOcN/q7XbO+rpsSAI7Kacr/tfAyukCms+k1Wnusyya6h6LUuIPa3EDOFN7+0a5f+ljW7Xxg+uzpo2Ju+zuArFGLkiD5Lmkl97Jh72vZUIZt7VNAGti1baTEXWGX3jOTJpFx/wQU0NbrAoo= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1609781887; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Id:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:References:Sender:Subject:To; bh=MvD7pOVOLYCgaowajp99Sk23irpatVxZbwF4X7REqdw=; b=bntnrWnjsn3PHz6e0i2zLZa52gxYSm25vghtgRCbytq+VCberY+8gmYHO+nutd2xmV1nVELQShVzCqcXuXwMJFN2Dz8N3LqqOdj5cvuSV6VGY/04HNUU41ich4fOWVSjh/zPVh4dMWEZJIHjSoCnucPVOq3m1Y+h1AYhDaEOcak= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+69609+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) header.from= Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 1609781887539840.6276418686025; Mon, 4 Jan 2021 09:38:07 -0800 (PST) Return-Path: X-Received: by 127.0.0.2 with SMTP id JHbsYY1788612xIbXucfryza; Mon, 04 Jan 2021 09:38:07 -0800 X-Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web10.17118.1609781876126923985 for ; Mon, 04 Jan 2021 09:37:56 -0800 X-Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id CBE43113E; Mon, 4 Jan 2021 09:37:55 -0800 (PST) X-Received: from mammon-tx2.austin.arm.com (mammon-tx2.austin.arm.com [10.118.28.62]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id C5C183F66E; Mon, 4 Jan 2021 09:37:55 -0800 (PST) From: "Jeremy Linton" To: devel@edk2.groups.io Cc: ard.biesheuvel@arm.com, leif@nuviainc.com, pete@akeo.ie, samer.el-haj-mahmoud@arm.com, Jeremy Linton , Andrei Warkentin Subject: [edk2-devel] [PATCH v3 5/7] Platform/RaspberryPi/Arasan: Select the correct base frequency Date: Mon, 4 Jan 2021 11:37:29 -0600 Message-Id: <20210104173731.1413044-6-jeremy.linton@arm.com> In-Reply-To: <20210104173731.1413044-1-jeremy.linton@arm.com> References: <20210104173731.1413044-1-jeremy.linton@arm.com> MIME-Version: 1.0 Precedence: Bulk List-Unsubscribe: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,jeremy.linton@arm.com X-Gm-Message-State: Dqc7egHFTqO0ZLi7AWw6Xbtax1787277AA= Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1609781887; bh=UPMGha1mvck7/yWPcWyqCc7uxQQG0JlJYcty7GAqnzw=; h=Cc:Date:From:Reply-To:Subject:To; b=bmx5BaN/iQjmfL2Pxktmb6ILh0U3G+D3YeG6qSC+5/Wz2yrx1LnM69byW+jty4Kb+BT dCLILMSxMiTVpASnL/oxbrDGyy850QACirW5wKy97PM9/P2R75WKF4h5QKZ11gy1SR2qK NfCWCUj6Uhmf+qKz7umaTikcBPO4YwSAj1g= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" The firmware reports the emmc2 frequency with a slightly different mailbox command, lets select the correct one based on which controller we are binding to. Signed-off-by: Jeremy Linton Reviewed-by: Andrei Warkentin --- .../RaspberryPi/Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe.c | 10 ++++++= +--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/Platform/RaspberryPi/Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe= .c b/Platform/RaspberryPi/Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe.c index a7b538a91a..b0f03cefc9 100644 --- a/Platform/RaspberryPi/Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe.c +++ b/Platform/RaspberryPi/Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe.c @@ -250,7 +250,11 @@ CalculateClockFrequencyDivisor ( UINT32 Divisor; UINT32 BaseFrequency =3D 0; =20 - Status =3D mFwProtocol->GetClockRate (RPI_MBOX_CLOCK_RATE_EMMC, &BaseFre= quency); + if (PcdGet32 (PcdSdIsArasan)) { + Status =3D mFwProtocol->GetClockRate (RPI_MBOX_CLOCK_RATE_EMMC, &BaseF= requency); + } else { + Status =3D mFwProtocol->GetClockRate (RPI_MBOX_CLOCK_RATE_EMMC2, &Base= Frequency); + } if (EFI_ERROR (Status)) { DEBUG ((DEBUG_ERROR, "Couldn't get RPI_MBOX_CLOCK_RATE_EMMC\n")); return Status; @@ -472,8 +476,8 @@ MMCNotifyState ( switch (State) { case MmcHwInitializationState: { - EFI_STATUS Status; - UINT32 Divisor; + + DEBUG ((DEBUG_MMCHOST_SD, "ArasanMMCHost: current divisor %x\n", Mmi= oRead32(MMCHS_SYSCTL))); =20 Status =3D SoftReset (SRA); if (EFI_ERROR (Status)) { --=20 2.13.7 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#69609): https://edk2.groups.io/g/devel/message/69609 Mute This Topic: https://groups.io/mt/79429632/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- From nobody Fri Apr 19 13:01:51 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) client-ip=66.175.222.108; envelope-from=bounce+27952+69610+1787277+3901457@groups.io; helo=mail02.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+69610+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=arm.com ARC-Seal: i=1; a=rsa-sha256; t=1609781886; cv=none; d=zohomail.com; s=zohoarc; b=hsnsMPCxQIXAVOSTEpWQ5tDsMolvUG5tDX4cAdZJ9V1Pd6IhtFxK/byrHoZ1kKnydgk7IGh8/Mx16v+g3IvuS4CPZF4xK/G7BdFWt11UJXzR5aPM1LR6AtoPz3YImJG7qEMGJA0S7uC6I20BmV2Idg/UxBnDSFfdQPtAAfWePu8= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1609781886; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Id:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:References:Sender:Subject:To; bh=0aYK/39ziEbVxLRhB23LbU33UXmx+d/86euqogK2JxI=; b=UpRH7JEkgv987HXRqFLdY722O7MW502/e0WA3ak+NOcrtiSTB0bg7Sx90+A81h7l5u9oqLxgLb+wM/2+aZ4JGfzD0ksVzSS30NG/x4ebarN3z7ecv0DOuaaqgXfNiUiGnqVogLiCM3LcN2oc83zq8iRz4bzAGC6dI1b2JR/pWtA= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+69610+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) header.from= Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 1609781886168831.9275142980473; Mon, 4 Jan 2021 09:38:06 -0800 (PST) Return-Path: X-Received: by 127.0.0.2 with SMTP id Nne0YY1788612xG2uLEDWb7y; Mon, 04 Jan 2021 09:38:00 -0800 X-Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web08.17074.1609781876750647973 for ; Mon, 04 Jan 2021 09:37:56 -0800 X-Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 666CA11D4; Mon, 4 Jan 2021 09:37:56 -0800 (PST) X-Received: from mammon-tx2.austin.arm.com (mammon-tx2.austin.arm.com [10.118.28.62]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id 6037F3F66E; Mon, 4 Jan 2021 09:37:56 -0800 (PST) From: "Jeremy Linton" To: devel@edk2.groups.io Cc: ard.biesheuvel@arm.com, leif@nuviainc.com, pete@akeo.ie, samer.el-haj-mahmoud@arm.com, Jeremy Linton , Andrei Warkentin Subject: [edk2-devel] [PATCH v3 6/7] Platform/RaspberryPi: Power up sd, and tweak GPIOs Date: Mon, 4 Jan 2021 11:37:30 -0600 Message-Id: <20210104173731.1413044-7-jeremy.linton@arm.com> In-Reply-To: <20210104173731.1413044-1-jeremy.linton@arm.com> References: <20210104173731.1413044-1-jeremy.linton@arm.com> MIME-Version: 1.0 Precedence: Bulk List-Unsubscribe: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,jeremy.linton@arm.com X-Gm-Message-State: NlF189TgfDCkdftkEImlXkb8x1787277AA= Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1609781880; bh=cMX3cZYbPLjrIBhti7Om/J6t54uAVNt61Qp1oXQgPwk=; h=Cc:Date:From:Reply-To:Subject:To; b=hrNpUQT5S/O7FtDfqY6Nwy+9ltnTVqS9YHqemqsBRTHjZLDdDIPA13NMKO3yc5rpFhk wV6B5+4b91TttFLGT3Y3Mw/o6yys6mkBF+2CLQlroF9D0xsBxoyH0eEH2uOJoITgeOd8o sFVvvB5OshsIzjFsd00myJPqmcdYttJIgII= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" It seems we should be powering up the sd cards, and possibly the clocks as well to assure they are setup properly before we attempt to access the controller. Signed-off-by: Jeremy Linton Reviewed-by: Andrei Warkentin --- Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxe.c | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxe.c b/Platform/= RaspberryPi/Drivers/ConfigDxe/ConfigDxe.c index 28f57438c5..e55ed63efa 100644 --- a/Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxe.c +++ b/Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxe.c @@ -552,6 +552,16 @@ ApplyVariables ( GpioPinFuncSet (37, GPIO_FSEL_ALT3); GpioPinFuncSet (38, GPIO_FSEL_ALT3); GpioPinFuncSet (39, GPIO_FSEL_ALT3); + + /* + * power and clock everything by default + */ + Status =3D mFwProtocol->SetPowerState (RPI_MBOX_POWER_STATE_SDHCI, + TRUE, TRUE); //SD on with wait + Status =3D mFwProtocol->SetGpioConfig (RPI_EXP_GPIO_SD_VOLT,=20 + RPI_EXP_GPIO_DIR_OUT, TRUE); //= 3.3v + Status =3D mFwProtocol->SetClockState (RPI_MBOX_CLOCK_RATE_EMMC2, TR= UE); + Status =3D mFwProtocol->SetClockState (RPI_MBOX_CLOCK_RATE_EMMC, TRU= E); } } else { DEBUG ((DEBUG_ERROR, "Model Family %d not supported...\n", mModelFamil= y)); --=20 2.13.7 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#69610): https://edk2.groups.io/g/devel/message/69610 Mute This Topic: https://groups.io/mt/79429634/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- From nobody Fri Apr 19 13:01:51 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) client-ip=66.175.222.108; envelope-from=bounce+27952+69611+1787277+3901457@groups.io; helo=mail02.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+69611+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=arm.com ARC-Seal: i=1; a=rsa-sha256; t=1609781887; cv=none; d=zohomail.com; s=zohoarc; b=gb3v69ijSDiJKyXreCoDSAylAQ5cfy0uim2BUGWR95RrLVtqIDCe7Ds4BVGAmNXg76W6cqlWQrMnED1Krs4pTuwghh9j/Nzew0Tjgyt6tvss1Iyvbt6vNql5xkMuv6IPL1EjHdivfxSSWVPw6hHRNX7jIFgAtiRcuOL5P3P5DKk= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1609781887; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Id:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:References:Sender:Subject:To; bh=Iy41h1Zy9gkwmF5ltKsRs2nx69V92EqAmM9ZYVgZK2E=; b=IsTc+i5kCTsVk1/rttvZ1i6Zp0bA9wfw95WkvVuIIclA4TxwspTyNcexlAzpS1SAR+GFwWcRBk6cwm6t9DpYIy3Oum0+SGVcdmqcNhWMEN+ySjgAeqlaHYCHaJj1gfuFz+hO071LKaj7xtcVXKKo5bLrhD8tkn8AwL4t7BbeHU8= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+69611+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) header.from= Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 1609781887101743.9669659564596; Mon, 4 Jan 2021 09:38:07 -0800 (PST) Return-Path: X-Received: by 127.0.0.2 with SMTP id TDpzYY1788612xtdefHMbFtk; Mon, 04 Jan 2021 09:38:06 -0800 X-Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mx.groups.io with SMTP id smtpd.web12.17290.1609781877318043635 for ; Mon, 04 Jan 2021 09:37:57 -0800 X-Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 0F8CE12FC; Mon, 4 Jan 2021 09:37:57 -0800 (PST) X-Received: from mammon-tx2.austin.arm.com (mammon-tx2.austin.arm.com [10.118.28.62]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id F41A83F66E; Mon, 4 Jan 2021 09:37:56 -0800 (PST) From: "Jeremy Linton" To: devel@edk2.groups.io Cc: ard.biesheuvel@arm.com, leif@nuviainc.com, pete@akeo.ie, samer.el-haj-mahmoud@arm.com, Jeremy Linton , Andrei Warkentin Subject: [edk2-devel] [PATCH v3 7/7] Platform/RaspberryPi: Correct device path removal. Date: Mon, 4 Jan 2021 11:37:31 -0600 Message-Id: <20210104173731.1413044-8-jeremy.linton@arm.com> In-Reply-To: <20210104173731.1413044-1-jeremy.linton@arm.com> References: <20210104173731.1413044-1-jeremy.linton@arm.com> MIME-Version: 1.0 Precedence: Bulk List-Unsubscribe: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,jeremy.linton@arm.com X-Gm-Message-State: 2okDzW6BjncxOJshPm4qcES3x1787277AA= Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1609781886; bh=6fL67PzLm7z22ePCqAVJE18a7Tc0MJ4SysLJp5CEydc=; h=Cc:Date:From:Reply-To:Subject:To; b=IHithoHCvcv0jL1BX4uL5b0z5IuXWgSFFJAARAeVzBlyzzoilDroNCwB5e6flSztu1k FMHo1NVW9ZadNyTGqs9V0DL2hhpvzPkk1XdUoRTY2PQLvBRHIFTBSww0JE/iCUnTa9lGd AqGxZyyq1RRF8MuNm9WecmlzB7+3z/e4mDA= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" The "arasan" driver now works with the emmc2 device. This means that both the PcdSdIsArasan and the !PcdSdIsArasan result in valid SD controllers on the rpi4. Lets avoid removing the "stale" boot entry, in this case which also has the side effect of avoiding a boot assert when emmc2 is selected. Signed-off-by: Jeremy Linton Reviewed-by: Andrei Warkentin --- Platform/RaspberryPi/Library/PlatformBootManagerLib/PlatformBm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Platform/RaspberryPi/Library/PlatformBootManagerLib/PlatformBm= .c b/Platform/RaspberryPi/Library/PlatformBootManagerLib/PlatformBm.c index fa46be7917..c2fc40b8ea 100644 --- a/Platform/RaspberryPi/Library/PlatformBootManagerLib/PlatformBm.c +++ b/Platform/RaspberryPi/Library/PlatformBootManagerLib/PlatformBm.c @@ -431,7 +431,7 @@ RemoveStaleBootOptions ( EFI_DEVICE_PATH_PROTOCOL *DevicePath =3D BootOptions[Index].FilePath; =20 if (CompareMem (&mArasan, DevicePath, GetDevicePathSize (DevicePath)) = =3D=3D 0) { - if (PcdGet32 (PcdSdIsArasan)) { + if (PcdGet32 (PcdSdIsArasan) || RPI_MODEL =3D=3D 4) { continue; } } else if (CompareMem (&mSDHost, DevicePath, GetDevicePathSize (Device= Path)) =3D=3D 0) { --=20 2.13.7 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#69611): https://edk2.groups.io/g/devel/message/69611 Mute This Topic: https://groups.io/mt/79429635/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-