Page MenuHomeFreeBSD

mmc: Fix regression in 8a8166e5bcfb breaking Stratix 10 boot
ClosedPublic

Authored by jrtc27 on Sep 28 2021, 6:19 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Jun 24, 10:28 PM
Unknown Object (File)
Fri, Jun 21, 11:19 PM
Unknown Object (File)
Fri, Jun 21, 8:40 AM
Unknown Object (File)
Fri, Jun 21, 8:23 AM
Unknown Object (File)
Fri, Jun 21, 8:00 AM
Unknown Object (File)
Sun, Jun 16, 8:30 AM
Unknown Object (File)
Fri, Jun 7, 2:21 AM
Unknown Object (File)
May 26 2024, 12:08 PM
Subscribers

Details

Summary

The refactoring in 8a8166e5bcfb introduced a functional change that
breaks booting on the Stratix 10, hanging when it should be attaching
da0. Previously OF_getencprop was called with a pointer to host->f_max,
so if it wasn't present then the existing value was left untouched, but
after that commit it will instead clobber the value with 0. The dwmmc
driver, as used on the Stratix 10, sets a default value before calling
mmc_fdt_parse and so was broken by this functional change. It appears
that aw_mmc also does the same thing, so was presumably also broken on
some boards.

Fixes: 8a8166e5bcfb ("mmc: switch mmc_helper to device_ api")

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 41816
Build 38704: arc lint + arc unit