On Sat, Jun 29, 2019 at 07:23:26AM -0400, Narendra K wrote: > From: Narendra K <Narendra.K@xxxxxxxx> > > System firmware advertises the address of the 'Runtime > Configuration Interface table version 2 (RCI2)' via > an EFI Configuration Table entry. This code retrieves the RCI2 > table from the address and exports it to sysfs as a binary > attribute 'rci2' under /sys/firmware/efi/tables directory. > The approach adopted is similar to the attribute 'DMI' under > /sys/firmware/dmi/tables. > > RCI2 table contains BIOS HII in XML format and is used to populate > BIOS setup page in Dell EMC OpenManage Server Administrator tool. > The BIOS setup page contains BIOS tokens which can be configured. > > Signed-off-by: Narendra K <Narendra.K@xxxxxxxx> > --- Hi, apologies. I missed mentioning that the patch is created on the 'next' branch of efi tree. > RFC -> v0: > > - Removed rci2 table from struct efi and defined it in rci2_table.c similar to > the way uv_systab_phys is define in dmesg. Correcting the mistake above. It is defined similar to the way uv_systab_phys is defined in arch/x86/platform/uv/bios_uv.c . > - Removed the oem_tables array and added rci2 to common_tables array > - Removed the string 'rci2' from the common_tables array so that it is > not printed in dmesg. > - Merged function 'efi_rci2_table_init' into 'efi_rci2_sysfs_init' function to > avoid calling early_memremap/unmap functions. > > Also, a note unrelated to this patch - compilation error is observed when > testing with make defconfig related to uv_systab_phys in > arch/x86/platform/efi/efi.c. It seems like it needs to be protected with > CONFIG_X86_UV in efi_tables array. [...] -- With regards, Narendra K