Page MenuHomeFreeBSD

sdhci_xenon: add ACPI support
ClosedPublic

Authored by bag_semihalf.com on Aug 18 2021, 8:52 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Oct 21, 4:42 PM
Unknown Object (File)
Sat, Oct 12, 7:20 PM
Unknown Object (File)
Oct 1 2024, 1:52 PM
Unknown Object (File)
Sep 27 2024, 7:54 AM
Unknown Object (File)
Sep 26 2024, 10:20 PM
Unknown Object (File)
Sep 23 2024, 3:07 PM
Unknown Object (File)
Sep 23 2024, 4:42 AM
Unknown Object (File)
Sep 22 2024, 5:09 PM

Details

Summary

Add support for ACPI device probing for SDHCI controller found on Marvell chips.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

sys/conf/files.arm64
281

Add in alphabetical order.

sys/dev/sdhci/sdhci_xenon_acpi.c
87

Don't mix assignments with declarations - please revisit entire file.

100

Add comment, that in ACPI world the firmware is responsible for configuring the regulators.

103

This will disappear after moving max_clk handling to sdhci_xenon_attach

104

slot->quirks = 0; is done twice.

110

This will disappear after moving max_clk handling to sdhci_xenon_attach

119

No need to assign NULL values - sc fields are zeroed by default.

mw requested changes to this revision.Sep 23 2021, 11:01 AM
This revision now requires changes to proceed.Sep 23 2021, 11:01 AM
bag_semihalf.com marked 7 inline comments as done.

LGTM + tested on (with DT and ACPI) on: MacchiatoBin, CN913x-DB and CN913x-CEx7 Evaluation Board

sys/dev/sdhci/sdhci_xenon_acpi.c
100

Above assignments are done in a common part, I'll remove while committing.

This revision is now accepted and ready to land.Sep 29 2021, 1:23 PM
This revision was automatically updated to reflect the committed changes.