Page MenuHomeFreeBSD

Enable the SDHCI controller on ARMADA38X
ClosedPublic

Authored by loos on May 5 2017, 3:18 AM.
Tags
None
Referenced Files
F105768528: D10606.diff
Fri, Dec 20, 10:30 AM
Unknown Object (File)
Sun, Dec 1, 8:51 AM
Unknown Object (File)
Mon, Nov 25, 4:58 PM
Unknown Object (File)
Sat, Nov 23, 8:34 PM
Unknown Object (File)
Fri, Nov 22, 3:25 PM
Unknown Object (File)
Nov 7 2024, 8:02 AM
Unknown Object (File)
Oct 21 2024, 7:30 AM
Unknown Object (File)
Oct 18 2024, 7:28 AM
Subscribers

Details

Summary

Add the Marvell SDHCI controller the list of supported devices in sdhci_fdt.

Enable the SDHCI controller, bus and devices on ARMADA38X kernel.

Depends on: D10601, D10602
Sponsored by: Rubicon Communications, LLC (Netgate)

Test Plan

Tested on ClearFog Pro.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

loos retitled this revision from Enable the SDHCI controller for ARMADA38X to Enable the SDHCI controller on ARMADA38X.May 5 2017, 5:19 AM
sys/dev/sdhci/sdhci_fdt.c
196 ↗(On Diff #28046)

Hi,

Have you checked other quirks with linux drivers/mmc/host/sdhci-pxav3.c?

Also, I think this driver is also missing setting bus width - I guess it's always "4", right?

sys/dev/sdhci/sdhci_fdt.c
196 ↗(On Diff #28046)

Yes, the other quirks are not applicable for our SDHCI layer (we are still using SDMA) or are not applicable to this hardware.

The quirks were also checked against the NetBSD drivers (where I found the missing bits): https://nxr.netbsd.org/xref/src/sys/arch/arm/marvell/mvsoc_sdhc.c#83

The SDHCI code seems to be doing the right thing with respect to bus-width. If necessary we can handle it based on FDT properties.

sys/dev/sdhci/sdhci_fdt.c
196 ↗(On Diff #28046)

Thanks. I think we can add bus-width handling and parsing in the separate patch (a38x supports 8-bits as well and with current code we won't make use of it). Other than that I'm ok with the patch, thanks!

This revision is now accepted and ready to land.May 10 2017, 7:05 PM
This revision was automatically updated to reflect the committed changes.