<div dir="ltr"><div dir="ltr">This version of MiniComm will work without vcredist (compiled with /MT): <a href="https://drive.google.com/file/d/1v04RHUEIqlhrg2TUU_d_G4Z6lbZ1l44g/view?usp=sharing">https://drive.google.com/file/d/1v04RHUEIqlhrg2TUU_d_G4Z6lbZ1l44g/view?usp=sharing</a></div><div dir="ltr"> </div><div>--Mario</div></div> <div class="gmail_quote"><div dir="ltr" class="gmail_attr">În lun., 7 iun. 2021 la 11:31, Sunny Wang <<a href="mailto:Sunny.Wang@arm.com">Sunny.Wang@arm.com</a>> a scris: </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> <div lang="EN-US" style="overflow-wrap: break-word;"> <div class="gmail-m_-8588933396907808617WordSection1"> Thanks for all the valuable comments, Mario. I learned a lot from your comments. I just sent v4 for addressing your comments. By the way, for <a href="https://github.com/ms-iot/samples/tree/develop/MinComm" target="_blank">samples/MinComm at develop · ms-iot/samples (github.com)</a> <ol style="margin-top:0in" start="1" type="1"> <li class="gmail-m_-8588933396907808617MsoListParagraph" style="margin-left:0in">I successfully build it. However, when I ran MinComm.exe on RPi4, I saw the “vcruntime140.dll was not found” error message, and couldn’t fix it by installing the latest Microsoft Visual C++ Redistributable (<a href="https://aka.ms/vs/16/release/VC_redist.arm64.exe" target="_blank">vc_redist.arm64.exe</a>) from <a href="https://support.microsoft.com/en-us/topic/the-latest-supported-visual-c-downloads-2647da03-1eea-4433-9aff-95f26a218cc0" target="_blank"> https://support.microsoft.com/en-us/topic/the-latest-supported-visual-c-downloads-2647da03-1eea-4433-9aff-95f26a218cc0</a>. </li><li class="gmail-m_-8588933396907808617MsoListParagraph" style="color:rgb(68,114,196);margin-left:0in"> Just a suggestion. I think it would be good to upload the precompiled binary somewhere. </li></ol> Best Regards, Sunny Wang <div style="border-right:none;border-bottom:none;border-left:none;border-top:1pt solid rgb(225,225,225);padding:3pt 0in 0in"> From: <a href="mailto:devel@edk2.groups.io" target="_blank">devel@edk2.groups.io</a> <<a href="mailto:devel@edk2.groups.io" target="_blank">devel@edk2.groups.io</a>> On Behalf Of Mario Balanica via <a href="http://groups.io" target="_blank">groups.io</a> Sent: Tuesday, June 1, 2021 12:36 AM To: Sunny Wang <<a href="mailto:Sunny.Wang@arm.com" target="_blank">Sunny.Wang@arm.com</a>> Cc: <a href="mailto:devel@edk2.groups.io" target="_blank">devel@edk2.groups.io</a>; Samer El-Haj-Mahmoud <<a href="mailto:Samer.El-Haj-Mahmoud@arm.com" target="_blank">Samer.El-Haj-Mahmoud@arm.com</a>>; Sami Mujawar <<a href="mailto:Sami.Mujawar@arm.com" target="_blank">Sami.Mujawar@arm.com</a>>; Jeremy Linton <<a href="mailto:Jeremy.Linton@arm.com" target="_blank">Jeremy.Linton@arm.com</a>>; Pete Batard <<a href="mailto:pete@akeo.ie" target="_blank">pete@akeo.ie</a>>; Ard Biesheuvel <<a href="mailto:ardb%2Btianocore@kernel.org" target="_blank">ardb+tianocore@kernel.org</a>> Subject: Re: [edk2-devel] [PATCH v3 2/2] Platform/RaspberryPi: Enable Bluetooth and UART in Windows OS </div> <div> <div> <div> <div> <div> edit: "GpioPinSet (31, 0);" should be "GpioPinSet (31, FALSE);" and maybe add the "fake the CTS signal as we don't support HW flow control yet" comment too. </div> <div> </div> <div> When this patch gets merged, I'll add my Bluetooth changes to <a href="https://github.com/raspberrypi/windows-drivers" target="_blank">raspberrypi/windows-drivers: Windows IOT drivers (github.com)</a> and submit another patch here to enable hardware flow control. </div> <div> </div> <div> --Mario </div> </div> </div> </div> </div> <div> <div> În lun., 31 mai 2021 la 18:56, Mario Bălănică <<a href="mailto:mariobalanica02@gmail.com" target="_blank">mariobalanica02@gmail.com</a>> a scris: </div> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> <div> <div> <div> <div> <div> <div> <div> <div> <div> <div> <div> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> If there is no COM port for PL011 UART, how can I check PL011 UART’s functionality? How can I use a telnet tool like PuTTY to send messages from RPi4 to my laptop? </blockquote> <div> </div> <div> You can use: <a href="https://github.com/ms-iot/samples/tree/develop/MinComm" target="_blank">samples/MinComm at develop · ms-iot/samples (github.com)</a> (let me know if you need a precompiled binary) </div> <div> </div> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> As for why I mux both UARTs to the BT chip, I was trying to address your comment “This always assumes that PL011 is used for Bluetooth”. Apparently, it didn’t address your comment. No matter whether I add “PinFunction (Exclusive, PullDown, BCM_ALT5, "<a>\\_SB.GDV0.GPI0</a>", 0, ResourceConsumer, , ) { 32, 33 }” or not, the Bluetooth has NOT worked when I configure PL011 UART as the primary serial console. Bluetooth only works when I configure Mini UART as the Primary serial console. I also confirmed that this problem also exists with RPi4 Release FW 1.26 + edk2-platforms-raspberrypi-pl011-bth-noflow.diff in <a href="https://github.com/worproject/RPi-Bluetooth-Testing/" target="_blank">https://github.com/worproject/RPi-Bluetooth-Testing/</a>. Do you have any idea to make the Bluetooth work with configuring PL011 UART as the primary serial console? </blockquote> <div> </div> <div> Bluetooth doesn't really work with the mini UART driver at the moment. The chip stalls after a few transfers, as far as I remember. Haven't looked much into it. </div> <div> </div> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> By the way, it is good to know the loading sequence. Do you know where I can quickly find this information without using WinDbg? </blockquote> <div> </div> <div> I'm not aware of any way to see the driver loading order without WinDbg. Also, verbose mode must be enabled before boot: <a href="https://docs.microsoft.com/en-us/windows-hardware/drivers/debugger/view---verbose-output" target="_blank">View Verbose Output - Windows drivers | Microsoft Docs</a> </div> <div> </div> <div> You could also test if Bluetooth will break by uninstalling / re-installing the mini UART driver while the system is running, then try to scan for nearby BT devices. </div> <div> </div> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> 1. What is the connection between GPIO 128 and Bluetooth (BT_ON)? Why did we add this line in the beginning? I just removed it by your request and according to the change in worproject/RPi-Bluetooth-Testing/, but I’m still worried about if we did this for some other purposes. </blockquote> <div> </div> <div> That is a leftover from the MS-IOT tables. It tries to toggle BT_ON from the GPIO expander (I2C) through RPIQ (mailbox), but the driver has no knowledge of it. And since the firmware blob turns on Bluetooth by default, the line is completely useless. </div> <div> </div> <div> We may want to have something like this in the future, for power management of the BT chip (will require some driver changes too). </div> <div> </div> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> 2. Why do we need to configure GPIO 32 and 33? Is the Bluetooth using GPIO 32 and 33? Or Is Windows OS checking GPIO 32 and 33 for Bluetooth? Where can I find the information? </blockquote> <div> </div> <div> Yes, the Bluetooth chip is available on pins 32, 33 (TX / RX). Also 30, 31 for hardware flow control (CTS / RTS). More info can be found in the <a href="https://datasheets.raspberrypi.org/bcm2711/bcm2711-peripherals.pdf" target="_blank"> BCM2711 datasheet</a> (5.3. Alternative Function Assignments) </div> <div> </div> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> 3. As for your comment about GPIO 31, I was confused, so I didn’t update anything for addressing this comment. My understanding was that It is intended to override the BCM_ALT2’s default setting from PullLow to PullNone to fack the CTS signal. If we change it to PullLow, it will be used for HW flow control, which is unwanted, isn’t it? What did you want me to change? Could you directly paste the code change you wanted here for my reference? </blockquote> <div> </div> <div> The BCM_ALT2 thing is a dirty hack. Pin 31 must be held LOW so that we can talk to the BT chip without flow control. It seems this can't be described with PinFunction, so I've relied on the default value of ALT2 for pin 31 (which turns out to be always LOW). </div> <div> </div> <div> My proposal is to move the pin muxing stuff in ConfigDxe, like this: </div> <div> </div> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> STATIC VOID </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> ApplyVariables ( </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> VOID </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> ) </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> { </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> ... </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> if (FanOnGpio) { </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> DEBUG ((DEBUG_INFO, "Fan enabled on GPIO %d\n", FanOnGpio)); </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> GpioPinFuncSet (FanOnGpio, GPIO_FSEL_OUTPUT); </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> } </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> /* </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> * Bluetooth pin muxing </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> */ </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> GpioPinFuncSet (31, GPIO_FSEL_OUTPUT); </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> GpioPinSet (31, 0); </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> if ((PcdGet32 (PcdUartInUse) == PL011_UART_IN_USE)) { </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> GpioPinFuncSet (32, GPIO_FSEL_ALT3); </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> GpioPinFuncSet (33, GPIO_FSEL_ALT3); </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> } else { </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> GpioPinFuncSet (32, GPIO_FSEL_ALT5); </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> GpioPinFuncSet (33, GPIO_FSEL_ALT5); </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> } </blockquote> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> } </blockquote> </blockquote> <div> </div> <div> --Mario </div> </div> </div> </div> </div> </div> </div> </div> </div> </div> </div> </div> <div> <div> În lun., 31 mai 2021 la 16:26, Sunny Wang <<a href="mailto:Sunny.Wang@arm.com" target="_blank">Sunny.Wang@arm.com</a>> a scris: </div> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in"> <div> <div> Hi Mario, Thanks for checking this. Yeah, the problem is NO COM port for PL011 UART in Windows IOT’s device manager, so I thought it doesn’t work. If there is no COM port for PL011 UART, how can I check PL011 UART’s functionality? How can I use a telnet tool like PuTTY to send messages from RPi4 to my laptop? How to use SerCx2? Is there a guideline for using PL011 UART on RPi4 with Windows IoT? As for why I mux both UARTs to the BT chip, I was trying to address your comment “This always assumes that PL011 is used for Bluetooth”. Apparently, it didn’t address your comment. No matter whether I add “PinFunction (Exclusive, PullDown, BCM_ALT5, "<a>\\_SB.GDV0.GPI0</a>", 0, ResourceConsumer, , ) { 32, 33 }” or not, the Bluetooth has NOT worked when I configure PL011 UART as the primary serial console. Bluetooth only works when I configure Mini UART as the Primary serial console. I also confirmed that this problem also exists with RPi4 Release FW 1.26 + edk2-platforms-raspberrypi-pl011-bth-noflow.diff in <a href="https://github.com/worproject/RPi-Bluetooth-Testing/" target="_blank"> https://github.com/worproject/RPi-Bluetooth-Testing/</a>. Do you have any idea to make the Bluetooth work with configuring PL011 UART as the primary serial console? By the way, it is good to know the loading sequence. Do you know where I can quickly find this information without using WinDbg? Actually, the original code change (Patch 2/2) is the patch (edk2-platforms-raspberrypi-pl011-bth-noflow.diff) on <a href="https://github.com/worproject/RPi-Bluetooth-Testing/" target="_blank">https://github.com/worproject/RPi-Bluetooth-Testing/</a>. I’m not familiar with Windows drivers’ behavior, so I had no idea about why we need to change them and couldn’t find more information about the changes’ background either. It looks like you know the details. Could you share with me more information about the background of these changes? At least, I have some questions below: 1. What is the connection between GPIO 128 and Bluetooth (BT_ON)? Why did we add this line in the beginning? I just removed it by your request and according to the change in worproject/RPi-Bluetooth-Testing/, but I’m still worried about if we did this for some other purposes. 2. Why do we need to configure GPIO 32 and 33? Is the Bluetooth using GPIO 32 and 33? Or Is Windows OS checking GPIO 32 and 33 for Bluetooth? Where can I find the information? 3. As for your comment about GPIO 31, I was confused, so I didn’t update anything for addressing this comment. My understanding was that It is intended to override the BCM_ALT2’s default setting from PullLow to PullNone to fack the CTS signal. If we change it to PullLow, it will be used for HW flow control, which is unwanted, isn’t it? What did you want me to change? Could you directly paste the code change you wanted here for my reference? Moreover, for the changes you want, could you also paste the code change? Best Regards, Sunny Wang <div style="border-right:none;border-bottom:none;border-left:none;border-top:1pt solid rgb(225,225,225);padding:3pt 0in 0in"> From: <a href="mailto:devel@edk2.groups.io" target="_blank">devel@edk2.groups.io</a> <<a href="mailto:devel@edk2.groups.io" target="_blank">devel@edk2.groups.io</a>> On Behalf Of Mario Balanica via <a href="http://groups.io" target="_blank"> groups.io</a> Sent: Monday, May 31, 2021 8:41 PM To: Sunny Wang <<a href="mailto:Sunny.Wang@arm.com" target="_blank">Sunny.Wang@arm.com</a>> Cc: <a href="mailto:devel@edk2.groups.io" target="_blank">devel@edk2.groups.io</a>; Samer El-Haj-Mahmoud <<a href="mailto:Samer.El-Haj-Mahmoud@arm.com" target="_blank">Samer.El-Haj-Mahmoud@arm.com</a>>; Sami Mujawar <<a href="mailto:Sami.Mujawar@arm.com" target="_blank">Sami.Mujawar@arm.com</a>>; Jeremy Linton <<a href="mailto:Jeremy.Linton@arm.com" target="_blank">Jeremy.Linton@arm.com</a>>; Pete Batard <<a href="mailto:pete@akeo.ie" target="_blank">pete@akeo.ie</a>>; Ard Biesheuvel <<a href="mailto:ardb%2Btianocore@kernel.org" target="_blank">ardb+tianocore@kernel.org</a>> Subject: Re: [edk2-devel] [PATCH v3 2/2] Platform/RaspberryPi: Enable Bluetooth and UART in Windows OS </div> <div> <div> <div> Hi Sunny, <div> </div> <div> What issues are you seeing with the PL011 UART in Windows? Last time I checked, it worked fine, and the fact that Bluetooth works also confirms this. </div> <div> It won't show up as a COM port (like mini UART does) as it's built using SerCx2. </div> <div> </div> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin:5pt 0in 5pt 4.8pt"> @@ -30,6 +30,12 @@ Device (URT0) { MEMORY32FIXED (ReadWrite, 0, BCM2836_PL011_UART_LENGTH, RMEM) Interrupt (ResourceConsumer, Level, ActiveHigh, Exclusive) { BCM2836_PL011_UART_INTERRUPT } + + PinFunction (Exclusive, PullDown, BCM_ALT3, "<a>\\_SB.GDV0.GPI0</a>", 0, ResourceConsumer, , ) { 32, 33 } + </blockquote> <div> </div> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin:5pt 0in 5pt 4.8pt"> @@ -79,6 +85,11 @@ Device (URTM) // from muxing the pins away. // PinFunction (Exclusive, PullDown, BCM_ALT5, "<a>\\_SB.GPI0</a>", 0, ResourceConsumer, , ) { 14, 15 } + PinFunction (Exclusive, PullDown, BCM_ALT5, "<a>\\_SB.GDV0.GPI0</a>", 0, ResourceConsumer, , ) { 32, 33 } + </blockquote> <div> </div> <div> What is the reason for trying to mux both UARTs to the BT chip? If PL011 is used for Bluetooth and the mini UART driver loads after it, wouldn't it mux away the pins and break Bluetooth? </div> <div> </div> <div> </div> <div> </div> </div> </div> </div> <div> <div> În lun., 31 mai 2021 la 11:23, Sunny Wang <<a href="mailto:Sunny.Wang@arm.com" target="_blank">Sunny.Wang@arm.com</a>> a scris: </div> <blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin:5pt 0in 5pt 4.8pt"> This change is based on edk2-platforms-raspberrypi-pl011-bth-noflow.diff in <a href="https://github.com/worproject/RPi-Bluetooth-Testing/" target="_blank"> https://github.com/worproject/RPi-Bluetooth-Testing/</a> for enabling Bluetooth and serial port (Mini UART) in Windows IOT. Note that PL011 UART still doesn't work with Windows 10 IOT with this change, but PL011 UART works fine with VMware ESXi-Arm Fling v1.3. Therefore, there should be no issue with PL011 UART related changes, and we will still need a Windows expert to take a further look in the future. Making PL011 UART work properly with Windows 10 IOT may require additional changes to Windows driver or firmware's ACPI table. Testing Done: - Successfully booted Windows Windows 10 IOT (20279.1) on SD (made by WOR) with the RPi-Windows-Drivers release ver 0.5 downloaded from <a href="https://github.com/worproject/RPi-Windows-Drivers/releases" target="_blank"> https://github.com/worproject/RPi-Windows-Drivers/releases</a> and checked that both Bluetooth and serial port (Mini UART) can work fine. - Successfully booted VMware ESXi-Arm Fling v1.3 with only serial console connection (PL011 UART). Cc: Samer El-Haj-Mahmoud <<a href="mailto:samer.el-haj-mahmoud@arm.com" target="_blank">samer.el-haj-mahmoud@arm.com</a>> Cc: Sami Mujawar <<a href="mailto:sami.mujawar@arm.com" target="_blank">sami.mujawar@arm.com</a>> Cc: Jeremy Linton <<a href="mailto:jeremy.linton@arm.com" target="_blank">jeremy.linton@arm.com</a>> Cc: Pete Batard <<a href="mailto:pete@akeo.ie" target="_blank">pete@akeo.ie</a>> Cc: Ard Biesheuvel <<a href="mailto:ardb%2Btianocore@kernel.org" target="_blank">ardb+tianocore@kernel.org</a>> Cc: Mario Bălănică <<a href="mailto:mariobalanica02@gmail.com" target="_blank">mariobalanica02@gmail.com</a>> Signed-off-by: Sunny Wang <<a href="mailto:sunny.wang@arm.com" target="_blank">sunny.wang@arm.com</a>> --- Platform/RaspberryPi/AcpiTables/Uart.asl | 19 +++++++++++-------- 1 file changed, 11 insertions(+), 8 deletions(-) diff --git a/Platform/RaspberryPi/AcpiTables/Uart.asl b/Platform/RaspberryPi/AcpiTables/Uart.asl index bac9d791eb..cb99086d39 100644 --- a/Platform/RaspberryPi/AcpiTables/Uart.asl +++ b/Platform/RaspberryPi/AcpiTables/Uart.asl @@ -30,6 +30,12 @@ Device (URT0) { MEMORY32FIXED (ReadWrite, 0, BCM2836_PL011_UART_LENGTH, RMEM) Interrupt (ResourceConsumer, Level, ActiveHigh, Exclusive) { BCM2836_PL011_UART_INTERRUPT } + + PinFunction (Exclusive, PullDown, BCM_ALT3, "<a>\\_SB.GDV0.GPI0</a>", 0, ResourceConsumer, , ) { 32, 33 } + + // fake the CTS signal as we don't support HW flow control yet + // BCM_ALT2 is set as output (low) by default + PinFunction (Exclusive, PullNone, BCM_ALT2, "<a>\\_SB.GDV0.GPI0</a>", 0, ResourceConsumer, , ) { 31 } }) Method (_CRS, 0x0, Serialized) { @@ -79,6 +85,11 @@ Device (URTM) // from muxing the pins away. // PinFunction (Exclusive, PullDown, BCM_ALT5, "<a>\\_SB.GPI0</a>", 0, ResourceConsumer, , ) { 14, 15 } + PinFunction (Exclusive, PullDown, BCM_ALT5, "<a>\\_SB.GDV0.GPI0</a>", 0, ResourceConsumer, , ) { 32, 33 } + + // fake the CTS signal as we don't support HW flow control yet + // BCM_ALT2 is set as output (low) by default + PinFunction (Exclusive, PullNone, BCM_ALT2, "<a>\\_SB.GDV0.GPI0</a>", 0, ResourceConsumer, , ) { 31 } }) Method (_CRS, 0x0, Serialized) { @@ -143,10 +154,6 @@ Device(BTH0) UAR0, // DescriptorName: creates name // for offset of resource descriptor ) // Vendor data - // - // RPIQ connection for BT_ON/OFF - // - GpioIO (Shared, PullUp, 0, 0, IoRestrictionNone, "<a>\\_SB.GDV0.RPIQ</a>", 0, ResourceConsumer, , ) { 128 } }) // @@ -190,10 +197,6 @@ Device(BTH0) UARM, // DescriptorName: creates name // for offset of resource descriptor ) // Vendor data - // - // RPIQ connection for BT_ON/OFF - // - GpioIO (Shared, PullUp, 0, 0, IoRestrictionNone, "<a>\\_SB.GDV0.RPIQ</a>", 0, ResourceConsumer, , ) { 128 } }) Method (_CRS, 0x0, Serialized) -- 2.31.0.windows.1 </blockquote> </div> </div> IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. </div> </blockquote> </div> </blockquote> </div> <div> </div> </div> IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. </div> </blockquote></div> <div width="1" style="color:white;clear:both">_._,_._,_</div> <hr> Groups.io Links: You receive all messages sent to this group. <a target="_blank" href="https://edk2.groups.io/g/devel/message/76134">View/Reply Online (#76134)</a> | | <a target="_blank" href="https://groups.io/mt/83205747/1813853">Mute This Topic</a> | <a href="https://edk2.groups.io/g/devel/post">New Topic</a> <a href="https://edk2.groups.io/g/devel/editsub/1813853">Your Subscription</a> | <a href="mailto:devel+owner@edk2.groups.io">Contact Group Owner</a> | <a href="https://edk2.groups.io/g/devel/unsub">Unsubscribe</a> [edk2-devel-archive@redhat.com] <div width="1" style="color:white;clear:both">_._,_._,_</div>