Page MenuHomeFreeBSD

mmc_helper: add mmc_helper and rename mmc_fdt_helper to mmc_helper
ClosedPublic

Authored by bag_semihalf.com on Aug 18 2021, 8:49 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 24, 9:46 PM
Unknown Object (File)
Sun, Apr 21, 9:34 PM
Unknown Object (File)
Sat, Apr 20, 6:52 AM
Unknown Object (File)
Sat, Apr 20, 6:52 AM
Unknown Object (File)
Sat, Apr 20, 6:52 AM
Unknown Object (File)
Sat, Apr 20, 6:52 AM
Unknown Object (File)
Sat, Apr 20, 6:01 AM
Unknown Object (File)
Sun, Apr 7, 11:08 PM

Details

Summary

Add generic mmc_helper which uses device_get_property api.

Rename mmc_fdt_helper structure to mmc_helper to get rid of redundant code.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Code looks ok but that will break out of tree drivers (even if I'm sure that no such out of tree drivers using mmc_fdt_helpers exists) so I'm not too sure about this change.

I can add a macro like this somewhere in mmc_fdt_helper.h header, to keep it backwards compatible.

#define mmc_fdt_helper mmc_helper

I can add a macro like this somewhere in mmc_fdt_helper.h header, to keep it backwards compatible.

#define mmc_fdt_helper mmc_helper

Yes please.

I've added the macro. Should I revert the changes in other drivers? I didn't do it now, cause I think this could encourage people to use the new structure.

This revision is now accepted and ready to land.Aug 26 2021, 8:34 AM

I've added the macro. Should I revert the changes in other drivers? I didn't do it now, cause I think this could encourage people to use the new structure.

No, all drivers should use the new struct.

This revision was automatically updated to reflect the committed changes.