David Howells <dhowells@xxxxxxxxxx> wrote: > That makes it less clear. I think something like this makes it much more > obvious: > > static efi_status_t get_efi_var(const efi_char16_t *name, > const efi_guid_t *vendor, > u32 *attr, > unsigned long *data_size, void *data) > { > return efi_call_runtime(get_variable, > (efi_char16_t *)name, (efi_guid_t *)vendor, > attr, data_size, data); > } > > And then doing: > > status = get_efi_var(efi_SecureBoot_name, &efi_variable_guid, > NULL, &size, &val); > > which the compiler will inline. Of course, it has to be a macro because efi_call_runtime() has an undeclared argument on ARM... David -- To unsubscribe from this list: send the line "unsubscribe linux-efi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html