On Fri, Dec 02, 2016 at 04:15:43PM +0800, Hanjun Guo wrote: > On 2016/12/2 11:46, Dongdong Liu wrote: > > Hi Bjorn > > > > 在 2016/12/2 0:31, Bjorn Helgaas 写道: > >> On Thu, Dec 01, 2016 at 10:04:48PM +0800, Dongdong Liu wrote: > >>> ... > >>> Aftet fixing the compile errors. I tested on HiSilicon D03 board. It works ok. > >>> The dmesg is as below > >> > >> Thanks very much for testing this! A few comments below. > >> > >>> root@(none)$ dmesg > >>> [ 0.000000] Booting Linux on physical CPU 0x10000 > >>> [ 0.000000] Linux version 4.9.0-rc1-gaf05ef2-dirty (l00290354@linux-ioko) (gcc version 4.9.3 20150211 (prerelease) (20150316) ) #257 SMP PREEMPT Thu Dec 1 22:13:05 CST 2016 > >>> [ 0.000000] Boot CPU: AArch64 Processor [411fd071] > >>> [ 0.000000] earlycon: hisilpcuart0 at MMIO 0x00000000a01b0000 (options '0,0x2f8') > >>> [ 0.000000] bootconsole [hisilpcuart0] enabled > >>> [ 0.000000] efi: Getting EFI parameters from FDT: > >>> [ 0.000000] efi: EFI v2.60 by EDK II > >>> [ 0.000000] efi: SMBIOS=0x3f110000 SMBIOS 3.0=0x39ce0000 ACPI=0x39db0000 ACPI 2.0=0x39db0014 MEMATTR=0x3c945018 > >> > >> On x86 we normally have info about the machine and BIOS, e.g., > >> > >> DMI: LENOVO 20FCS12V03/20FCS12V03, BIOS N1FET40W (1.14 ) 04/18/2016 > >> > >> I think this comes from dmi_present(), and it looks like we should > >> call that via arm64_dmi_init(). Any idea why we don't see that info? > >> Is it just missing from the SMBIOS table? I think it's useful to > >> have for debugging problems. > > > > I need to ask my colleagues about this, then reply. > > git grep -w "DMI:" > drivers/firmware/dmi_scan.c: printk(KERN_DEBUG "DMI: %s\n", dmi_ids_string); > drivers/firmware/dmi_scan.c: pr_debug("DMI: %s\n", dmi_ids_string); > > It's a pr_debug information, seems we didn't enable the debug print? Yeah, probably. Looks like somebody else was deceived into thinking that pr_debug() is equivalent to printk(KERN_DEBUG). Bjorn -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html