Messages in this thread | | | Subject | Re: [PATCH v2] EDAC, mellanox: Add ECC support for BlueField DDR4 | From | Randy Dunlap <> | Date | Thu, 21 Mar 2019 08:23:23 -0700 |
| |
On 3/21/19 7:30 AM, Junhan Zhou wrote: >>> +config EDAC_BLUEFIELD >>> + tristate "Mellanox BlueField Memory ECC" >>> + depends on (MELLANOX_PLATFORM && ARM64 && ACPI) || >> COMPILE_TEST >> >> Hi, >> >> While I am in favor of using COMPILE_TEST whenever possible, I don't see >> how that is possible here if: >> >> # MELLANOX_PLATFORM is not set >> # ARM64 is not set >> # ACPI is not set >> COMPILE_TEST=y >> >> Can you please explain? >> >> Thanks. >> >> ~Randy > > Good point. I tried compiling it against the x86 architecture and it fails because it can't find the definition for arm_smccc_smc(). But it would compile for the ARM64 architecture with both ACPI and MELLANOX _PLATFORM not set, so I'll change it to ARM64 && ((MELLANOX_PLATFORM && ACPI) || COMPILE_TEST) instead. > The dependency on ACPI is because our BlueField SoC firmware only uses ACPI to pass the device mappings, and MELLANOX_PLATFORM because this device is only seen on the Mellanox BlueField SoC so it would be pointless to be used elsewhere, but these doesn't prevent it from being compiled. > > -Junhan >
Sounds fine. Thanks.
-- ~Randy
| |