On Tue, 17 Jul 2018, Chao Fan wrote: > + > +/* Search efi table for rsdp table. */ > +static bool efi_get_rsdp_addr(acpi_physical_address *rsdp_addr) > +{ > + efi_system_table_t *systab; > + bool find_rsdp = false; > + bool acpi_20 = false; > + bool efi_64 = false; > + void *config_tables; > + struct efi_info *e; > + char *sig; > + int size; > + int i; > + > +#ifndef CONFIG_EFI > + return false; > +#endif
Please no. Wrap the whole function into CONFIG_EFI annd have a stub for the non EFI case returning false. Thanks, tglx