History log of /freebsd-current/sys/arm/broadcom/bcm2835/bcm2838_pci.c
Revision Date Author Comments
# 10e0c34b 16-Apr-2024 HP van Braam <hp@tmm.cx>

Enable L1SS handling on RPI4 pcib

Thanks to @kevans91 for pointing me in the right direction. FreeBSD had
the same bug as Linux (see
https://bugzilla.kernel.org/show_bug.cgi?id=217276) where the ultimate
solution was to honor the brcm,enable-l1ss FDT property.

In current versions of the dtb files this property has been added by
default.

Without this on many, many pcie addin cards the pcib will Serror when
trying to assert the clreq# pin on the pcie bus. Many cards do not have
these hooked up.

PR: 260131, 277638, 277605
Reviewed-by: emaste
Signed-off-by: HP van Braam <hp@tmm.cx>
Pull-request: https://github.com/freebsd/freebsd-src/pull/1179


# fdafd315 24-Nov-2023 Warner Losh <imp@FreeBSD.org>

sys: Automated cleanup of cdefs and other formatting

Apply the following automated changes to try to eliminate
no-longer-needed sys/cdefs.h includes as well as now-empty
blank lines in a row.

Remove /^#if.*\n#endif.*\n#include\s+<sys/cdefs.h>.*\n/
Remove /\n+#include\s+<sys/cdefs.h>.*\n+#if.*\n#endif.*\n+/
Remove /\n+#if.*\n#endif.*\n+/
Remove /^#if.*\n#endif.*\n/
Remove /\n+#include\s+<sys/cdefs.h>\n#include\s+<sys/types.h>/
Remove /\n+#include\s+<sys/cdefs.h>\n#include\s+<sys/param.h>/
Remove /\n+#include\s+<sys/cdefs.h>\n#include\s+<sys/capsicum.h>/

Sponsored by: Netflix


# 685dc743 16-Aug-2023 Warner Losh <imp@FreeBSD.org>

sys: Remove $FreeBSD$: one-line .c pattern

Remove /^[\s*]*__FBSDID\("\$FreeBSD\$"\);?\s*\n/


# 95ee2897 16-Aug-2023 Warner Losh <imp@FreeBSD.org>

sys: Remove $FreeBSD$: two-line .h pattern

Remove /^\s*\*\n \*\s+\$FreeBSD\$$\n/


# 8bc94f25 23-Apr-2023 Andrew Turner <andrew@FreeBSD.org>

Remove redundant data from pci host generic

The bus tag and handle fields are already stored in the resource. Use
this with the bus_read/bus_write helper macros.

Sponsored by: Arm Ltd


# f16e3816 23-May-2022 Andrew Turner <andrew@FreeBSD.org>

Fix an indentation bug in the bcm pci driver

Sponsored by: The FreeBSD Foundation


# 5749fdc5 23-May-2022 Andrew Turner <andrew@FreeBSD.org>

Use the error given in the bcm PCI driver

When given an error return it and not just ENXIO as it may contain
information usefule to debug an error.

Sponsored by: The FreeBSD Foundation


# c3147104 23-May-2022 Andrew Turner <andrew@FreeBSD.org>

Error is not a bool in bcm2838_pci, check for != 0

Fix for style an check if error != 0 in the bcm2838 pci driver.

Sponsored by: The FreeBSD Foundation


# b25d7f74 23-May-2022 Andrew Turner <andrew@FreeBSD.org>

Fix the style of bcm_pcib_msi_attach a little

Move the definition of error to the top of the function.

Sponsored by: The FreeBSD Foundation


# 82d4dc06 09-May-2022 John Baldwin <jhb@FreeBSD.org>

arm/arm64 broadcom: Remove unused devclass arguments to DRIVER_MODULE.


# ac89220b 12-Sep-2020 Mike Karels <karels@FreeBSD.org>

bcm2838_pci.c: Respect DMA limits of controller.

Fixes for Raspberry Pi 4B PCIe / USB:
- Pass through a DMA tag for the controller.
- In theory the controller can access the lower 3 GB, but testing found
that unreliable. OpenBSD also restricts DMA to the lowest 960 MiB.
- Rename some constants to be a bit more meaningful.

Submitted by: Robert Crowston, crowston at protonmail.com
Reviewed by: mkarels, outside reviewers
Differential Revision: https://reviews.freebsd.org/D26344


# 65454883 01-Sep-2020 Mateusz Guzik <mjg@FreeBSD.org>

arm: clean up empty lines in .c and .h files


# 6e986252 12-Jul-2020 Michal Meloun <mmel@FreeBSD.org>

Fix the module name for some arm drivers.
Module name (unlike of the of driver name) must be system wide unique.

Reported by: Mark Millard(bcm_pci), andrew(mvebu_gpio)
MFC with: r362954, r362385


# eed8b80f 06-Jul-2020 Andrew Turner <andrew@FreeBSD.org>

Add a driver for bcm2838 PCI express controller

This adds support for the Broadcom bcm2711 PCI express controller, found
on the Raspberry Pi 4 (aka the bcm2838 SoC). The driver has only been
developed against the soldered-on VIA XHCI controller and not tested
with other end points.

Submitted by: Robert Crowston <crowston_protonmail.com>
Differential Revision: https://reviews.freebsd.org/D25068