Re: [PATCH v7 01/11] kconfig: Add PicoSAT interface
From: Luis Chamberlain
Date: Mon Feb 10 2025 - 10:37:57 EST
On Sat, Feb 08, 2025 at 05:39:49PM +0100, Ole Schuerks wrote:
> +bool load_picosat(void)
> +{
> + void *handle = NULL;
> + bool failed = false;
> +
> + /*
> + * Try different names for the .so library. This is necessary since
> + * all packages don't use the same versioning.
> + */
> + for (int i = 0; i < ARRAY_SIZE(picosat_lib_names) && !handle; ++i)
> + handle = dlopen(picosat_lib_names[i], RTLD_LAZY);
> + if (!handle) {
> + printd("%s\n", dlerror());
> + return false;
> + }
This will only detect an error if the last dlopen() failed.
Other than that:
Reviewed-by: Luis Chamberlain <mcgrof@xxxxxxxxxx>
Luis