diff mbox series

ARM: bcm283x: change the virtual address of the XHCI PCI device base

Message ID 20210617092203.19825-1-m.szyprowski@samsung.com
State Accepted
Commit d28e127171768c618fb4d03ec4d3249527d7dccd
Headers show
Series ARM: bcm283x: change the virtual address of the XHCI PCI device base | expand

Commit Message

Marek Szyprowski June 17, 2021, 9:22 a.m. UTC
Move the XHCI PCI device base up in the virtual address space. This fixes
initialization failure observed with newer Raspberry Pi firmware, later
than 63b1922311 ("firmware: arm_loader: Update armstubs with those from
PR 117). It looks that chosing 0xff800000 as the XHCI PCI device base
conflicts with the updated ARM/VideoCore firmware.

This also requires to reduce the size of the mapped PCI device region
from 8MiB to 4MiB to fit into 32bit address space. This is still enough
for the XHCI PCI device.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

---
This fixes the issue observed on ARM 32bit after upgrading the RPi4
firmware files, described some time ago here:
https://lists.denx.de/pipermail/u-boot/2021-February/442317.html
---
 arch/arm/mach-bcm283x/init.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

-- 
2.17.1

Comments

Jaehoon Chung June 17, 2021, 9:44 p.m. UTC | #1
On 6/17/21 6:22 PM, Marek Szyprowski wrote:
> Move the XHCI PCI device base up in the virtual address space. This fixes

> initialization failure observed with newer Raspberry Pi firmware, later

> than 63b1922311 ("firmware: arm_loader: Update armstubs with those from

> PR 117). It looks that chosing 0xff800000 as the XHCI PCI device base

> conflicts with the updated ARM/VideoCore firmware.

> 

> This also requires to reduce the size of the mapped PCI device region

> from 8MiB to 4MiB to fit into 32bit address space. This is still enough

> for the XHCI PCI device.

> 

> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>


Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>


Best Regards,
Jaehoon Chung

> ---

> This fixes the issue observed on ARM 32bit after upgrading the RPi4

> firmware files, described some time ago here:

> https://lists.denx.de/pipermail/u-boot/2021-February/442317.html

> ---

>  arch/arm/mach-bcm283x/init.c | 4 ++--

>  1 file changed, 2 insertions(+), 2 deletions(-)

> 

> diff --git a/arch/arm/mach-bcm283x/init.c b/arch/arm/mach-bcm283x/init.c

> index 49027ce0a2..9803499985 100644

> --- a/arch/arm/mach-bcm283x/init.c

> +++ b/arch/arm/mach-bcm283x/init.c

> @@ -14,7 +14,7 @@

>  #include <asm/global_data.h>

>  

>  #define BCM2711_RPI4_PCIE_XHCI_MMIO_PHYS	0x600000000UL

> -#define BCM2711_RPI4_PCIE_XHCI_MMIO_SIZE	0x800000UL

> +#define BCM2711_RPI4_PCIE_XHCI_MMIO_SIZE	0x400000UL

>  

>  #ifdef CONFIG_ARM64

>  #include <asm/armv8/mmu.h>

> @@ -148,7 +148,7 @@ int mach_cpu_init(void)

>  

>  #ifdef CONFIG_ARMV7_LPAE

>  #ifdef CONFIG_TARGET_RPI_4_32B

> -#define BCM2711_RPI4_PCIE_XHCI_MMIO_VIRT	0xff800000UL

> +#define BCM2711_RPI4_PCIE_XHCI_MMIO_VIRT	0xffc00000UL

>  #include <addr_map.h>

>  #include <asm/system.h>

>  

>
nicolas saenz julienne June 18, 2021, 9 a.m. UTC | #2
On Thu, 2021-06-17 at 11:22 +0200, Marek Szyprowski wrote:
> Move the XHCI PCI device base up in the virtual address space. This fixes

> initialization failure observed with newer Raspberry Pi firmware, later

> than 63b1922311 ("firmware: arm_loader: Update armstubs with those from

> PR 117). It looks that chosing 0xff800000 as the XHCI PCI device base

> conflicts with the updated ARM/VideoCore firmware.

> 

> This also requires to reduce the size of the mapped PCI device region

> from 8MiB to 4MiB to fit into 32bit address space. This is still enough

> for the XHCI PCI device.

> 

> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

> Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>

> ---

> This fixes the issue observed on ARM 32bit after upgrading the RPi4

> firmware files, described some time ago here:

> https://lists.denx.de/pipermail/u-boot/2021-February/442317.html

> ---


Reviewed-by: Nicolas Saenz Julienne <nsaenz@kernel.org>


Regards,
Nicolas
Stefan Agner Sept. 3, 2021, 2:23 p.m. UTC | #3
On 2021-06-17 11:22, Marek Szyprowski wrote:
> Move the XHCI PCI device base up in the virtual address space. This fixes

> initialization failure observed with newer Raspberry Pi firmware, later

> than 63b1922311 ("firmware: arm_loader: Update armstubs with those from

> PR 117). It looks that chosing 0xff800000 as the XHCI PCI device base

> conflicts with the updated ARM/VideoCore firmware.

> 

> This also requires to reduce the size of the mapped PCI device region

> from 8MiB to 4MiB to fit into 32bit address space. This is still enough

> for the XHCI PCI device.

> 

> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>

> ---

> This fixes the issue observed on ARM 32bit after upgrading the RPi4

> firmware files, described some time ago here:

> https://lists.denx.de/pipermail/u-boot/2021-February/442317.html


This also fixes an issue we are seeing in Home Assistant:
https://github.com/home-assistant/operating-system/issues/1490

It does seem that it depends on the binary size (since USB enumeration
worked with 2021.04 at one point, but after enabling more configurations
it caused issues).

So:
Tested-by: Stefan Agner <stefan@agner.ch>


I've seen that this did not make it into master yet, which tree is this
expected to get merged through?

--
Stefan


> ---

>  arch/arm/mach-bcm283x/init.c | 4 ++--

>  1 file changed, 2 insertions(+), 2 deletions(-)

> 

> diff --git a/arch/arm/mach-bcm283x/init.c b/arch/arm/mach-bcm283x/init.c

> index 49027ce0a2..9803499985 100644

> --- a/arch/arm/mach-bcm283x/init.c

> +++ b/arch/arm/mach-bcm283x/init.c

> @@ -14,7 +14,7 @@

>  #include <asm/global_data.h>

>  

>  #define BCM2711_RPI4_PCIE_XHCI_MMIO_PHYS	0x600000000UL

> -#define BCM2711_RPI4_PCIE_XHCI_MMIO_SIZE	0x800000UL

> +#define BCM2711_RPI4_PCIE_XHCI_MMIO_SIZE	0x400000UL

>  

>  #ifdef CONFIG_ARM64

>  #include <asm/armv8/mmu.h>

> @@ -148,7 +148,7 @@ int mach_cpu_init(void)

>  

>  #ifdef CONFIG_ARMV7_LPAE

>  #ifdef CONFIG_TARGET_RPI_4_32B

> -#define BCM2711_RPI4_PCIE_XHCI_MMIO_VIRT	0xff800000UL

> +#define BCM2711_RPI4_PCIE_XHCI_MMIO_VIRT	0xffc00000UL

>  #include <addr_map.h>

>  #include <asm/system.h>
Matthias Brugger Sept. 15, 2021, 11:22 a.m. UTC | #4
On 17/06/2021 11:22, Marek Szyprowski wrote:
> Move the XHCI PCI device base up in the virtual address space. This fixes

> initialization failure observed with newer Raspberry Pi firmware, later

> than 63b1922311 ("firmware: arm_loader: Update armstubs with those from

> PR 117). It looks that chosing 0xff800000 as the XHCI PCI device base

> conflicts with the updated ARM/VideoCore firmware.

> 

> This also requires to reduce the size of the mapped PCI device region

> from 8MiB to 4MiB to fit into 32bit address space. This is still enough

> for the XHCI PCI device.

> 

> Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>


Queued in rpi-next now.

Thanks,
Matthias

> ---

> This fixes the issue observed on ARM 32bit after upgrading the RPi4

> firmware files, described some time ago here:

> https://lists.denx.de/pipermail/u-boot/2021-February/442317.html

> ---

>   arch/arm/mach-bcm283x/init.c | 4 ++--

>   1 file changed, 2 insertions(+), 2 deletions(-)

> 

> diff --git a/arch/arm/mach-bcm283x/init.c b/arch/arm/mach-bcm283x/init.c

> index 49027ce0a2..9803499985 100644

> --- a/arch/arm/mach-bcm283x/init.c

> +++ b/arch/arm/mach-bcm283x/init.c

> @@ -14,7 +14,7 @@

>   #include <asm/global_data.h>

>   

>   #define BCM2711_RPI4_PCIE_XHCI_MMIO_PHYS	0x600000000UL

> -#define BCM2711_RPI4_PCIE_XHCI_MMIO_SIZE	0x800000UL

> +#define BCM2711_RPI4_PCIE_XHCI_MMIO_SIZE	0x400000UL

>   

>   #ifdef CONFIG_ARM64

>   #include <asm/armv8/mmu.h>

> @@ -148,7 +148,7 @@ int mach_cpu_init(void)

>   

>   #ifdef CONFIG_ARMV7_LPAE

>   #ifdef CONFIG_TARGET_RPI_4_32B

> -#define BCM2711_RPI4_PCIE_XHCI_MMIO_VIRT	0xff800000UL

> +#define BCM2711_RPI4_PCIE_XHCI_MMIO_VIRT	0xffc00000UL

>   #include <addr_map.h>

>   #include <asm/system.h>

>   

>
diff mbox series

Patch

diff --git a/arch/arm/mach-bcm283x/init.c b/arch/arm/mach-bcm283x/init.c
index 49027ce0a2..9803499985 100644
--- a/arch/arm/mach-bcm283x/init.c
+++ b/arch/arm/mach-bcm283x/init.c
@@ -14,7 +14,7 @@ 
 #include <asm/global_data.h>
 
 #define BCM2711_RPI4_PCIE_XHCI_MMIO_PHYS	0x600000000UL
-#define BCM2711_RPI4_PCIE_XHCI_MMIO_SIZE	0x800000UL
+#define BCM2711_RPI4_PCIE_XHCI_MMIO_SIZE	0x400000UL
 
 #ifdef CONFIG_ARM64
 #include <asm/armv8/mmu.h>
@@ -148,7 +148,7 @@  int mach_cpu_init(void)
 
 #ifdef CONFIG_ARMV7_LPAE
 #ifdef CONFIG_TARGET_RPI_4_32B
-#define BCM2711_RPI4_PCIE_XHCI_MMIO_VIRT	0xff800000UL
+#define BCM2711_RPI4_PCIE_XHCI_MMIO_VIRT	0xffc00000UL
 #include <addr_map.h>
 #include <asm/system.h>