<feed xmlns='http://www.w3.org/2005/Atom'>
<title>src/sys/dev/sdhci, branch main</title>
<subtitle>FreeBSD source tree</subtitle>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/'/>
<entry>
<title>sdhci_fdt_rockchip: Don't return 0 from probe routine</title>
<updated>2026-02-09T16:27:23+00:00</updated>
<author>
<name>John Baldwin</name>
<email>jhb@FreeBSD.org</email>
</author>
<published>2026-02-09T16:27:23+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=3f18463745c9c75e1fdcfacc710799838ef5c9d9'/>
<id>3f18463745c9c75e1fdcfacc710799838ef5c9d9</id>
<content type='text'>
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
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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
</pre>
</div>
</content>
</entry>
<entry>
<title>sdhci_fdt_rockchip: Move ofw_bus_status_okay check</title>
<updated>2026-02-09T16:27:04+00:00</updated>
<author>
<name>John Baldwin</name>
<email>jhb@FreeBSD.org</email>
</author>
<published>2026-02-09T16:27:04+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=efdd30c94e0ac4007b52b6f40afdab52fecf2aa5'/>
<id>efdd30c94e0ac4007b52b6f40afdab52fecf2aa5</id>
<content type='text'>
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
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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
</pre>
</div>
</content>
</entry>
<entry>
<title>sdhci_pci(4): Remove incorrect __unused</title>
<updated>2026-01-08T01:20:54+00:00</updated>
<author>
<name>Marius Strobl</name>
<email>marius@FreeBSD.org</email>
</author>
<published>2026-01-07T22:18:21+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=9b5d6ab1be32772ba86021e50e709db9a1659226'/>
<id>9b5d6ab1be32772ba86021e50e709db9a1659226</id>
<content type='text'>
The slot parameter actually is used as index into the memory resource.

Fixes:	c11bbc7d
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The slot parameter actually is used as index into the memory resource.

Fixes:	c11bbc7d
</pre>
</div>
</content>
</entry>
<entry>
<title>sdhci: Try to complete the last transaction if dumping</title>
<updated>2025-12-17T14:17:06+00:00</updated>
<author>
<name>Justin Hibbits</name>
<email>jhibbits@FreeBSD.org</email>
</author>
<published>2025-12-17T14:15:40+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=1fad49baf390cb52f238e6c352d0bc0893c008c3'/>
<id>1fad49baf390cb52f238e6c352d0bc0893c008c3</id>
<content type='text'>
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
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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
</pre>
</div>
</content>
</entry>
<entry>
<title>gpio_alloc_intr_resource: Pass rid by value</title>
<updated>2025-12-09T20:03:03+00:00</updated>
<author>
<name>John Baldwin</name>
<email>jhb@FreeBSD.org</email>
</author>
<published>2025-12-09T20:03:03+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=01dcb4e6645f1a551e52763d58ba4375efbc5428'/>
<id>01dcb4e6645f1a551e52763d58ba4375efbc5428</id>
<content type='text'>
Reviewed by:	imp
Differential Revision:	https://reviews.freebsd.org/D53407
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Reviewed by:	imp
Differential Revision:	https://reviews.freebsd.org/D53407
</pre>
</div>
</content>
</entry>
<entry>
<title>sdhci: Add driver for the cvitek sdhci controller</title>
<updated>2025-08-01T12:35:54+00:00</updated>
<author>
<name>Bojan Novković</name>
<email>bnovkov@FreeBSD.org</email>
</author>
<published>2025-01-18T19:03:25+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=89e650eacc9bb3960b38d704abae06694fad08e6'/>
<id>89e650eacc9bb3960b38d704abae06694fad08e6</id>
<content type='text'>
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
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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
</pre>
</div>
</content>
</entry>
<entry>
<title>machine/stdarg.h -&gt; sys/stdarg.h</title>
<updated>2025-06-11T16:39:02+00:00</updated>
<author>
<name>Brooks Davis</name>
<email>brooks@FreeBSD.org</email>
</author>
<published>2025-06-11T16:39:02+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=e453e498cbb88570a3ff7b3679de65c88707da95'/>
<id>e453e498cbb88570a3ff7b3679de65c88707da95</id>
<content type='text'>
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
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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
</pre>
</div>
</content>
</entry>
<entry>
<title>sdhci: Use bus_topo_lock and taskqueue_bus for hotplug events</title>
<updated>2025-03-10T17:33:49+00:00</updated>
<author>
<name>John Baldwin</name>
<email>jhb@FreeBSD.org</email>
</author>
<published>2025-03-10T17:33:49+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=aaf0a7302d10912e62dcd8e047798b4a2aefa039'/>
<id>aaf0a7302d10912e62dcd8e047798b4a2aefa039</id>
<content type='text'>
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
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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
</pre>
</div>
</content>
</entry>
<entry>
<title>sdhci: Fixes for sdhci_fdt_rockchip.c</title>
<updated>2025-02-16T10:41:43+00:00</updated>
<author>
<name>Bojan Novković</name>
<email>bnovkov@FreeBSD.org</email>
</author>
<published>2025-02-16T10:41:43+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=19f202f859b101507e26fe181aaf9f0b99f4d59c'/>
<id>19f202f859b101507e26fe181aaf9f0b99f4d59c</id>
<content type='text'>
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)
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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)
</pre>
</div>
</content>
</entry>
<entry>
<title>sdhci: Refactor the generic FDT driver</title>
<updated>2025-02-07T09:06:07+00:00</updated>
<author>
<name>Bojan Novković</name>
<email>bnovkov@FreeBSD.org</email>
</author>
<published>2025-01-18T19:02:51+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.freebsd.org/src/commit/?id=e17e33f997d63107e3a6859cfe3c19eba041b424'/>
<id>e17e33f997d63107e3a6859cfe3c19eba041b424</id>
<content type='text'>
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
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
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
</pre>
</div>
</content>
</entry>
</feed>
