src/sys/dev/sdhci, branch mainFreeBSD source treehttps://cgit-dev.freebsd.org/src/atom?h=main2026-02-09T16:27:23Zsdhci_fdt_rockchip: Don't return 0 from probe routine2026-02-09T16:27:23ZJohn Baldwinjhb@FreeBSD.org2026-02-09T16:27:23Zurn:sha1:3f18463745c9c75e1fdcfacc710799838ef5c9d9
The only write to the softc is a no-op (softcs are initialized to all
zeroes), so remove that and just use BUS_PROBE_DEFAULT.
Reviewed by: bnovkov
Differential Revision: https://reviews.freebsd.org/D55159
sdhci_fdt_rockchip: Move ofw_bus_status_okay check2026-02-09T16:27:04ZJohn Baldwinjhb@FreeBSD.org2026-02-09T16:27:04Zurn:sha1:efdd30c94e0ac4007b52b6f40afdab52fecf2aa5
This fixes a warning from GCC:
sys/dev/sdhci/sdhci_fdt_rockchip.c: In function 'sdhci_fdt_rockchip_probe':
sys/dev/sdhci/sdhci_fdt_rockchip.c:120:14: error: statement will never be executed [-Werror=switch-unreachable]
120 | if (!ofw_bus_status_okay(dev))
| ^~~~~~~~~~~~~~~~~~~~~~~~
Reviewed by: bnovkov, imp, emaste
Differential Revision: https://reviews.freebsd.org/D55158
sdhci_pci(4): Remove incorrect __unused2026-01-08T01:20:54ZMarius Stroblmarius@FreeBSD.org2026-01-07T22:18:21Zurn:sha1:9b5d6ab1be32772ba86021e50e709db9a1659226
The slot parameter actually is used as index into the memory resource.
Fixes: c11bbc7d
sdhci: Try to complete the last transaction if dumping2025-12-17T14:17:06ZJustin Hibbitsjhibbits@FreeBSD.org2025-12-17T14:15:40Zurn:sha1:1fad49baf390cb52f238e6c352d0bc0893c008c3
If the kernel panics while a thread is in the middle of an SDHCI
transaction, trying to dump to a dump target on the MMC would result in
a hang. Fix this by completing the transaction first.
Reviewed by: imp
Obtained from: Hewlett Packard Enterprise
Differential Revision: https://reviews.freebsd.org/D54255
gpio_alloc_intr_resource: Pass rid by value2025-12-09T20:03:03ZJohn Baldwinjhb@FreeBSD.org2025-12-09T20:03:03Zurn:sha1:01dcb4e6645f1a551e52763d58ba4375efbc5428
Reviewed by: imp
Differential Revision: https://reviews.freebsd.org/D53407
sdhci: Add driver for the cvitek sdhci controller2025-08-01T12:35:54ZBojan Novkovićbnovkov@FreeBSD.org2025-01-18T19:03:25Zurn:sha1:89e650eacc9bb3960b38d704abae06694fad08e6
This patch add support for cvitek SDHCI controller (only SD card mode).
This controller is present on Milk-V's riscv SoCs.
Reviewed by: manu
Differential Revision: https://reviews.freebsd.org/D48528
machine/stdarg.h -> sys/stdarg.h2025-06-11T16:39:02ZBrooks Davisbrooks@FreeBSD.org2025-06-11T16:39:02Zurn:sha1:e453e498cbb88570a3ff7b3679de65c88707da95
Switch to using sys/stdarg.h for va_list type and va_* builtins.
Make an attempt to insert the include in a sensible place. Where
style(9) was followed this is easy, where it was ignored, aim for the
first block of sys/*.h headers and don't get too fussy or try to fix
other style bugs.
Reviewed by: imp
Exp-run by: antoine (PR 286274)
Pull Request: https://github.com/freebsd/freebsd-src/pull/1595
sdhci: Use bus_topo_lock and taskqueue_bus for hotplug events2025-03-10T17:33:49ZJohn Baldwinjhb@FreeBSD.org2025-03-10T17:33:49Zurn:sha1:aaf0a7302d10912e62dcd8e047798b4a2aefa039
Drop SDHCI_LOCK and instead acquire bus_topo_lock when adding and
removing new-bus devices.
Reviewed by: imp
Differential Revision: https://reviews.freebsd.org/D49271
sdhci: Fixes for sdhci_fdt_rockchip.c2025-02-16T10:41:43ZBojan Novkovićbnovkov@FreeBSD.org2025-02-16T10:41:43Zurn:sha1:19f202f859b101507e26fe181aaf9f0b99f4d59c
This change fixes a couple of issues in the Rockchip SDHCI driver:
- Fix a panic caused by sdhci_fdt_rockchip_attach not populating the
softc's dev variable before initializing clocks
- Fix a bug where sdhci_fdt_rockchip_set_clock fails to call
sdhci_fdt_set_clock
Fixes: e17e33f997d6
Reported by: Alonso Cárdenas Márquez (acardenas@bsd-peru.org)
sdhci: Refactor the generic FDT driver2025-02-07T09:06:07ZBojan Novkovićbnovkov@FreeBSD.org2025-01-18T19:02:51Zurn:sha1:e17e33f997d63107e3a6859cfe3c19eba041b424
This patch refactors the 'sdhci_fdt.c' driver by moving all vendor
specific routines into separate files and making the base 'sdhci_fdt'
driver subclassable. The goal is to make adding new FDT-based drivers
easier and more maintainable. No functional change intended.
Reviewed by: manu, imp
Differential Revision: https://reviews.freebsd.org/D48527